Repository: wicket
Updated Branches:
  refs/heads/master 7006771da -> b6db8d977


Fix the condition check after adding "if (logger.isDebugEnabled())"


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/b6db8d97
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/b6db8d97
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/b6db8d97

Branch: refs/heads/master
Commit: b6db8d97761bcc640d2e03500ffd283c833c21db
Parents: 7006771
Author: Martin Tzvetanov Grigorov <[email protected]>
Authored: Wed Feb 26 11:41:50 2014 +0200
Committer: Martin Tzvetanov Grigorov <[email protected]>
Committed: Wed Feb 26 11:41:50 2014 +0200

----------------------------------------------------------------------
 .../org/apache/wicket/markup/html/form/FormComponent.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/b6db8d97/wicket-core/src/main/java/org/apache/wicket/markup/html/form/FormComponent.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/FormComponent.java
 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/FormComponent.java
index d5eea24..2b2ae49 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/FormComponent.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/FormComponent.java
@@ -1655,14 +1655,14 @@ public abstract class FormComponent<T> extends 
LabeledWebMarkupContainer impleme
                        }
                        catch (Exception noSetter)
                        {
-                               if (modified && logger.isDebugEnabled())
+                               if (!modified)
                                {
-                                       logger.debug("An error occurred while 
trying to set the collection attached to "
+                                       throw new WicketRuntimeException("An 
error occurred while trying to set the collection attached to "
                                                        + formComponent, 
noSetter);
                                }
-                               else
+                               else if (logger.isDebugEnabled())
                                {
-                                       throw new WicketRuntimeException("An 
error occurred while trying to set the collection attached to "
+                                       logger.debug("An error occurred while 
trying to set the collection attached to "
                                                        + formComponent, 
noSetter);
                                }
                        }

Reply via email to