summaryrefslogtreecommitdiffstats
path: root/core/modules/node/src
diff options
context:
space:
mode:
authorNathaniel Catchpole2017-09-29 15:27:59 (GMT)
committerNathaniel Catchpole2017-09-29 15:27:59 (GMT)
commit2df0fc02eff1d9a1e8664b72c0985c9e1da8d07e (patch)
tree09720f63d0c0c422d636fb9df0054b38bdce12c4 /core/modules/node/src
parent8685e7c72f70c9c1171a920e56bd45c80b8365a5 (diff)
Issue #2850085 by maxocub, heddn, Jo Fitzgerald, Gábor Hojtsy, catch, larowlan, mikeryan: Redirects for translation set migration path in Drupal 6 and 7
Diffstat (limited to 'core/modules/node/src')
-rw-r--r--core/modules/node/src/EventSubscriber/NodeTranslationExceptionSubscriber.php128
-rw-r--r--core/modules/node/src/EventSubscriber/NodeTranslationMigrateSubscriber.php113
-rw-r--r--core/modules/node/src/NodeServiceProvider.php42
3 files changed, 283 insertions, 0 deletions
diff --git a/core/modules/node/src/EventSubscriber/NodeTranslationExceptionSubscriber.php b/core/modules/node/src/EventSubscriber/NodeTranslationExceptionSubscriber.php
new file mode 100644
index 0000000..795670f
--- /dev/null
+++ b/core/modules/node/src/EventSubscriber/NodeTranslationExceptionSubscriber.php
@@ -0,0 +1,128 @@
+<?php
+
+namespace Drupal\node\EventSubscriber;
+
+use Drupal\Core\KeyValueStore\KeyValueFactoryInterface;
+use Drupal\Core\Language\LanguageManagerInterface;
+use Drupal\Core\ParamConverter\ParamNotConvertedException;
+use Drupal\Core\Routing\UrlGeneratorInterface;
+use Drupal\Core\State\StateInterface;
+use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+use Symfony\Component\HttpFoundation\RedirectResponse;
+use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
+use Symfony\Component\HttpKernel\KernelEvents;
+
+/**
+ * Redirect node translations that have been consolidated by migration.
+ *
+ * If we migrated node translations from Drupal 6 or 7, these nodes are now
+ * combined with their source language node. Since there still might be
+ * references to the URLs of these now consolidated nodes, this service catches
+ * the 404s and try to redirect them to the right node in the right language.
+ *
+ * The mapping of the old nids to the new ones is made by the
+ * NodeTranslationMigrateSubscriber class during the migration and is stored
+ * in the "node_translation_redirect" key/value collection.
+ *
+ * @see \Drupal\node\NodeServiceProvider
+ * @see \Drupal\node\EventSubscriber\NodeTranslationMigrateSubscriber
+ */
+class NodeTranslationExceptionSubscriber implements EventSubscriberInterface {
+
+ /**
+ * The key value factory.
+ *
+ * @var \Drupal\Core\KeyValueStore\KeyValueFactoryInterface
+ */
+ protected $keyValue;
+
+ /**
+ * The language manager.
+ *
+ * @var \Drupal\Core\Language\LanguageManagerInterface
+ */
+ protected $languageManager;
+
+ /**
+ * The URL generator.
+ *
+ * @var \Drupal\Core\Routing\UrlGeneratorInterface
+ */
+ protected $urlGenerator;
+
+ /**
+ * The state service.
+ *
+ * @var \Drupal\Core\State\StateInterface
+ */
+ protected $state;
+
+ /**
+ * Constructs the NodeTranslationExceptionSubscriber.
+ *
+ * @param \Drupal\Core\KeyValueStore\KeyValueFactoryInterface $key_value
+ * The key value factory.
+ * @param \Drupal\Core\Language\LanguageManagerInterface $language_manager
+ * The language manager.
+ * @param \Drupal\Core\Routing\UrlGeneratorInterface $url_generator
+ * The URL generator.
+ * @param \Drupal\Core\State\StateInterface $state
+ * The state service.
+ */
+ public function __construct(KeyValueFactoryInterface $key_value, LanguageManagerInterface $language_manager, UrlGeneratorInterface $url_generator, StateInterface $state) {
+ $this->keyValue = $key_value;
+ $this->languageManager = $language_manager;
+ $this->urlGenerator = $url_generator;
+ $this->state = $state;
+ }
+
+ /**
+ * Redirects not found node translations using the key value collection.
+ *
+ * @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event
+ * The exception event.
+ */
+ public function onException(GetResponseForExceptionEvent $event) {
+ $exception = $event->getException();
+
+ // If this is not a 404, we don't need to check for a redirection.
+ if (!($exception instanceof NotFoundHttpException)) {
+ return;
+ }
+
+ $previous_exception = $exception->getPrevious();
+ if ($previous_exception instanceof ParamNotConvertedException) {
+ $route_name = $previous_exception->getRouteName();
+ $parameters = $previous_exception->getRawParameters();
+ if ($route_name === 'entity.node.canonical' && isset($parameters['node'])) {
+ // If the node_translation_redirect state is not set, we don't need to check
+ // for a redirection.
+ if (!$this->state->get('node_translation_redirect')) {
+ return;
+ }
+ $old_nid = $parameters['node'];
+ $collection = $this->keyValue->get('node_translation_redirect');
+ if ($old_nid && $value = $collection->get($old_nid)) {
+ list($nid, $langcode) = $value;
+ $language = $this->languageManager->getLanguage($langcode);
+ $url = $this->urlGenerator->generateFromRoute('entity.node.canonical', ['node' => $nid], ['language' => $language]);
+ $response = new RedirectResponse($url, 301);
+ $event->setResponse($response);
+ }
+ }
+ }
+ }
+
+ /**
+ * {@inheritdoc}
+ */
+ public static function getSubscribedEvents() {
+ $events = [];
+
+ $events[KernelEvents::EXCEPTION] = ['onException'];
+
+ return $events;
+ }
+
+}
diff --git a/core/modules/node/src/EventSubscriber/NodeTranslationMigrateSubscriber.php b/core/modules/node/src/EventSubscriber/NodeTranslationMigrateSubscriber.php
new file mode 100644
index 0000000..5911f0e
--- /dev/null
+++ b/core/modules/node/src/EventSubscriber/NodeTranslationMigrateSubscriber.php
@@ -0,0 +1,113 @@
+<?php
+
+namespace Drupal\node\EventSubscriber;
+
+use Drupal\Core\KeyValueStore\KeyValueFactoryInterface;
+use Drupal\Core\State\StateInterface;
+use Drupal\migrate\Event\EventBase;
+use Drupal\migrate\Event\MigrateEvents;
+use Drupal\migrate\Event\MigrateImportEvent;
+use Drupal\migrate\Event\MigratePostRowSaveEvent;
+use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+
+/**
+ * Creates a key value collection for migrated node translation redirections.
+ *
+ * If we are migrating node translations from Drupal 6 or 7, these nodes will be
+ * combined with their source node. Since there still might be references to the
+ * URLs of these now consolidated nodes, this service saves the mapping between
+ * the old nids to the new ones to be able to redirect them to the right node in
+ * the right language.
+ *
+ * The mapping is stored in the "node_translation_redirect" key/value collection
+ * and the redirection is made by the NodeTranslationExceptionSubscriber class.
+ *
+ * @see \Drupal\node\NodeServiceProvider
+ * @see \Drupal\node\EventSubscriber\NodeTranslationExceptionSubscriber
+ */
+class NodeTranslationMigrateSubscriber implements EventSubscriberInterface {
+
+ /**
+ * The key value factory.
+ *
+ * @var \Drupal\Core\KeyValueStore\KeyValueFactoryInterface
+ */
+ protected $keyValue;
+
+ /**
+ * The state service.
+ *
+ * @var \Drupal\Core\State\StateInterface
+ */
+ protected $state;
+
+ /**
+ * Constructs the NodeTranslationMigrateSubscriber.
+ *
+ * @param \Drupal\Core\KeyValueStore\KeyValueFactoryInterface $key_value
+ * The key value factory.
+ * @param \Drupal\Core\State\StateInterface $state
+ * The state service.
+ */
+ public function __construct(KeyValueFactoryInterface $key_value, StateInterface $state) {
+ $this->keyValue = $key_value;
+ $this->state = $state;
+ }
+
+ /**
+ * Helper method to check if we are migrating translated nodes.
+ *
+ * @param \Drupal\migrate\Event\EventBase $event
+ * The migrate event.
+ *
+ * @return bool
+ * True if we are migrating translated nodes, false otherwise.
+ */
+ protected function isNodeTranslationsMigration(EventBase $event) {
+ $migration = $event->getMigration();
+ $source_configuration = $migration->getSourceConfiguration();
+ $destination_configuration = $migration->getDestinationConfiguration();
+ return !empty($source_configuration['translations']) && $destination_configuration['plugin'] === 'entity:node';
+ }
+
+ /**
+ * Maps the old nid to the new one in the key value collection.
+ *
+ * @param \Drupal\migrate\Event\MigratePostRowSaveEvent $event
+ * The migrate post row save event.
+ */
+ public function onPostRowSave(MigratePostRowSaveEvent $event) {
+ if ($this->isNodeTranslationsMigration($event)) {
+ $row = $event->getRow();
+ $source = $row->getSource();
+ $destination = $row->getDestination();
+ $collection = $this->keyValue->get('node_translation_redirect');
+ $collection->set($source['nid'], [$destination['nid'], $destination['langcode']]);
+ }
+ }
+
+ /**
+ * Set the node_translation_redirect state to enable the redirections.
+ *
+ * @param \Drupal\migrate\Event\MigrateImportEvent $event
+ * The migrate import event.
+ */
+ public function onPostImport(MigrateImportEvent $event) {
+ if ($this->isNodeTranslationsMigration($event)) {
+ $this->state->set('node_translation_redirect', TRUE);
+ }
+ }
+
+ /**
+ * {@inheritdoc}
+ */
+ public static function getSubscribedEvents() {
+ $events = [];
+
+ $events[MigrateEvents::POST_ROW_SAVE] = ['onPostRowSave'];
+ $events[MigrateEvents::POST_IMPORT] = ['onPostImport'];
+
+ return $events;
+ }
+
+}
diff --git a/core/modules/node/src/NodeServiceProvider.php b/core/modules/node/src/NodeServiceProvider.php
new file mode 100644
index 0000000..8d25817
--- /dev/null
+++ b/core/modules/node/src/NodeServiceProvider.php
@@ -0,0 +1,42 @@
+<?php
+
+namespace Drupal\node;
+
+use Drupal\Core\DependencyInjection\ContainerBuilder;
+use Drupal\Core\DependencyInjection\ServiceProviderInterface;
+use Drupal\node\EventSubscriber\NodeTranslationExceptionSubscriber;
+use Drupal\node\EventSubscriber\NodeTranslationMigrateSubscriber;
+use Symfony\Component\DependencyInjection\Reference;
+
+/**
+ * Registers services in the container.
+ */
+class NodeServiceProvider implements ServiceProviderInterface {
+
+ /**
+ * {@inheritdoc}
+ */
+ public function register(ContainerBuilder $container) {
+ // Register the node.node_translation_migrate service in the container if
+ // the migrate and language modules are enabled.
+ $modules = $container->getParameter('container.modules');
+ if (isset($modules['migrate']) && isset($modules['language'])) {
+ $container->register('node.node_translation_migrate', NodeTranslationMigrateSubscriber::class)
+ ->addTag('event_subscriber')
+ ->addArgument(new Reference('keyvalue'))
+ ->addArgument(new Reference('state'));
+ }
+
+ // Register the node.node_translation_exception service in the container if
+ // the language module is enabled.
+ if (isset($modules['language'])) {
+ $container->register('node.node_translation_exception', NodeTranslationExceptionSubscriber::class)
+ ->addTag('event_subscriber')
+ ->addArgument(new Reference('keyvalue'))
+ ->addArgument(new Reference('language_manager'))
+ ->addArgument(new Reference('url_generator'))
+ ->addArgument(new Reference('state'));
+ }
+ }
+
+}