fix issue 81

diff --git a/gwtquery-core/src/main/java/com/google/gwt/query/client/GQuery.java b/gwtquery-core/src/main/java/com/google/gwt/query/client/GQuery.java
index f2b58c460ed5680b7039732a79302b865a304f2c..78e405d2de1c702b241eca5bc3ff82cb88162965 100644 (file)
--- a/gwtquery-core/src/main/java/com/google/gwt/query/client/GQuery.java
+++ b/gwtquery-core/src/main/java/com/google/gwt/query/client/GQuery.java
@@ -569,8 +569,11 @@ public class GQuery implements Lazy<GQuery, LazyGQuery> {
    */\r
   public GQuery addClass(String... classes) {\r
     for (Element e : elements) {\r
-      for (String clz : classes) {\r
-        e.addClassName(clz);\r
+      //issue 81 : ensure that the element is an Element node.\r
+      if (Element.is(e)){\r
+        for (String clz : classes) {\r
+          e.addClassName(clz);\r
+        }\r
       }\r
     }\r
     return this;\r
@@ -3024,8 +3027,10 @@ public class GQuery implements Lazy<GQuery, LazyGQuery> {
    */\r
   public GQuery removeClass(String... classes) {\r
     for (Element e : elements) {\r
-      for (String clz : classes) {\r
-        e.removeClassName(clz);\r
+      if (Element.is(e)){\r
+        for (String clz : classes) {\r
+          e.removeClassName(clz);\r
+        }\r
       }\r
     }\r
     return this;\r

diff --git a/gwtquery-core/src/test/java/com/google/gwt/query/client/GQueryCoreTest.java b/gwtquery-core/src/test/java/com/google/gwt/query/client/GQueryCoreTest.java
index eb6b36dc9e7eb0a28a7d25bb8304fa7ede05e303..653dd93e521708f1f03c33b59ae5e4d09b1baf4d 100644 (file)
--- a/gwtquery-core/src/test/java/com/google/gwt/query/client/GQueryCoreTest.java
+++ b/gwtquery-core/src/test/java/com/google/gwt/query/client/GQueryCoreTest.java
@@ -1639,5 +1639,12 @@ public class GQueryCoreTest extends GWTTestCase {
       assertEquals($(selector).toString(), $(xselector).toString());
     }
   }
+  
+  public void testIssue81(){
+    GQuery test = $("     <div>blop</div><!-- comment --> <p>test2</p>    ");
+    test.addClass("test");
+    test.removeClass("test");
+    
+  }
 
 }