There is supposed to be a warning above the "Delete aliases now!" button -- this warning does not appear in D7 or D8. The warning reads:

Note: there is no confirmation. Be sure of your action before clicking the "Delete aliases now!" button.
You may want to make a backup of the database and/or the url_alias table prior to using this feature.

The warning message is already present in the code, it’s just not calling the form api correctly to display it.

Some background here:
https://www.drupal.org/project/pathauto/issues/256298#comment-3253406

CommentFileSizeAuthor
#9 restore_delete_aliases_now_warning-2967617-9.patch1.4 KBidebr
#6 d8_drupal_modules_pathauto.png7.89 KBakshay kashyap
#4 afterpatch.png67.2 KBbandanasharma
#3 restore_delete_aliases_now_warning-d7-2967617-3.patch952 bytesdaniel.nitsche
#2 restore_delete_aliases_now_warning-2967617-2.patch1.01 KBdaniel.nitsche

Comments

daniel.nitsche created an issue. See original summary.

daniel.nitsche’s picture

daniel.nitsche commented
StatusFileSize
new restore_delete_aliases_now_warning-2967617-2.patch1.01 KB

This patch should restore the warning in D8.

daniel.nitsche’s picture

daniel.nitsche commented
StatusFileSize
new restore_delete_aliases_now_warning-d7-2967617-3.patch952 bytes

This patch should restore the warning in D7.

bandanasharma’s picture

bandanasharma commented
Status: Active » Needs review
StatusFileSize
new afterpatch.png67.2 KB

#2 patch for drupal 8 is apply cleanly. UI changes reflect on admin/config/search/path/delete_bulk page. Attached the after patch screen shot.

daniel.nitsche’s picture

daniel.nitsche commented
Issue summary: View changes
akshay kashyap’s picture

akshay kashyap commented
StatusFileSize
new d8_drupal_modules_pathauto.png7.89 KB

#2 patch applied successfully and working fine for 8.x-1.x-dev version.

akshay kashyap’s picture

akshay kashyap commented
Status: Needs review » Reviewed & tested by the community
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Needs work

Won't apply anymore.

idebr’s picture

idebr commented
Status: Needs work » Needs review
StatusFileSize
new restore_delete_aliases_now_warning-2967617-9.patch1.4 KB

Reroll against 8.x-1.x after #3029311: Use short array syntax was committed.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Thanks.

  • Berdir committed 7325915 on 8.x-1.x authored by idebr 
    Issue #2967617 by daniel.nitsche, idebr, Akshay kashyap, bandanasharma:...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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