[Koha-bugs] [Bug 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-09-13 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

Lucas Gass  changed:

   What|Removed |Added

 Status|Pushed to oldstable |RESOLVED
 CC||lu...@bywatersolutions.com
 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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #34 from Pedro Amorim  ---
Nice work everyone!

Pushed to 22.11.x for next release

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Pedro Amorim  changed:

   What|Removed |Added

 Version(s)|23.11.00,23.05.03   |23.11.00,23.05.03,22.11.09
released in||
 Status|Pushed to stable|Pushed to oldstable

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Fridolin Somers  changed:

   What|Removed |Added

 CC||fridolin.som...@biblibre.co
   ||m
 Version(s)|23.11.00|23.11.00,23.05.03
released in||
 Status|Pushed to master|Pushed to stable

--- Comment #33 from Fridolin Somers  ---
Pushed to 23.05.x for 23.05.03

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #32 from Tomás Cohen Arazi  ---
Pushed to master for 23.11.

Nice work everyone, thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Tomás Cohen Arazi  changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Michelle Spinney  changed:

   What|Removed |Added

 CC||mspin...@clamsnet.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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Martin Renvoize  changed:

   What|Removed |Added

 QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe
   |y.org   |.com

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Martin Renvoize  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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #31 from Marcel de Rooy  ---
(In reply to Marcel de Rooy from comment #30)
> timethis 500:  1 wallclock secs ( 0.64 usr +  0.06 sys =  0.70 CPU) @
> 714.29/s (n=500)
> timethis 500:  1 wallclock secs ( 0.73 usr +  0.02 sys =  0.75 CPU) @
> 666.67/s (n=500)
> timethis 500:  1 wallclock secs ( 0.67 usr +  0.03 sys =  0.70 CPU) @
> 714.29/s (n=500)
> timethis 500:  1 wallclock secs ( 0.72 usr +  0.04 sys =  0.76 CPU) @
> 657.89/s (n=500)

Not from this patch. But we should catch the output.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #30 from Marcel de Rooy  ---
timethis 500:  1 wallclock secs ( 0.64 usr +  0.06 sys =  0.70 CPU) @ 714.29/s
(n=500)
timethis 500:  1 wallclock secs ( 0.73 usr +  0.02 sys =  0.75 CPU) @ 666.67/s
(n=500)
timethis 500:  1 wallclock secs ( 0.67 usr +  0.03 sys =  0.70 CPU) @ 714.29/s
(n=500)
timethis 500:  1 wallclock secs ( 0.72 usr +  0.04 sys =  0.76 CPU) @ 657.89/s
(n=500)

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Marcel de Rooy  changed:

   What|Removed |Added

 CC||m.de.r...@rijksmuseum.nl

--- Comment #29 from Marcel de Rooy  ---
Having a look here too today

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #28 from Martin Renvoize  ---
Created attachment 153642
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153642=edit
Bug 32271: (QA follow-up) Restore $Price filtering for valid values

Signed-off-by: Martin Renvoize 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #27 from Martin Renvoize  ---
Created attachment 153641
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153641=edit
Bug 32271: (QA follow-up) Correction to 'can_be_blank' logic

The logic introducing the can_be_blank check into the monetary decimal
format check was flawed and meant we were no longer checking decimal
formatting in the majority of cases. This patch updates that so we pass
the unit tests and correctly check any value passed that's not an empty
string.

Signed-off-by: Martin Renvoize 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #26 from Martin Renvoize  ---
Created attachment 153640
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153640=edit
Bug 32271: Unit test

Add a unit test for the allow blank option added to the monetary check
in CirculationRules.

Signed-off-by: Martin Renvoize 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #153489|0   |1
is obsolete||

--- Comment #25 from Martin Renvoize  ---
Created attachment 153639
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153639=edit
Bug 32271: (QA follow-up) Convert all positive numbers to monetary float and
convert all 0 equivilents to blank value

Signed-off-by: Katrin Fischer 
Signed-off-by: Martin Renvoize 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #153488|0   |1
is obsolete||

--- Comment #24 from Martin Renvoize  ---
Created attachment 153638
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153638=edit
Bug 32271: Fix overdue fines cap (amount) set to 0.00 when editing rule.

Signed-off-by: Anneli Österman 
Signed-off-by: Katrin Fischer 
Signed-off-by: Martin Renvoize 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #23 from Katrin Fischer  ---
Now we have the following behaviour:

overdue fines cap  = ofd

* ofd empty on editing, stays empty after saving
* ofd 0.00 on editing, will be empty after saving
* ofd changed to 0.00, will be empty after saving
* ofd changed to 0, will be empty after saving

I believe with bug 34279 that will give us some nice consistency in behaviour
again.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=34279

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Attachment #144528|0   |1
is obsolete||

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Attachment #153487|0   |1
is obsolete||

--- Comment #22 from Katrin Fischer  ---
Created attachment 153489
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153489=edit
Bug 32271: (QA follow-up) Convert all positive numbers to monetary float and
convert all 0 equivilents to blank value

Signed-off-by: Katrin Fischer 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #21 from Katrin Fischer  ---
Created attachment 153488
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153488=edit
Bug 32271: Fix overdue fines cap (amount) set to 0.00 when editing rule.

Signed-off-by: Anneli Österman 
Signed-off-by: Katrin Fischer 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Attachment #153486|0   |1
is obsolete||

--- Comment #20 from Katrin Fischer  ---
Created attachment 153487
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153487=edit
Bug 32271: (QA follow-up) Convert all positive numbers to monetary float and
convert all 0 equivilents to blank value

Signed-off-by: Katrin Fischer 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off
   Patch complexity|--- |Small 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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #19 from Kyle M Hall  ---
Created attachment 153486
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153486=edit
Bug 32271: (QA follow-up) Convert all positive numbers to monetary float and
convert all 0 equivilents to blank value

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Kyle M Hall  changed:

   What|Removed |Added

 Status|NEW |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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #18 from Katrin Fischer  ---
I have the same result as Kyle:

Exception 'Koha::Exceptions::BadParameter' thrown 'set_rule expected decimal'
 at /usr/share/perl5/Exception/Class/Base.pm line 88

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #17 from Kyle M Hall  ---
When I enter "0" with this patch, I get the error:
Exception 'Koha::Exceptions::BadParameter' thrown 'set_rule expected decimal'
 at /usr/share/perl5/Exception/Class/Base.pm line 88

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 CC||k...@bywatersolutions.com

--- Comment #16 from Katrin Fischer  ---
*** Bug 34279 has been marked as a duplicate of this bug. ***

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #15 from Katrin Fischer  ---
It  would be great to get more eyes on this one. We are now nearly there with
bug 33028 and I think this is the missing piece to get us back on track again.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Status|REOPENED|NEW

--- Comment #14 from Katrin Fischer  ---
I don't believe the patch attached fully works. I made this change on bug 33028
and I still see the 0.00. We need to dig a little deeper.

But I am splitting off this bug from 33028 again, because I hope to get that
resolved as it's almost there. 

Both patch sets would need backporting later.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #13 from Katrin Fischer  ---
To summarize (and because I already had that typed up for the duplicate bug:

* In 20.11 (already using circulation_rules), if you left overdue fines cap
empty, it remained empty on saving. This meant that fines would be calculated
without a limit or only limited by the MaxFine system preference.

* In master and other later versions we have started to convert empty values on
saving to 0.00 which means: no fines will calculate.

This is especially an issue for older databases that have the empty values and
unknowingly change them when editing other values in circulation rules. ->
chaos.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 CC||mishrav...@gmail.com

--- Comment #12 from Katrin Fischer  ---
*** Bug 32905 has been marked as a duplicate of this bug. ***

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Depends on||33028


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33028
[Bug 33028] Wrongly formatted monetary amounts in circulation rules break
scripts and calculations
-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Katrin Fischer  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|DUPLICATE   |---

--- Comment #11 from Katrin Fischer  ---
(In reply to Martin Renvoize from comment #10)
> I think bug 33028 is going in the right direction here... marking this one
> as a duplicate.
> 
> *** This bug has been marked as a duplicate of bug 33028 ***

It's actually not. The problem remains. I am deduplicating.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Martin Renvoize  changed:

   What|Removed |Added

 Status|In Discussion   |RESOLVED
 CC||martin.renvoize@ptfs-europe
   ||.com
 Resolution|--- |DUPLICATE

--- Comment #10 from Martin Renvoize  ---
I think bug 33028 is going in the right direction here... marking this one as a
duplicate.

*** This bug has been marked as a duplicate of bug 33028 ***

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-02-27 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

Patrick Robitaille  changed:

   What|Removed |Added

 CC||patrick.robitaille@collecto
   ||.ca

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-02-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

--- Comment #9 from Blou  ---
We are open to anything but this needs some direction.  If two QAs put the bz
on pause to discuss, and a third put it "In Discussion", we mere mortals can't
add much.  

Our team can code this many ways, but we need a patch to get through.  Sooner
than later.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-02-21 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Status|Signed Off  |In Discussion

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-02-09 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

he...@bywatersolutions.com changed:

   What|Removed |Added

 CC||he...@bywatersolutions.com

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #8 from Katrin Fischer  ---
I think the problem is not fixed in the right spot. Displaying the fine with
comma is one thing, but the problem appears on editing. When the rule values
are copied to the inputs for editing, they need to be "unformatted".

Does that make sense?

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #7 from Jonathan Druart  ---
There is also recall_overdue_fine, but it should not be a problem as it's not a
max.
However, shouldn't we revert the changes made to bug 20395 and restore the
formatting instead of completely removing it?

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Jonathan Druart  changed:

   What|Removed |Added

 Depends on||20395
 CC||jonathan.druart+koha@gmail.
   ||com


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20395
[Bug 20395] Use Price formatter in more templates (paycollect, request, parcel,
smart-rules)
-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Tomás Cohen Arazi  changed:

   What|Removed |Added

 CC||tomasco...@gmail.com
   Severity|normal  |major

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-02-01 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

Hans Pålsson  changed:

   What|Removed |Added

 CC||hans.pals...@hkr.se

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

2023-01-30 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271

Katrin Fischer  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |geraud.frapp...@inlibro.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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Noémie Labine  changed:

   What|Removed |Added

 CC||noemie.lab...@collecto.ca

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Anneli Österman  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off
 CC||anneli.osterman@koha-suomi.
   ||fi

--- Comment #6 from Anneli Österman  ---
I tested with system preference CalculateFinesOnReturn.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

PTFS Europe Sandboxes  changed:

   What|Removed |Added

 Attachment #144430|0   |1
is obsolete||

--- Comment #5 from PTFS Europe Sandboxes  ---
Created attachment 144528
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144528=edit
BUG 32271 : Fix overdue fines cap (amount) set to 0.00 when editing rule.

Signed-off-by: Anneli Österman 

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #4 from geraud.frapp...@inlibro.com ---
Created attachment 144430
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144430=edit
BUG 32271 : Fix overdue fines cap (amount) set to 0.00 when editing rule.

To test:

0. Preliminary settings
   0.1. finesMode system preference must be set to Calculate and charge
   0.2. MaxFine system preference must be empty
   0.3. In Tools > Calendar, today must not be a holiday

1. Add a circulation rule with fines but no fines cap
   1.1. Go to Administration > Circulation and fines rules
   1.2. Add or modify a circulation rule, make sure
-  There is a quantity in "Current checkouts allowed"
-  There is a number of days in "Loan period"
-  There is an amount in "Fine amount"
-  There is a number in "Fine charging interval" (usually 1)
-  "Overdue fines cap (amount)" is **empty**
   1.3. Save the circulation rule
   --> Note that the amount in "Overdue fines cap (amount)" is displayed as
0.00

2. Checkout an item with a date in the past to create an overdue
   2.1. Find an item barcode
2.1.1. Go to Item search
2.2.2. Click Search
2.2.3. Copy a barcode from the search results
   2.2. Go to a patron's file
2.2.1. Go to Patrons
2.2.2. Click on "Search"
2.2.3. Click on a patron's name
   2.3. Click on the "Check out" tab on the left of the patron's file
   2.4. Click on the settings icon in the "Enter item barcode" field
   2.5. Pick a due date in the past
   2.6. Paste the item barcode in the field
   2.7. Click "Check out" or press Enter
   2.8. Click "Yes, check out"
   --> Patron should have an overdue

3. Run fines.pl
   3.1. In a terminal, run

   ./misc/cronjobs/fines.pl

4. Check that the patron has fines
   4.1. Go to the patron's file
   4.2. Click on the "Accounting" tab on the left
   --> Patron should have a fine (accruing)

5. Modify the circulation rule, change any field except "Overdue fines cap
(amount)" (let's say the quantity of current checkouts has gone up)
   5.1. Go to Administration > Circulation and fines rules
   5.2. Click "Edit" next to the rule created in step 1
   --> Note that the amount in "Overdue fines cap (amount)" is 0.00
   5.3. Change any setting except "Overdue fines cap (amount)", e.g. change
Current checkouts allowed to another number
   5.4. Click "Save"
   --> Note that the amount in "Overdue fines cap (amount)" is displayed as
0.00

Apply patch : 
6. Redo step 2

7. Redo step 3

8. Redo step 4
--> Note that the patron has a fine

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

geraud.frapp...@inlibro.com changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Caroline Cyr La Rose  changed:

   What|Removed |Added

 CC||philippe.blo...@inlibro.com

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #3 from Caroline Cyr La Rose  ---
(In reply to Sara Brown from comment #2)
> The same thing is happening on 21.11.11. However, it isn't (consistently?)
> preventing patrons from accruing fines for overdue items - I'm seeing
> patrons who are accruing fines as they should be even though the applicable
> rule in the matrix shows the 0.00 fine cap.

Hi Sara,

If you didn't edit the rule, the matrix will show 0.00, but in the database,
the overdue fines cap is actually empty. So your patrons will still accrue (?)
fines normally.

Only beware if you need to edit the rule, make sure to empty the overdue fines
cap field.

Caroline

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Sara Brown  changed:

   What|Removed |Added

 CC||sbr...@bywatersolutions.com

--- Comment #2 from Sara Brown  ---
The same thing is happening on 21.11.11. However, it isn't (consistently?)
preventing patrons from accruing fines for overdue items - I'm seeing patrons
who are accruing fines as they should be even though the applicable rule in the
matrix shows the 0.00 fine cap.

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

Caroline Cyr La Rose  changed:

   What|Removed |Added

   Severity|enhancement |normal

-- 
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 32271] Overdue fines cap (amount) set to 0.00 when editing rule

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

--- Comment #1 from Caroline Cyr La Rose  ---
I noticed this as far back as 22.05 (did not check 21.11, but 21.05 is OK).

-- 
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/