[Koha-bugs] [Bug 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-05-09 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Jared Camins-Esakov jcam...@cpbibliography.com changed:

   What|Removed |Added

 Status|Pushed to Master|Pushed to Stable

--- Comment #20 from Jared Camins-Esakov jcam...@cpbibliography.com ---
This bug will be included in the Koha 3.6.5 release.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-21 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

--- Comment #16 from Paul Poulain paul.poul...@biblibre.com ---
(In reply to comment #12)
 I have also set UseICU = 1 in updatedatabase.pl, since that will preserve
 the current
 behaviour of Koha, but UseICU = 0 in sysprefs.sql, since most new
 installations will
 not have ICU activated.

mmm... If I understand correctly, it means that someone not using ICU don't has
a problem before your patch, and will have after, because the default value is
1 ?
In this case, it's a bad idea : we should have libraries that have a working
behaviour still working, and those with a buggy behaviour before the patch be
warned that, if they change the new syspref, it will fix the behaviour

Please confirm i'm wrong or change the default value ;-)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-21 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

--- Comment #17 from Jared Camins-Esakov jcam...@cpbibliography.com ---
Paul,

(In reply to comment #16)
 (In reply to comment #12)
  I have also set UseICU = 1 in updatedatabase.pl, since that will preserve
  the current
  behaviour of Koha, but UseICU = 0 in sysprefs.sql, since most new
  installations will
  not have ICU activated.
 
 mmm... If I understand correctly, it means that someone not using ICU don't
 has a problem before your patch, and will have after, because the default
 value is 1 ?
 In this case, it's a bad idea : we should have libraries that have a working
 behaviour still working, and those with a buggy behaviour before the patch
 be warned that, if they change the new syspref, it will fix the behaviour
 
 Please confirm i'm wrong or change the default value ;-)

Not exactly. All existing Koha installations have tracings working as if UseICU
is set to 1. While this may be causing problems that would be solved by
changing UseICU to 0, setting UseICU to 1 will not break any existing
installations that are working, while setting UseICU to 0 might. So the problem
you described with the update breaking working installations is exactly why
updatedatabase needs to set UseICU to 1. ;)

Jared

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-21 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

--- Comment #18 from Magnus Enger mag...@enger.priv.no ---
Thanks Jared, that was exactly what I was going to say. :-)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-21 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Paul Poulain paul.poul...@biblibre.com changed:

   What|Removed |Added

 Status|Passed QA   |Pushed to Master

--- Comment #19 from Paul Poulain paul.poul...@biblibre.com ---
thanks for the confirmation i was wrong ;-)

patch pushed (I haven't tested it, just checked that the updatedatabase is OK,
te sysprefs.sql file can be loaded and the syspref can be set)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-16 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

   Attachment #8140|0   |1
is obsolete||

--- Comment #12 from Magnus Enger mag...@enger.priv.no ---
Created attachment 8225
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8225action=edit
Bug 7092 - Complete-subfield searches TraceCompleteSubfields syspref not
working correctly

This patch adds a syspref TracingQuotes for toggling between using  and {}
for quoting in
subject tracings. With normal Zebra indexing,  must be used. With ICU Zebra
indexing, {}
must be used.

This syspref interacts with TraceCompleteSubfields and
TraceSubjectSubdivisions.

To test:

- marcflavour = MARC21
- make sure you have a record with a complex subject, like
  Internet -- Law and legislation

Toggle TracingQuotes, TraceCompleteSubfields and TraceSubjectSubdivisions and
check that you
get the expected results for the clickable links in the OPAC detail view:

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su:{Internet}

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su,complete-subfield:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su,complete-subfield:{Internet}

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su:Internet) and (su:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su:{Internet}) and (su:{Law and legislation.})

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su,complete-subfield:Internet) and
(su,complete-subfield:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su,complete-subfield:{Internet}) and
(su,complete-subfield:{Law and legislation.})

Check that clicking on the generated links gives the expected search results,
bot with normal and ICU
indexing. (I have actually not tested this with ICU, so doing that before
signing off is crucial.)

UPDATE 2012-03-16:

Changed the name of the syspref from TracingQuotes to UseICU after a suggestion
from Paul P., to
make it more generally useful.

I have used Using/Not using instead of Use/Don't use in the description of
the syspref, to
reflect the fact that this syspref will not change how Zebra does its indexing
- it's just
a way of telling Koha how the indexing is actually done.

I have also set UseICU = 1 in updatedatabase.pl, since that will preserve the
current
behaviour of Koha, but UseICU = 0 in sysprefs.sql, since most new installations
will
not have ICU activated.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-16 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

--- Comment #13 from Magnus Enger mag...@enger.priv.no ---
New patch attached, changing the name of the syspref from TracingQuotes to
UseICU.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-14 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Ian Walls koha.sek...@gmail.com changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA
 CC||koha.sek...@gmail.com

--- Comment #10 from Ian Walls koha.sek...@gmail.com ---
Despite the length test plan, the change here is really pretty simple.  It just
adds a new system preference on how to quote tracings, and then implements that
wrapper in the MARC21 Details XSLT using variables.  Change to C4/XSLT.pm is
just to make the module aware of the new preference.

