Scarab commit: svn commit: r10559 - trunk/src/java/org/tigris/scarab: om util/xmlissues

[email protected]
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: ronvoe122
Date: 2008-05-05 14:12:24-0700
New Revision: 10559

Modified:
   trunk/src/java/org/tigris/scarab/om/Issue.java
   trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java

Log:
Reverted r10555.

Modified: trunk/src/java/org/tigris/scarab/om/Issue.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/Issue.java?view=diff&rev=10559&p1=trunk/src/java/org/tigris/scarab/om/Issue.java&p2=trunk/src/java/org/tigris/scarab/om/Issue.java&r1=10558&r2=10559
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/Issue.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/Issue.java	2008-05-05 14:12:24-0700
@@ -2082,6 +2082,11 @@
                 }
             }
         }
+        if(  getActivitySetRelatedByLastTransId()==null
+           ||getActivitySetRelatedByCreatedTransId()==null)
+        {
+            throw new RuntimeException("Created or Last ActivitySet must not be null.");
+        }
         super.save(dbCon);
     }
 
@@ -2374,12 +2379,12 @@
         {
             newIssue = this;
             newIssue.setIssueType(newIssueType);
+            newIssue.save();
         }
         else
         {
             newIssue = newModule.getNewIssue(newIssueType);
         }
-        newIssue.save();
 
         if (newIssue != this)
         {
@@ -4004,4 +4009,12 @@
     {
         return ActivityPeer.getNewIssueUniqueId(this);
     }
+
+    public void setCreatedTransId(Long createdTransId) 
+        throws TorqueException
+    {
+        super.setCreatedTransId(createdTransId);
+        if(super.getLastTransId()==null)
+            super.setLastTransId(createdTransId);
+    }
 }

Modified: trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java?view=diff&rev=10559&p1=trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java&p2=trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java&r1=10558&r2=10559
==============================================================================
--- trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java	(original)
+++ trunk/src/java/org/tigris/scarab/util/xmlissues/ScarabIssues.java	2008-05-05 14:12:24-0700
@@ -912,7 +912,6 @@
             issueOM.setFederatedId(id);
         }
         // create the issue in the database
-        issueOM.save();
 
         // Add the mapping between the issue id and the id that was created.
         // This mapping is used dependency checking and printing out in
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.