Bug 20008

Summary: Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr
Product: Koha Reporter: Kyle M Hall <kyle>
Component: PatronsAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, gmcharlt, jonathan.druart, kyle.m.hall, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20178
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20008 - Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr
Bug 20008 - Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr
Bug 20008 - Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr
Bug 20008: Fix restrictions added from memberentry.pl if TimeFormat is 12hr

Description Kyle M Hall 2018-01-17 19:29:26 UTC
To recreate:
1) Set TimeFormat to 12 hour
2) Add a restriction with an expiration date via memberentry.pl
3) Note the restriction exists, but has no expiration date
Comment 1 Kyle M Hall 2018-01-17 19:31:19 UTC
Created attachment 70635 [details] [review]
Bug 20008 - Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr

Test Plan:
1) Set TimeFormat to 12 hour
2) Add a restriction with an expiration date via memberentry.pl
3) Note the restriction exists, but has no expiration date
4) Apply this patch
5) Repeat step 2
6) Note the restriction exists and has an expiration date!
Comment 2 Kyle M Hall 2018-01-17 19:40:32 UTC
Created attachment 70636 [details] [review]
Bug 20008 - Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr

Test Plan:
1) Set TimeFormat to 12 hour
2) Add a restriction with an expiration date via memberentry.pl
3) Note the restriction exists, but has no expiration date
4) Apply this patch
5) Repeat step 2
6) Note the restriction exists and has an expiration date!
Comment 3 Roch D'Amour 2018-02-09 20:58:56 UTC
Created attachment 71401 [details] [review]
Bug 20008 - Restrictions added from memberentry.pl have expiration date ignored if TimeFormat is 12hr

Test Plan:
1) Set TimeFormat to 12 hour
2) Add a restriction with an expiration date via memberentry.pl
3) Note the restriction exists, but has no expiration date
4) Apply this patch
5) Repeat step 2
6) Note the restriction exists and has an expiration date!

Signed-off-by: Roch D'Amour <roch.damour@inlibro.com>
Comment 4 Katrin Fischer 2018-02-11 15:38:14 UTC
Created attachment 71405 [details] [review]
Bug 20008: Fix restrictions added from memberentry.pl if TimeFormat is 12hr

Test Plan:
1) Set TimeFormat to 12 hour
2) Add a restriction with an expiration date via memberentry.pl
3) Note the restriction exists, but has no expiration date
4) Apply this patch
5) Repeat step 2
6) Note the restriction exists and has an expiration date!

Signed-off-by: Roch D'Amour <roch.damour@inlibro.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2018-02-12 18:50:00 UTC
Related to bug 20178
Comment 6 Jonathan Druart 2018-02-12 19:07:45 UTC
It seems that we must be the module instead.

Kyle can you take a look at bug 20178 and tell me if the patch fixes your issue?

*** This bug has been marked as a duplicate of bug 20178 ***
Comment 7 Jonathan Druart 2018-02-28 17:37:50 UTC
I will reevaluate this later but I think we should provide a global solution, see bug 20178.
Comment 8 Jonathan Druart 2018-04-04 18:49:05 UTC
This patch is not perfect but fixes the problem.

I still think it must be fixed globally.
Comment 9 Jonathan Druart 2018-04-04 19:08:12 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 10 Fridolin Somers 2018-07-18 14:19:21 UTC
Pushed to 17.11.x for 17.11.08