subject:"\[Koha\-bugs\] \[Bug 12063\] Change date calculation for reserve expiration to skip all holidays"

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2020-01-07 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Katrin Fischer  changed:

   What|Removed |Added

 CC||tmis...@gmail.com

--- Comment #92 from Katrin Fischer  ---
*** Bug 14307 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 12063] Change date calculation for reserve expiration to skip all holidays

2019-10-14 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Martin Renvoize  changed:

   What|Removed |Added

 Blocks||23809


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23809
[Bug 23809] Update to DB revision 16.12.00.032 fails: `Not a HASH reference at
/usr/share/koha/intranet/cgi-bin/installer/data/mysql/updatedatabase.pl line
14371.`
-- 
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 12063] Change date calculation for reserve expiration to skip all holidays

2019-08-05 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Jonathan Druart  changed:

   What|Removed |Added

 Blocks||23265


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23265
[Bug 23265] Update to DB revision 16.12.00.032 fails: Unknown column
'me.item_level_hold'
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2019-01-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #91 from Katrin Fischer  ---
*** Bug 12353 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
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2018-05-09 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Jonathan Druart  changed:

   What|Removed |Added

 Blocks||20724


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20724
[Bug 20724] ReservesNeedReturns syspref breaks "Holds awaiting pickup"
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-12-19 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Katrin Fischer  changed:

   What|Removed |Added

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

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-11-15 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Nick Clemens  changed:

   What|Removed |Added

 Blocks||19626


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19626
[Bug 19626] Database update for bug 12063 incorrectly calculates expirationdate
for holds
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-09-28 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Jonathan Druart  changed:

   What|Removed |Added

 Blocks||19260


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19260
[Bug 19260] Reservations / holds marked as problems being seen as expired ones
and deleted wrongly.
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-08-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marc Véron  changed:

   What|Removed |Added

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

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-14 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #90 from Katrin Fischer  ---
Also, what changes for libraries not using the cronjob to automatically expire
holds? 

We don't use this because libraries often want to leave the book on a few days
longer during holidays etc. and because the slip for the next patron is not
printed automatically and there is no clue for the library that they have to
print a new one.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-14 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #89 from Katrin Fischer  ---
I think it would be good to have the "Text to go in the release notes" filled
for this one, describing the change to the libraries updating.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-14 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Katrin Fischer  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|Pushed to Master|RESOLVED

--- Comment #88 from Katrin Fischer  ---
This won't get ported back to 16.11.x as it is an enhancement.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-09 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Kyle M Hall  changed:

   What|Removed |Added

 Status|Passed QA   |Pushed to Master
 CC||k...@bywatersolutions.com

--- Comment #87 from Kyle M Hall  ---
Pushed to master for 17.05, thanks Alex, Marcel!

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

--- Comment #86 from Marcel de Rooy  ---
Final QA Comment:
After a longer process, we reached Passed QA now. Added two small follow-ups.
One serious problem in the db rev fixed.

Still wanting to mention two points without triggering discussion again:
[1] The construction around expirationdate still feels little bit unappropriate
to me personally, but after some discussion we found a compromize.
[2] The change in Letters.pm is still arguable. Previously, expirationdate was
calculated here and could be used on a notice. Now, we do no longer need to
calculate it theoretically. But take e.g. the hold slip; the notice is
generated before the expiration date is set and therefore still empty.
I recommend to have a look at this problem on a new report. I do not consider
it as a blocker.

Thanks, Alex.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #85 from Marcel de Rooy  ---
Created attachment 63099
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63099=edit
Bug 12063: [QA Follow-up] Small change of two test scripts

Removing dbh from one script, changing rollback in the other.
Schema is leading now.

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #63057|0   |1
is obsolete||

--- Comment #83 from Marcel de Rooy  ---
Created attachment 63097
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63097=edit
Bug 12063 - Remove checking of ExpireReservesMaxPickUpDelay in
CancelExpiredReserves(). Koha::Hold::set_waiting calculate expiration date from
today instead of hold's waiting date.

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #84 from Marcel de Rooy  ---
Created attachment 63098
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63098=edit
Bug 12063: [QA Follow-up] Adjustments to db revision

Replace use by require.
Add require Koha::Calendar (seriously needed to prevent crashing).
Base expirationdate on today not on waitingdate (just as in Reserves).
Remove if ( waitingdate ) construction. Not needed anymore.
Remove $cancel_on_holidays. Not used. Note that the if construction around
the new pref is not strictly needed at upgrade time (pref still zero).
Fill the dbrev description.

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #62002|0   |1
is obsolete||

--- Comment #78 from Marcel de Rooy  ---
Created attachment 63092
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63092=edit
Bug 12063 - Remove Koha::Hold::waiting_expires_on since dateexpiration is set
on database

Signed-off-by: sonia BOUIS 

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #62004|0   |1
is obsolete||

--- Comment #80 from Marcel de Rooy  ---
Created attachment 63094
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63094=edit
Bug 12063 - Fix unit tests

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #62474|0   |1
is obsolete||

--- Comment #82 from Marcel de Rooy  ---
Created attachment 63096
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63096=edit
Bug 12063 - make perl atomic update follow skeleton.perl

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #62005|0   |1
is obsolete||

--- Comment #81 from Marcel de Rooy  ---
Created attachment 63095
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63095=edit
Bug 12063 - Fix QA failures

- Remove expiration date calculation in C4::Letter since it's done
when setting the reserve waiting,
  - remove expiration date calculation in circ/waitingreserves.pl. Use
