Hi Tibor,
Fixed in CVS. Thanks for the patch (note that I modified it, because you should not really replace c and c_name, only c_printable)
[...]
Great! Your fix was of course better. Testing it, I found we forgot another snippet, when an error message is displayed. I've also added a cosmetic <em></em> in the list of collection names, because I feel it is easier to read, but you decide. Thanks again, Ferran
Index: cds-invenio-0.91.0.20061116/modules/websearch/lib/search_engine.py =================================================================== --- cds-invenio-0.91.0.20061116.orig/modules/websearch/lib/search_engine.py 2006-12-01 10:52:10.000000000 +0100 +++ cds-invenio-0.91.0.20061116/modules/websearch/lib/search_engine.py 2006-12-01 10:59:55.000000000 +0100 @@ -1562,11 +1562,13 @@ t1 = os.times()[4] results = {} results_nbhits = 0 + colls_printable = [] for coll in colls: results[coll] = HitSet() results[coll]._set = Numeric.bitwise_and(hitset_in_any_collection._set, get_collection_reclist(coll)._set) results[coll].calculate_nbhits() results_nbhits += results[coll]._nbhits + colls_printable.append(get_coll_i18nname(coll, ln)) if results_nbhits == 0: # no hits found, try to search in Home: results_in_Home = HitSet() @@ -1577,7 +1579,7 @@ if of.startswith("h"): url = websearch_templates.build_search_url(req.argd, cc=cdsname, c=[]) print_warning(req, _("No match found in collection %(x_collection)s. Other public collections gave %(x_url_open)s%(x_nb_hits)d hits%(x_url_close)s.") %\ - {'x_collection': string.join(colls, ','), + {'x_collection': '<em>' + string.join(colls_printable, ',') + '</em>', 'x_url_open': '<a class="nearestterms" href="%s">' % (url), 'x_nb_hits': results_in_Home._nbhits, 'x_url_close': '</a>'})