Apparently the current version is not aware of multilingual sites.
Worse, if you have a multilingual site, not one alias will be created.
This is because of request_path() is also including the language prefix, current_path() is not.

Secondly, when an alias has been created, but you update the pattern afterwards, no new alias will be created.

I refactored page_manager_pathauto_ctools_render_alter() from scratch.
Patch attached.

CommentFileSizeAuthor
#5 page_manager_pathauto-not_lang_pattern_change_aware-2607436-5.patch2.98 KBdrupov
#3 language_pattern_updates_refactor-2607436-3.patch2.82 KBbetz
#2 language_pattern_updates_refactor-2607436.patch2.87 KBbetz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

betz created an issue. See original summary.

betz’s picture

betz CreditAttribution: betz as a volunteer commented
FileSize
language_pattern_updates_refactor-2607436.patch2.87 KB

patch

betz’s picture

betz CreditAttribution: betz as a volunteer commented
FileSize
language_pattern_updates_refactor-2607436-3.patch2.82 KB

Added a drupal_goto() when a path has been saved.

kopeboy’s picture

kopeboy CreditAttribution: kopeboy commented

Hello?

I need this to work well on a multilingual site, but I am no a developer (yet).

drupov’s picture

drupov CreditAttribution: drupov commented
FileSize
page_manager_pathauto-not_lang_pattern_change_aware-2607436-5.patch2.98 KB

@betz good work.

There is one small change needed though: if a pattern changes the path should be updated - currently it gets created anew. The patch adds that.

drupov’s picture

drupov CreditAttribution: drupov commented
Status: Active » Needs review
drupov’s picture

drupov CreditAttribution: drupov commented

@kopeboy if you need translations for your page manager paths, please check out https://www.drupal.org/node/2701187#comment-11045169

cimo75’s picture

cimo75 CreditAttribution: cimo75 commented

Thanks you working fine here.

cimo75’s picture

cimo75 CreditAttribution: cimo75 commented

I am getting a
Drush command terminated abnormally due to an unrecoverable error.
error now, it may related to the goto as stated in
https://www.drupal.org/node/1140842
it should be wrapped in drupal_is_cli.
S.