[Koha-bugs] [Bug 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #33 from Jonathan Druart --- Marcel, this change has terrible side-effects: commit 7e222bb0796196f1e466b73d909fd28d529cab75 Bug 18811: Change visibility checks in authorities.pl next if $tagslib->{$tag}->{$subfield}->{hidden}; I was trying to test the AuthDisplayHierarchy behaviour and notice that the $9 was not filled with the related (broader) authority id. Indeed it is hidden. Can you fix ASAP? -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #34 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #33) > Marcel, this change has terrible side-effects: > > commit 7e222bb0796196f1e466b73d909fd28d529cab75 > Bug 18811: Change visibility checks in authorities.pl > > next if $tagslib->{$tag}->{$subfield}->{hidden}; > > I was trying to test the AuthDisplayHierarchy behaviour and notice that the > $9 was not filled with the related (broader) authority id. Indeed it is > hidden. > Can you fix ASAP? I am not really familiar with AuthDisplayHierarchy. The third patch added a script auth_show_hidden_data.pl to detect hidden fields, prompting the user to change them in the frameworks. At the time we considered this to be enough.. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #35 from Marcel de Rooy --- We could more or less revert the change but return to the older inconsistent situation.. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #36 from Katrin Fischer --- Make an exception for $9 as a special use case? Not sure if that is a valid option. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #37 from Marcel de Rooy --- (In reply to Katrin Fischer from comment #36) > Make an exception for $9 as a special use case? Not sure if that is a valid > option. Yeah that might not be a bad idea.. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #38 from Marcel de Rooy --- Created attachment 70614 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70614&action=edit Bug 18811: (QA follow-up) Allow hidden $9 subfields Although we could argue that the user should make all $9 subfields visible, this might be a good compromise. It selectively reverts the change from the first patch by accepting $9 subfields without checking the hidden flag in the framework. Test plan: [1] Make a subfield $9 visible for some authority field. [2] Put a value in it somewhere in authorities.pl [3] Make subfield $9 invisible in the auth framework. [4] Open and save the adjusted authority record. [5] Check the marcxml of this record. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #39 from Marcel de Rooy --- Could you test this one, Jonathan ? -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #40 from Jonathan Druart --- (In reply to Marcel de Rooy from comment #39) > Could you test this one, Jonathan ? It fixes the problem, but it sounds like there is something else to fix: On a new DB: MariaDB [koha_kohadev]> select tagfield, tagsubfield, hidden from auth_subfield_structure where tagfield=551 and tagsubfield=9 and authtypecode="GEOGR_NAME"\G *** 1. row *** tagfield: 551 tagsubfield: 9 hidden: 1 Edit the subfields for GEOGR_NAME /admin/auth_subfields_structure.pl?op=add_form&tagfield=551&authtypecode=GEOGR_NAME Possible value for "Select to display or not"/hidden are -5 and 0, so "Show all" is displayed. Save without modifying anything: hidden is now '0' -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Jonathan Druart changed: What|Removed |Added Blocks||20063 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20063 [Bug 20063] $9 is lost when cataloguing authority records -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #41 from Jonathan Druart --- (In reply to Marcel de Rooy from comment #38) > Created attachment 70614 [details] [review] > Bug 18811: (QA follow-up) Allow hidden $9 subfields Patch moved to bug 20063 -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #42 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #40) > It fixes the problem, but it sounds like there is something else to fix: > > On a new DB: > MariaDB [koha_kohadev]> select tagfield, tagsubfield, hidden from > auth_subfield_structure where tagfield=551 and tagsubfield=9 and > authtypecode="GEOGR_NAME"\G > *** 1. row *** >tagfield: 551 > tagsubfield: 9 > hidden: 1 > > Edit the subfields for GEOGR_NAME > /admin/auth_subfields_structure. > pl?op=add_form&tagfield=551&authtypecode=GEOGR_NAME > > Possible value for "Select to display or not"/hidden are -5 and 0, so "Show > all" is displayed. > Save without modifying anything: hidden is now '0' Moved this to a new report: bug 20074 -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Marcel de Rooy changed: What|Removed |Added Blocks||20074 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 [Bug 20074] Auth_subfield_structure changes hidden attribute -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #6 from Marcel de Rooy --- Created attachment 64542 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64542&action=edit Bug 18811: Add a script for checking authority data in hidden fields If you edit an authority record while having data in hidden fields or subfields, that data will be lost now. This script can help you to unhide some fields and prevent data loss. Test plan: [1] Add a PERSO_NAME record. Fill e.g. 100b. [2] Hide 100b in the PERSO_NAME framework. [3] Run auth_show_hidden_data.pl and verify that it reports 100b in the PERSO_NAME framework. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #5 from Marcel de Rooy --- Created attachment 64541 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64541&action=edit Bug 18811: Add Koha Objects for authority tags and subfields Trivial copy and paste activity. Will be used in a later patch. No test plan needed. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Marcel de Rooy changed: What|Removed |Added Attachment #64314|0 |1 is obsolete|| --- Comment #7 from Marcel de Rooy --- Created attachment 64570 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64570&action=edit Bug 18811: Change visibility checks in authorities.pl The check is now <=-4 or >=5, but the framework uses 0 for Show all and -5 for Hide all. (Note that sql installer scripts also use 8.) When modifying an authority, the script also showed hidden fields when filled, since it did not check the hidden field but only the tab field. NOTE: The proposed solution restores consistency, but will remove hidden fields from the MARC record. Test plan: [1] Set field 942a to Show all in an authority framework. [2] Open a new record in this framework and verify that you see 942a. [3] Edit an existing record in this framework and verify again. [4] Set field 942a now to Hide all in this framework. [5] Open a new record in this framework and verify that 942a is hidden. [6] Edit an existing record in this framework and verify again. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Marcel de Rooy changed: What|Removed |Added Attachment #64542|0 |1 is obsolete|| --- Comment #9 from Marcel de Rooy --- Created attachment 64572 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64572&action=edit Bug 18811: Add a script for checking authority data in hidden fields If you edit an authority record while having data in hidden fields or subfields, that data will be lost now. This script can help you to unhide some fields and prevent data loss. Test plan: [1] Add a PERSO_NAME record. Fill e.g. 100b. [2] Hide 100b in the PERSO_NAME framework. [3] Run auth_show_hidden_data.pl and verify that it reports 100b in the PERSO_NAME framework. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Marcel de Rooy changed: What|Removed |Added Attachment #64541|0 |1 is obsolete|| --- Comment #8 from Marcel de Rooy --- Created attachment 64571 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64571&action=edit Bug 18811: Add Koha Objects for authority tags and subfields Trivial copy and paste activity. Will be used in a later patch. No test plan needed. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #10 from Marcel de Rooy --- Created attachment 64573 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64573&action=edit Bug 18811: Atomic update to print warning at upgrade time The warning encourages people to run the auth_show_hidden_data script to check for data in hidden fields and adjust their frameworks. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #11 from Marcel de Rooy --- In order to overcome the possible risk of data loss in hidden fields, I have added a maintenance script to search these fields. A user can decide to make some of these fields visible in the respective frameworks. Also a warning is printed at upgrade time. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Michael Cabus changed: What|Removed |Added CC||mich...@bywatersolutions.co ||m --- Comment #12 from Michael Cabus --- When I run perl misc/maintenance/auth_show_hidden_data.pl -c I get this error: DBIx::Class::ResultSet::find(): find() expects either a column/value hashref, or a list of values corresponding to the columns of the specified unique constraint 'primary' at /home/vagrant/kohaclone/Koha/Objects.pm line 85 Checked Authorities schema and all seemed well. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #13 from Marcel de Rooy --- (In reply to Michael Cabus from comment #12) > When I run perl misc/maintenance/auth_show_hidden_data.pl -c I get this > error: > > DBIx::Class::ResultSet::find(): find() expects either a column/value > hashref, or a list of values corresponding to the columns of the specified > unique constraint 'primary' at /home/vagrant/kohaclone/Koha/Objects.pm line > 85 > > Checked Authorities schema and all seemed well. Hi Michael, Thanks for testing. I guess that you are using kohadevbox to test, but I wonder if you have a configuration issue. Line 85 of Koha/Objects in master is not the DBIx find call, but only the first line of the sub with me. Please check if your paths are in order. The perl library path should also point to current master not to the older instance installed on your devbox. Do you use dev_install in your koha-conf? Marcel -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Jonathan Druart changed: What|Removed |Added Blocks||21037 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21037 [Bug 21037] auth_show_hidden_data.pl gives false positives about 942$a and 152$b fields. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Jonathan Druart changed: What|Removed |Added Blocks||21039 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21039 [Bug 21039] auth_show_hidden_data.pl should also check if the subfield is ignored -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Josef Moravec changed: What|Removed |Added Attachment #64570|0 |1 is obsolete|| Attachment #64571|0 |1 is obsolete|| Attachment #64572|0 |1 is obsolete|| Attachment #64573|0 |1 is obsolete|| --- Comment #14 from Josef Moravec --- Created attachment 66309 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66309&action=edit [SIGNED-OFF] Bug 18811: Change visibility checks in authorities.pl The check is now <=-4 or >=5, but the framework uses 0 for Show all and -5 for Hide all. (Note that sql installer scripts also use 8.) When modifying an authority, the script also showed hidden fields when filled, since it did not check the hidden field but only the tab field. NOTE: The proposed solution restores consistency, but will remove hidden fields from the MARC record. Test plan: [1] Set field 942a to Show all in an authority framework. [2] Open a new record in this framework and verify that you see 942a. [3] Edit an existing record in this framework and verify again. [4] Set field 942a now to Hide all in this framework. [5] Open a new record in this framework and verify that 942a is hidden. [6] Edit an existing record in this framework and verify again. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Josef Moravec 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 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #17 from Josef Moravec --- Created attachment 66312 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66312&action=edit [SIGNED-OFF] Bug 18811: Atomic update to print warning at upgrade time The warning encourages people to run the auth_show_hidden_data script to check for data in hidden fields and adjust their frameworks. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #15 from Josef Moravec --- Created attachment 66310 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66310&action=edit [SIGNED-OFF] Bug 18811: Add Koha Objects for authority tags and subfields Trivial copy and paste activity. Will be used in a later patch. No test plan needed. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #16 from Josef Moravec --- Created attachment 66311 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66311&action=edit [SIGNED-OFF] Bug 18811: Add a script for checking authority data in hidden fields If you edit an authority record while having data in hidden fields or subfields, that data will be lost now. This script can help you to unhide some fields and prevent data loss. Test plan: [1] Add a PERSO_NAME record. Fill e.g. 100b. [2] Hide 100b in the PERSO_NAME framework. [3] Run auth_show_hidden_data.pl and verify that it reports 100b in the PERSO_NAME framework. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Nick Clemens changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |n...@bywatersolutions.com |y.org | -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 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 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Nick Clemens changed: What|Removed |Added Attachment #66309|0 |1 is obsolete|| Attachment #66310|0 |1 is obsolete|| Attachment #66311|0 |1 is obsolete|| Attachment #66312|0 |1 is obsolete|| --- Comment #18 from Nick Clemens --- Created attachment 66708 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66708&action=edit Bug 18811: Change visibility checks in authorities.pl The check is now <=-4 or >=5, but the framework uses 0 for Show all and -5 for Hide all. (Note that sql installer scripts also use 8.) When modifying an authority, the script also showed hidden fields when filled, since it did not check the hidden field but only the tab field. NOTE: The proposed solution restores consistency, but will remove hidden fields from the MARC record. Test plan: [1] Set field 942a to Show all in an authority framework. [2] Open a new record in this framework and verify that you see 942a. [3] Edit an existing record in this framework and verify again. [4] Set field 942a now to Hide all in this framework. [5] Open a new record in this framework and verify that 942a is hidden. [6] Edit an existing record in this framework and verify again. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec 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 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #21 from Nick Clemens --- Created attachment 66711 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66711&action=edit Bug 18811: Atomic update to print warning at upgrade time The warning encourages people to run the auth_show_hidden_data script to check for data in hidden fields and adjust their frameworks. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec 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 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #20 from Nick Clemens --- Created attachment 66710 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66710&action=edit Bug 18811: Add a script for checking authority data in hidden fields If you edit an authority record while having data in hidden fields or subfields, that data will be lost now. This script can help you to unhide some fields and prevent data loss. Test plan: [1] Add a PERSO_NAME record. Fill e.g. 100b. [2] Hide 100b in the PERSO_NAME framework. [3] Run auth_show_hidden_data.pl and verify that it reports 100b in the PERSO_NAME framework. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec 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 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #19 from Nick Clemens --- Created attachment 66709 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66709&action=edit Bug 18811: Add Koha Objects for authority tags and subfields Trivial copy and paste activity. Will be used in a later patch. No test plan needed. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec 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 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Failed QA --- Comment #22 from Jonathan Druart --- Please provide tests for new modules (based on t/db_dependent/Koha/Cities.t) -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Marcel de Rooy changed: What|Removed |Added Status|Failed QA |Passed QA -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #23 from Marcel de Rooy --- Created attachment 66793 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66793&action=edit Bug 18811: [QA Follow-up] Add tests for Koha::Authority::Subfields/Tags Renaming t/db_dependent/Authorities to Authority Adding Subfields.t and Tags.t Test plan: [1] prove t/db_dependent/Authority Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #24 from Marcel de Rooy --- Created attachment 66794 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66794&action=edit Bug 18811: [QA Follow-up] Resolved POD warnings Trivial changes to silence qa tools on POD warnings for Subfields and Tags. Note: Since Subfield and Tag only contain one sub which is considered as private by Pod::Coverage, these modules are listed as unrated (no public symbols defined) and trigger a FAIL on pod coverage in qa tools. This fail can be ignored. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #25 from Marcel de Rooy --- (In reply to Marcel de Rooy from comment #24) > Note: Since Subfield and Tag only contain one sub which is considered as > private by Pod::Coverage, these modules are listed as unrated (no public > symbols defined) and trigger a FAIL on pod coverage in qa tools. This fail > can be ignored. This is true for all 'empty' singular Koha objects. Could be resolved by renaming _type to type, or adding another public symbol. Should be done on another report, if we really feel the need to do so. We can just ignore them too. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #26 from Jonathan Druart --- Pushed to master for 17.11, thanks to everybody involved! -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Fridolin SOMERS changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m Status|Pushed to Master|Pushed to Stable --- Comment #27 from Fridolin SOMERS --- Pushed to 17.05.x, will be in 17.05.06. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Katrin Fischer changed: What|Removed |Added CC||katrin.fisc...@bsz-bw.de --- Comment #28 from Katrin Fischer --- I am not sure about this comment: NOTE: The proposed solution restores consistency, but will remove hidden fields from the MARC record. Nick or Marcel, can you please explain and advise if this should go into 16.11.x? -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #29 from Marcel de Rooy --- (In reply to Katrin Fischer from comment #28) > I am not sure about this comment: > > NOTE: The proposed solution restores consistency, but will remove hidden > fields from the MARC record. > > Nick or Marcel, can you please explain and advise if this should go into > 16.11.x? Comment11 addresses this concern. I would recommend to push it in order to keep in line with 17.05 too. Note too that fields are not removed rightaway but could be lost when you save your (authority) record again in such a framework. A warning is printed and a script is provided. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #30 from Marcel de Rooy --- Applying: Bug 18811: [QA Follow-up] Add tests for Koha::Authority::Subfields/Tags fatal: mode change for t/db_dependent/Authorities/MergeRequests.t, which is not in current HEAD -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 --- Comment #31 from Marcel de Rooy --- (In reply to Marcel de Rooy from comment #30) > Applying: Bug 18811: [QA Follow-up] Add tests for > Koha::Authority::Subfields/Tags > fatal: mode change for t/db_dependent/Authorities/MergeRequests.t, which is > not in current HEAD This might change the game. It is related to 9988. -- You are receiving this mail because: 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 18811] Visibility settings inconsistent between framework and authority editor
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 Katrin Fischer changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to Stable|RESOLVED --- Comment #32 from Katrin Fischer --- This won't get pushed into 16.11.x, closing. -- You are receiving this mail because: 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/