the one in DB,
  - add a new atomic update that calculate expiration date for
waiting reserves,
  - add tests for days_foward function and fix the infinite loop.

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #62001|0   |1
is obsolete||

--- Comment #77 from Marcel de Rooy  ---
Created attachment 63091
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63091=edit
Bug 12063: Change date calculation for reserve expiration to skip all holiday

This patch makes koha automatically set expiration date when reserves become
waitting. Also it adds a new syspref "ExcludeHolidaysFromMaxPickUpDelay" that
allows to
take holidays into account while calculating expiration date.

Test plan:

  - Install this patch and run updatedatabase.pl script,
  - allow ExpireReservesMaxPickUpDelay in system preferences,
  - set ReservesMaxPickUpDelay to 5.

  - Place an hold on a checked out item and check in this item:
The hold's expiration date should be today + 5.

  - Allow ExcludeHolidaysFromMaxPickUpDelay in system preferences,
  - add holiday during this pickup delay period,
  - Create a new hold and make it comes waitting:
The hold's expiration date should be today + 5 + number of closed
day(s).

Also:
  - Check that ExpireReservesOnHolidays syspref works again
without ExcludeHolidaysFromMaxPickUpDelay.
  - Check that cancel fees apply again if wanted.

Signed-off-by: sonia BOUIS 

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

  Attachment #62003|0   |1
is obsolete||

--- Comment #79 from Marcel de Rooy  ---
Created attachment 63093
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63093=edit
Bug 12063 - Keep patron's requested expiration date if it is prior to the
calculated one

Signed-off-by: Andreas Hedström Mace 

