After http://drupal.org/node/154517 is committed I will create a patch that extends the current path alias UI to allow users with proper permissions to delete multiple aliases at once.
This was already done in http://drupal.org/files/issues/path_lang_1.patch but the original issue should be committed first, befor adding new features.
Screenshots:
http://www.flickr.com/photo_zoom.gne?id=1708807310&size=o
http://www.flickr.com/photo_zoom.gne?id=1707954833&size=o
http://www.flickr.com/photo_zoom.gne?id=1707955611&size=o
Comment | File | Size | Author |
---|---|---|---|
#13 | path_ui_alias_table.png | 6.22 KB | stBorchert |
#10 | path_ui-186571-8[1].patch | 10.69 KB | Robin Monks |
#8 | path_ui-186571-8.patch | 10.3 KB | floretan |
#7 | path_ui.patch | 10.56 KB | stBorchert |
#5 | snapshot2.png | 15.09 KB | floretan |
Comments
Comment #1
catchsubscribing.
Comment #2
stBorchertUh, forgot about this issue.
Here's a patch against 6.1. Perhaps I should do this for 7.x?
Comment #3
catchYeah this'll need to be against 7.x
Comment #4
stBorchert"Ported" the patch to 7.x.
Comment #5
floretan CreditAttribution: floretan commentedThe patch seems to work but the HTML output needs to be fixed (see screenshot).
Comment #6
stBorchertYou have to clear your cache to load the new theme function for the overview form.
Go to [admin/settings/performance] and click on "Clear cached data" on the bottom of the page.
Comment #7
stBorchertNew patch:
- hide "Delete selected" button if no aliases are found
Comment #8
floretan CreditAttribution: floretan commentedUpdate patch for singular/plural in notification message ("The URL alias has been deleted." / "The URL aliases have been deleted").
Code looks good and passes coder module validation.
Comment #9
Robin Monks CreditAttribution: Robin Monks commentedNo longer applies to HEAD.
Robin
Comment #10
Robin Monks CreditAttribution: Robin Monks commentedFree untested re-roll.
Robin
Comment #11
Robin Monks CreditAttribution: Robin Monks commentedYeah, code's broken and needs to be updated. flobruit, interested? :)
Robin
Comment #12
drawk CreditAttribution: drawk commentedsubscribing.
Comment #13
stBorchertPatch applies but there is one major bug: the path alias table is displayed with empty cells (see screenshot).
In function
theme_path_admin_overview_form()
(around #310)drupal_render()
returns an empty string for$form['paths'][$key]
,$form['dst'][$key]
and$form['src'][$key]
so there seems to be something wrong with the form array (I searched but couldn't find any eye-catching error).Comment #14
EllECTRONC CreditAttribution: EllECTRONC commentedsubscribing...
need to 6.x and 7.x
Comment #15
Dave ReidToo late for 7.x
Comment #16
yoroy CreditAttribution: yoroy commentedtagging
Comment #29
smustgrave CreditAttribution: smustgrave at Mobomo commentedThis seems like a feature that could still be useful to many.
Will need an issue summary update and appropriate test coverage.