Author: bpapez
Date: Mon Jul  2 16:30:36 2007
New Revision: 17858

URL: https://svndev.jahia.net/websvn/listing.php?sc=3D1&rev=3D17858&repname=
=3Djahia
Log:
minor refactorings

Modified:
    branches/JAHIA-5-0-SP-BRANCH/core/src/java/org/jahia/data/fields/JahiaP=
ageField.java

Modified: branches/JAHIA-5-0-SP-BRANCH/core/src/java/org/jahia/data/fields/=
JahiaPageField.java
URL: https://svndev.jahia.net/websvn/diff.php?path=3D/branches/JAHIA-5-0-SP=
-BRANCH/core/src/java/org/jahia/data/fields/JahiaPageField.java&rev=3D17858=
&repname=3Djahia
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D
--- branches/JAHIA-5-0-SP-BRANCH/core/src/java/org/jahia/data/fields/JahiaP=
ageField.java (original)
+++ branches/JAHIA-5-0-SP-BRANCH/core/src/java/org/jahia/data/fields/JahiaP=
ageField.java Mon Jul  2 16:30:36 2007
@@ -181,7 +181,7 @@
 =

                                     if (def !=3D null) {
                                         //if ((def !=3D null) && (def.isVi=
sible())) {
-                                        this.setValue(thePage.getUrl(jPara=
ms));
+                                        this.setValue(thePage.getURL(jPara=
ms));
                                     } else {
                                         // FIXME : NK
                                         // If a direct page has its templa=
te deleted,
@@ -194,7 +194,7 @@
                                         }*/
 =

                                         // set the url to the page , but w=
e sure have a page template not found when accessing it
-                                        this.setValue(thePage.getUrl(jPara=
ms));
+                                        this.setValue(thePage.getURL(jPara=
ms));
 =

                                     }
                                     if ( ProcessingContext.COMPARE.equals(=
jParams.getOpMode()) ){
@@ -302,7 +302,6 @@
         } catch (NumberFormatException e) {
         }
 =

-        boolean pageUpdated =3D false;
         String tmpVal =3D this.getValue();
         if (!tmpVal.equals("<empty>")) {
             int pageID =3D -1;
@@ -317,7 +316,6 @@
                 if (thePage !=3D null) {
 =

                     thePage.commitChanges(true, jParams.getUser());
-                    pageUpdated =3D true;
                     ContentPage contentPage =3D ContentPage.getPage(thePag=
e.getID());
                     JahiaEvent objectCreatedEvent =3D new JahiaEvent(this,=
 jParams, contentPage);
                     ServicesRegistry.getInstance().getJahiaEventService()
@@ -327,14 +325,10 @@
                     //        getJahiaPageService().savePageInfo( thePage =
);
 =

                     this.setValue(Integer.toString(thePage.getID()));
-                    tmpVal =3D Integer.toString(thePage.getID()); // Very =
useful !???
                 } else {
                     this.setValue("");
-                    tmpVal =3D "<No Page>"; // Very useful !???
                 }
             }
-        } else {
-            tmpVal =3D "<No Page>"; // Very useful !???
         }
         int pageIDToSet;
         try {
@@ -454,13 +448,6 @@
             clonedField.setObject(this.getObject());
         }
 =

-        //toDebug("cloneField(): value =3D "+this.getValue());
-        if (clonedField =3D=3D null) {
-            throw new JahiaException("Could not clone field.",
-                    "Could not instanciate a new JahiaField object.",
-                    JahiaException.PAGE_ERROR, JahiaException.CRITICAL_SEV=
ERITY);
-        }
-
         clonedField.setRawValue(this.getRawValue());
         clonedField.setProperties((Properties) (this.getProperties()).clon=
e());
         clonedField.setForComparisonOnly(this.isForComparisonOnly());
@@ -480,7 +467,7 @@
                 new Integer(aclID),
                 new Integer(versionID),
                 new Integer(workflowState),
-                new String(getLanguageCode()));
+                getLanguageCode());
         pf.setObject(objItem);
         pf.setForComparisonOnly(this.isForComparisonOnly());
         return pf;

_______________________________________________
cvs_list mailing list
[email protected]
http://lists.jahia.org/cgi-bin/mailman/listinfo/cvs_list

Reply via email to