You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by ga...@apache.org on 2010/12/28 14:38:58 UTC

svn commit: r1053348 - /shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/ImageResizeRewriter.java

Author: gagan
Date: Tue Dec 28 13:38:57 2010
New Revision: 1053348

URL: http://svn.apache.org/viewvc?rev=1053348&view=rev
Log:
Patch by satya3656 | Issue 3798045: Small fix to ImageResizeRewriter to make it backward compatible with java 1.5 | http://codereview.appspot.com/3798045/

Modified:
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/ImageResizeRewriter.java

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/ImageResizeRewriter.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/ImageResizeRewriter.java?rev=1053348&r1=1053347&r2=1053348&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/ImageResizeRewriter.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/ImageResizeRewriter.java Tue Dec 28 13:38:57 2010
@@ -84,7 +84,7 @@ public class ImageResizeRewriter extends
     }
 
     private boolean isEmpty(Element element, String attribute) {
-      return element.getAttribute(attribute).isEmpty();
+      return "".equals(element.getAttribute(attribute));
     }
 
     public boolean revisit(Gadget gadget, List<Node> nodes) throws RewritingException {
@@ -174,7 +174,7 @@ public class ImageResizeRewriter extends
       }
 
       Integer value = null;
-      if (!integerPrefix.isEmpty()) {
+      if (!"".equals(integerPrefix)) {
         try {
           value = NumberUtils.createInteger(integerPrefix);
         } catch (NumberFormatException e) {