[Koha-bugs] [Bug 37354] Bookings should respect circulation rules for max loan periods

2026-01-23 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks|39584   |


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39584
[Bug 39584] Booking post-processing time cuts into circulation period
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2026-01-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks|37363   |


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37363
[Bug 37363] Booking dates should respect closed days
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2026-01-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks|37618   |


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37618
[Bug 37618] Select patron's, then item's home library by default for pickup
location when creating bookings
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2025-10-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354
Bug 37354 depends on bug 34440, which changed state.

Bug 34440 Summary: Add warm-up and cool-down periods to bookings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34440

   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
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2025-07-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Thibaud Guillot (thibaud_g)  changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2025-05-21 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks||37363


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37363
[Bug 37363] Booking dates should respect closed days
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2025-05-14 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks||39584


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39584
[Bug 39584] Booking post-processing time cuts into circulation period
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2025-01-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Aude Charillon  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|Needs documenting   |RESOLVED

--- Comment #36 from Aude Charillon  ---
Included in the Koha Manual.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2025-01-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Aude Charillon  changed:

   What|Removed |Added

  Documentation||Aude Charillon
contact||
 CC||aude.charillon@ptfs-europe.
   ||com
  Documentation||https://gitlab.com/koha-com
 submission||munity/koha-manual/-/merge_
   ||requests/1009

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 CC||[email protected]
 Status|Pushed to main  |Needs documenting

--- Comment #35 from Lucas Gass (lukeg)  ---
Enhancement, no 24.05.x backport.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-19 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks||36271


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36271
[Bug 36271] Bookings should have circulation rules
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

  Text to go in the|This enhancement builds on  |This enhancement builds on
  release notes|the bookings functionality  |the bookings functionality
   |added in the last cycle. We |added in the last cycle. We
   |now prevent a booking from  |now prevent a booking from
   |exceeding the maximum   |exceeding the maximum
   |period laid out by the  |period laid out by the
   |circulation rules for that  |circulation rules for that
   |item.
  |item.
   |
   |
   |We highlight the|We highlight the
   |loan period and renewal |loan period and renewal
   |periods in the bookings |periods in the bookings
   |calendar upon selection of  |calendar upon selection of
   |the booking start date and  |the booking start date and
   |disallow bookings that  |disallow bookings that
   |exceed the loan period +|exceed the loan period +
   |renewal period * max|renewal period * max
   |renewals rules. |renewals rules.
   ||
   ||**Sponsored by** *PTFS
   ||Europe*

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

   Keywords|release-notes-needed|

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

  Text to go in the||This enhancement builds on
  release notes||the bookings functionality
   ||added in the last cycle. We
   ||now prevent a booking from
   ||exceeding the maximum
   ||period laid out by the
   ||circulation rules for that
   ||item.
   ||
   ||We highlight the
   ||loan period and renewal
   ||periods in the bookings
   ||calendar upon selection of
   ||the booking start date and
   ||disallow bookings that
   ||exceed the loan period +
   ||renewal period * max
   ||renewals rules.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-11 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Katrin Fischer  changed:

   What|Removed |Added

   Keywords|rel_24_11_candidate |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Katrin Fischer  changed:

   What|Removed |Added

 Version(s)||24.11.00
released in||
 Status|Passed QA   |Pushed to main

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #34 from Katrin Fischer  ---
Pushed for 24.11!

Well done everyone, thank you!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  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
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #33 from Martin Renvoize (ashimema) 
 ---
Created attachment 174279
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174279&action=edit
Bug 37354: (QA follow-up) Reset state on clear

We were missing a reset for flatPickr state when using the clear option
for flatPickr.

Signed-off-by: Martin Renvoize 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Attachment #173623|0   |1
is obsolete||

--- Comment #32 from Martin Renvoize (ashimema) 
 ---
Created attachment 174278
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174278&action=edit
Bug 37354: (QA follow-up) Reset periodPicker on rule change

This patch clears the periodPicker values when the applied circulation
rules change. This prevents selection of one booking period against one
set of rules and then a change of item to an itemtype where different
rules should apply but the old selected period remaining and thus 'rule
breaking' being allowed.

Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Attachment #173622|0   |1
is obsolete||

--- Comment #31 from Martin Renvoize (ashimema) 
 ---
Created attachment 174277
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174277&action=edit
Bug 37354: Fix circ rule selections

Only expose the bookings calendar picker input if we have enough detail
already defined to pick the corresponding circulation rules.

We currently use the item pickup location for selecting which rules to
apply, this corresponds to hard coding CircControl=ItemHomeLibrary +
ItemHomeOrHolding=holdingbranch for this use case. We can't easily
change this to match those rules dynamically without rethinking the 'Any
item' functionality and displays in the flatpickr.  We may revisit this
at some point in the future should that become a requirement.

Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Attachment #173621|0   |1
is obsolete||

