[Koha-bugs] [Bug 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-05-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Katrin Fischer  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #12 from Katrin Fischer  ---
I am sorry, but I don't believe this is a good solution. 
This will fail the second someone translates the authorised value (trust me, we
do that! :) )

Looking at this, I wonder if we should handle this differently:

We run into this issue because as soon as something is linked to an authorised
value, the code will be replaced by the description in the XSLT. And this
should not happen:

XSLT.pm (see bug 14377):

+  # Replace the field value with the authorised value *except* for
MARC21/NORMARC field 942$n (suppression in opac)
if ( !( $tag eq '942' && $subfield eq 'n' ) || $marcflavour eq 'UNIMARC' ) {
 $value = GetAuthorisedValueDesc( $tag, $letter, $value, '', $tagslib )

So even when linked to an authorised value, we should still be seeing a 1 and
the check should be working.

Please check why this is not working as expected.

I also feel like the depends on bug 26116 is wrong here - should be a blocks.

-- 
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-05-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Phil Ringnalda  changed:

   What|Removed |Added

 CC||p...@chetcolibrary.org

-- 
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-05-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Phil Ringnalda  changed:

   What|Removed |Added

 Attachment #120213|0   |1
is obsolete||

--- Comment #11 from Phil Ringnalda  ---
Created attachment 120719
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120719=edit
Bug 23406: Test for 1 or 'Yes' in record suppression

In marc21 we now link the 942$n to 'YES_NO' authorised values. When displaying
the details page in the staff client the authorised values in a record are
translated before parsing the record with XSLT

The '1' for YES_NO is translated to 'Yes' - we simply need to test for this
additional
value in the XSLT to ensure we display the suppression message

To test:
 1 - Confirm 952$n in default framework is tied ot authorised value 'YES_NO' or
do so
 2  - Set OpacSuppression to 'Hide'
 3 - Edit a record in the default template, changing 942$n to 'Yes'
 4 - Confirm the record is suppressed in opac
 5 - Note there is no suppression notice in the record in staff client
 6 - Apply patch
 7 - Reload and confirm staff side shows suppression notice
 8 - Confirm record still suppressed in OPAC
 9 - Edit record, set 942$n to 'no'
10 - Confirm there is no suppression notice in staff client
11 - Confirm you can view the record in OPAC

Signed-off-by: Phil Ringnalda 

-- 
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-05-07 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Phil Ringnalda  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

-- 
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-04-27 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Nick Clemens  changed:

   What|Removed |Added

 CC||n...@bywatersolutions.com
   Assignee|koha-b...@lists.koha-commun |n...@bywatersolutions.com
   |ity.org |

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-04-27 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #10 from Nick Clemens  ---
Created attachment 120213
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120213=edit
Bug 23406: Test for 1 or 'Yes' in record suppression

In marc21 we now link the 942$n to 'YES_NO' authorised values. When displaying
the details page in the staff client the authorised values in a record are
translated before parsing the record with XSLT

The '1' for YES_NO is translated to 'Yes' - we simply need to test for this
additional
value in the XSLT to ensure we display the suppression message

To test:
 1 - Confirm 952$n in default framework is tied ot authorised value 'YES_NO' or
do so
 2  - Set OpacSuppression to 'Hide'
 3 - Edit a record in the default template, changing 942$n to 'Yes'
 4 - Confirm the record is suppressed in opac
 5 - Note there is no suppression notice in the record in staff client
 6 - Apply patch
 7 - Reload and confirm staff side shows suppression notice
 8 - Confirm record still suppressed in OPAC
 9 - Edit record, set 942$n to 'no'
10 - Confirm there is no suppression notice in staff client
11 - Confirm you can view the record in OPAC

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-04-27 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Nick Clemens  changed:

   What|Removed |Added

   Patch complexity|--- |Trivial patch
 Status|NEW |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2021-04-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Andrew Fuerste-Henry  changed:

   What|Removed |Added

Version|19.05   |master

--- Comment #9 from Andrew Fuerste-Henry  ---
I've just confirmed that this problem exists in master and 20.05. To recreate:
- set OpacSuppression to Hide
- Edit a bib, set 942$n=1
- confirm bib shows "Suppressed in OPAC" in the intranet and will not open on
OPAC
- Edit the framework used by this bib, connect the 942$n to the YES_NO auth
value
- Reload the bib without editing it, confirm "Suppressed in OPAC" message no
longer shows
- try to open record in OPAC, confirm it's still suppressed
- edit record, set 942$n to No, confirm no apparent change on intranet, record
now opens on opac
- edit record, set 942$n to Yes, confirm no apparent change on intranet, record
is now suppressed on opac
- Edit the framework again, unlink the auth value
- reload bib without editing, confirm bib shows "Suppressed in OPAC" in the
intranet and will not open on OPAC

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-07-31 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Depends on|25335   |26116


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25335
[Bug 25335] Use of an authorised value in a marc subfield causes strict mode
SQL error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26116
[Bug 26116] Allow multiple suppression values
-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-05-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #8 from Andrew Fuerste-Henry  ---
Also, agreed that !=0 would be a better check for suppression, but I'm not
certain the query builder isn't looking for 1 specifically. That's a whole
other bug I'm hoping to get around to filing and working on.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-05-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #7 from Andrew Fuerste-Henry  ---
Oop, there's the issue. OpacSuppression wasn't turned on in my testing docker.
That's so standard for us that I forget it's off by default.
So this does work in both master and 19.11.

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-05-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Andrew Fuerste-Henry  changed:

   What|Removed |Added

Version|18.11   |19.05

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-05-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #6 from Katrin Fischer  ---
Yes, I am using default - freshly set up database without any changes. What's
your authorised value?

I wonder about this line:

[marc:subfield[@code='n'] = '1']">

I would have expected != ''.

But YES_NO is 0 or 1 so that makes sense. Are you using other values by chance?
OpacSuppression activated?

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-05-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #5 from Andrew Fuerste-Henry  ---
I'm on master using Koha testing docker, so those should be up to date. Here's
the relevant bit from MARC21slim2intranetResults.xsl (line 285):



Suppressed in
OPAC



Katrin, is that the same file you're using?

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-04-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Katrin Fischer  changed:

   What|Removed |Added

   Severity|enhancement |minor

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-04-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #4 from Katrin Fischer  ---
(In reply to Andrew Fuerste-Henry from comment #3)
> But it doesn't work in master, oddly enough. Also, adding a dependency where
> strict mode causes some trouble in addbiblio

It does work for me in master, even if the formatting of the notes could be
improved. I used the YES_NO and tested with the advanced and the normal editor
in staff detail and result pages.

Could it be your XSLT files are custom and outdated? (ours were...)

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-04-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

--- Comment #3 from Andrew Fuerste-Henry  ---
But it doesn't work in master, oddly enough. Also, adding a dependency where
strict mode causes some trouble in addbiblio

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-04-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 Depends on||25335


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25335
[Bug 25335] Use of an authorised value in a marc subfield causes strict mode
SQL error
-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2020-04-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Andrew Fuerste-Henry  changed:

   What|Removed |Added

 CC||and...@bywatersolutions.com

--- Comment #2 from Andrew Fuerste-Henry  ---
This appears to be fixed in 19.11. Haven't figured out what's causing it in
19.05 and previous.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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 23406] Authorized Value for Suppression when added to Frameworks doesn't show Suppress in Staff Client

2019-08-13 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23406

Katrin Fischer  changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

--- Comment #1 from Katrin Fischer  ---
Hi Kelly, I think it actually should show (bug 14377). Is your test user using
custom XSLT by chance? (Couldn't test it yet, just a thought for now)

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
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/