Google Scholar is no longer accessible via <a href="http://www.scholar.google.com/scholar?q={TITLE}" target="_blank">Google Scholar</a>. It should be changed to the current url <a href="https://scholar.google.com/scholar?q={TITLE}" target="_blank">Google Scholar</a>.
This will require an update to the default system preferences SQL (variable OPACSearchForTitleIn), and probably a check during update for customized values.
Created attachment 70545 [details] [review] Bug 17682:Change URL for Google Scholar in OPACSearchForTitleIn 1)apply patch 2) go into administration --> search for OPACSearchForTitleIn 3) *click to edit* 4) look at the link and read the google scholar one. 5) check that the start link is https://scholar.google.com. 6) http://www.scholar.google is incorrect
Created attachment 70546 [details] [review] Bug 17682:Change URL for Google Scholar in OPACSearchForTitleIn 1)apply patch 2) go into administration --> search for OPACSearchForTitleIn 3) *click to edit* 4) look at the link and read the google scholar one. 5) check that the start link is https://scholar.google.com. 6) http://www.scholar.google is incorrect Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
I think the update needs to target only http://www.scholar.google.com Current code also updates bookfinder URL and breaks it (https seems to work, but requires keeping the 'www.') We can update both above links specifically so we don't hit any extra custom ones
Created attachment 70568 [details] [review] Bug 17682: Change URL for Google Scholar in OPACSearchForTitleIn Amended the sql statement so it only affects the http://www.scholar.google.com url Current status: Failed QA
This is ready for testing.
Created attachment 70738 [details] [review] Bug 17682:Change URL for Google Scholar in OPACSearchForTitleIn 1)apply patch 2) go into administration --> search for OPACSearchForTitleIn 3) *click to edit* 4) look at the link and read the google scholar one. 5) check that the start link is https://scholar.google.com. 6) http://www.scholar.google is incorrect Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 70739 [details] [review] Bug 17682: Change URL for Google Scholar in OPACSearchForTitleIn Amended the sql statement so it only affects the http://www.scholar.google.com url Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Small change, SO and QA in one step
Created attachment 70770 [details] [review] Bug 17682: Replace 2 occurrences in de-DE and nb-NO
Patches pushed to master for 18.05. Congratulations Charlotte for your first patches in!
Awesome work all! Pushed to Stable for 17.11.02
Pushed to 17.05.x for v17.05.09