[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-08-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #21 from Fridolin Somers  ---
(In reply to David Cook from comment #3)
> In Master:
> 1) Turn off QueryAutoTruncate
> 2) Go to
> http://localhost:8081/cgi-bin/koha/catalogue/search.pl?q=GEN=location
> 3) Note "No results found"
> 
> The problem is that QueryWeightFields adds a "phr" qualifier which is a
> problem since "location" doesn't have a phrase index defined...:
> 
> (rk=( location,ext,r1=" GEN " or location,phr,r3=" GEN " or
> location,wrdl,r6=" GEN " or location,wrdl,fuzzy,r8=" GEN " or
> location,wrdl,rt,r9=""gen"? "))

We have seen other problems like that.
We should add phrase search to all search fields to fix once and for all.

But for this cas, it disables autotruncation of location code, so its a good
fix.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-05-05 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Lucas Gass  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|Pushed to stable|RESOLVED
 CC||lu...@bywatersolutions.com

--- Comment #20 from Lucas Gass  ---
Doesn't apply cleanly to 22.05.x, no backport.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-04-16 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #19 from Jacob O'Mara  ---
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-04-16 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Jacob O'Mara  changed:

   What|Removed |Added

 Status|Pushed to master|Pushed to stable
 Version(s)|23.05.00|23.05.00,22.11.05
released in||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-04-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #18 from Tomás Cohen Arazi  ---
Pushed to master for 23.05.

Nice work everyone, thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-04-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Version(s)||23.05.00
released in||
 Status|Passed QA   |Pushed to master

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Nick Clemens  changed:

   What|Removed |Added

 Attachment #148856|0   |1
is obsolete||

--- Comment #17 from Nick Clemens  ---
Created attachment 148916
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148916=edit
Bug 31841: Use location,wrdl when searching Shelving location

The "location" index only has a word index defined, so we need
to specify the wrdl qualifier when searching it via the staff
interface dropdown menu

Test plan:
1) Change QueryAutoTruncate to "only if * is added"
2) Change IntranetCatalogSearchPulldown to "Show"
3) Search for "Shelving location" of "GEN" in catalogue
search bar
4) Note no results returned

5) Apply patch
6) koha-plack --restart kohadev

7) Repeat Step #3
8) Note that results appear for 'location,wrdl: GEN'

Signed-off-by: Magnus Enger 
Works as advertised.
Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Nick Clemens  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Magnus Enger  changed:

   What|Removed |Added

 Attachment #148848|0   |1
is obsolete||

--- Comment #16 from Magnus Enger  ---
Created attachment 148856
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148856=edit
Bug 31841: Use location,wrdl when searching Shelving location

The "location" index only has a word index defined, so we need
to specify the wrdl qualifier when searching it via the staff
interface dropdown menu

Test plan:
1) Change QueryAutoTruncate to "only if * is added"
2) Change IntranetCatalogSearchPulldown to "Show"
3) Search for "Shelving location" of "GEN" in catalogue
search bar
4) Note no results returned

5) Apply patch
6) koha-plack --restart kohadev

7) Repeat Step #3
8) Note that results appear for 'location,wrdl: GEN'

Signed-off-by: Magnus Enger 
Works as advertised.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Magnus Enger  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off
   Patch complexity|--- |Trivial patch

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-29 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #15 from Fridolin Somers  ---
(In reply to David Cook from comment #13)
> (In reply to Fridolin Somers from comment #9)
> > > since "location" doesn't have a phrase index defined...
> > Why not add it to Zebra MARC21 configuration ?
> > 
> > Even if unlikely :
> > Imagine two locations "foo bar" and "bar foo", words list type search "wrdl"
> > will match both.
> > 
> > This indexing can be useful for other cases.
> 
> It might be worth adding that the status quo right now is doing a "wrdl"
> search for "location", so I'm just trying to make the status quo explicit
> and functional in all cases.

OKi no problem.
A fix that works is a good fix ;)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-28 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

David Cook  changed:

   What|Removed |Added

 Attachment #141950|0   |1
is obsolete||

--- Comment #14 from David Cook  ---
Created attachment 148848
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148848=edit
Bug 31841: Use location,wrdl when searching Shelving location

The "location" index only has a word index defined, so we need
to specify the wrdl qualifier when searching it via the staff
interface dropdown menu

Test plan:
1) Change QueryAutoTruncate to "only if * is added"
2) Change IntranetCatalogSearchPulldown to "Show"
3) Search for "Shelving location" of "GEN" in catalogue
search bar
4) Note no results returned

