You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ojb-dev@db.apache.org by ar...@apache.org on 2007/10/29 11:15:50 UTC

svn commit: r589571 - in /db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker: core/PersistenceCheckerOptImpl.java metadata/fieldaccess/PersistentFieldFactory.java

Author: arminw
Date: Mon Oct 29 03:15:49 2007
New Revision: 589571

URL: http://svn.apache.org/viewvc?rev=589571&view=rev
Log:
update comment, remove out-commented lines

Modified:
    db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/core/PersistenceCheckerOptImpl.java
    db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldFactory.java

Modified: db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/core/PersistenceCheckerOptImpl.java
URL: http://svn.apache.org/viewvc/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/core/PersistenceCheckerOptImpl.java?rev=589571&r1=589570&r2=589571&view=diff
==============================================================================
--- db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/core/PersistenceCheckerOptImpl.java (original)
+++ db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/core/PersistenceCheckerOptImpl.java Mon Oct 29 03:15:49 2007
@@ -65,7 +65,7 @@
         /*
         arminw: The Identity object doesn't guarantee that method Identity.isPersistent()
         reliably identify transient objects (only if the id generation is managed by OJB),
-        thus we have to further checks for objects without autoincrement PK values
+        thus we have to do further checks for objects without autoincrement PK values
         */
         if(oid != null)
         {

Modified: db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldFactory.java
URL: http://svn.apache.org/viewvc/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldFactory.java?rev=589571&r1=589570&r2=589571&view=diff
==============================================================================
--- db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldFactory.java (original)
+++ db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldFactory.java Mon Oct 29 03:15:49 2007
@@ -30,6 +30,8 @@
 import org.apache.ojb.broker.util.logging.LoggerFactory;
 
 /**
+ * Factory to create {@link org.apache.ojb.broker.metadata.fieldaccess.PersistentField} instances.
+ *
  * @version $Id$
  */
 
@@ -39,9 +41,6 @@
     private static final Class DEFAULT_PERSISTENT_FIELD_IMPL = PersistentFieldDirectImpl.class;
     private static final Class[] METHOD_PARAMETER_TYPES = {Class.class, String.class};
 
-//    private static boolean usesAccessorsAndMutators = false;
-//    private static boolean usesAccessorsAndMutatorsCheck = false;
-
     /**
      * @throws MetadataException if an erros occours when creating the PersistenteField
      */
@@ -72,33 +71,6 @@
                     attributeType.getName() + ", " + attributeName, ex);
         }
     }
-
-//    public static boolean usesAccessorsAndMutators()
-//    {
-//        boolean retval = false;
-//        if (usesAccessorsAndMutatorsCheck)
-//            retval = usesAccessorsAndMutators;
-//        else
-//        {
-//            String className = getDefaultPersistentFieldClassName();
-//            PersistentField field = null;
-//            try
-//            {
-//                field = (PersistentField) ClassHelper.newInstance(className);
-//                usesAccessorsAndMutators = field.usesAccessorsAndMutators();
-//                retval = usesAccessorsAndMutators;
-//            }
-//            catch (Exception e)
-//            {
-//                log.error("Cannot verify 'usesAccessorsAndMutators' attribute for class " + className, e);
-//            }
-//            finally
-//            {
-//                usesAccessorsAndMutatorsCheck = true;
-//            }
-//        }
-//        return retval;
-//    }
 
     private static String getDefaultPersistentFieldClassName()
     {



---------------------------------------------------------------------
To unsubscribe, e-mail: ojb-dev-unsubscribe@db.apache.org
For additional commands, e-mail: ojb-dev-help@db.apache.org