https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31652

Katrin Fischer <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Passed QA                   |Failed QA

--- Comment #23 from Katrin Fischer <[email protected]> ---
Please check:

1) Are the changes to name / label in kohastructure intended?

   `id` int(11) NOT NULL AUTO_INCREMENT,
-  `name` varchar(255) NOT NULL COMMENT 'the name of the field as it will be
stored in the search engine',
-  `label` varchar(255) NOT NULL COMMENT 'the human readable name of the field,
for display',
-  `type`
enum('','string','date','number','boolean','sum','isbn','stdno','year','callnumber')
NOT NULL COMMENT 'what type of data this holds, relevant when storing it in the
search engine',
+  `name` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL COMMENT 'the name of
the field as it will be stored in the search engine',
+  `label` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL COMMENT 'the human
readable name of the field, for display',
+  `type`
enum('','string','date','number','boolean','sum','isbn','stdno','year','callnumber','geo_point')
COLLATE utf8mb4_unicode_ci NOT NULL COMMENT 'what type of data this holds,
relevant when storing it in the search engine',

2) System preference GeoSearchEnabled

I believe this should be removed and due to rewrites? There is no other mention
of this pref.

+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/searching.pref
@@ -99,6 +99,13 @@ Searching:
             - "List of search fields (separated by | or ,) that should not be
autotruncated by Elasticsearch even if <a
href='/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=QueryAutoTruncate'>QueryAutoTruncate</a>
is set to 'Yes':"
             - pref: ESPreventAutoTruncate
               class: long
+        -
+            - pref: GeoSearchEnabled
+              type: boolean
+              choices:
+                  1: Enable
+                  0: Disable
+            - 'GeoSearch via Elasticsearch'
     Search form:

3) Capitalization

Usually I just fix these sneakily, but so we don't forget with the other
points:

+                                              <option value="geo_point"
selected="selected">Geo Point</option>

We only capitalize the first word, so this should read: Geo Point (I know Call
Number in the same spot is wrong as well, but no need to add more)


Please fix!

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to