Hello Jermoe,

Thanks for you bug report and fix (in the future however, please use
Bugzilla for this, as then it can be tracked by others as well). I will take
a look and make the necessary fix in the coming build(s).

Best regards,
Emil

On Wed, Feb 4, 2009 at 2:29 PM, Jérôme Combes <
[email protected]> wrote:

> Hello,
>
> I'am using the CalendarCombo widget and I have a bug with it and I have a
> solution to correct this problem. (See the attached file).
>
> The bug is the following. If I select in the calendar popup 'Today', then
> 'None', then 'Today', I have the following exception (I added a "try ...
> catch" to see this exception) :
> java.lang.StringIndexOutOfBoundsException: String index out of range: -1
>   at java.lang.String.<init>(String.java:202)
>   at org.eclipse.swt.internal.win32.TCHAR.toString(TCHAR.java:115)
>   at org.eclipse.swt.widgets.Combo.getText(Combo.java:846)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarCombo.setText(CalendarCombo.java:1127)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarCombo.updateDate(CalendarCombo.java:1566)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarCombo.access$20(CalendarCombo.java:1565)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarCombo$17.dateChanged(CalendarCombo.java:1461)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarComposite.notifyListeners(CalendarComposite.java:732)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarComposite.notifyListeners(CalendarComposite.java:745)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarComposite.clickedTodayButton(CalendarComposite.java:214)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarComposite.externalClick(CalendarComposite.java:770)
>   at
> org.eclipse.nebula.widgets.calendarcombo.CalendarCombo$9.handleEvent(CalendarCombo.java:839)
>
> I have this exception because in the class 'CalendarCombo' the method
> 'Combo.setText(String )' is used to set the text in the Combo. I think that
> the method 'Combo.add(...)' (like in the method 'CalendarCombosetText(...)')
> must be always used to set the text.
> In attached file, there is the CalendarCombo with a solution : add a method
> 'setComboText(String )' to set the text for the Combo (flat or not) and I am
> using only this method in the CalendarCombo class.
>
> Jérôme
>
>
>
_______________________________________________
nebula-dev mailing list
[email protected]
https://dev.eclipse.org/mailman/listinfo/nebula-dev

Reply via email to