diff --git a/core/modules/config_translation/src/ConfigEntityMapper.php b/core/modules/config_translation/src/ConfigEntityMapper.php index fd7fc537edf..541101bdcd3 100644 --- a/core/modules/config_translation/src/ConfigEntityMapper.php +++ b/core/modules/config_translation/src/ConfigEntityMapper.php @@ -13,12 +13,12 @@ use Drupal\Core\Config\TypedConfigManagerInterface; use Drupal\Core\Entity\EntityManagerInterface; use Drupal\Core\Language\LanguageManagerInterface; use Drupal\Core\Routing\RouteMatch; +use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\Routing\RouteProviderInterface; use Drupal\Core\StringTranslation\TranslationInterface; use Drupal\Core\Url; use Drupal\locale\LocaleConfigManager; use Symfony\Component\DependencyInjection\ContainerInterface; -use Symfony\Component\HttpFoundation\Request; use Symfony\Component\Routing\Route; /** @@ -110,9 +110,9 @@ class ConfigEntityMapper extends ConfigNamesMapper { /** * {@inheritdoc} */ - public function populateFromRequest(Request $request) { - parent::populateFromRequest($request); - $entity = $request->attributes->get($this->entityType); + public function populateFromRouteMatch(RouteMatchInterface $route_match) { + parent::populateFromRouteMatch($route_match); + $entity = $route_match->getParameter($this->entityType); $this->setEntity($entity); } diff --git a/core/modules/config_translation/src/ConfigMapperInterface.php b/core/modules/config_translation/src/ConfigMapperInterface.php index f794a10583c..9815f85785c 100644 --- a/core/modules/config_translation/src/ConfigMapperInterface.php +++ b/core/modules/config_translation/src/ConfigMapperInterface.php @@ -8,6 +8,7 @@ namespace Drupal\config_translation; use Drupal\Core\Language\LanguageInterface; +use Drupal\Core\Routing\RouteMatchInterface; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\Routing\RouteCollection; @@ -204,6 +205,16 @@ interface ConfigMapperInterface { */ public function getLangcode(); + /** + * Sets the original language code. + * + * @param string $langcode + * The langcode. + * + * @return $this + */ + public function setLangcode($langcode); + /** * Returns the name of the type of data the mapper encapsulates. * @@ -267,10 +278,10 @@ interface ConfigMapperInterface { * * @todo Replace $request with RouteMatch https://www.drupal.org/node/2295255. * - * @param \Symfony\Component\HttpFoundation\Request $request - * Page request object. + * @param \Drupal\Core\Routing\RouteMatchInterface $route_match + * The route match. */ - public function populateFromRequest(Request $request); + public function populateFromRouteMatch(RouteMatchInterface $route_match); /** * Returns the name of the contextual link group to add contextual links to. diff --git a/core/modules/config_translation/src/ConfigNamesMapper.php b/core/modules/config_translation/src/ConfigNamesMapper.php index ca4e8d8d2b3..6a51be64b9e 100644 --- a/core/modules/config_translation/src/ConfigNamesMapper.php +++ b/core/modules/config_translation/src/ConfigNamesMapper.php @@ -13,13 +13,13 @@ use Drupal\Core\Language\Language; use Drupal\Core\Language\LanguageInterface; use Drupal\Core\Language\LanguageManagerInterface; use Drupal\Core\Plugin\PluginBase; +use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\Routing\RouteProviderInterface; use Drupal\Core\StringTranslation\TranslationInterface; use Drupal\Core\Plugin\ContainerFactoryPluginInterface; use Drupal\Core\Url; use Drupal\locale\LocaleConfigManager; use Symfony\Component\DependencyInjection\ContainerInterface; -use Symfony\Component\HttpFoundation\Request; use Symfony\Component\Routing\Route; use Symfony\Component\Routing\RouteCollection; @@ -371,13 +371,8 @@ class ConfigNamesMapper extends PluginBase implements ConfigMapperInterface, Con /** * {@inheritdoc} */ - public function populateFromRequest(Request $request) { - if ($request->attributes->has('langcode')) { - $this->langcode = $request->attributes->get('langcode'); - } - else { - $this->langcode = NULL; - } + public function populateFromRouteMatch(RouteMatchInterface $route_match) { + $this->langcode = $route_match->getParameter('langcode'); } /** @@ -407,6 +402,14 @@ class ConfigNamesMapper extends PluginBase implements ConfigMapperInterface, Con return reset($langcodes); } + /** + * {@inheritdoc} + */ + public function setLangcode($langcode) { + $this->langcode = $langcode; + return $this; + } + /** * {@inheritdoc} */ diff --git a/core/modules/config_translation/src/Controller/ConfigTranslationController.php b/core/modules/config_translation/src/Controller/ConfigTranslationController.php index 0340758b96d..98ab72b50ef 100644 --- a/core/modules/config_translation/src/Controller/ConfigTranslationController.php +++ b/core/modules/config_translation/src/Controller/ConfigTranslationController.php @@ -13,6 +13,7 @@ use Drupal\Core\Controller\ControllerBase; use Drupal\Core\Language\Language; use Drupal\Core\Language\LanguageManagerInterface; use Drupal\Core\PathProcessor\InboundPathProcessorInterface; +use Drupal\Core\Routing\RouteMatch; use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\Session\AccountInterface; use Drupal\Core\Url; @@ -122,7 +123,7 @@ class ConfigTranslationController extends ControllerBase { public function itemPage(Request $request, RouteMatchInterface $route_match, $plugin_id) { /** @var \Drupal\config_translation\ConfigMapperInterface $mapper */ $mapper = $this->configMapperManager->createInstance($plugin_id); - $mapper->populateFromRequest($request); + $mapper->populateFromRouteMatch($route_match); $page = array(); $page['#title'] = $this->t('Translations for %label', array('%label' => $mapper->getTitle())); @@ -140,7 +141,7 @@ class ConfigTranslationController extends ControllerBase { } // We create a fake request object to pass into - // ConfigMapperInterface::populateFromRequest() for the different languages. + // ConfigMapperInterface::populateFromRouteMatch() for the different languages. // Creating a separate request for each language and route is neither easily // possible nor performant. $fake_request = $request->duplicate(); @@ -155,8 +156,9 @@ class ConfigTranslationController extends ControllerBase { // This is needed because // ConfigMapperInterface::getAddRouteParameters(), for example, // needs to return the correct language code for each table row. - $fake_request->attributes->set('langcode', $langcode); - $mapper->populateFromRequest($fake_request); + $fake_route_match = RouteMatch::createFromRequest($fake_request); + $mapper->populateFromRouteMatch($fake_route_match); + $mapper->setLangcode($langcode); // Prepare the language name and the operations depending on whether this // is the original language or not. diff --git a/core/modules/config_translation/src/Form/ConfigTranslationAddForm.php b/core/modules/config_translation/src/Form/ConfigTranslationAddForm.php index 883574dfd0e..5961c475894 100644 --- a/core/modules/config_translation/src/Form/ConfigTranslationAddForm.php +++ b/core/modules/config_translation/src/Form/ConfigTranslationAddForm.php @@ -8,7 +8,7 @@ namespace Drupal\config_translation\Form; use Drupal\Core\Form\FormStateInterface; -use Symfony\Component\HttpFoundation\Request; +use Drupal\Core\Routing\RouteMatchInterface; /** * Defines a form for adding configuration translations. @@ -25,8 +25,8 @@ class ConfigTranslationAddForm extends ConfigTranslationFormBase { /** * {@inheritdoc} */ - public function buildForm(array $form, FormStateInterface $form_state, Request $request = NULL, $plugin_id = NULL, $langcode = NULL) { - $form = parent::buildForm($form, $form_state, $request, $plugin_id, $langcode); + public function buildForm(array $form, FormStateInterface $form_state, RouteMatchInterface $route_match = NULL, $plugin_id = NULL, $langcode = NULL) { + $form = parent::buildForm($form, $form_state, $route_match, $plugin_id, $langcode); $form['#title'] = $this->t('Add @language translation for %label', array( '%label' => $this->mapper->getTitle(), '@language' => $this->language->getName(), diff --git a/core/modules/config_translation/src/Form/ConfigTranslationDeleteForm.php b/core/modules/config_translation/src/Form/ConfigTranslationDeleteForm.php index 0707a74c59a..8e2b669c09d 100644 --- a/core/modules/config_translation/src/Form/ConfigTranslationDeleteForm.php +++ b/core/modules/config_translation/src/Form/ConfigTranslationDeleteForm.php @@ -12,6 +12,7 @@ use Drupal\Core\Cache\Cache; use Drupal\Core\Extension\ModuleHandlerInterface; use Drupal\Core\Form\ConfirmFormBase; use Drupal\Core\Form\FormStateInterface; +use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\Url; use Drupal\language\ConfigurableLanguageManagerInterface; use Symfony\Component\DependencyInjection\ContainerInterface; @@ -116,10 +117,10 @@ class ConfigTranslationDeleteForm extends ConfirmFormBase { /** * {@inheritdoc} */ - public function buildForm(array $form, FormStateInterface $form_state, Request $request = NULL, $plugin_id = NULL, $langcode = NULL) { + public function buildForm(array $form, FormStateInterface $form_state, RouteMatchInterface $route_match = NULL, $plugin_id = NULL, $langcode = NULL) { /** @var \Drupal\config_translation\ConfigMapperInterface $mapper */ $mapper = $this->configMapperManager->createInstance($plugin_id); - $mapper->populateFromRequest($request); + $mapper->populateFromRouteMatch($route_match); $language = $this->languageManager->getLanguage($langcode); if (!$language) { diff --git a/core/modules/config_translation/src/Form/ConfigTranslationEditForm.php b/core/modules/config_translation/src/Form/ConfigTranslationEditForm.php index 2c1fc3c2778..bafbfa2f5e5 100644 --- a/core/modules/config_translation/src/Form/ConfigTranslationEditForm.php +++ b/core/modules/config_translation/src/Form/ConfigTranslationEditForm.php @@ -8,7 +8,7 @@ namespace Drupal\config_translation\Form; use Drupal\Core\Form\FormStateInterface; -use Symfony\Component\HttpFoundation\Request; +use Drupal\Core\Routing\RouteMatchInterface; /** * Defines a form for editing configuration translations. @@ -25,8 +25,8 @@ class ConfigTranslationEditForm extends ConfigTranslationFormBase { /** * {@inheritdoc} */ - public function buildForm(array $form, FormStateInterface $form_state, Request $request = NULL, $plugin_id = NULL, $langcode = NULL) { - $form = parent::buildForm($form, $form_state, $request, $plugin_id, $langcode); + public function buildForm(array $form, FormStateInterface $form_state, RouteMatchInterface $route_match = NULL, $plugin_id = NULL, $langcode = NULL) { + $form = parent::buildForm($form, $form_state, $route_match, $plugin_id, $langcode); $form['#title'] = $this->t('Edit @language translation for %label', array( '%label' => $this->mapper->getTitle(), '@language' => $this->language->getName(), diff --git a/core/modules/config_translation/src/Form/ConfigTranslationFormBase.php b/core/modules/config_translation/src/Form/ConfigTranslationFormBase.php index 9e5372f0589..f7b0497c037 100644 --- a/core/modules/config_translation/src/Form/ConfigTranslationFormBase.php +++ b/core/modules/config_translation/src/Form/ConfigTranslationFormBase.php @@ -9,6 +9,7 @@ namespace Drupal\config_translation\Form; use Drupal\config_translation\ConfigMapperManagerInterface; use Drupal\Core\Config\TypedConfigManagerInterface; +use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\TypedData\TypedDataInterface; use Drupal\Core\TypedData\TraversableTypedDataInterface; use Drupal\Core\Form\BaseFormIdInterface; @@ -16,7 +17,6 @@ use Drupal\Core\Form\FormBase; use Drupal\Core\Form\FormStateInterface; use Drupal\language\ConfigurableLanguageManagerInterface; use Symfony\Component\DependencyInjection\ContainerInterface; -use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpKernel\Exception\NotFoundHttpException; /** @@ -132,10 +132,10 @@ abstract class ConfigTranslationFormBase extends FormBase implements BaseFormIdI * Throws an exception if the language code provided as a query parameter in * the request does not match an active language. */ - public function buildForm(array $form, FormStateInterface $form_state, Request $request = NULL, $plugin_id = NULL, $langcode = NULL) { + public function buildForm(array $form, FormStateInterface $form_state, RouteMatchInterface $route_match = NULL, $plugin_id = NULL, $langcode = NULL) { /** @var \Drupal\config_translation\ConfigMapperInterface $mapper */ $mapper = $this->configMapperManager->createInstance($plugin_id); - $mapper->populateFromRequest($request); + $mapper->populateFromRouteMatch($route_match); $language = $this->languageManager->getLanguage($langcode); if (!$language) { diff --git a/core/modules/config_translation/src/Tests/ConfigTranslationFormTest.php b/core/modules/config_translation/src/Tests/ConfigTranslationFormTest.php index b162b867138..074b65413c2 100644 --- a/core/modules/config_translation/src/Tests/ConfigTranslationFormTest.php +++ b/core/modules/config_translation/src/Tests/ConfigTranslationFormTest.php @@ -55,8 +55,8 @@ class ConfigTranslationFormTest extends WebTestBase { */ public function testConfigTranslationFormAlter() { $form_builder = \Drupal::formBuilder(); - $add_form = $form_builder->getForm('Drupal\config_translation\Form\ConfigTranslationAddForm', \Drupal::request(), $this->pluginId, $this->langcode); - $edit_form = $form_builder->getForm('Drupal\config_translation\Form\ConfigTranslationEditForm', \Drupal::request(), $this->pluginId, $this->langcode); + $add_form = $form_builder->getForm('Drupal\config_translation\Form\ConfigTranslationAddForm', \Drupal::routeMatch(), $this->pluginId, $this->langcode); + $edit_form = $form_builder->getForm('Drupal\config_translation\Form\ConfigTranslationEditForm', \Drupal::routeMatch(), $this->pluginId, $this->langcode); // Test that hook_form_BASE_FORM_ID_alter() was called for the base form ID // 'config_translation_form'. diff --git a/core/modules/config_translation/tests/src/Unit/ConfigNamesMapperTest.php b/core/modules/config_translation/tests/src/Unit/ConfigNamesMapperTest.php index fce0d674bc8..9ecc32b3c2e 100644 --- a/core/modules/config_translation/tests/src/Unit/ConfigNamesMapperTest.php +++ b/core/modules/config_translation/tests/src/Unit/ConfigNamesMapperTest.php @@ -11,6 +11,7 @@ use Drupal\config_translation\ConfigNamesMapper; use Drupal\Core\Config\ConfigFactoryInterface; use Drupal\Core\DependencyInjection\ContainerBuilder; use Drupal\Core\Language\Language; +use Drupal\Core\Routing\RouteMatch; use Drupal\Core\Url; use Drupal\Tests\UnitTestCase; use Symfony\Component\Routing\Route; @@ -239,9 +240,8 @@ class ConfigNamesMapperTest extends UnitTestCase { * Tests ConfigNamesMapper::getAddRouteParameters(). */ public function testGetAddRouteParameters() { - $request = Request::create(''); - $request->attributes->set('langcode', 'xx'); - $this->configNamesMapper->populateFromRequest($request); + $route_match = new RouteMatch('example', new Route('/test/{langcode}'), ['langcode' => 'xx']); + $this->configNamesMapper->populateFromRouteMatch($route_match); $expected = array('langcode' => 'xx'); $result = $this->configNamesMapper->getAddRouteParameters(); @@ -278,9 +278,8 @@ class ConfigNamesMapperTest extends UnitTestCase { * Tests ConfigNamesMapper::getEditRouteParameters(). */ public function testGetEditRouteParameters() { - $request = Request::create(''); - $request->attributes->set('langcode', 'xx'); - $this->configNamesMapper->populateFromRequest($request); + $route_match = new RouteMatch('example', new Route('/test/{langcode}'), ['langcode' => 'xx']); + $this->configNamesMapper->populateFromRouteMatch($route_match); $expected = array('langcode' => 'xx'); $result = $this->configNamesMapper->getEditRouteParameters(); @@ -317,9 +316,8 @@ class ConfigNamesMapperTest extends UnitTestCase { * Tests ConfigNamesMapper::getDeleteRouteParameters(). */ public function testGetDeleteRouteParameters() { - $request = Request::create(''); - $request->attributes->set('langcode', 'xx'); - $this->configNamesMapper->populateFromRequest($request); + $route_match = new RouteMatch('example', new Route('/test/{langcode}'), ['langcode' => 'xx']); + $this->configNamesMapper->populateFromRouteMatch($route_match); $expected = array('langcode' => 'xx'); $result = $this->configNamesMapper->getDeleteRouteParameters(); $this->assertSame($expected, $result); @@ -370,25 +368,25 @@ class ConfigNamesMapperTest extends UnitTestCase { } /** - * Tests ConfigNamesMapper::populateFromRequest(). + * Tests ConfigNamesMapper::populateFromRouteMatch(). */ - public function testPopulateFromRequest() { + public function testPopulateFromRouteMatch() { // Make sure the language code is not set initially. $this->assertSame(NULL, $this->configNamesMapper->getInternalLangcode()); // Test that an empty request does not set the language code. - $request = Request::create(''); - $this->configNamesMapper->populateFromRequest($request); + $route_match = new RouteMatch('example', new Route('/test/{langcode}')); + $this->configNamesMapper->populateFromRouteMatch($route_match); $this->assertSame(NULL, $this->configNamesMapper->getInternalLangcode()); // Test that a request with a 'langcode' attribute sets the language code. - $request->attributes->set('langcode', 'xx'); - $this->configNamesMapper->populateFromRequest($request); + $route_match = new RouteMatch('example', new Route('/test/{langcode}'), ['langcode' => 'xx']); + $this->configNamesMapper->populateFromRouteMatch($route_match); $this->assertSame('xx', $this->configNamesMapper->getInternalLangcode()); // Test that the language code gets unset with the wrong request. - $request->attributes->remove('langcode'); - $this->configNamesMapper->populateFromRequest($request); + $route_match = new RouteMatch('example', new Route('/test/{langcode}')); + $this->configNamesMapper->populateFromRouteMatch($route_match); $this->assertSame(NULL, $this->configNamesMapper->getInternalLangcode()); }