Tweaks

Author: Samu Juvonen

autoslug.module

@@ -24,10 +24,7 @@ function autoslug_entity_insert(EntityInterface $entity) {
 function autoslug_entity_update(EntityInterface $entity) {
   if (autoslug_is_entity_published($entity)) {
     try {
-      $aliases = Drupal::service('autoslug.alias_generator');
-      if (!$aliases->entityAliasExists($entity)) {
-        $aliases->createAlias($entity);
-      }
+      Drupal::service('autoslug.alias_generator')->createAllAliases($entity);
     } catch (RuntimeException $e) {
       // pass
     }

src/AliasGenerator.php

@@ -30,21 +30,29 @@ public function entityAliasExists(EntityInterface $entity) {
     return $this->fetchExistingAlias($entity) != NULL;
   }
 
-  public function createAlias(EntityInterface $entity) {
-    foreach ($this->sluggers as $slugger) {
-      if ($slugger->applies($entity)) {
-        $langcode = $entity->language()->getId();
-        $alias = $slugger->build($entity);
-        $alias = $this->ensureAliasUnique($alias, $langcode);
-        $cache_key = '/' . $entity->urlInfo()->getInternalPath();
-        $this->aliasStorage->save($cache_key, $alias, $langcode);
-        return TRUE;
+  public function createAlias(EntityInterface $entity, $force = FALSE) {
+    if (!$this->entityAliasExists($entity) || $force) {
+      foreach ($this->sluggers as $slugger) {
+        if ($slugger->applies($entity)) {
+          $langcode = $entity->language()->getId();
+          $alias = $slugger->build($entity);
+          $alias = $this->ensureAliasUnique($alias, $langcode);
+          $cache_key = '/' . $entity->urlInfo()->getInternalPath();
+          $this->aliasStorage->save($cache_key, $alias, $langcode);
+          return TRUE;
+        }
       }
     }
 
     return FALSE;
   }
 
+  public function createAllAliases(EntityInterface $entity) {
+    foreach ($entity->getTranslationLanguages() as $language) {
+      $this->createAlias($entity->getTranslation($language->getId()));
+    }
+  }
+
   public function ensureAliasUnique($base, $langcode) {
     $alias = $base;
     for ($i = 1; $this->aliasStorage->lookupPathSource($alias, $langcode); $i++) {

src/Controller/SlugController.php

@@ -49,10 +49,7 @@ public function aliases($type) {
     });
 
     foreach ($iterator as $entity) {
-      foreach ($entity->getTranslationLanguages() as $language) {
-        $translated = $entity->getTranslation($language->getId());
-        $this->aliases->createAlias($translated);
-      }
+      $this->aliases->createAllAliases($entity);
     }
 
     exit('finished ' . $type);