[Koha-bugs] [Bug 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m Resolution|--- |FIXED Status|Pushed to master|RESOLVED --- Comment #23 from Fridolin Somers --- Enhancement not pushed to 23.11.x -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #22 from Katrin Fischer --- Pushed for 24.05! Well done everyone, thank you! -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Katrin Fischer changed: What|Removed |Added Version(s)||24.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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #21 from Katrin Fischer --- Nice code clean-up! -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #20 from Tomás Cohen Arazi --- Created attachment 162384 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=162384=edit Bug 33431: Fix remaining cases This patch tweaks three remaining cases, that are not covered by tests. To test: 1. Apply this patch 2. Make use of those places => SUCCESS: No behavior change Signed-off-by: David Nind Signed-off-by: Pedro Amorim -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #19 from Tomás Cohen Arazi --- Created attachment 162382 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=162382=edit Bug 33431: Make C4::Record use C4::Context->yaml_preference This patch makes what the title says. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Rec* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi Signed-off-by: David Nind Signed-off-by: Pedro Amorim -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi changed: What|Removed |Added Attachment #160392|0 |1 is obsolete|| Attachment #160393|0 |1 is obsolete|| Attachment #160394|0 |1 is obsolete|| --- Comment #18 from Tomás Cohen Arazi --- Created attachment 162381 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=162381=edit Bug 33431: Make C4::Circulation use C4::Context->yaml_preference This patch removes manual YAML handling for sysprefs in C4::Circulation. It also makes C4::Context->yaml_preference not warn when undef is retrieved from the sysprefs. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Circulation* => SUCCESS: Tests pass! 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: David Nind Signed-off-by: Pedro Amorim -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi changed: What|Removed |Added Depends on||35293 --- Comment #17 from Tomás Cohen Arazi --- (In reply to Katrin Fischer from comment #16) > I am sorry, this no longer applies. I'll put it on my list to check and push > with higher priority: Caused by bug 35293. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35293 [Bug 35293] Regression: Bug 33390 (QA follow-up) patch overwrote the template changes to bug 25560 -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Katrin Fischer changed: What|Removed |Added Status|Passed QA |Patch doesn't apply --- Comment #16 from Katrin Fischer --- I am sorry, this no longer applies. I'll put it on my list to check and push with higher priority: Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 33431: Make C4::Circulation use C4::Context->yaml_preference Using index info to reconstruct a base tree... M C4/Circulation.pm Falling back to patching base and 3-way merge... Auto-merging C4/Circulation.pm CONFLICT (content): Merge conflict in C4/Circulation.pm error: Failed to merge in the changes. Patch failed at 0001 Bug 33431: Make C4::Circulation use C4::Context->yaml_preference 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-33431-Make-C4Circulation-use-C4Context-yamlpre-issx6rg8.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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Pedro Amorim changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=35680 -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Pedro Amorim changed: What|Removed |Added Attachment #160283|0 |1 is obsolete|| --- Comment #15 from Pedro Amorim --- Created attachment 160394 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160394=edit Bug 33431: Fix remaining cases This patch tweaks three remaining cases, that are not covered by tests. To test: 1. Apply this patch 2. Make use of those places => SUCCESS: No behavior change Signed-off-by: David Nind Signed-off-by: Pedro Amorim -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Pedro Amorim changed: What|Removed |Added Attachment #160282|0 |1 is obsolete|| --- Comment #14 from Pedro Amorim --- Created attachment 160393 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160393=edit Bug 33431: Make C4::Record use C4::Context->yaml_preference This patch makes what the title says. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Rec* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi Signed-off-by: David Nind Signed-off-by: Pedro Amorim -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Pedro Amorim changed: What|Removed |Added Attachment #160281|0 |1 is obsolete|| --- Comment #13 from Pedro Amorim --- Created attachment 160392 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160392=edit Bug 33431: Make C4::Circulation use C4::Context->yaml_preference This patch removes manual YAML handling for sysprefs in C4::Circulation. It also makes C4::Context->yaml_preference not warn when undef is retrieved from the sysprefs. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Circulation* => SUCCESS: Tests pass! 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: David Nind Signed-off-by: Pedro Amorim -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Pedro Amorim 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #12 from David Nind --- Testing notes (using KTD) - for patch 3 (see comment #8): 1. Add LC: 010$a to RisExportAdditionalFields and export a record as RIS. It should still work as usual and be the same before and after the patch. I used record 262 (Perl programming). 2. Have something set on `MarcFieldsToOrder` and add an order using an ISO2709 (mrc) file. Should work as usual. 2.1 I used the steps from comment 22 and sample files for bug 34645 3. Have something set for `UpdateItemWhenLostFromHoldList` and a couple pending holds. See this ByWater Solutions tutorial "Alert Patrons When Hold is Cancelled" https://bywatersolutions.com/education/monday-minutes-alert-patrons-when-hold-is-cancelled -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 David Nind changed: What|Removed |Added Attachment #160247|0 |1 is obsolete|| --- Comment #11 from David Nind --- Created attachment 160283 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160283=edit Bug 33431: Fix remaining cases This patch tweaks three remaining cases, that are not covered by tests. To test: 1. Apply this patch 2. Make use of those places => SUCCESS: No behavior change Signed-off-by: David Nind -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 David Nind changed: What|Removed |Added Attachment #160246|0 |1 is obsolete|| --- Comment #10 from David Nind --- Created attachment 160282 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160282=edit Bug 33431: Make C4::Record use C4::Context->yaml_preference This patch makes what the title says. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Rec* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi Signed-off-by: David Nind -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 David Nind changed: What|Removed |Added Attachment #160245|0 |1 is obsolete|| --- Comment #9 from David Nind --- Created attachment 160281 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160281=edit Bug 33431: Make C4::Circulation use C4::Context->yaml_preference This patch removes manual YAML handling for sysprefs in C4::Circulation. It also makes C4::Context->yaml_preference not warn when undef is retrieved from the sysprefs. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Circulation* => SUCCESS: Tests pass! 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: David Nind -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 David Nind 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #8 from Tomás Cohen Arazi --- (In reply to David Nind from comment #4) > The tests for the first two patches pass, both before and after the patches > are applied. > > I'm not sure for the third patch what features/things to test to make sure > they work as expected. Hi, this changes are pretty trivial code-wise. For the third patch you would need to: * Have something set on `RisExportAdditionalFields` and export a record as RIS- It should still work as usual. * Have something set on `MarcFieldsToOrder` and add an order using an ISO2709 (mrc) file. Should work as usual * Have something set for `UpdateItemWhenLostFromHoldList` and a couple pending holds. I'm not sure how to trigger it but it seems to be possible to mark a hold as lost from the pending list, and it should honor the setting. -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #7 from Tomás Cohen Arazi --- Created attachment 160247 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160247=edit Bug 33431: Fix remaining cases This patch tweaks three remaining cases, that are not covered by tests. To test: 1. Apply this patch 2. Make use of those places => SUCCESS: No behavior change -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #6 from Tomás Cohen Arazi --- Created attachment 160246 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160246=edit Bug 33431: Make C4::Record use C4::Context->yaml_preference This patch makes what the title says. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Rec* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi changed: What|Removed |Added Attachment #149236|0 |1 is obsolete|| Attachment #149237|0 |1 is obsolete|| Attachment #149238|0 |1 is obsolete|| --- Comment #5 from Tomás Cohen Arazi --- Created attachment 160245 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160245=edit Bug 33431: Make C4::Circulation use C4::Context->yaml_preference This patch removes manual YAML handling for sysprefs in C4::Circulation. It also makes C4::Context->yaml_preference not warn when undef is retrieved from the sysprefs. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Circulation* => SUCCESS: Tests pass! 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Pedro Amorim changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply CC||pedro.amo...@ptfs-europe.co ||m -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Bug 33431 depends on bug 30403, which changed state. Bug 30403 Summary: Update notforloan status also on check out https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30403 What|Removed |Added Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 David Nind changed: What|Removed |Added CC||da...@davidnind.com --- Comment #4 from David Nind --- The tests for the first two patches pass, both before and after the patches are applied. I'm not sure for the third patch what features/things to test to make sure they work as expected. -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #3 from Tomás Cohen Arazi --- Created attachment 149238 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149238=edit Bug 33431: Fix remaining cases This patch tweaks three remaining cases, that are not covered by tests. To test: 1. Apply this patch 2. Make use of those places => SUCCESS: No behavior change -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #2 from Tomás Cohen Arazi --- Created attachment 149237 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149237=edit Bug 33431: Make C4::Record use C4::Context->yaml_preference This patch makes what the title says. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Rec* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 --- Comment #1 from Tomás Cohen Arazi --- Created attachment 149236 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149236=edit Bug 33431: Make C4::Circulation use C4::Context->yaml_preference This patch removes manual YAML handling for sysprefs in C4::Circulation. It also makes C4::Context->yaml_preference not warn when undef is retrieved from the sysprefs. To test: 1. Run: $ ktd --shell k$ prove t/db_dependent/Circulation* => SUCCESS: Tests pass! 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests pass! 4. Sign off :-D -- 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi changed: What|Removed |Added Status|ASSIGNED|Needs Signoff 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 33431] Make code use C4::Context->yaml_preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33431 Tomás Cohen Arazi changed: What|Removed |Added Depends on||30403, 23580 Assignee|koha-b...@lists.koha-commun |tomasco...@gmail.com |ity.org | Status|NEW |ASSIGNED CC||tomasco...@gmail.com Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23580 [Bug 23580] Add C4::Context->yaml_preference https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30403 [Bug 30403] Update notforloan status also on check out -- 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/