Hi Klaus,

Thanks for the detailed report and fix! I've rolled this into the date picker and made a new release (2.1.1) with the fixed code:

http://jquery.com/plugins/project/datePicker

Thanks again,

Kelvin :)

Klaus Hartl wrote:
Hi,

I just came across a hard to find bug in Opera 9.23 (Win), that made
the datepicker not work properly. In short, Opera failed on number-
only keys in objects, even if you convert the number to a string*.

Thus I made some changes to use the date as a string itself, which
worked properly with Firefox 2, Opera 9.23, Safari 2 & 3, IE 6 & 7...

Changes were made in the selected date getter and setter and is
methods:


*** setSelected:

this.selectedDates[d.getTime()] = v;

=> this.selectedDates[d.toString()] = v;


*** isSelected:

return this.selectedDates[t];

=> return this.selectedDates[d.toString()];

(exchanged argument t with d to indicate a date)


getSelected:

r.push(new Date(Number(t)));

=> r.push(new Date(Date.parse(s)));

(exchanged t with s to indicate a string)


* did not work: d.getTime() + ''
  did work: d.getTime() + '_foo'


--Klaus

Reply via email to