https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Owen Leonard changed:
What|Removed |Added
Status|Pushed to Stable|RESOLVED
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Fridolin SOMERS changed:
What|Removed |Added
Blocks||22524
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Ere Maijala changed:
What|Removed |Added
See Also||https://bugs.koha-community
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #104 from Ere Maijala ---
Claire, the publication date mapping will be fixed in bug 22524.
--
You are receiving this mail because:
You are watching all bug changes.
___
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #103 from Katrin Fischer ---
(In reply to claire.hernan...@biblibre.com from comment #102)
> I see this line:
> $dbh->do( "UPDATE `search_field` SET `name` = 'date-of-publication', `label`
> = 'date-of-publication'
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
claire.hernan...@biblibre.com changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Fridolin SOMERS changed:
What|Removed |Added
Blocks||20390
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Jonathan Druart changed:
What|Removed |Added
Blocks||22495
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Lucas Gass changed:
What|Removed |Added
CC||lu...@bywatersolutions.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #100 from Ere Maijala ---
Looks to me like the original confusion might stem from the fact that
queryparser.yaml defines su-geo as an alias for subject. I think that might be
a mistake in queryparser.yaml. It was added
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #99 from Ere Maijala ---
I filed bug 22495 for the su-geo fix.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Ere Maijala changed:
What|Removed |Added
See Also||https://bugs.koha-community
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #98 from David Gustafsson ---
I had a look at the script again and now remembered it does not migrate zebra
facets declared in biblio-koha-indexdefs.xml, that might be the reason it got
dropped. The other facet fields
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #97 from David Gustafsson ---
That might have been a mistake. The mappings where generated from this script
from Zebra mapping definitions:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Josef Moravec changed:
What|Removed |Added
CC||josef.mora...@gmail.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Status|Pushed to Master|Pushed to Stable
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Status|Passed QA |Pushed to Master
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Text to go in the||This patchset makes some
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #93 from Nick Clemens ---
(In reply to Nick Clemens from comment #92)
Talked with Ere, he verified these make sense
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #92 from Nick Clemens ---
(In reply to David Gustafsson from comment #91)
> Created attachment 85247 [details] [review]
> Bug 19575: Don't lower case fields in db and use update script instead of
> raw SQL
Hi David,
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #91 from David Gustafsson ---
Created attachment 85247
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85247=edit
Bug 19575: Don't lower case fields in db and use update script instead of raw
SQL
--
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #90 from David Gustafsson ---
Yes, the lowercase update statement is pretty pointless since the mixed case
field names will be restored on reset. Field names are lower-cased in
_foreach_mapping, so the case of field
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #89 from Nick Clemens ---
I looked at this today, but the update statement is a little confusing.
Should we update a library's mappings in updates? or simply require that they
reset mappings when upgrading?
I am okay
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #88 from Nick Clemens ---
Created attachment 85178
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85178=edit
Bug 19575: (RM follow-up) Fix mapping typo
Found with QA tools, simple fix
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #87 from Nick Clemens ---
*** Bug 9 has been marked as a duplicate of this bug. ***
--
You are receiving this mail because:
You are watching all bug changes.
___
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Attachment #85105|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Status|Signed Off |Passed QA
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Attachment #85029|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Attachment #85062|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Attachment #85124|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #82 from David Gustafsson ---
Ok! Since it was some time ago I don't really remember how to parse the Zebra
field definitions anymore, but the authority-number entry looked a little bit
strange and I might have made some
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #81 from David Gustafsson ---
Created attachment 85124
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85124=edit
Bug 19575: Keep authority-number as alias and fix query
Keep authority-number as alias
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #80 from Nick Clemens ---
(In reply to David Gustafsson from comment #75)
> (In reply to Nick Clemens from comment #69)
> > Extra note: I found that indexes like 'se' and 'an' were not searchable - no
> > results were
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #79 from Nick Clemens ---
Created attachment 85105
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85105=edit
Bug 19575: Fix an search field
--
You are receiving this mail because:
You are watching
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #78 from David Gustafsson ---
(And in many other places). Should perhaps not matter though if "an" never
contains whitespace.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #77 from David Gustafsson ---
Could the cause be that the value of an: is unquoted in clause where
UseAuthoritiesForTracings is set in the excerpt below
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #76 from David Gustafsson ---
Oops, bug 22325 I mean, perhaps you linked to the wrong bug number in the
previous comment?
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #75 from David Gustafsson ---
(In reply to Nick Clemens from comment #69)
> Extra note: I found that indexes like 'se' and 'an' were not searchable - no
> results were returned from faceting or searchign like
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #74 from Nick Clemens ---
(In reply to David Gustafsson from comment #71)
> (In reply to Nick Clemens from comment #67)
> > Created attachment 85030 [details] [review] [review]
> > Bug 19575: Rebase and fix tests
> >
>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #73 from Nick Clemens ---
(In reply to Katrin Fischer from comment #70)
> (In reply to Nick Clemens from comment #69)
> > Extra note: I found that indexes like 'se' and 'an' were not searchable - no
> > results were
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Attachment #85030|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #71 from David Gustafsson ---
(In reply to Nick Clemens from comment #67)
> Created attachment 85030 [details] [review]
> Bug 19575: Rebase and fix tests
>
> This patch lower cases the sort by fields to normalize
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #70 from Katrin Fischer ---
(In reply to Nick Clemens from comment #69)
> Extra note: I found that indexes like 'se' and 'an' were not searchable - no
> results were returned from faceting or searchign like "se:pengiun"
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #69 from Nick Clemens ---
Extra note: I found that indexes like 'se' and 'an' were not searchable - no
results were returned from faceting or searchign like "se:pengiun" changing to
title-series (or even 'ser' in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #68 from Nick Clemens ---
Rebased and updated, I think in spirit of original set - set to Signed off,
would like another QA run and Ere's okay
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Status|Failed QA |Signed Off
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #67 from Nick Clemens ---
Created attachment 85030
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85030=edit
Bug 19575: Rebase and fix tests
This patch lower cases the sort by fields to normalize
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Attachment #83710|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #65 from David Gustafsson ---
Unfortunately I have have been a little bit too busy lately to have a look at
this, but hope will have the time to do so some time next week.
--
You are receiving this mail because:
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Katrin Fischer changed:
What|Removed |Added
See Also|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #64 from Ere Maijala ---
David, will you be able to address the test issues?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Status|Passed QA |Failed QA
--- Comment #63
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
QA Contact||ere.maij...@helsinki.fi
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Status|Signed Off |Passed QA
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
Attachment #83704|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Ere Maijala changed:
What|Removed |Added
CC||frank.han...@ub.lu.se
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #59 from Ere Maijala ---
Awesome, thanks David. This was actually on my list of things to work on when
time permits, and it had been bugging me.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Ere Maijala changed:
What|Removed |Added
Attachment #82794|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
CC||ere.maij...@helsinki.fi
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Martin Renvoize changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #81304|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #56 from David Gustafsson ---
Rebase against master.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Katrin Fischer changed:
What|Removed |Added
CC||katrin.fisc...@bsz-bw.de
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #81303|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #81299|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #81297|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #75115|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #49 from David Gustafsson ---
Will fix this. Test plan is a litte bit tricky since this corrects a lot of
small stuff. But will try to backtrack to list most of what is fixed.
--
You are receiving this mail because:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Julian Maurice changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #72673|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #47 from David Gustafsson ---
Rebased against master.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Jonathan Druart changed:
What|Removed |Added
Summary|ES: Use
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Status|Needs Signoff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Nick Clemens changed:
What|Removed |Added
Attachment #71966|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #44 from David Gustafsson ---
Removed incorrect alias/field name conversions (ctype => "content-type"), and
some aliases with identical source and target values.
--
You are receiving this mail
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #71112|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #42 from David Gustafsson ---
*('an' to 'authority-number')
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #41 from David Gustafsson ---
Fixed hard coded field name in Koha/SearchEngine/ElasticSearch/Search.pm ('au'
to 'authority-number')
--
You are receiving this mail because:
You are watching all bug
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #71090|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #39 from David Gustafsson ---
Found one more hard coded field value in authorities search that needed
lower-casing.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #70558|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #37 from David Gustafsson ---
Found one (hopefully) last place that needed case-adjustment. New diff:
https://github.com/ub-digit/Koha/commit/a6fda51b59c65c2459d460fd374377b56b5f10cc
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #70495|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #35 from David Gustafsson ---
Here is a github-diff that might be easier on the eyes:
https://github.com/ub-digit/Koha/commit/8be850d2cdc93e5d121afa19c5eed1e1cc683f1c
--
You are receiving this mail
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #70494|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #34 from David Gustafsson ---
Found some more places where hard coded fields needed to be changed to
lower-case.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #32 from David Gustafsson ---
Case insensitive field names in searches is now fixed. A few changes I made
could use some extra motivation/explanation:
# sub _convert_index_strings_freeform() (line
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #69507|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #30 from David Gustafsson ---
Just a caveat, just discovered that some field names hard coded in
xsl-templates have names that differer in case from mappings.yaml.
(control-number for example). Zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #29 from David Gustafsson ---
And, now I saw there was a dropdown, but below the comment area.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Status|Patch doesn't apply
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #28 from David Gustafsson ---
Aha, I looked for a status drop down but manage to miss it since it wasn't a
drop down. :) Will fix.
--
You are receiving this mail because:
You are watching all bug
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #27 from Jonathan Druart
---
(In reply to David Gustafsson from comment #26)
> Yes should be ready for sign off, have discovered no more issues since the
> most recent patch.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #26 from David Gustafsson ---
Yes should be ready for sign off, have discovered no more issues since the most
recent patch.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #25 from Jonathan Druart
---
(In reply to David Gustafsson from comment #22)
> Think git auto-merged some stuff incorrectly, but this should now be sorted
> out.
So, ready to be
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #24 from David Gustafsson ---
Field name 'Date/time-last-modified' does not work with elastic since "/" will
create subfields, koha will also not tokanize this correctly, changed this
(back) to
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #69480|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #22 from David Gustafsson ---
Think git auto-merged some stuff incorrectly, but this should now be sorted
out.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
David Gustafsson changed:
What|Removed |Added
Attachment #69479|0 |1
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #20 from David Gustafsson ---
Reviewing the commit-diff once again, I think I made an error in the merge.
Will upload again.
--
You are receiving this mail because:
You are watching all bug changes.
1 - 100 of 121 matches
Mail list logo