Scarab commit: svn commit: r10718 - branches/release/0.22/src/java/org/tigris/scarab/screens/ViewIssue.java

Author: jhoech
Date: 2009-05-12 01:27:36-0700
New Revision: 10718

Modified:
   branches/release/0.22/src/java/org/tigris/scarab/screens/ViewIssue.java

Log:
FIX - no null pointer on invalid issue id (bug was detected after re-login with old url for issue)

Modified: branches/release/0.22/src/java/org/tigris/scarab/screens/ViewIssue.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/0.22/src/java/org/tigris/scarab/screens/ViewIssue.java?view=diff&pathrev=10718&r1=10717&r2=10718
==============================================================================
--- branches/release/0.22/src/java/org/tigris/scarab/screens/ViewIssue.java	(original)
+++ branches/release/0.22/src/java/org/tigris/scarab/screens/ViewIssue.java	2009-05-12 01:27:36-0700
@@ -86,7 +86,13 @@
         super.doBuildTemplate(data, context);
         ScarabRequestTool scarabR = getScarabRequestTool(context);
         Issue issue = null;
-        String id = data.getParameters().getString("id").trim();
+        
+        String id = data.getParameters().getString("id");
+        if(id != null){
+        	id = id.trim();
+        	issue = getReferredIssue(id, (ScarabModule)scarabR.getCurrentModule());
+        }
+        
         issue = getReferredIssue(id, (ScarabModule)scarabR.getCurrentModule());
         boolean hasViewPermission = false;
         boolean hasDeletePermission = false;

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2207791