Made Integer/Long comparisons happen through intValue()/longValue() methods.
authorMarco Zanon <info@marcozanon.com>
Sun, 24 Jun 2012 16:30:11 +0000 (16:30 +0000)
committerMarco Zanon <info@marcozanon.com>
Sun, 24 Jun 2012 16:30:11 +0000 (16:30 +0000)
src/java/com/marcozanon/macaco/attic/web/ui/MWebColor.java
src/java/com/marcozanon/macaco/attic/web/ui/MWebTable.java
src/java/com/marcozanon/macaco/attic/web/ui/MWebTextBox.java

index aa886ffdb5d5fd6f9b1c271c53a19622fe8b0082..546d9768b2bf380f5c6c385f344de3e63a793e14 100644 (file)
@@ -20,7 +20,7 @@ public class MWebColor extends MObject {
         if (null == color) {
             throw new IllegalArgumentException("Invalid 'color': null.");
         }
-        else if ((color < 0) || (color > 0xFFFFFF)) {
+        else if ((color.intValue() < 0) || (color.intValue() > 0xFFFFFF)) {
             throw new IllegalArgumentException(String.format("Invalid 'color': %s.", color));
         }
         //
index 2332d943fd64e59c07a97ac95144f902e6f08045..25b9f4fbf94eec137700cf836d1f523d6a32be2f 100644 (file)
@@ -173,7 +173,7 @@ public abstract class MWebTable extends MWebDirectWidget {
             this.selectedPage = 1;
             return;
         }
-        else if ((null != selectedPage) && ((selectedPage < 1) || (selectedPage > (int)Math.ceil((float)this.getRowCount() / (float)this.getRowsPerPage())))) {
+        else if ((null != selectedPage) && ((selectedPage.intValue() < 1) || (selectedPage.intValue() > (int)Math.ceil((float)this.getRowCount() / (float)this.getRowsPerPage())))) {
             throw new IllegalArgumentException(String.format("Invalid 'selectedPage': %s: out of range.", selectedPage));
         }
         //
@@ -210,10 +210,10 @@ public abstract class MWebTable extends MWebDirectWidget {
         else if (0 == this.getRowCount()) {
             this.setSelectedPage(null);
         }
-        else if (selectedPage < 1) {
+        else if (selectedPage.intValue() < 1) {
             this.setSelectedPage(1);
         }
-        else if (selectedPage > (int)Math.ceil((float)this.getRowCount() / (float)this.getRowsPerPage())) {
+        else if (selectedPage.intValue() > (int)Math.ceil((float)this.getRowCount() / (float)this.getRowsPerPage())) {
             this.setSelectedPage((int)Math.ceil((float)this.getRowCount() / (float)this.getRowsPerPage()));
         }
     }
@@ -248,8 +248,8 @@ public abstract class MWebTable extends MWebDirectWidget {
         int lastRow = this.getRowCount() - 1;
         Integer selectedPage = this.getSelectedPage();
         if (null != selectedPage) {
-            firstRow = (selectedPage - 1) * this.getRowsPerPage();
-            lastRow = Math.min((selectedPage * this.getRowsPerPage()) - 1, lastRow);
+            firstRow = (selectedPage.intValue() - 1) * this.getRowsPerPage();
+            lastRow = Math.min((selectedPage.intValue() * this.getRowsPerPage()) - 1, lastRow);
         }
         LinkedList<LinkedHashMap<String, String>> tmpRowSubset = new LinkedList<LinkedHashMap<String, String>>();
         for (int r = firstRow; r < (lastRow + 1); r++) {
index 339cb035eaba67bafa298e46d00a327a67d80ed9..eb18d1c8ac52c03c0dedba2e1d33997ccc3945ca 100644 (file)
@@ -126,7 +126,7 @@ public abstract class MWebTextBox extends MWebDirectWidget {
     }
 
     protected void setMaximumLength(Integer maximumLength, boolean refreshMode) {
-        if ((null != maximumLength) && (0 > maximumLength)) {
+        if ((null != maximumLength) && (0 > maximumLength.intValue())) {
             throw new IllegalArgumentException("Invalid 'maximumLength': negative number.");
         }
         //