Button: Callback method selected randomly when multiple methods match

[Artur Signell]

{
Button b = new Button("Testbutton", this, "method"); 
}

public void method() {
    method("no parameter");
}

public void method(String parameter) {
    System.out.println("Method called with "+parameter);
}

->

IllegalArgumentException? is thrown from public ListenerMethod?(Class eventType, Object object, String methodName) if method(String) is selected to be the callback method (depends on what the result of

for (int i = 0; i < methods.length; i++)
			if (methods[i].getName().equals(methodName))
				method = methods[i];

is). Changing order of method() methods in source file possibly gives different results.

[Joonas Lehtinen]

IMO throwing exception when registering listener is the correct way to go

[Marc Englund]

Suggestion: remove all string-reflect -type constructors & addListener -methods. Really. They are error prone, and prevent refactoring. In-line (anonymous) listeners are trivial to create and errors are caught at compile-time:

new Button("Close", new Button.ClickListener() {
            public void buttonClick(ClickEvent event) {
               close();
            }
        });

[Jani Laakso]

Fixed, using reflect methods for listeners is now deprecated

[Joonas Lehtinen]

Milestone User Interface Library 5.1.2 deleted