You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by le...@apache.org on 2010/12/12 21:42:00 UTC

svn commit: r1044915 - /ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/screen/ConfigureItem.java

Author: lektran
Date: Sun Dec 12 20:41:59 2010
New Revision: 1044915

URL: http://svn.apache.org/viewvc?rev=1044915&view=rev
Log:
Replace empty if block with a condition for the else block

Modified:
    ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/screen/ConfigureItem.java

Modified: ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/screen/ConfigureItem.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/screen/ConfigureItem.java?rev=1044915&r1=1044914&r2=1044915&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/screen/ConfigureItem.java (original)
+++ ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/screen/ConfigureItem.java Sun Dec 12 20:41:59 2010
@@ -179,8 +179,8 @@ public class ConfigureItem extends XPage
         questionHashMap = FastMap.newInstance();
 
         List<ConfigItem> questions = m_pcw.getQuestions();
-        if (questions==null) ; // no questions, we shouldn't be here
-        else{
+        
+        if (questions != null) {
             Iterator<ConfigItem> iter = questions.iterator();
             Iterator<XButton> buttons = m_buttonList.iterator();
             while (iter.hasNext()) {