Scarab commit: svn commit: r10848 - branches/release/0.22/src/java/org/tigris/scarab/actions/ReportIssue.java

Johannes Höchstädter <[email protected]>
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: jhoech
Date: 2009-09-16 04:49:25-0700
New Revision: 10848

Modified:
   branches/release/0.22/src/java/org/tigris/scarab/actions/ReportIssue.java

Log:
FIX - SCB2619 Intake parameter "NumericValue" wasn't regarded on reporting a new issue.

Modified: branches/release/0.22/src/java/org/tigris/scarab/actions/ReportIssue.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/0.22/src/java/org/tigris/scarab/actions/ReportIssue.java?view=diff&pathrev=10848&r1=10847&r2=10848
==============================================================================
--- branches/release/0.22/src/java/org/tigris/scarab/actions/ReportIssue.java	(original)
+++ branches/release/0.22/src/java/org/tigris/scarab/actions/ReportIssue.java	2009-09-16 04:49:25-0700
@@ -64,6 +64,7 @@
 import org.apache.turbine.tool.IntakeTool;
 import org.tigris.scarab.actions.base.RequireLoginFirstAction;
 import org.tigris.scarab.attribute.DateAttribute;
+import org.tigris.scarab.attribute.IntegerAttribute;
 import org.tigris.scarab.attribute.OptionAttribute;
 import org.tigris.scarab.attribute.UserAttribute;
 import org.tigris.scarab.notification.NotificationManagerFactory;
@@ -293,6 +294,10 @@
                         selectedOptions.add(fieldValue);
                     }                    
                 }
+                else if (aval instanceof IntegerAttribute) 
+                {
+                    field = group.get("NumericValue");
+                }
                 else if (aval instanceof UserAttribute) 
                 {
                     field = group.get("UserId");
@@ -383,8 +388,18 @@
                     intake.get("AttributeValue", aval.getQueryKey(), false);
                 if (group != null) 
                 {
-                    Field field = group.get(aval instanceof OptionAttribute ?
-                                            "OptionId" : "Value");
+                	
+                    Field field = null;
+                    if(aval instanceof OptionAttribute){
+                    	field = group.get("OptionId");
+                    }
+                    else if(aval instanceof IntegerAttribute){
+                    	field = group.get("NumericValue");
+                    } 
+                    else{
+                    	field = group.get("Value");
+                    }
+                    	
                     String value = field.toString();
 
                     if (value != null && value.length() > 0)

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2395471
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.