I built a module to set redirects from a node and to list node redirects, as I couldn’t see this in this module previously. The listing of node redirects functionality of my module https://www.drupal.org/project/bundle_redirect has now been combined into redirect module if so is this making my module obsolete? Please confirm so I can take steps on my module.

Also my module has an additional feature (adding redirect from the node itself) like D7 feature.

The screen shots of my module as well as the redirect module (with URL redirects) are attached herewith.

CommentFileSizeAuthor
#5 Screen Shot 2018-01-09 at 13.17.23.png55.67 KBchandraraj
bundle_redirect.png30.92 KBchandraraj
redirect_and_bundle_redirect.png126.61 KBchandraraj

Comments

chandraraj created an issue. See original summary.

idebr’s picture

idebr commented

Hi chandraraj,

I can confirm the patch had a similar goal as the contrib module. The issue that added the vertical tab is available at #2702353: Vertical tab for listing redirects to content/entities missing

chandraraj’s picture

chandraraj commented

Hi idebr,

Thanks for your response. The other additional features of my module are:

* Ability to add a redirect from the node itself (like D7)
* Redirect links are clickable

Please let me know if you want a patch created for this, and I will make my project obsolete.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Adding a redirect seems useful, but I'm not sure about clickable redirect links. Why would you need to do that? It jut adds more risk that users click on it and lose data from the current node form.

chandraraj’s picture

chandraraj commented
StatusFileSize
new Screen Shot 2018-01-09 at 13.17.23.png55.67 KB

Also if the source node has a query string. Then redirect module is not listing it. For e.g. the redirect is 'aaf-18' to the source node which has the path

/node/5?utm_medium=print&utm_source=prospectus&utm_campaign=ug%2Bprospectus%2Bbrand&utm_content=2018

Please see the screen grab of the above example in both our modules.

chandraraj’s picture

chandraraj commented

Hi Berdir,

The D7 version of redirect module provides the same clickable facility. So it's included in my module and also it's the client requirement when their project is migrated from D7 to D8.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Well, just because something existed in 7.x is IMHO not reason enough, I don't see why it would be necessary to display a link to a page that just redirects back to the current node? Maybe it was just to have the full URL accessible, IIRC we implemented that as a title attribute that shows on hover.

idebr’s picture

idebr commented
Status: Active » Closed (duplicate)

I suggest we discuss further improvements to the node form element in #2931770: Improve usability of redirect operation links in node form