You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2017/04/07 13:31:13 UTC

[30/50] [abbrv] groovy git commit: More useful error message in DOMCategory.replaceNode method

More useful error message in DOMCategory.replaceNode method

Related to change in PR #516


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

Branch: refs/heads/parrot
Commit: 293e1790fec7a446de763c42e1dd1243c0218d11
Parents: 97a1a48
Author: John Wagenleitner <jw...@apache.org>
Authored: Sun Mar 26 11:40:24 2017 -0700
Committer: John Wagenleitner <jw...@apache.org>
Committed: Sun Mar 26 11:40:24 2017 -0700

----------------------------------------------------------------------
 .../groovy-xml/src/main/java/groovy/xml/dom/DOMCategory.java    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/293e1790/subprojects/groovy-xml/src/main/java/groovy/xml/dom/DOMCategory.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-xml/src/main/java/groovy/xml/dom/DOMCategory.java b/subprojects/groovy-xml/src/main/java/groovy/xml/dom/DOMCategory.java
index d2f0d6d..71fb4d3 100644
--- a/subprojects/groovy-xml/src/main/java/groovy/xml/dom/DOMCategory.java
+++ b/subprojects/groovy-xml/src/main/java/groovy/xml/dom/DOMCategory.java
@@ -355,7 +355,10 @@ public class DOMCategory {
 
     public static Node replaceNode(NodesHolder self, Closure c) {
         if (self.getLength() <= 0 || self.getLength() > 1) {
-            throw new GroovyRuntimeException("replaceNode() can only be used to replace a single element.");
+            throw new GroovyRuntimeException(
+                    "replaceNode() can only be used to replace a single element, " +
+                    "but was applied to " + self.getLength() + " elements."
+            );
         }
         return replaceNode(self.item(0), c);
     }