OPACBaseurl is still being used in opac-results.tt and opac-opensearch.tt It also shows up in many Poodle files. Note: opac-results.tt doesn't currently get passed OPACBaseURL anyway, but it's still probably a good idea to have it spelled correctly.
(In reply to comment #0) > OPACBaseurl is still being used in opac-results.tt and opac-opensearch.tt > > It also shows up in many Poodle files. > > Note: opac-results.tt doesn't currently get passed OPACBaseURL anyway, but > it's still probably a good idea to have it spelled correctly. Pootle :P
Created attachment 12486 [details] [review] Bug 8766 - OPACBaseURL still called as OPACBaseurl in many files This patch fixes incorrect capitalization of the OPACBaseURL variable in three templates. The fix can be confirmed in opac-results.tt by inspecting the link attached to the RSS icon on the OPAC's search results screen. The same fix is included in opac-opensearch.tt and search/results.tt but I'm not sure how you confirm that this change fixes anything, if it does.
Created attachment 12579 [details] [review] [SIGNED-OFF] Bug 8766 - OPACBaseURL still called as OPACBaseurl in many files This patch fixes incorrect capitalization of the OPACBaseURL variable in three templates. The fix can be confirmed in opac-results.tt by inspecting the link attached to the RSS icon on the OPAC's search results screen. The same fix is included in opac-opensearch.tt and search/results.tt but I'm not sure how you confirm that this change fixes anything, if it does. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
OPACBaseURL is set by get_template_and_user so having a script in the opac directory set it as well is superfluous, but not harmful.
QA comment: tiny patch, just change capitalization
Patch pushed to master
Patch doesn't apply on 3.8.x, please rebase and submit if it is needed.
(In reply to comment #7) > Patch doesn't apply on 3.8.x, please rebase and submit if it is needed. It seems to me that the patch is getting hung up on opac-tmpl\prog\en\modules\search\results.tt, which doesn't exist in 3.8.x as it is a file specific to BibLibre's work with Solr in master. Wouldn't this be a case where the commit needs to be cherrypicked to 3.8.x from master and the bit about search\results.tt removed while the rest of the patch remains relevant?
(In reply to comment #8) > (In reply to comment #7) > > Patch doesn't apply on 3.8.x, please rebase and submit if it is needed. > > It seems to me that the patch is getting hung up on > opac-tmpl\prog\en\modules\search\results.tt, which doesn't exist in 3.8.x as > it is a file specific to BibLibre's work with Solr in master. > > Wouldn't this be a case where the commit needs to be cherrypicked to 3.8.x > from master and the bit about search\results.tt removed while the rest of > the patch remains relevant? Exactly, but what I want is the patch author to do that, so and attach a 3.8.x patch, so I don't muck it up :)
Still waiting on a 3.8.x patch for this
(In reply to Kyle M Hall from comment #4) > OPACBaseURL is set by get_template_and_user so having a script in the opac > directory set it as well is superfluous, but not harmful. At the time, I don't think I 100% understood what you meant by this, but I do now. I suppose it's not harmful so long as it doesn't tamper with the https detection happening in Auth.pm. That said, at the moment, I think the OPACBaseURL set by get_template_and_user is using an environmental "SERVER_NAME" variable (which appears to be the domain name that a user puts into their browser, rather than the ServerName in the Apache conf), rather than the OPACBaseURL system preference. While this might not be an issue in many cases, it appears to cause issues for Koha instances sitting behind proxy servers... I'm opening bug 11575 to deal with that...