Scarab commit: svn commit: r10131 - trunk/src/java/org/tigris/scarab/om/AttributeValue.java

Author: dabbous
Date: 2006-06-01 06:50:56-0700
New Revision: 10131

Modified:
   trunk/src/java/org/tigris/scarab/om/AttributeValue.java

Log:
nullpointer exception during creation of a report.

casued when "no value" for an attribute was selected

unintentionally. the nullpointer is detected now .

the fix seems correct, not yet tested in depths though.

Modified: trunk/src/java/org/tigris/scarab/om/AttributeValue.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/AttributeValue.java?view=diff&rev=10131&p1=trunk/src/java/org/tigris/scarab/om/AttributeValue.java&p2=trunk/src/java/org/tigris/scarab/om/AttributeValue.java&r1=10130&r2=10131
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/AttributeValue.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/AttributeValue.java	2006-06-01 06:50:56-0700
@@ -440,15 +440,18 @@
                         throw new TorqueException(e); //EXCEPTION
                     }
                 }
-                for (int i=options.size()-1; i>=0; i--) 
+                if(options != null)
                 {
-                    RModuleOption option = (RModuleOption)options.get(i);
-                    if (option.getOptionId().equals(optionId)) 
+                    for (int i = options.size() - 1; i >= 0; i--)
                     {
-                        setValueOnly(option.getDisplayValue());
-                        break;
+                        RModuleOption option = (RModuleOption) options.get(i);
+                        if (option.getOptionId().equals(optionId))
+                        {
+                            setValueOnly(option.getDisplayValue());
+                            break;
+                        }
                     }
-                }   
+                }
             }            
         }
         else