Signed-off-by: Marcel de Rooy 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #76 from Alex Arnaud  ---
Created attachment 63057
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63057=edit
Bug 12063 - Remove checking of ExpireReservesMaxPickUpDelay in
CancelExpiredReserves(). Koha::Hold::set_waiting calculate expiration date from
today instead of hold's waiting date.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #75 from Marcel de Rooy  ---
(In reply to Alex Arnaud from comment #73)
> And note that CancelExpiredReserves is only caled by the script
> misc/cronjobs/holds/cancel_expired_holds.pl. Nowhere else if i'm not wrong.
> So if we remove the check of ExpireReservesMaxPickUpDelay from this sub, we
> get a kind of "force cancel reserves".

I do not understand. CancelExpiredReserves should just do what it says. Note
that this patch would change behavior otherwise.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #74 from Marcel de Rooy  ---
(In reply to Alex Arnaud from comment #72)
> (In reply to Marcel de Rooy from comment #71)
> > The situation I described is independent of this pref. You could use the
> > cron job to remove expired reserves based on expiration dates entered by
> > patrons.
> > 
> 
> Difficult to decide the issue. The behavior of sub CancelExpiredReserves was
> before:
>   - Cancel holds with an expiration date (asked by the patron afaik),
>   - Cancel other exipred holds only if ExpireReservesMaxPickUpDelay is set
> to allow
> 
> So now we have all waiting reerves with an expiration date, we can't make
> the difference between those with expiration ask by the patron and those
> with expiration calculated.

True, but not in all cases. If you disabled ExpireReservesMaxPickUpDelay, all
expired reserves must be 'patron-originated'.
But the question is: do we still need to make a distinction? The reserve has
expired for some reason, we can cancel it. We can just remove the line imo.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #73 from Alex Arnaud  ---
And note that CancelExpiredReserves is only caled by the script
misc/cronjobs/holds/cancel_expired_holds.pl. Nowhere else if i'm not wrong. So
if we remove the check of ExpireReservesMaxPickUpDelay from this sub, we get a
kind of "force cancel reserves".

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #72 from Alex Arnaud  ---
(In reply to Marcel de Rooy from comment #71)
> The situation I described is independent of this pref. You could use the
> cron job to remove expired reserves based on expiration dates entered by
> patrons.
> 

Difficult to decide the issue. The behavior of sub CancelExpiredReserves was
before:
  - Cancel holds with an expiration date (asked by the patron afaik),
  - Cancel other exipred holds only if ExpireReservesMaxPickUpDelay is set to
allow

So now we have all waiting reerves with an expiration date, we can't make the
difference between those with expiration ask by the patron and those with
expiration calculated.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #71 from Marcel de Rooy  ---
(In reply to Alex Arnaud from comment #69)
> (In reply to Marcel de Rooy from comment #68)
> >  sub CancelExpiredReserves {
> > +return unless C4::Context->preference("ExpireReservesMaxPickUpDelay");
> > This may have a unwanted side-effect. If we do not use the pickup delay, but
> > still want to cancel expired reserves. 
> 
> You mean: If ExpireReservesMaxPickUpDelay is set to don't allow ? If yes,
> this is because we don't want to cancel expired reserves right ? Or i don't
> understand the syspref.
The situation I described is independent of this pref. You could use the cron
job to remove expired reserves based on expiration dates entered by patrons.

> > sub set_waiting {
> > [...]
> > +if ( C4::Context->preference("ExcludeHolidaysFromMaxPickUpDelay") ) {
> > +$expirationdate = $calendar->days_forward(
> > dt_from_string($self->waitingdate), $max_pickup_delay );
> > +}
> > This may cause something weird. If you do a second 
> 
> Is it possible that the code of set_waiting (after tranferToDo) should be
> called 2 times for the same reserve ?
As mentioned, this is exceptional. But it can be easily simulated. Check this
book in today and perhaps by accident tomorrow (it was still on the desk or
something like that). Two separate checkins, two calls.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #70 from Alex Arnaud  ---
(In reply to Alex Arnaud from comment #69)
> (In reply to Marcel de Rooy from comment #68)

> > sub set_waiting {
> > [...]
> > +if ( C4::Context->preference("ExcludeHolidaysFromMaxPickUpDelay") ) {
> > +$expirationdate = $calendar->days_forward(
> > dt_from_string($self->waitingdate), $max_pickup_delay );
> > +}
> > This may cause something weird. If you do a second 
> 
> Is it possible that the code of set_waiting (after tranferToDo) should be
> called 2 times for the same reserve ?

Anyway it doesn't hurt to pass today to days_forward. So i'll fix it

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #69 from Alex Arnaud  ---
(In reply to Marcel de Rooy from comment #68)
> Still working on this one now.
> Preliminary results:
> 
>  sub CancelExpiredReserves {
> +return unless C4::Context->preference("ExpireReservesMaxPickUpDelay");
> This may have a unwanted side-effect. If we do not use the pickup delay, but
> still want to cancel expired reserves. 

You mean: If ExpireReservesMaxPickUpDelay is set to don't allow ? If yes, this
is because we don't want to cancel expired reserves right ? Or i don't
understand the syspref.

> 
> sub set_waiting {
> [...]
> +if ( C4::Context->preference("ExcludeHolidaysFromMaxPickUpDelay") ) {
> +$expirationdate = $calendar->days_forward(
> dt_from_string($self->waitingdate), $max_pickup_delay );
> +}
> This may cause something weird. If you do a second 

Is it possible that the code of set_waiting (after tranferToDo) should be
called 2 times for the same reserve ?

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-05-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #68 from Marcel de Rooy  ---
Still working on this one now.
Preliminary results:

 sub CancelExpiredReserves {
+return unless C4::Context->preference("ExpireReservesMaxPickUpDelay");
This may have a unwanted side-effect. If we do not use the pickup delay, but
still want to cancel expired reserves. It is no longer possible. (Note that
patrons may have entered expiration dates too.)
Not marking this as a blocker, since this is probably exceptional. Do you have
an easy fix or can you open a new report for it?

sub set_waiting {
[...]
+if ( C4::Context->preference("ExcludeHolidaysFromMaxPickUpDelay") ) {
+$expirationdate = $calendar->days_forward(
dt_from_string($self->waitingdate), $max_pickup_delay );
+}
This may cause something weird. If you do a second checkin while the
waitingdate is say yesterday, the waitingdate is set to today but the
expiration date will be based upon the old waitingdate. Again probably
exceptional, but still something to fix. I would opt here for basing the expiry
on today, since we reset the waitingdate too.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-20 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #67 from Alex Arnaud  ---
Created attachment 62474
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62474=edit
Bug 12063 - make perl atomic update follow skeleton.perl

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-20 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #66 from Marcel de Rooy  ---
Could you please have another look at comment60 for the two atomic update
files?

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #61735|0   |1
is obsolete||

--- Comment #65 from Alex Arnaud  ---
Created attachment 62005
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62005=edit
Bug 12063 - Fix QA failures

- Remove expiration date calculation in C4::Letter since it's done
when setting the reserve waiting,
  - remove expiration date calculation in circ/waitingreserves.pl. Use
the one in DB,
  - add a new atomic update that calculate expiration date for
waiting reserves,
  - add tests for days_foward function and fix the infinite loop.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #61461|0   |1
is obsolete||

--- Comment #64 from Alex Arnaud  ---
Created attachment 62004
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62004=edit
Bug 12063 - Fix unit tests

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #59162|0   |1
is obsolete||

--- Comment #62 from Alex Arnaud  ---
Created attachment 62002
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62002=edit
Bug 12063 - Remove Koha::Hold::waiting_expires_on since dateexpiration is set
on database

Signed-off-by: sonia BOUIS 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #61401|0   |1
is obsolete||

--- Comment #63 from Alex Arnaud  ---
Created attachment 62003
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62003=edit
Bug 12063 - Keep patron's requested expiration date if it is prior to the
calculated one

Signed-off-by: Andreas Hedström Mace 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #59161|0   |1
is obsolete||

--- Comment #61 from Alex Arnaud  ---
Created attachment 62001
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62001=edit
Bug 12063: Change date calculation for reserve expiration to skip all holiday

This patch makes koha automatically set expiration date when reserves become
waitting. Also it adds a new syspref "ExcludeHolidaysFromMaxPickUpDelay" that
allows to
take holidays into account while calculating expiration date.

Test plan:

  - Install this patch and run updatedatabase.pl script,
  - allow ExpireReservesMaxPickUpDelay in system preferences,
  - set ReservesMaxPickUpDelay to 5.

  - Place an hold on a checked out item and check in this item:
The hold's expiration date should be today + 5.

  - Allow ExcludeHolidaysFromMaxPickUpDelay in system preferences,
  - add holiday during this pickup delay period,
  - Create a new hold and make it comes waitting:
The hold's expiration date should be today + 5 + number of closed
day(s).

Also:
  - Check that ExpireReservesOnHolidays syspref works again
without ExcludeHolidaysFromMaxPickUpDelay.
  - Check that cancel fees apply again if wanted.

Signed-off-by: sonia BOUIS 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-04-07 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #60 from Marcel de Rooy  ---
perl t/db_dependent/Holds/CancelReserves.t
1..5
ok 1 - use C4::Reserves;
ok 2 - Reserve 1 should not be canceled.
not ok 3 - Reserve 2 should be canceled.
#   Failed test 'Reserve 2 should be canceled.'
#   at t/db_dependent/Holds/CancelReserves.t line 66.
#  got: 'Koha::Hold=HASH(0x905fce8)'
# expected: undef
ok 4 - Reserve 3 should not be canceled.
not ok 5 - Reserve 3 should be canceled.
#   Failed test 'Reserve 3 should be canceled.'
#   at t/db_dependent/Holds/CancelReserves.t line 101.
#  got: 'Koha::Hold=HASH(0x8f502b8)'
# expected: undef
# Looks like you failed 2 tests of 5.

bug_12063-define_expirationdate_for_waitting_reserves.perl
Please adjust conform skeleton file in this same directory (DBVersion test,
SetVersion and print)

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-30 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

--- Comment #59 from Alex Arnaud  ---
Patch above fix Marcel's QA comments. Also, i added some minor changes:

in the sub Koha::Hold::set_waiting:
 => Remove the test "if (
C4::Context->preference("ExpireReservesMaxPickUpDelay") )".
I think we want to calculate the expiration date even if
ExpireReservesMaxPickUpDelay is set to dont't allow

in C4::Reserves::CancelExpiredReserves:
return if ExpireReservesMaxPickUpDelay is set to don't allow. Do nothing

turn this BZ back to Signed off status. Tell me if the changes make i should
pass it to Needs Signoff again.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-30 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #58 from Alex Arnaud  ---
Created attachment 61735
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61735=edit
Bug 12063 - Fix QA failures

- Remove expiration date calculation in C4::Letter since it's done
when setting the reserve waiting,
  - remove expiration date calculation in circ/waitingreserves.pl. Use
the one in DB,
  - add a new atomic update that calculate expiration date for
waiting reserves,
  - add tests for days_foward function and fix the infinite loop.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #57 from Alex Arnaud  ---
Yup, good catch Marcel. Will fix that asap

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #55 from Marcel de Rooy  ---
About the atomic update (db revision):
You add the pref, but we should also take into account all waiting reserves now
and look at/calculate their expiration date.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #56 from Marcel de Rooy  ---
New sub days_foward:
Please add a unit test for this routine. In Calendar.t? Or Holidays.t?
Also note that the while loop is very dangerous if you would pass a negative
number (infinite loop). Please check the value of the days.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #54 from Marcel de Rooy  ---
Also please look here in circ/waitingreserves.pl
( $waiting_year, $waiting_month, $waiting_day ) =
  Add_Delta_Days( $waiting_year, $waiting_month, $waiting_day,
$max_pickup_delay);
my $calcDate = Date_to_Days( $waiting_year, $waiting_month, $waiting_day );
This date is compared with today and holds may be cancelled. We should use the
new expirationdate here too. 
An interesting aspect is this one too in the same script: 
$max_pickup_delay-- if C4::Context->preference('ExpireReservesMaxPickUpDelay');
This is kind of strange btw. You should either use the delay or not use it.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #53 from Marcel de Rooy  ---
Please look at this code in C4/Letters.pm: 
if ( $table eq 'reserves' && $values->{'waitingdate'} ) {
my @waitingdate = split /-/, $values->{'waitingdate'};

$values->{'expirationdate'} = '';
if ( C4::Context->preference('ReservesMaxPickUpDelay') ) {
my $dt = dt_from_string();
$dt->add( days => C4::Context->preference('ReservesMaxPickUpDelay')
);
$values->{'expirationdate'} = output_pref( { dt => $dt, dateonly =>
1 } );
}
This code should be adjusted too in line with the changes in
$hold->set_waiting. Actually, if there is a waiting date, we must assume now
that it is no longer needed to calculate an expiration date. In the above code
the holiday was not respected either.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #52 from Marcel de Rooy  ---
Well, holds patches are not simple at least. Submitting a few comments
collected so far.
Do not give up. We are improving on the way.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-23 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #51 from Marcel de Rooy  ---
QA: Looking here now

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #50 from Alex Arnaud  ---
It should be ok now (Sorry Marcel and ths for the review)

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #61440|0   |1
is obsolete||

--- Comment #49 from Alex Arnaud  ---
Created attachment 61461
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61461=edit
Bug 12063 - Fix unit tests

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #48 from Andreas Hedström Mace  ---
(In reply to Marcel de Rooy from comment #44)
> (In reply to Andreas Hedström Mace from comment #41)
> > And yes that is the same settings I was using. Since it works for both of
> > you, I'm going to assume that either it is an issue with my devbox or I made
> > a mistake while testing. Going to sign off.
> Andreas:
> This is not the best introduction for a signoff btw :)

Haha, yes, sorry about that. Just wanted to mentioned that I looked at this
(cancelation fee) again and got it to work properly. It was an issue with
date/time on debian, where it was not correct set.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #47 from Marcel de Rooy  ---
Still something wrong?

1..11
ok 1 - use C4::Reserves;
Violation of unique constraint in Letter at
/usr/share/koha/masterclone/t/lib/TestBuilder.pm line 202.
ok 2 - Waiting date should be set to today
ok 3 - Expiration date should be set to today + 6
ok 4 - Reserve status is now "waiting"
ok 5 - Priority should be 0
ok 6 - Item number should be set correctly
not ok 7 - 2nd reserve - Reserve status is now "To transfer"
#   Failed test '2nd reserve - Reserve status is now "To transfer"'
#   at t/db_dependent/Holds/WaitingReserves.t line 175.
#  got: 'W'
# expected: 'T'
ok 8 - 2nd reserve - Priority should be 0
ok 9 - 2nd reserve - Item number should be set correctly
ok 10 - Expiration date should be set to today + 7
ok 11 - Requested expiration date should be kept
# Looks like you failed 1 test of 11.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #46 from Alex Arnaud  ---
Created attachment 61440
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61440=edit
Bug 12063 - Fix unit tests

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-22 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

 Status|Failed QA   |Signed Off

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Signed Off  |Failed QA

--- Comment #45 from Marcel de Rooy  ---
Prelimary results (QA can be an iterative process):

perl t/db_dependent/Holds/CancelReserves.t
1..5
ok 1 - use C4::Reserves;
ok 2 - Reserve 1 should not be canceled.
ok 3 - Reserve 2 should be canceled.
Can't locate object method "get_instance" via package "Koha::Cache" at
t/db_dependent/Holds/CancelReserves.t line 81.
# Looks like your test exited with 255 just after 3.

perl t/db_dependent/Holds/WaitingReserves.t
1..11
ok 1 - use C4::Reserves;
ok 2 - Waiting date should be set to today
ok 3 - Expiration date should be set to today + 6
ok 4 - Reserve status is now "waiting"
ok 5 - Priority should be 0
ok 6 - Item number should be set correctly
ok 7 - 2nd reserve - Reserve status is now "To transfer"
ok 8 - 2nd reserve - Priority should be 0
ok 9 - 2nd reserve - Item number should be set correctly
not ok 10 - Expiration date should be set to today + 7
#   Failed test 'Expiration date should be set to today + 7'
#   at t/db_dependent/Holds/WaitingReserves.t line 218.
#  got: '2017-03-27'
# expected: '2017-03-29'
ok 11 - Requested expiration date should be kept
# Looks like you failed 1 test of 11.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #44 from Marcel de Rooy  ---
(In reply to Andreas Hedström Mace from comment #41)
> And yes that is the same settings I was using. Since it works for both of
> you, I'm going to assume that either it is an issue with my devbox or I made
> a mistake while testing. Going to sign off.
Andreas:
This is not the best introduction for a signoff btw :)

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Andreas Hedström Mace  changed:

   What|Removed |Added

  Attachment #59165|0   |1
is obsolete||

--- Comment #42 from Andreas Hedström Mace  ---
Created attachment 61401
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61401=edit
Bug 12063 - Keep patron's requested expiration date if it is prior to the
calculated one

Signed-off-by: Andreas Hedström Mace 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #43 from Marcel de Rooy  ---
Ah great. I will be looking again at this one very soon.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Andreas Hedström Mace  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Andreas Hedström Mace  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

--- Comment #41 from Andreas Hedström Mace  ---
Thanks for testing Alex and Sonia!

And yes that is the same settings I was using. Since it works for both of you,
I'm going to assume that either it is an issue with my devbox or I made a
mistake while testing. Going to sign off.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #40 from Koha Team Lyon 3  ---
hello Andreas,
I have tested one more time but I can't reproduce your issue.

you said : "one might want to exclude holidays but still apply a fee for those
who doesn’t pick up a reserved book"

I try with those syspref : 
ExcludeHolidaysFromMaxPickUpDelay = allow
ExpireReservesMaxPickUpDelay= allow
ExpireReservesMaxPickUpDelayCharge=1
ReservesMaxPickUpDelay=1

That seems to be the setting translation of your sentence.
After launching misc/cronjobs/holds/cancel_expired_holds.pl, holds expired are
canceled and a fee of 1 € is charged to borrowers.

Perhaps, I haven't really understood what is the problem..


(I assume that the script misc/cronjobs/holds/cancel_expired_holds.pl will have
no effect if ExpireReservesMaxPickUpDelay is set to "don't allow")

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-03-20 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #39 from Alex Arnaud  ---
(In reply to Andreas Hedström Mace from comment #37)
> Tested and everything works as intended, except that it is not possible to
> cancel holds with the cronjob in
> misc/cronjobs/holds/cancel_expired_holds.pl, and thus not possible to set a
> cancelation fee.
> 
> However, with the ExpireReservesOnHolidays syspref set to allow, and without
> ExcludeHolidaysFromMaxPickUpDelay (set to don’t allow), the hold is
> cancelled and sets the fee. 
> 
> I don't think this is the desired behaviour (one might want to exclude
> holidays but still apply a fee for those who doesn’t pick up a reserved
> book), so I'm setting this to patch doesn't apply.
> 
> If someone disagrees, feel free to change the status.

Hello Andreas,

I didn't managed to reproduce this issue.
I have an expired hold. ExpireReservesOnHolidays is set to "don't allow".
ExcludeHolidaysFromMaxPickUpDelay is set to "allow", i'm not on a holiday
period and ExpireReservesMaxPickUpDelayCharge is set to 11.

Launch misc/cronjobs/holds/cancel_expired_holds.pl:
  1) hold is canceled,
  2) related borrower is charged

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-02-13 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Andreas Hedström Mace  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Failed QA

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-02-13 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #38 from Marcel de Rooy  ---
(In reply to Andreas Hedström Mace from comment #37)
> Tested and everything works as intended, except that it is not possible to
> cancel holds with the cronjob in
> misc/cronjobs/holds/cancel_expired_holds.pl, and thus not possible to set a
> cancelation fee.
> 
> However, with the ExpireReservesOnHolidays syspref set to allow, and without
> ExcludeHolidaysFromMaxPickUpDelay (set to don’t allow), the hold is
> cancelled and sets the fee. 
> 
> I don't think this is the desired behaviour (one might want to exclude
> holidays but still apply a fee for those who doesn’t pick up a reserved
> book), so I'm setting this to patch doesn't apply.
> 
> If someone disagrees, feel free to change the status.

Sounds like a Failed QA.
Patch doesnt apply is meant for patches needing a rebase.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-02-13 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Andreas Hedström Mace  changed:

   What|Removed |Added

 Status|Needs Signoff   |Patch doesn't apply

--- Comment #37 from Andreas Hedström Mace  ---
Tested and everything works as intended, except that it is not possible to
cancel holds with the cronjob in misc/cronjobs/holds/cancel_expired_holds.pl,
and thus not possible to set a cancelation fee.

However, with the ExpireReservesOnHolidays syspref set to allow, and without
ExcludeHolidaysFromMaxPickUpDelay (set to don’t allow), the hold is cancelled
and sets the fee. 

I don't think this is the desired behaviour (one might want to exclude holidays
but still apply a fee for those who doesn’t pick up a reserved book), so I'm
setting this to patch doesn't apply.

If someone disagrees, feel free to change the status.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-18 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #59163|0   |1
is obsolete||

--- Comment #36 from Alex Arnaud  ---
Created attachment 59165
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59165=edit
Bug 12063 - Keep patron's requested expiration date if it is prior to the
calculated one

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-18 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

 QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl
   |y.org   |

--- Comment #35 from Marcel de Rooy  ---
I would like to QA this patch set. Setting QA contact

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-18 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

 Status|In Discussion   |Needs Signoff

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-18 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #53100|0   |1
is obsolete||

--- Comment #32 from Alex Arnaud  ---
Created attachment 59161
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59161=edit
Bug 12063: Change date calculation for reserve expiration to skip all holiday

This patch makes koha automatically set expiration date when reserves become
waitting. Also it adds a new syspref "ExcludeHolidaysFromMaxPickUpDelay" that
allows to
take holidays into account while calculating expiration date.

Test plan:

  - Install this patch and run updatedatabase.pl script,
  - allow ExpireReservesMaxPickUpDelay in system preferences,
  - set ReservesMaxPickUpDelay to 5.

  - Place an hold on a checked out item and check in this item:
The hold's expiration date should be today + 5.

  - Allow ExcludeHolidaysFromMaxPickUpDelay in system preferences,
  - add holiday during this pickup delay period,
  - Create a new hold and make it comes waitting:
The hold's expiration date should be today + 5 + number of closed
day(s).

Also:
  - Check that ExpireReservesOnHolidays syspref works again
without ExcludeHolidaysFromMaxPickUpDelay.
  - Check that cancel fees apply again if wanted.

Signed-off-by: sonia BOUIS 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-18 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #34 from Alex Arnaud  ---
Created attachment 59163
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59163=edit
Bug 12063 - Keep patron's requested expiration date if it is prior to the
calculated one

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-18 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #53101|0   |1
is obsolete||

--- Comment #33 from Alex Arnaud  ---
Created attachment 59162
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59162=edit
Bug 12063 - Remove Koha::Hold::waiting_expires_on since dateexpiration is set
on database

Signed-off-by: sonia BOUIS 

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-17 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #31 from Andreas Hedström Mace  ---
Brilliant! I hope this is doable, and we are also able to test and (possibly)
sign off!

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-17 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #30 from Koha Team Lyon 3  ---
Hurrah! We have an agreement ;-)
Is it difficult to add this verification to the patch ?
We can re-test it and it can pursue his way toward the "Pushed to master"
status...

[in aparte, I'm not totally sure that it's really understandable for a user..
a borrower who said that he doesn't need his hold after friday 20th January
(for exemple), and this item is available the thursday 19th january with a
pickup delay of 3 days (rules of the library. In our Library, this what we send
in hold notice). The item will return in stacks the 20th January instead of
monday the 23th. 
We would see after this patch how our users react and we could open a new
ticket if necessary]

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-17 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #29 from Marcel de Rooy  ---
(In reply to Alex Arnaud from comment #28)
> 
> Hello Sonia,
> 
> I'm not comfortable with having two expirationdate fields. And i think we
> can spare ourselves from adding a new field by doing what Marcel suggests
> previously:
>   override the expiration date only if the one requested by the patron is
> before the calculated one.
> 
> My two cents

Yeah, sounds better :)

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-17 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #28 from Alex Arnaud  ---
(In reply to Koha Team Lyon 3 from comment #27)
> Hello,
> I'm afraid that we are mired in a never ending discussion.
> For me, as I have already sayd, there's no reason to keep expirationday
> after the hold is available and waiting to be picked up, as this date is
> meaning for users "I don't need this hold if it isn't available for me after
> this date".
> 
> But, I can see that it bothers Marcel so I propose that we try to find new
> solutions to help this feature to be pushed in Koha (because it's really
> important for many libraries).
> 
> So, I don't know if it's valuable but it seems to me that it could at least
> be simple, what do you think about creating a new field where we can stock a
> "pickupexpirationdate" or something like that ?
> 
> Marcel, Alex ? Is it possible ?

Hello Sonia,

I'm not comfortable with having two expirationdate fields. And i think we can
spare ourselves from adding a new field by doing what Marcel suggests
previously:
  override the expiration date only if the one requested by the patron is
before the calculated one.

My two cents

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-17 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #27 from Koha Team Lyon 3  ---
Hello,
I'm afraid that we are mired in a never ending discussion.
For me, as I have already sayd, there's no reason to keep expirationday after
the hold is available and waiting to be picked up, as this date is meaning for
users "I don't need this hold if it isn't available for me after this date".

But, I can see that it bothers Marcel so I propose that we try to find new
solutions to help this feature to be pushed in Koha (because it's really
important for many libraries).

So, I don't know if it's valuable but it seems to me that it could at least be
simple, what do you think about creating a new field where we can stock a
"pickupexpirationdate" or something like that ?

Marcel, Alex ? Is it possible ?

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Christopher Davis  changed:

   What|Removed |Added

 CC||cgda...@uintah.utah.gov

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Lisette  changed:

   What|Removed |Added

 CC||lisettesla...@gmail.com

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #26 from Alex Arnaud  ---
(In reply to Viccy Kemp from comment #25)
> I do not understand what Alex and Marcel are working on here. All I know is
> it appears that reserves do not take into account the dates when the library
> is closed. Today, the holds are showing they expire on January 16, but the
> library is closed that day and this is defined in the calendar. 
> WHY IS THIS NOT TAKEN INTO ACCOUNT ON THE EXPIRY DATE?

Hello Viccy,

The purpose of this patch is that the expiration date of reserves being
calculated one time when the item is checked in and that this date take into
account the pickup delay set by the library *AND* the potential closed days
during the period until the delay.

Let's admit:

  - a pickup delay set to 10 (days),
  - an item checked in at 01/01/2017,
  - so hold expires at 11/01/2017,
  - but library is closed from 04/01/2017 to 06/01/2017 (3 days),
  - if ExcludeHolidaysFromMaxPickUpDelay system preference is enabled, the
expiration date will be extended with these 3 days. So until 14/01/2017.

But, the case of holds expiring on a closed day is already handled in koha by
disabling ExpireReservesOnHolidays if i'm not wrong.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #25 from Viccy Kemp  ---
I do not understand what Alex and Marcel are working on here. All I know is it
appears that reserves do not take into account the dates when the library is
closed. Today, the holds are showing they expire on January 16, but the library
is closed that day and this is defined in the calendar. 
WHY IS THIS NOT TAKEN INTO ACCOUNT ON THE EXPIRY DATE?

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #24 from Alex Arnaud  ---
(In reply to Marcel de Rooy from comment #23)
> (In reply to Alex Arnaud from comment #22)
> > For me there is two case:
> > 
> > 1) The expiration date set by the patron is higher than the calculated
> > expiration date (that take into account pickup delay and holiday). In this
> > case, i think it is quite fair to overwrite the patron expiration date since
> > he is not supposed to keep the item waiting beyond the library delay. This
> > is what this patch does.
> > 
> > 2) The patron expiration date is before the calculated expiration date. As
> > the patron don't need the item after its own date, we could overwrite
> > expirationdate with the one calculated. We need a adjustment to have this
> > behavior.
> 
> As indicated before, I would not overwrite the expiration date.
> What about comment19 ?

Not sure to understand. (In reply to Marcel de Rooy from comment #19)
> (In reply to Andreas Hedström Mace from comment #18)
> > Without having actually tested this bug, only reading the discussion, I
> > would say that I have to agree with Sonia. 
> > 
> > As this new date is set after the material is checked in, it doesn't appear
> > (at least to me) that the expire date set by the patron would be valuable
> > anymore. And the added value of excluding dates that the library is closed
> > from the number of day in ReservesMaxPickUpDelay, seem greater than any
> > negative impact.
> 
> Hi Andreas,
> Thx for your feedback.
> I read it again too now. This patch tries to achieve its goal by modifying
> the expirationdate, while it actually should change this select in
> CancelExpiredReserves?
> 
> my $query = "SELECT * FROM reserves WHERE TO_DAYS( NOW() ) - TO_DAYS(
> waitingdate ) > ? AND found = 'W' AND priority = 0";
> 
> If the holiday stuff is taken into account here, I am having the impression
> that we achieve the same. Mabye Sonia or Alex can respond to that?

The code here is just simplified since the expirationdate (with max pickup
delay + holiday) has already been set at the check in time. So we just need to
fetch reserves whose expirationdate is over.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #23 from Marcel de Rooy  ---
(In reply to Alex Arnaud from comment #22)
> For me there is two case:
> 
> 1) The expiration date set by the patron is higher than the calculated
> expiration date (that take into account pickup delay and holiday). In this
> case, i think it is quite fair to overwrite the patron expiration date since
> he is not supposed to keep the item waiting beyond the library delay. This
> is what this patch does.
> 
> 2) The patron expiration date is before the calculated expiration date. As
> the patron don't need the item after its own date, we could overwrite
> expirationdate with the one calculated. We need a adjustment to have this
> behavior.

As indicated before, I would not overwrite the expiration date.
What about comment19 ?

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #22 from Alex Arnaud  ---
For me there is two case:

1) The expiration date set by the patron is higher than the calculated
expiration date (that take into account pickup delay and holiday). In this
case, i think it is quite fair to overwrite the patron expiration date since he
is not supposed to keep the item waiting beyond the library delay. This is what
this patch does.

2) The patron expiration date is before the calculated expiration date. As the
patron don't need the item after its own date, we could overwrite
expirationdate with the one calculated. We need a adjustment to have this
behavior.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-10 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #21 from Andreas Hedström Mace  ---
(In reply to Marcel de Rooy from comment #19)
> Hi Andreas,
> Thx for your feedback.
> I read it again too now. This patch tries to achieve its goal by modifying
> the expirationdate, while it actually should change this select in
> CancelExpiredReserves?
> 
> my $query = "SELECT * FROM reserves WHERE TO_DAYS( NOW() ) - TO_DAYS(
> waitingdate ) > ? AND found = 'W' AND priority = 0";
> 
> If the holiday stuff is taken into account here, I am having the impression
> that we achieve the same. Mabye Sonia or Alex can respond to that?

Any word from Sonia or Alex regarding this (the suggestion from Marcel?)? We
would very much like this to move forward, as is holiday is now creating
problems with holds waiting on the shelves.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-04 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Sally Healey  changed:

   What|Removed |Added

 CC||sally.healey@cheshireshared
   ||services.gov.uk

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2017-01-03 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Nick Clemens  changed:

   What|Removed |Added

 CC||n...@bywatersolutions.com

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-12-21 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Viccy Kemp  changed:

   What|Removed |Added

 CC||victoria.kemp@flower-mound.
   ||com

--- Comment #20 from Viccy Kemp  ---
Flower Mound Public Library has noticed that reserves DO NOT TAKE INTO ACCOUNT
closed holidays. What exactly is the point of marking up a calendar if the
software does not know what to do with it. That is nuts. We have been live on
Koha for four and a half months now and during both Thanksgiving and now
Christmas the closed days are not being accounted for in the holds. Please,
someone, figure out how to make this work.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-11-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #19 from Marcel de Rooy  ---
(In reply to Andreas Hedström Mace from comment #18)
> Without having actually tested this bug, only reading the discussion, I
> would say that I have to agree with Sonia. 
> 
> As this new date is set after the material is checked in, it doesn't appear
> (at least to me) that the expire date set by the patron would be valuable
> anymore. And the added value of excluding dates that the library is closed
> from the number of day in ReservesMaxPickUpDelay, seem greater than any
> negative impact.

Hi Andreas,
Thx for your feedback.
I read it again too now. This patch tries to achieve its goal by modifying the
expirationdate, while it actually should change this select in
CancelExpiredReserves?

my $query = "SELECT * FROM reserves WHERE TO_DAYS( NOW() ) - TO_DAYS(
waitingdate ) > ? AND found = 'W' AND priority = 0";

If the holiday stuff is taken into account here, I am having the impression
that we achieve the same. Mabye Sonia or Alex can respond to that?

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-11-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

   Patch complexity|--- |Small patch
   Assignee|koha-b...@lists.koha-commun |alex.arn...@biblibre.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
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-11-11 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Ola Andersson  changed:

   What|Removed |Added

 CC||ola.anders...@ltu.se

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-10-20 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Andreas Hedström Mace  changed:

   What|Removed |Added

 CC||andreas.hedstrom.mace@sub.s
   ||u.se

--- Comment #18 from Andreas Hedström Mace  ---
Without having actually tested this bug, only reading the discussion, I would
say that I have to agree with Sonia. 

As this new date is set after the material is checked in, it doesn't appear (at
least to me) that the expire date set by the patron would be valuable anymore.
And the added value of excluding dates that the library is closed from the
number of day in ReservesMaxPickUpDelay, seem greater than any negative impact.

-- 
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
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-08-29 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marjorie Barry-Vila  changed:

   What|Removed |Added

 CC||marjorie.barry-v...@ccsr.qc
   ||.ca

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-07-20 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #17 from Koha Team Lyon 3  ---
Hi Katrin, Hi Marcel,
>From what I have understood, but I may be wrong, patrons can put an
expirationdate to say that they're not interested in this reservation if it's
not available for them after this date.
Here, we change the date after the item was check-in, so available for the
patron. In this case, the first information isn't useful anymore.
I haven't tested if there's a change of behaviour if the patron isn't the first
in the holding list, but I don't think so.
>From a functional point of view, I don't think that's an issue to overwrite
expirationdate. 
But, It can be perhaps a problem if the library want to make statistics to know
an average of the time between the expirationdate and the waitingdate... or
some tricky things we sometimes wants to know.
And, I can understand that it may be not in the "coding line" and that's a
dirty way of doing... you can say that better than me.

Sonia

-- 
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
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-07-20 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #16 from Koha Team Lyon 3  ---
Hi Katrin, Hi Marcel,
For what I have understood, but I maybe wrong, patrons can put an
expirationdate to say that they're not interstesting in this reservation if
it's not available for them after this date.
Here, we change the date after the item was check-in, so available for the
patron. In this case, the first information isn't useful anymore.
I haven't tested if there's a change of behaviour if the patron isn't the first
in the holding list, but I don't think so.
For a functional point of view, I don't think that's an issue to overwrite
expirationdate. 
But, It can be perhaps a problem if the library want to make statistics to know
an average of the time between the expirationdate and the waitingdate... or
some tricky things we sometimes want to know.
And, I can undertstand that it m

-- 
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
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-07-19 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

--- Comment #15 from Katrin Fischer  ---
I wonder if it could be irritatng to the user to find the date entered changed. 
I am not sure if libraries print those in notices, but they show up in the
OPAC.

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

[Koha-bugs] [Bug 12063] Change date calculation for reserve expiration to skip all holidays

2016-07-19 Thread bugzilla-daemon

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12063

Marcel de Rooy  changed:

   What|Removed |Added

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

--- Comment #14 from Marcel de Rooy  ---
Hm. I am not in favor of blindly overwriting expiredate for reserves that
already have an expiry date, entered by the patron himself.

In this case we could probably make a compromise: if expire < today + delay
then leave it as-is, else expire = today + delay.
Although the else should not be necessary, the cronjob is intelligent too :)

This change is not dramatic, but I would prefer an adjustment.

-- 
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
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

1 2 >

1 - 100 of 118 matches

Mail list logo