Hi Tibor,

I've seen that you still haven't applied my tiny patch below, and 0.92
is approaching.  Could you consider it, please?

Thanks,

Ferran
Display i18n collection names when an error message is shown.  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.

Index: cds-invenio/modules/websearch/lib/search_engine.py
===================================================================
--- cds-invenio.orig/modules/websearch/lib/search_engine.py	2006-12-15 08:55:25.875425044 +0100
+++ cds-invenio/modules/websearch/lib/search_engine.py	2006-12-15 08:55:42.098305099 +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>'})

Reply via email to