Closed (fixed)
Project:
Path redirect
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
27 Jan 2009 at 17:38 UTC
Updated:
3 Jan 2014 at 00:07 UTC
Jump to comment: Most recent
Comments
Comment #1
mrfelton commentedI'd like to add a reference to this ticket I just created here too:
#429310: Filter option like in admin/build/path
Comment #2
gkallenbergI'd like to add that the text under the From text field be a bit more generic. On first reading, I took away that one needed an internal Drupal path to exist in order to provide a redirect. Since we're migrating a site from static/dynamic pages to the Drupal system, I was unclear whether we could use this module for all redirects that now exist on our current site.
"Enter an interal Drupal path or path alias to redirect (e.g. node/123 or taxonomy/term/123). Fragment anchors (e.g. #anchor) are not allowed."
This line could be changed to "Enter any path in your current path structure, an internal Drupal path or path alias...(e.g. /index.html, node/123..."
I had to have a colleague explain that you could input anything as a From path which could be redirected to a Drupal path, path alias, etc. I guess it seems obvious, but I was a bit slow on the uptake thinking we had to add all of our rules in the .htaccess file.
Comment #3
guillaumeduveauMeta-issuing... Every issue mentionned including the last one is a duplicate
Comment #4
dave reidPlease don't remove the issue from my tracker my making it a duplicate. #365242: Add a filter to admin/build/redirect is still active.
Comment #5
guillaumeduveauVery sorry, didn't realize you were the maintainer !
Comment #6
dave reidAnother change that needs to be made is that I want very much to get rid of the three separate fields for 'redirect to' when there only needs to be one textfield. This will also help simplify the code a little bit. Marking this issue as a release blocker as well.
Comment #7
dave reidWhat's been finished:
- #168877: Path-redirect as an option on the node form (add or edit)
- #365242: Add a filter to admin/build/redirect
- #396198: Bulk Delete Redirects
- Combining the three 'to' fields into one field and split the values on validation.
Todo:
- #615008: "Forced" path redirect - override the existing path or alias
Comment #8
dave reidOnly one remaining issue with 615008, so marking as fixed.