Problem/Motivation

Tested the update function and found two bugs

* the bundle config needs to be saved so that the bundles are key and value, not just value.
* it creates empty patterns for top-level types that didn't have any.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 pathauto-update-2654930-3.patch3.27 KBBerdir
#2 pathauto-update-2654930-2.patch537 bytesBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Active » Needs review
FileSize
pathauto-update-2654930-2.patch537 bytes

Fixed the first for now. There might be a second case that needs to be updated for languages.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
FileSize
pathauto-update-2654930-3.patch3.27 KB

The other bundle config is already OK. Changed an isset() to a !empty() check and did a bunch of cleanup and added cache clears. This runs fine now.

  • Berdir committed 47926b3 on 8.x-1.x 
    Issue #2654930 by Berdir: Fixed bugs in pathauto_update_8100
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.