Closed (fixed)
Project:
Apache Solr Search
Version:
6.x-3.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
15 Aug 2012 at 16:03 UTC
Updated:
2 Oct 2012 at 01:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
cpliakas commentedFixing quotes in title.
Comment #2
pwolanin commentedI thought at some point we did have an option to show extra info to admins - maybe we should see if that got broken, or needs to be expanded.
Comment #3
cpliakas commented@pwolanin, I definitely could have missed something, but the only thing I can see is the "Extra help messages for administrators" options which only adds notices to a page whenever Drupal changed content that needs reindexing.
Comment #4
pwolanin commentedOk, I was thinking of that option, but not remembering what it does - we could maybe use that as the flag?
Comment #5
cpliakas commentedYes, that seems very reasonable to me. As a side note, I still think we should reconsider the wording of the error message as well. I understand it can be overridden in settings.php, but since it is more of an end user-facing message, I'm not sure it is the best option to mention things like "Apache Solr" and "Server". Maybe saying something even more generic like "search is currently unavailable" or something similar, that would be OK if we coupled it with the detailed information using the technique you mentioned. If we agree that this is the right path forward and can also agree on a message, I don't mind taking a crack at a patch.
Comment #6
cpliakas commentedThis message is the bane of my existence. Assigning to myself.
Comment #7
cpliakas commentedProposed message change attached. The words "Apache Solr" probably shouldn't be displayed to end users.
Comment #8
pwolanin commentedLooks fine to me.
Comment #9
nick_vhLooking great, committed to 7.x-1.x with giving all credit to cpliakas! :)
Comment #10
cpliakas commentedExcellent. So when there is an inevitable spelling error, we know who did it :-P.
Comment #11
pwolanin commentedComment #12
pwolanin commentedcommitted