Re: [vote] release Wicket 1.4.14

2010-11-24 Thread Andrea Del Bene
+1 for release. I can confirm that memory leak bug is solved. bye! This vote is to release wicket 1.4.14. This is a bugfix release on the 1.4.x (stable) branch. Branch: http://svn.apache.org/repos/asf/wicket/branches/wicket-1.4.14/ Artifacts:

Re: [vote] release Wicket 1.4.14

2010-11-24 Thread Steve Swinsburg
+1 cheers, Steve On 24/11/2010, at 2:22 PM, Jeremy Thomerson wrote: This vote is to release wicket 1.4.14. This is a bugfix release on the 1.4.x (stable) branch. Branch: http://svn.apache.org/repos/asf/wicket/branches/wicket-1.4.14/ Artifacts:

Re: [vote] change ibehavior from interface to abstract class in trunk

2010-11-24 Thread Johan Compagner
+1 On Wed, Nov 24, 2010 at 03:22, Igor Vaynberg igor.vaynb...@gmail.com wrote: the ibehavior interface has become somewhat cluttered and a lot of methods in it can have a nice default implementation that works for 99% of the usecases. there is no longer a point to having it as an interface.

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Igor Vaynberg
i added boolean canCallListenerInterface(Component component) to ibehavior -igor On Wed, Nov 24, 2010 at 12:54 AM, Johan Compagner jcompag...@gmail.com wrote: which method name? I currently only have a tagging interface that tags this behavior... On Wed, Nov 24, 2010 at 03:00, Igor Vaynberg

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Johan Compagner
yes that i did see in 1.5 but in 1.4 i guess the only thing we can do is just introduce that IIgnoreDisabledComponentBehavior interface (that doesnt have any method) (or something the same with a different name, any recomendations?) johan On Wed, Nov 24, 2010 at 16:57, Igor Vaynberg

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Martijn Dashorst
IWontDoAnythingWhenTheComponentIsDisabledBehavior IMASlackerBehavior Martijn On Wed, Nov 24, 2010 at 5:12 PM, Johan Compagner jcompag...@gmail.com wrote: yes that i did see in 1.5 but in 1.4 i guess the only thing we can do is just introduce that IIgnoreDisabledComponentBehavior interface

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Pedro Santos
Why don't simple change the AbstractBehavior default isEnabled implementation to respect the component state, like: public boolean isEnabled(Component component) { return component.isEnabled(); } It would solve the problem at the ticket. IMO the proposed interface is prolix

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Johan Compagner
that first one is a perfect name, except that the intent is that it works just the opposite! :) On Wed, Nov 24, 2010 at 17:20, Martijn Dashorst martijn.dasho...@gmail.com wrote: IWontDoAnythingWhenTheComponentIsDisabledBehavior IMASlackerBehavior Martijn On Wed, Nov 24, 2010 at 5:12 PM,

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Igor Vaynberg
i dont think the name of the mixin matters much because 1.5 is out soonish. IListenerInterfaceTarget { canCallListenerInterface()} is not so bad. -igor On Wed, Nov 24, 2010 at 8:12 AM, Johan Compagner jcompag...@gmail.com wrote: yes that i did see in 1.5 but in 1.4 i guess the only thing we

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Johan Compagner
And then completely remove the check if the component is disabled i guess? That would work for me but then if users dont really call super() when they have there own enable logic that would mean it would always be called.. (because now it just returns true so people could think i dont need to

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Johan Compagner
ok then i check my changes in for 1.4 So that at least people that use 1.4 (like us) can use that one for now. I can add a note in the doc that this will be replaced by 1.5 completely.. On Wed, Nov 24, 2010 at 17:15, Igor Vaynberg igor.vaynb...@gmail.com wrote: i dont think the name of the

Re: disabled components/behaviors fixes of WICKET-3098

2010-11-24 Thread Pedro Santos
On Wed, Nov 24, 2010 at 2:24 PM, Johan Compagner jcompag...@gmail.comwrote: And then completely remove the check if the component is disabled i guess? exactly That would work for me but then if users dont really call super() when they have there own enable logic that would mean it would

Re: svn commit: r1038752 - in /wicket/branches/wicket-1.4.14/wicket/src: main/java/org/apache/wicket/util/io/WicketObjectOutputStream.java test/java/org/apache/wicket/resource/DummyPage.java test/java

2010-11-24 Thread Igor Vaynberg
no need to commit into 1.4.14 branch. the release has already been cut. if we have to recut it we would do it from 1.4.x branch. -igor On Wed, Nov 24, 2010 at 10:27 AM, pe...@apache.org wrote: Author: pedro Date: Wed Nov 24 18:27:33 2010 New Revision: 1038752 URL:

several tests are failing on trunk

2010-11-24 Thread Juergen Donnerstag
is it just me were several tests on trunk are failing? Juergen

getMarkupIdFromMarkup problem

2010-11-24 Thread Martin Grigorov
Hi, I just updated my app to latest 1.5 and the following exception happened: org.apache.wicket.WicketRuntimeException: No Page found for component [ShortcutsPanel [Component id = componentItem]] at org.apache.wicket.Component.getPage(Component.java:1787) at

Re: several tests are failing on trunk

2010-11-24 Thread Major Péter
Hi, for me this is the build output: Failed tests: testLicenseHeaders(org.apache.wicket.util.license.ApacheLicenceHeaderTest) Tests in error: testDisabledBehaviorRequest(org.apache.wicket.BehaviorRequestTest) testDisabled(org.apache.wicket.DisabledComponentTest)

Re: several tests are failing on trunk

2010-11-24 Thread Martin Grigorov
Same here. It's the new logic for disabled behaviors. Too late for me to take a look today... On Wed, Nov 24, 2010 at 10:00 PM, Juergen Donnerstag juergen.donners...@gmail.com wrote: is it just me were several tests on trunk are failing? Juergen

Re: getMarkupIdFromMarkup problem

2010-11-24 Thread Juergen Donnerstag
Excepted is: - if a component provides its own markup either via subclassing getMarkup() or is expected to have associated markup (e.g. Panel) = markup available in ctor - Component.onMarkupAttached() is invoked as soon as the markup is available. E.g. a child to a Panel. child.onMarkupAttached()