Marking as Passed QA.  Still need followups for the other MARC flavours.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-10 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Jared Camins-Esakov jcam...@cpbibliography.com changed:

   What|Removed |Added

   Attachment #8118|0   |1
is obsolete||

--- Comment #8 from Jared Camins-Esakov jcam...@cpbibliography.com ---
Created attachment 8139
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8139action=edit
Bug 7092 - Complete-subfield searches TraceCompleteSubfields syspref not
working correctly

This patch adds a syspref TracingQuotes for toggling between using  and {}
for quoting in
subject tracings. With normal Zebra indexing,  must be used. With ICU Zebra
indexing, {}
must be used.

This syspref interacts with TraceCompleteSubfields and
TraceSubjectSubdivisions.

To test:

- marcflavour = MARC21
- make sure you have a record with a complex subject, like
  Internet -- Law and legislation

Toggle TracingQuotes, TraceCompleteSubfields and TraceSubjectSubdivisions and
check that you
get the expected results for the clickable links in the OPAC *and* intranet
detail views:

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su:{Internet}

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su,complete-subfield:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su,complete-subfield:{Internet}

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su:Internet) and (su:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su:{Internet}) and (su:{Law and legislation.})

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su,complete-subfield:Internet) and
(su,complete-subfield:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su,complete-subfield:{Internet}) and
(su,complete-subfield:{Law and legislation.})

Check that clicking on the generated links gives the expected search results,
bot with normal and ICU
indexing. (I have actually not tested this with ICU, so doing that before
signing off is crucial.)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-10 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Jared Camins-Esakov jcam...@cpbibliography.com changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-10 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Jared Camins-Esakov jcam...@cpbibliography.com changed:

   What|Removed |Added

   Attachment #8139|0   |1
is obsolete||

--- Comment #9 from Jared Camins-Esakov jcam...@cpbibliography.com ---
Created attachment 8140
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8140action=edit
Bug 7092 - Complete-subfield searches TraceCompleteSubfields syspref not
working correctly

This patch adds a syspref TracingQuotes for toggling between using  and {}
for quoting in
subject tracings. With normal Zebra indexing,  must be used. With ICU Zebra
indexing, {}
must be used.

This syspref interacts with TraceCompleteSubfields and
TraceSubjectSubdivisions.

To test:

- marcflavour = MARC21
- make sure you have a record with a complex subject, like
  Internet -- Law and legislation

Toggle TracingQuotes, TraceCompleteSubfields and TraceSubjectSubdivisions and
check that you
get the expected results for the clickable links in the OPAC *and* intranet
detail views:

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su:{Internet}

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su,complete-subfield:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su,complete-subfield:{Internet}

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su:Internet) and (su:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su:{Internet}) and (su:{Law and legislation.})

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su,complete-subfield:Internet) and
(su,complete-subfield:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su,complete-subfield:{Internet}) and
(su,complete-subfield:{Law and legislation.})

Check that clicking on the generated links gives the expected search results,
bot with normal and ICU
indexing. (I have actually not tested this with ICU, so doing that before
signing off is crucial.)

Signed-off-by: Jared Camins-Esakov jcam...@cpbibliography.com

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-09 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

   Attachment #8069|0   |1
is obsolete||

--- Comment #6 from Magnus Enger mag...@enger.priv.no ---
Created attachment 8118
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8118action=edit
Bug 7092 - Complete-subfield searches TraceCompleteSubfields syspref not
working correctly

This patch adds a syspref TracingQuotes for toggling between using  and {}
for quoting in
subject tracings. With normal Zebra indexing,  must be used. With ICU Zebra
indexing, {}
must be used.

This syspref interacts with TraceCompleteSubfields and
TraceSubjectSubdivisions.

To test:

- marcflavour = MARC21
- make sure you have a record with a complex subject, like
  Internet -- Law and legislation

Toggle TracingQuotes, TraceCompleteSubfields and TraceSubjectSubdivisions and
check that you
get the expected results for the clickable links in the OPAC *and* intranet
detail views:

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su:{Internet}

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su,complete-subfield:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su,complete-subfield:{Internet}

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su:Internet) and (su:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su:{Internet}) and (su:{Law and legislation.})

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su,complete-subfield:Internet) and
(su,complete-subfield:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su,complete-subfield:{Internet}) and
(su,complete-subfield:{Law and legislation.})

Check that clicking on the generated links gives the expected search results,
bot with normal and ICU
indexing. (I have actually not tested this with ICU, so doing that before
signing off is crucial.)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-09 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

 Status|Signed Off  |Needs Signoff

--- Comment #7 from Magnus Enger mag...@enger.priv.no ---
I forgot to implement TracingQuotes for the intranet. This new patch replaces
the old one, and implements it for both the OPAC and the intranet.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-07 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Jared Camins-Esakov jcam...@cpbibliography.com changed:

   What|Removed |Added

   Attachment #8010|0   |1
is obsolete||

--- Comment #5 from Jared Camins-Esakov jcam...@cpbibliography.com ---
Created attachment 8069
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8069action=edit
Bug 7092 - Complete-subfield searches TraceCompleteSubfields syspref not
working correctly

