I noticed that in the current dev and beta that the pathauto cleanup option was not working. I have provided a patch that seems to work without adverse effects in my testing. Ultimately it seems that the array being passed to filefield_paths_process_string() should be $settings['field_aliases']['options'].

CommentFileSizeAuthor
#2 file_aliases-file-aliases-options-2613660-1-D7.patch743 bytesiamjdcollins

Comments

iamjdcollins created an issue. See original summary.

iamjdcollins’s picture

iamjdcollins commented
StatusFileSize
new file_aliases-file-aliases-options-2613660-1-D7.patch743 bytes
iamjdcollins’s picture

iamjdcollins commented
Status: Active » Needs review
iamjdcollins’s picture

iamjdcollins commented
Assigned: iamjdcollins » Unassigned
deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Needs review » Needs work

Patch doesn't resolve the issue completely due to changes in File (Field) Paths. Test added for Pathauto which currently fails both with and without this patch.

Status: Needs work » Needs review

Deciphered queued 2: file_aliases-file-aliases-options-2613660-1-D7.patch for re-testing.

  • Deciphered committed 38929a0 on 7.x-1.x 
    #2613660 by iamjdcollins, Deciphered: Fixed issue with Pathauto...
deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Needs review » Fixed

Fixed and committed. Thanks for the patch.

Status: Fixed » Needs work

The last submitted patch, 2: file_aliases-file-aliases-options-2613660-1-D7.patch, failed testing.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Needs work » Fixed

You know nothing Drupal.org CI.

Status: Fixed » Closed (fixed)

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