You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lenya.apache.org by an...@apache.org on 2007/11/16 10:18:03 UTC

svn commit: r595602 - /lenya/trunk/src/impl/java/org/apache/lenya/cms/publication/DocumentImpl.java

Author: andreas
Date: Fri Nov 16 01:18:02 2007
New Revision: 595602

URL: http://svn.apache.org/viewvc?rev=595602&view=rev
Log:
[minor change] DocumentImpl: put null check first so that the fields are not set if the argument is null

Modified:
    lenya/trunk/src/impl/java/org/apache/lenya/cms/publication/DocumentImpl.java

Modified: lenya/trunk/src/impl/java/org/apache/lenya/cms/publication/DocumentImpl.java
URL: http://svn.apache.org/viewvc/lenya/trunk/src/impl/java/org/apache/lenya/cms/publication/DocumentImpl.java?rev=595602&r1=595601&r2=595602&view=diff
==============================================================================
--- lenya/trunk/src/impl/java/org/apache/lenya/cms/publication/DocumentImpl.java (original)
+++ lenya/trunk/src/impl/java/org/apache/lenya/cms/publication/DocumentImpl.java Fri Nov 16 01:18:02 2007
@@ -105,12 +105,12 @@
                             + "], language [" + identifier.getLanguage() + "]");
         }
 
-        this.manager = manager;
-        this.identifier = identifier;
         if (identifier.getUUID() == null) {
-            throw new IllegalArgumentException("The document ID must not be null!");
+            throw new IllegalArgumentException("The UUID must not be null!");
         }
-
+        
+        this.manager = manager;
+        this.identifier = identifier;
         this.factory = map;
         this.revision = revision;
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@lenya.apache.org
For additional commands, e-mail: commits-help@lenya.apache.org