5) Apply patch
6) koha-plack --restart kohadev

7) Repeat Step #3
8) Note that results appear for 'location,wrdl: GEN'

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-28 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

David Cook  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-28 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #13 from David Cook  ---
(In reply to Fridolin Somers from comment #9)
> > since "location" doesn't have a phrase index defined...
> Why not add it to Zebra MARC21 configuration ?
> 
> Even if unlikely :
> Imagine two locations "foo bar" and "bar foo", words list type search "wrdl"
> will match both.
> 
> This indexing can be useful for other cases.

It might be worth adding that the status quo right now is doing a "wrdl" search
for "location", so I'm just trying to make the status quo explicit and
functional in all cases.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-28 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #12 from David Cook  ---
Thanks for looking Magnus. It's a simple fix. I'll look at doing that now
actually...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2023-03-28 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Magnus Enger  changed:

   What|Removed |Added

 Status|Needs Signoff   |Patch doesn't apply
 CC||mag...@libriotech.no

--- Comment #11 from Magnus Enger  ---
Sorry, patch does not apply and I don't see how to resolve the conflict: 

$ git bz apply 31841

Bug 31841 - Shelving location search in staff interface sometimes creates
invalid Zebra query

141950 - Bug 31841: Use location,wrdl when searching Shelving location

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 31841: Use location,wrdl when searching Shelving location
Using index info to reconstruct a base tree...
M   koha-tmpl/intranet-tmpl/prog/en/includes/search_indexes.inc
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/search_indexes.inc
CONFLICT (content): Merge conflict in
koha-tmpl/intranet-tmpl/prog/en/includes/search_indexes.inc
error: Failed to merge in the changes.
Patch failed at 0001 Bug 31841: Use location,wrdl when searching Shelving
location
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem run "git bz apply --continue".
If you would prefer to skip this patch, instead run "git bz apply --skip".
To restore the original branch and stop patching run "git bz apply --abort".
Patch left in
/tmp/Bug-31841-Use-locationwrdl-when-searching-Shelving-qsJQXm.patch

<<< HEAD
[% IF ms_location %]
Shelving location (code)
===
[% IF ms_locationcommawrdl %]
Shelving location (code)
>>> Bug 31841: Use location,wrdl when searching Shelving location

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2022-12-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

Fridolin Somers  changed:

   What|Removed |Added

 CC||fridolin.som...@biblibre.co
   ||m

--- Comment #9 from Fridolin Somers  ---
> since "location" doesn't have a phrase index defined...
Why not add it to Zebra MARC21 configuration ?

Even if unlikely :
Imagine two locations "foo bar" and "bar foo", words list type search "wrdl"
will match both.

This indexing can be useful for other cases.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2022-11-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #8 from David Cook  ---
(In reply to Katrin Fischer from comment #7)
> Sorry, I got confused :)

No worries!

I'd love to someone to sign it off though heh.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2022-11-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #7 from Katrin Fischer  ---
(In reply to David Cook from comment #6)
> (In reply to Katrin Fischer from comment #5)
> > Hi David, did you mean to sign-off here?
> 
> I'm not sure I understand the question? 

Sorry, I got confused :)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2022-11-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #6 from David Cook  ---
(In reply to Katrin Fischer from comment #5)
> Hi David, did you mean to sign-off here?

I'm not sure I understand the question? 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/


[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query

2022-11-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841

--- Comment #5 from Katrin Fischer  ---
Hi David, did you mean to sign-off here?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/