This patch adds a syspref TracingQuotes for toggling between using  and {}
for quoting in
subject tracings. With normal Zebra indexing,  must be used. With ICU Zebra
indexing, {}
must be used.

This syspref interacts with TraceCompleteSubfields and
TraceSubjectSubdivisions.

To test:

- marcflavour = MARC21
- make sure you have a record with a complex subject, like
  Internet -- Law and legislation

Toggle TracingQuotes, TraceCompleteSubfields and TraceSubjectSubdivisions and
check that you
get the expected results for the clickable links in the OPAC detail view:

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su:{Internet}

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su,complete-subfield:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su,complete-subfield:{Internet}

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su:Internet) and (su:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su:{Internet}) and (su:{Law and legislation.})

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su,complete-subfield:Internet) and
(su,complete-subfield:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su,complete-subfield:{Internet}) and
(su,complete-subfield:{Law and legislation.})

Check that clicking on the generated links gives the expected search results,
bot with normal and ICU
indexing. (I have actually not tested this with ICU, so doing that before
signing off is crucial.)

Signed-off-by: Jared Camins-Esakov jcam...@cpbibliography.com

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-07 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Jared Camins-Esakov jcam...@cpbibliography.com changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off
 CC||jcam...@cpbibliography.com

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-05 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

   Assignee|gmcha...@gmail.com  |mag...@enger.priv.no

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-05 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

--- Comment #3 from Magnus Enger mag...@enger.priv.no ---
Looks like UNIMARC does not implement TraceSubjectSubdivisions and
TraceCompleteSubfields, so I'll just do MARC21. Please correct me if I'm
missing something!

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-05 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

--- Comment #4 from Magnus Enger mag...@enger.priv.no ---
Created attachment 8010
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8010action=edit
Bug 7092 - Complete-subfield searches TraceCompleteSubfields syspref not
working correctly

This patch adds a syspref TracingQuotes for toggling between using  and {}
for quoting in
subject tracings. With normal Zebra indexing,  must be used. With ICU Zebra
indexing, {}
must be used.

This syspref interacts with TraceCompleteSubfields and
TraceSubjectSubdivisions.

To test:

- marcflavour = MARC21
- make sure you have a record with a complex subject, like
  Internet -- Law and legislation

Toggle TracingQuotes, TraceCompleteSubfields and TraceSubjectSubdivisions and
check that you
get the expected results for the clickable links in the OPAC detail view:

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su:{Internet}

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Don't include
TracingQuotes = quotes
  opac-search.pl?q=su,complete-subfield:Internet
TracingQuotes = curlybrace
  opac-search.pl?q=su,complete-subfield:{Internet}

TraceCompleteSubfields   = Don't force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su:Internet) and (su:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su:{Internet}) and (su:{Law and legislation.})

TraceCompleteSubfields   = Force
TraceSubjectSubdivisions = Include
TracingQuotes = quotes
  opac-search.pl?q=(su,complete-subfield:Internet) and
(su,complete-subfield:Law and legislation.)
TracingQuotes = curlybrace
  opac-search.pl?q=(su,complete-subfield:{Internet}) and
(su,complete-subfield:{Law and legislation.})

Check that clicking on the generated links gives the expected search results,
bot with normal and ICU
indexing. (I have actually not tested this with ICU, so doing that before
signing off is crucial.)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-05 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-02 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

--- Comment #2 from Magnus Enger mag...@enger.priv.no ---
magnuse jcamins: got any hunches about bug 7092? like where to start digging? 
jcamins magnuse: yes. Try changing the {} quotes to .
jcamins magnuse: alternatively, use complete-field instead of
complete-subfield.
magnuse ok, that gives me somewhere to start - thank you very much! 
magnuse jcamins: chaning {} to  seems to solve it! 
jcamins magnuse: yeah, Search.pm doesn't play well with ICU.
magnuse jcamins: so a patch to replace {} with  in the XSLT would solve bug
7092? or is it more of a quick and dirty workaround? 
jcamins magnuse: alas, it will *cause* bug 7092 on the installations that are
still working.
magnuse jcamins: ?? 
magnuse sounds like catch 22 or something, then...
jcamins magnuse: C4::Search can't handle regular quotes with ICU enabled.
jcamins magnuse: without ICU, C4::Search apparently can't handle {}
magnuse ah, so what should be used depends on whether icu is used or not? 
jcamins That's what I've concluded.
magnuse nice
jcamins And, of course, you can't tell whether ICU is enabled.
magnuse what to do then? add a syspref to choose {} or ? 
jcamins magnuse: that's the best idea I can come up with.
magnuse hm, i might try and add the syspref, then
magnuse probably better to call it e.g. TraceCompleteSubfieldsQuoting than
UseICU, right? cause it wouldn't actually influence the indexing...
jcamins Better TracingQuotes or something like that.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 7092] Complete-subfield searches TraceCompleteSubfields syspref not working correctly

2012-03-01 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7092

Magnus Enger mag...@enger.priv.no changed:

   What|Removed |Added

 CC||mag...@enger.priv.no
 Blocks||7537

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA Contact for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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/