SMS 'Return to Record' link breaks after texting if user has to log in

Bug #1396263 reported by Terran McCanna
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.8
Fix Released
Undecided
Unassigned

Bug Description

If user is already logged into the OPAC when texting a call number, the 'Return to Record' link functions properly.

Example of good 'Return to Record' link:

https://next.gapines.org/eg/opac/record/2138068?bool=and;bool=and;bool=and;qtype=title;qtype=title;qtype=author;contains=starts;contains=contains;contains=contains;query=joy%20of%20cooking;query=;query=;_adv=1;locg=1;pubdate=is;expand=marchtml

However, if user clicks the Text link when not logged in and is prompted to log in first, it still sends the text message properly, but the 'Return to Record' link no longer works and returns the user to his/her account summary screen instead of the record.

Example of bad 'Return to Record' link:

https://next.gapines.org/eg/opac/login?redirect_to=%2Feg%2Fopac%2Fsms_cn%3Fbool%3Dand%3Bbool%3Dand%3Bbool%3Dand%3Bqtype%3Dtitle%3Bqtype%3Dtitle%3Bqtype%3Dauthor%3Bcontains%3Dstarts%3Bcontains%3Dcontains%3Bcontains%3Dcontains%3Bquery%3Djoy%2520of%2520cooking%3Bquery%3D%3Bquery%3D%3B_adv%3D1%3Blocg%3D1%3Bpubdate%3Dis%3Bcopy_id%3D7575706

This issue appears to be another aspect of:
https://bugs.launchpad.net/evergreen/+bug/1175730

Evergreen version: 2.7.1

Kathy Lussier (klussier)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: pullrequest
tags: removed: sms
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Kathy Lussier (klussier)
Changed in evergreen:
assignee: nobody → Kathy Lussier (klussier)
Revision history for this message
Kathy Lussier (klussier) wrote :
Changed in evergreen:
assignee: Kathy Lussier (klussier) → nobody
tags: added: signedoff
Revision history for this message
Ben Shum (bshum) wrote :

Merged to master and rel_2_8, but could not backport cleanly to rel_2_7...

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
no longer affects: evergreen/2.7
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.