| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 08787 | Bug reports | Survey taking | public | 2014-02-28 21:16 | 2014-03-02 12:29 |
| Reporter | bkgray | Assigned To | mfaber | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 2.05+ | ||||
| Fixed in Version | 2.05+ | ||||
| Summary | 08787: problems with date.js - DatePicker min/max limits do not consider timezone | ||||
| Description | In file scripts/date.js, the values for minDate and maxDate Picker options are not calculated correctly for time zones other than GMT. For -0600, a minimum date string of "2014-02-17" creates a Date object with value 2014-02-16 18:00. The maxDate value is also off by 6 hours, so the allowable dates are shifted by 1 day (see attached image). Also in date.js, in doPopusDate() there is a call to setPickerOptions without the required argument. | ||||
| Tags | No tags attached. | ||||
| Bug heat | 8 | ||||
| Complete LimeSurvey version number (& build) | 2.05+ 140226 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | Chrome | ||||
| Database type & version | MySql 5.6.14 | ||||
| Server OS (if known) | Win 2008 Server R2 | ||||
| Webserver software & version (if known) | Apache 2.4.7 | ||||
| PHP Version | 5.5.6 | ||||
 |
bkgray: I can see the problem in your screenshot but I was not able to replicate here. I changed two lines in date.js, so the calculation of mindate and maxdate is not so much dependent on the implementation in browsers. Could you replace your date.js with the version attached, clear your browser cache and see if this works for you? |
 |
bkgray, do you have set a time difference in global settings? |
|
|
...and if it does not work, please also attach your test case (.lss file). Thanks, M |
|
|
c_schmitz: the time difference is set to 0. The server time and calculated time displayed are correct for my time zone. mfaber: Thanks for the quick response - and the quick fix. Your modified version of date.js solves the problem for me. Since you weren't able to reproduce the problem, I'll attach the .lss file anyway. I'm seeing some odd behavior with time validation on the pop-up, but I haven't spent enough time with it yet. Could be code problems - but it could be my error, too. |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13952 |
|
|
bkgray: thanks for the feedback. Patch is part of next release. |
|
|
2.0+ Build 140302 released. |