--- Comment #30 from Martin Renvoize (ashimema) 
 ---
Created attachment 174276
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174276&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

Signed-off-by: Paul Derscheid 
Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Attachment #173620|0   |1
is obsolete||

--- Comment #29 from Martin Renvoize (ashimema) 
 ---
Created attachment 174275
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174275&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

Signed-off-by: Paul Derscheid 
Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Katrin Fischer  changed:

   What|Removed |Added

   Keywords||rel_24_11_candidate
 Status|Passed QA   |Failed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Katrin Fischer  changed:

   What|Removed |Added

   Keywords|rel_24_11_candidate |release-notes-needed

--- Comment #28 from Katrin Fischer  ---
I am sorry, but I found a glitch:

What I did:

* Made item bookable on item level:
http://localhost:8081/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=114&itemnumber=236#item236
* Itemtype is not bookable.
* Update all - all - all circulation rule to: 1 day loan period, 5 renewals of
1 day renewal period.
* Try to book it for Henry, pickup in Fairview, item type Books
* Select a start date, the dates after the calculated end date are greyed out.
* Realize you want a different date. Remove the current dates using X next to
field to clear.
* Click on calendar: dates are still locked.
* Close booking modal, reopen: dates are still locked.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-11-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Marie  changed:

   What|Removed |Added

 CC||[email protected]

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Katrin Fischer  changed:

   What|Removed |Added

   Keywords||rel_24_11_candidate

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #27 from Nick Clemens (kidclamp)  ---
Created attachment 173623
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173623&action=edit
Bug 37354: (QA follow-up) Reset periodPicker on rule change

This patch clears the periodPicker values when the applied circulation
rules change. This prevents selection of one booking period against one
set of rules and then a change of item to an itemtype where different
rules should apply but the old selected period remaining and thus 'rule
breaking' being allowed.

Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Nick Clemens (kidclamp)  changed:

   What|Removed |Added

 Attachment #172890|0   |1
is obsolete||
 Attachment #172891|0   |1
is obsolete||
 Attachment #173137|0   |1
is obsolete||
 Attachment #173210|0   |1
is obsolete||

--- Comment #24 from Nick Clemens (kidclamp)  ---
Created attachment 173620
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173620&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

Signed-off-by: Paul Derscheid 
Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #26 from Nick Clemens (kidclamp)  ---
Created attachment 173622
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173622&action=edit
Bug 37354: Fix circ rule selections

Only expose the bookings calendar picker input if we have enough detail
already defined to pick the corresponding circulation rules.

We currently use the item pickup location for selecting which rules to
apply, this corresponds to hard coding CircControl=ItemHomeLibrary +
ItemHomeOrHolding=holdingbranch for this use case. We can't easily
change this to match those rules dynamically without rethinking the 'Any
item' functionality and displays in the flatpickr.  We may revisit this
at some point in the future should that become a requirement.

Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #25 from Nick Clemens (kidclamp)  ---
Created attachment 173621
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173621&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

Signed-off-by: Paul Derscheid 
Signed-off-by: Nick Clemens 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Nick Clemens (kidclamp)  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
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-23 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #23 from Martin Renvoize (ashimema) 
 ---
Created attachment 173210
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173210&action=edit
Bug 37354: (QA follow-up) Reset periodPicker on rule change

This patch clears the periodPicker values when the applied circulation
rules change. This prevents selection of one booking period against one
set of rules and then a change of item to an itemtype where different
rules should apply but the old selected period remaining and thus 'rule
breaking' being allowed.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Blocks||37618


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37618
[Bug 37618] Select patron's, then item's home library by default for pickup
location when creating bookings
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #22 from Martin Renvoize (ashimema) 
 ---
Created attachment 173137
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173137&action=edit
Bug 37354: Fix circ rule selections

Only expose the bookings calendar picker input if we have enough detail
already defined to pick the corresponding circulation rules.

We currently use the item pickup location for selecting which rules to
apply, this corresponds to hard coding CircControl=ItemHomeLibrary +
ItemHomeOrHolding=holdingbranch for this use case. We can't easily
change this to match those rules dynamically without rethinking the 'Any
item' functionality and displays in the flatpickr.  We may revisit this
at some point in the future should that become a requirement.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-21 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 QA Contact|[email protected] |[email protected]
   |y.org   |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-17 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #21 from Martin Renvoize (ashimema) 
 ---
Just another rebase

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-17 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Attachment #170067|0   |1
is obsolete||

--- Comment #20 from Martin Renvoize (ashimema) 
 ---
Created attachment 172891
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172891&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

Signed-off-by: Paul Derscheid 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-10-17 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize (ashimema)  changed:

   What|Removed |Added

 Attachment #170066|0   |1
is obsolete||

--- Comment #19 from Martin Renvoize (ashimema) 
 ---
