svn commit: r634067 - /wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/behavior/AbstractAjaxBehavior.java

[ivaynberg]

Author: ivaynberg
Date: Wed Mar  5 14:38:08 2008
New Revision: 634067

URL: http://svn.apache.org/viewvc?rev=634067&view=rev
Log:
relaxed final

Modified:
    
wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/behavior/AbstractAjaxBehavior.java

Modified: 
wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/behavior/AbstractAjaxBehavior.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/behavior/AbstractAjaxBehavior.java?rev=634067&r1=634066&r2=634067&view=diff
==============================================================================
--- 
wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/behavior/AbstractAjaxBehavior.java
 (original)
+++ 
wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/behavior/AbstractAjaxBehavior.java
 Wed Mar  5 14:38:08 2008
@@ -32,9 +32,9 @@
  * @author Igor Vaynberg
  */
 public abstract class AbstractAjaxBehavior extends AbstractBehavior
-               implements
-                       IBehaviorListener,
-                       IHeaderContributor
+       implements
+               IBehaviorListener,
+               IHeaderContributor
 {
        /**
         * 
@@ -66,8 +66,8 @@
                if (component != null)
                {
                        throw new IllegalStateException("this kind of handler 
cannot be attached to " +
-                                       "multiple components; it is already 
attached to component " + component +
-                                       ", but component " + hostComponent + " 
wants to be attached too");
+                               "multiple components; it is already attached to 
component " + component +
+                               ", but component " + hostComponent + " wants to 
be attached too");
 
                }
 
@@ -82,6 +82,11 @@
         * 
         * @return the url that references this handler
         */
+       /*
+        * TODO 1.4 remove this and only keep the (boolean) variant. its a huge 
mess to have both
+        * because both need to be overridable and you never know which one an 
intermediary subclass
+        * overrides to add its behavior, or all subclasses must be made to 
override both :|
+        */
        public CharSequence getCallbackUrl()
        {
                return getCallbackUrl(true);
@@ -96,12 +101,12 @@
         * 
         * @return the url that references this handler
         */
-       public final CharSequence getCallbackUrl(final boolean 
onlyTargetActivePage)
+       public CharSequence getCallbackUrl(final boolean onlyTargetActivePage)
        {
                if (getComponent() == null)
                {
                        throw new IllegalArgumentException(
-                                       "Behavior must be bound to a component 
to create the URL");
+                               "Behavior must be bound to a component to 
create the URL");
                }
 
                final RequestListenerInterface rli;