Code

Opened 3 years ago

Closed 3 years ago

#16431 closed New feature (worksforme)

Redirects ignore query strings should be made optional.

Reported by: cnobile Owned by: nobody
Component: contrib.redirects Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Here at Scripps Publications we often get complaints from our sites that redirects do not pass the query string on to the redirected URI. I have looked at this code and except for ROM changes this code has remained the same since the 0.90 days. It seems that the Redirect model could have a Yes/No button to either tack on the query string or not. This would be a DB change and a code change, but not very difficult from my investigations.

I don't have a patch, but could probably work one up if you feel it is necessary.

Attachments (0)

Change History (1)

comment:1 Changed 3 years ago by aaugustin

  • Component changed from Core (Other) to contrib.redirects
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to worksforme
  • Status changed from new to closed

Since the query string is included in the original URL (see #14153) when searching for a redirect, can't you just add the query string to the target URL?

Please reopen if I missed something.

#9966 is related.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.