Scarab commit: svn commit: r10695 - trunk/src/webapp/WEB-INF/templates/screens/entry/Wizard3.vm

Hussayn Dabbous <[email protected]>
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: dabbous
Date: 2009-02-15 14:07:58-0800
New Revision: 10695

Modified:
   trunk/src/webapp/WEB-INF/templates/screens/entry/Wizard3.vm

Log:
When the reasonField was hidden, no reason "attachment" was created. This results in a NPE within ReportIssue.java. I have forced to create the required (empty) data and that fixed the problem. 

FIXME: A better solution would be to allow the database to store issues without empty reasons. 

This fix is related to SCB2818

Modified: trunk/src/webapp/WEB-INF/templates/screens/entry/Wizard3.vm
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/webapp/WEB-INF/templates/screens/entry/Wizard3.vm?view=diff&pathrev=10695&r1=10694&r2=10695
==============================================================================
--- trunk/src/webapp/WEB-INF/templates/screens/entry/Wizard3.vm	(original)
+++ trunk/src/webapp/WEB-INF/templates/screens/entry/Wizard3.vm	2009-02-15 14:07:58-0800
@@ -101,6 +101,7 @@
   <div class="axial">
   <table width="100%" cellpadding="3" cellspacing="2" border="1">
 
+    #set ($intakeAttachment = $intake.Attachment.setKey("_1"))
     #if ($selected == false)
       <tr>
         <th nowrap="nowrap">   
@@ -112,7 +113,6 @@
           $l10n.Reason 
         </th>
         <td>
-          #set ($intakeAttachment = $intake.Attachment.setKey("_1"))
           #if (!$reasonOptional)
             #fieldErrorMsg($intakeAttachment.Data "")
             <p><small><strong>$l10n.MissedRequired1</strong> $l10n.MissedRequired2</small></p>

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=1166685
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.