Bug 7261

Summary: System Preference to select the first day of week (Sunday or Monday) to use in the calendar (OPAC and Intranet)
Product: Koha Reporter: Juan Romay Sieira <juan.sieira>
Component: System AdministrationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P4 CC: chris, gmcharlt, m.de.rooy, nengard, paul.poulain
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 9166    
Bug Blocks:    
Attachments: Patch
Signed-off patch
Patch to apply in current master
Bug 7261 System Preference to select the first day of week
Followup to make tests consistent

Description Juan Romay Sieira 2011-11-24 16:50:06 UTC Comment hidden (obsolete)
Comment 1 Owen Leonard 2011-12-06 16:03:11 UTC Comment hidden (obsolete)
Comment 2 Marcel de Rooy 2012-01-04 15:17:07 UTC
Just a few questions/remarks before finishing QA:

1 Would it not be better to move this pref to Internationalization tab instead of Administration?
2 I would recommend to make the two tests on the preference in Auth.pm identical. Currently, it works, but things can change.
3 Is opac-reserve template the only one needing the firstDay parameter on the opac side? I notice that at the staff side the same change is in a common calendar-setup file. 

For the rest, things look very good to me.
Could you clarify or send a follow up for the points above? Thanks.
Comment 3 Owen Leonard 2012-01-04 15:41:34 UTC
> 3 Is opac-reserve template the only one needing the firstDay parameter on the
> opac side? I notice that at the staff side the same change is in a common
> calendar-setup file. 

I noticed the same thing, and submitted a follow-up to Bug 2307.
Comment 4 Marcel de Rooy 2012-01-13 15:16:47 UTC
Changing status to reflect pending state. See comment 2.
Comment 5 Juan Romay Sieira 2012-02-09 18:15:31 UTC Comment hidden (obsolete)
Comment 6 Juan Romay Sieira 2012-02-09 18:19:41 UTC
(In reply to comment #2)
> Just a few questions/remarks before finishing QA:
> 
> 1 Would it not be better to move this pref to Internationalization tab instead
> of Administration?
I changed it to the tab you mentioned above.

> 2 I would recommend to make the two tests on the preference in Auth.pm
> identical. Currently, it works, but things can change.
> 3 Is opac-reserve template the only one needing the firstDay parameter on the
> opac side? I notice that at the staff side the same change is in a common
> calendar-setup file. 
I did a search over the opac templates and I only found it in opac-reserve.tt

> 
> For the rest, things look very good to me.
> Could you clarify or send a follow up for the points above? Thanks.
Comment 7 Chris Cormack 2012-03-03 07:55:52 UTC
Created attachment 7999 [details] [review]
Bug 7261 System Preference to select the first day of week

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Comment 8 Marcel de Rooy 2012-03-12 14:29:32 UTC
Created attachment 8183 [details] [review]
Followup to make tests consistent
Comment 9 Marcel de Rooy 2012-03-12 14:30:52 UTC
Marking it as Passed QA, while adding a followup to make the two tests in Auth.pm the same.
Trivial merge conflict in sysprefs.sql
Comment 10 Paul Poulain 2012-03-14 13:19:16 UTC
I'm not sure this syspref won't be deprecated by the datepicker change to switch to a jquery datepicker, but until then... patch pushed !
Comment 11 Owen Leonard 2012-03-14 13:26:26 UTC
In fact we may be able to tie the configuration of the datepicker to the current localization settings. There are a lot of options built in:

http://jqueryui.com/demos/datepicker/#localization
Comment 12 Nicole C. Engard 2012-03-14 19:56:25 UTC
This says it was pushed to master, but changing this preference's value in master doesn't seem to do anything for me ....
Comment 13 Owen Leonard 2012-03-14 20:02:18 UTC
Nicole, try clearing your browser cache. The patch includes changes to js files which might need to be refreshed.
Comment 14 Nicole C. Engard 2012-03-14 20:25:45 UTC
Bingo! and good tip to include in the manual.

Nicole
Comment 15 Owen Leonard 2012-11-27 13:45:05 UTC
Working in the staff client, not on opac-reserve.pl.
Comment 16 Owen Leonard 2013-05-15 14:31:19 UTC
Looks like this is working in master.