You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@wicket.apache.org by Dipu <di...@googlemail.com> on 2009/02/16 11:00:04 UTC

Problem in Wicket Stuff - Jquery - DatePicker

In Wicket Stuff - Jquery - DatePicker - DatePickerBehavior

the following line in the onBind() method

 if (component instanceof ITextFormatProvider) {
            	format_ = ((ITextFormatProvider) component).getTextFormat();

should be

 if (component instanceof ITextFormatProvider) {
            	format_ = ((ITextFormatProvider)
component).getTextFormat().toLowerCase();


i have created a patch, can some one apply the patch please.


regards
dipu

Re: Problem in Wicket Stuff - Jquery - DatePicker

Posted by Dipu <di...@googlemail.com>.
Cheers

Dipu



On Tue, Feb 17, 2009 at 12:30 PM, Martin Grigorov <mc...@e-card.bg> wrote:
> Ok, I see. The JS library works with lower case formats.
> Fixed with r4575.
>
> Additionally I think you are using the old jquery. The one I fixed is at
> https://wicket-stuff.svn.sourceforge.net/svnroot/wicket-stuff/trunk/wicketstuff-core/jquery-parent/jquery/src/main/java/org/wicketstuff/jquery/datepicker
>
>
> El mar, 17-02-2009 a las 11:21 +0000, Dipu escribió:
>> if i supply the date pattern "dd/MM/yyyy"
>> the date picker will print wrong date like 10/MM/2009
>>
>>  i noticed sdf.toPattern().toLowerCase() and i thought some one missed
>> to call toLowerCase()  on
>> format_ = ((ITextFormatProvider) component).getTextFormat();
>>
>> i called toLowerCase()  and it solved the issue, i didn't dig deep into it.
>>
>> regards
>> dipu
>>
>> On Tue, Feb 17, 2009 at 10:57 AM, Martin Grigorov <mc...@e-card.bg> wrote:
>> > I could do it but why it should be lower cased ?
>> > What will happen with months (MM)? They will become minutes (mm). Is
>> > this correct ?
>> >
>> > I see there is sdf.toPattern().toLowerCase() below but this seems like a
>> > bug to me.
>> >
>> > Can you give more details because I'm not using this behavior.
>> >
>> > El lun, 16-02-2009 a las 10:00 +0000, Dipu escribió:
>> >> In Wicket Stuff - Jquery - DatePicker - DatePickerBehavior
>> >>
>> >> the following line in the onBind() method
>> >>
>> >>  if (component instanceof ITextFormatProvider) {
>> >>               format_ = ((ITextFormatProvider) component).getTextFormat();
>> >>
>> >> should be
>> >>
>> >>  if (component instanceof ITextFormatProvider) {
>> >>               format_ = ((ITextFormatProvider)
>> >> component).getTextFormat().toLowerCase();
>> >>
>> >>
>> >> i have created a patch, can some one apply the patch please.
>> >>
>> >>
>> >> regards
>> >> dipu
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> >> For additional commands, e-mail: users-help@wicket.apache.org
>> >
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> > For additional commands, e-mail: users-help@wicket.apache.org
>> >
>> >
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> For additional commands, e-mail: users-help@wicket.apache.org
>>
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org


Re: Problem in Wicket Stuff - Jquery - DatePicker

Posted by Martin Grigorov <mc...@e-card.bg>.
Ok, I see. The JS library works with lower case formats.
Fixed with r4575.

Additionally I think you are using the old jquery. The one I fixed is at
https://wicket-stuff.svn.sourceforge.net/svnroot/wicket-stuff/trunk/wicketstuff-core/jquery-parent/jquery/src/main/java/org/wicketstuff/jquery/datepicker


El mar, 17-02-2009 a las 11:21 +0000, Dipu escribió:
> if i supply the date pattern "dd/MM/yyyy"
> the date picker will print wrong date like 10/MM/2009
> 
>  i noticed sdf.toPattern().toLowerCase() and i thought some one missed
> to call toLowerCase()  on
> format_ = ((ITextFormatProvider) component).getTextFormat();
> 
> i called toLowerCase()  and it solved the issue, i didn't dig deep into it.
> 
> regards
> dipu
> 
> On Tue, Feb 17, 2009 at 10:57 AM, Martin Grigorov <mc...@e-card.bg> wrote:
> > I could do it but why it should be lower cased ?
> > What will happen with months (MM)? They will become minutes (mm). Is
> > this correct ?
> >
> > I see there is sdf.toPattern().toLowerCase() below but this seems like a
> > bug to me.
> >
> > Can you give more details because I'm not using this behavior.
> >
> > El lun, 16-02-2009 a las 10:00 +0000, Dipu escribió:
> >> In Wicket Stuff - Jquery - DatePicker - DatePickerBehavior
> >>
> >> the following line in the onBind() method
> >>
> >>  if (component instanceof ITextFormatProvider) {
> >>               format_ = ((ITextFormatProvider) component).getTextFormat();
> >>
> >> should be
> >>
> >>  if (component instanceof ITextFormatProvider) {
> >>               format_ = ((ITextFormatProvider)
> >> component).getTextFormat().toLowerCase();
> >>
> >>
> >> i have created a patch, can some one apply the patch please.
> >>
> >>
> >> regards
> >> dipu
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> For additional commands, e-mail: users-help@wicket.apache.org
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> > For additional commands, e-mail: users-help@wicket.apache.org
> >
> >
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org


Re: Problem in Wicket Stuff - Jquery - DatePicker

Posted by Dipu <di...@googlemail.com>.
if i supply the date pattern "dd/MM/yyyy"
the date picker will print wrong date like 10/MM/2009

 i noticed sdf.toPattern().toLowerCase() and i thought some one missed
to call toLowerCase()  on
format_ = ((ITextFormatProvider) component).getTextFormat();

i called toLowerCase()  and it solved the issue, i didn't dig deep into it.

regards
dipu

On Tue, Feb 17, 2009 at 10:57 AM, Martin Grigorov <mc...@e-card.bg> wrote:
> I could do it but why it should be lower cased ?
> What will happen with months (MM)? They will become minutes (mm). Is
> this correct ?
>
> I see there is sdf.toPattern().toLowerCase() below but this seems like a
> bug to me.
>
> Can you give more details because I'm not using this behavior.
>
> El lun, 16-02-2009 a las 10:00 +0000, Dipu escribió:
>> In Wicket Stuff - Jquery - DatePicker - DatePickerBehavior
>>
>> the following line in the onBind() method
>>
>>  if (component instanceof ITextFormatProvider) {
>>               format_ = ((ITextFormatProvider) component).getTextFormat();
>>
>> should be
>>
>>  if (component instanceof ITextFormatProvider) {
>>               format_ = ((ITextFormatProvider)
>> component).getTextFormat().toLowerCase();
>>
>>
>> i have created a patch, can some one apply the patch please.
>>
>>
>> regards
>> dipu
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> For additional commands, e-mail: users-help@wicket.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org


Re: Problem in Wicket Stuff - Jquery - DatePicker

Posted by Martin Grigorov <mc...@e-card.bg>.
I could do it but why it should be lower cased ?
What will happen with months (MM)? They will become minutes (mm). Is
this correct ?

I see there is sdf.toPattern().toLowerCase() below but this seems like a
bug to me.

Can you give more details because I'm not using this behavior.

El lun, 16-02-2009 a las 10:00 +0000, Dipu escribió:
> In Wicket Stuff - Jquery - DatePicker - DatePickerBehavior
> 
> the following line in the onBind() method
> 
>  if (component instanceof ITextFormatProvider) {
>             	format_ = ((ITextFormatProvider) component).getTextFormat();
> 
> should be
> 
>  if (component instanceof ITextFormatProvider) {
>             	format_ = ((ITextFormatProvider)
> component).getTextFormat().toLowerCase();
> 
> 
> i have created a patch, can some one apply the patch please.
> 
> 
> regards
> dipu
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org