Created attachment 172890
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172890&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

Signed-off-by: Paul Derscheid 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-08-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #18 from Martin Renvoize  ---
(In reply to Sam Lau from comment #15)
> In testing I noticed that if you don't select an item and just leave it to
> the 'Any item' selection you can select whatever booking dates. Is the
> expected behaviour? If so I can sign-off.

I'm afraid I couldn't replicate this.. maybe I've fixed it already
inadvertently in the work I've continued on during bug 34440.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-08-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Paul Derscheid  changed:

   What|Removed |Added

 CC||[email protected]
 Status|Needs Signoff   |Signed Off

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-08-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Paul Derscheid  changed:

   What|Removed |Added

 Attachment #169686|0   |1
is obsolete||

--- Comment #17 from Paul Derscheid  ---
Created attachment 170067
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=170067&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

Signed-off-by: Paul Derscheid 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-08-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Paul Derscheid  changed:

   What|Removed |Added

 Attachment #169685|0   |1
is obsolete||

--- Comment #16 from Paul Derscheid  ---
Created attachment 170066
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=170066&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

Signed-off-by: Paul Derscheid 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-26 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Sam Lau  changed:

   What|Removed |Added

 CC||[email protected]

--- Comment #15 from Sam Lau  ---
In testing I noticed that if you don't select an item and just leave it to the
'Any item' selection you can select whatever booking dates. Is the expected
behavior? If so I can sign-off.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-26 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #169591|0   |1
is obsolete||

--- Comment #14 from Martin Renvoize  ---
Created attachment 169686
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169686&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-26 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #169590|0   |1
is obsolete||

--- Comment #13 from Martin Renvoize  ---
Created attachment 169685
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169685&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #12 from Martin Renvoize  ---
Rebased and ready for testing again.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #168964|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #169009|0   |1
is obsolete||

--- Comment #11 from Martin Renvoize  ---
Created attachment 169591
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169591&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #169008|0   |1
is obsolete||

--- Comment #10 from Martin Renvoize  ---
Created attachment 169590
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169590&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #9 from Thibaud Guillot  ---
I would test it today but there is just a little conflict with a fresh main
branch in koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #168967|0   |1
is obsolete||

--- Comment #8 from Martin Renvoize  ---
Created attachment 169009
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169009&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #168966|0   |1
is obsolete||

--- Comment #7 from Martin Renvoize  ---
Created attachment 169008
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169008&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #168965|0   |1
is obsolete||

--- Comment #6 from Martin Renvoize  ---
Created attachment 168967
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168967&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

Test plan
1) Repeat the test plan for the previous patch
2) Note that once you have selected your first date, that date will
   become bold, the date a 'Loan period' after the start date will also
   be bold and you'll also have bold dates for the number of 'Renewals
   allowed' at each 'Renewal period' beyond the initial 'Loan period'
   date.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #5 from Martin Renvoize  ---
Created attachment 168966
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168966&action=edit
Bug 37354: Set maxDate based on issuelength + (renewals * renewalperiod)

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

Test plan
1) Set an item in a test biblio to 'bookable' from the items tab.
2) Note the itemtype of the above item
3) Add circulation rules for the above item type, in particular the
   'Loan period', 'Renewals allowed' and 'Renewal period' options.
4) Go back to your biblio and 'Place booking'. Select a user, pickup
   location and item.
5) Open the booking dates flatpickr.
6) Select a start date.
7) Note that the date that is 'Loan period' + 'Renewals allowed' *
'Renewal period' is now greyed out and cannot be selected or any date
beyond it either.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #4 from Martin Renvoize  ---
Created attachment 168965
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168965&action=edit
Bug 37354: Add visual hint at usual loan periods

This patch adds a visual hint to the datepicker to highlight dates that
line up with normal loan and renewal lengths.

We use bold to highlight such dates.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Attachment #168950|0   |1
is obsolete||

--- Comment #3 from Martin Renvoize  ---
Created attachment 168964
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168964&action=edit
Bug 37354: Set maxDate based on issuelength + renewals * renewalperiod

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-13 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #2 from Martin Renvoize  ---
We'll likely want to also enforce this in the API controller level.. or perhaps
at the koha object level.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-13 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-13 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

--- Comment #1 from Martin Renvoize  ---
Created attachment 168950
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168950&action=edit
Bug 37354: Set maxDate based on issuelength + renewals * renewalperiod

This patch sets the flatpickr maxDate in the bookings range picker to
the startdate selected + issuelength + renewals * renewalperiod
circulation rules.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-13 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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 37354] Bookings should respect circulation rules for max loan periods

2024-07-13 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37354

Martin Renvoize  changed:

   What|Removed |Added

   Assignee|[email protected] |martin.renvoize@ptfs-europe
   |ity.org |.com

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
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/