Bug 4900 - sys pref useDaysMode misses option Datedue
Summary: sys pref useDaysMode misses option Datedue
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: System Administration (show other bugs)
Version: rel_3_2
Hardware: All All
: PATCH-Sent (DO NOT USE) critical
Assignee: Nicole C. Engard
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-21 15:26 UTC by Katrin Fischer
Modified: 2019-06-27 09:24 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
patch (1.57 KB, patch)
2010-06-21 18:33 UTC, Nicole C. Engard 		Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2010-06-21 15:26:01 UTC 
The system preference useDaysMode has 3 options:

- days
- calendar
- datedue

The new sys pref editor is missing datedue.


- pref: useDaysMode
 choices:
 Days: Include
 Calendar: "Don't include"
- days when the library in closed when calculating due dates.

I checked the code, it seems the option is still present. There is code in calendar.pm sub addDate, which is used by Circulation.pm and sub CalcDateDue.

Our libraries currently use the option datedue for calculation of due dates.
Comment 1 Nicole C. Engard 2010-06-21 18:33:01 UTC 
Created attachment 2257 [details] [review]
patch
Comment 2 Galen Charlton 2010-06-24 02:33:07 UTC 
Patch pushed.  Please test and close.