svn commit: r492857 - /db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldBase.java

Author: arminw
Date: Thu Jan  4 18:23:34 2007
New Revision: 492857

URL: http://svn.apache.org/viewvc?view=rev&rev=492857
Log:
minor improvements

Modified:
    db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldBase.java

Modified: db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldBase.java
URL: http://svn.apache.org/viewvc/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldBase.java?view=diff&rev=492857&r1=492856&r2=492857
==============================================================================
--- db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldBase.java (original)
+++ db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/metadata/fieldaccess/PersistentFieldBase.java Thu Jan  4 18:23:34 2007
@@ -197,15 +197,14 @@
     {
         String eol = SystemUtils.LINE_SEPARATOR;
         StringBuffer buf = new StringBuffer();
-        buf
-                .append(eol + "[try to set 'object value' in 'target object'")
-                .append(eol + "  target obj class: " + (obj != null ? obj.getClass().getName() : null))
-                .append(eol + "  target field name: " + (aField != null ? aField.getName() : null))
-                .append(eol + "  target field type: " + (aField != null ? aField.getType() : null))
-                .append(eol + "  target field declared in: " + (aField != null ? aField.getDeclaringClass().getName() : null))
-                .append(eol + "  object value class: " + (value != null ? value.getClass().getName() : null))
-                .append(eol + "  object value: " + (value != null ? value : null))
-                .append(eol + "]");
+        buf.append(eol).append("[try to set 'object value' in 'target object'");
+        buf.append(eol).append("  target obj class: ").append(obj != null ? obj.getClass().getName() : null);
+        buf.append(eol).append("  target field name: ").append(aField != null ? aField.getName() : null);
+        buf.append(eol).append("  target field type: ").append(aField != null ? aField.getType() : null);
+        buf.append(eol).append("  target field declared in: ").append(aField != null ? aField.getDeclaringClass().getName() : null);
+        buf.append(eol).append("  object value class: ").append(value != null ? value.getClass().getName() : null);
+        buf.append(eol).append("  object value: ").append(value != null ? value : null);
+        buf.append(eol).append("]");
         return buf.toString();
     }
 
@@ -216,13 +215,12 @@
     {
         String eol = SystemUtils.LINE_SEPARATOR;
         StringBuffer buf = new StringBuffer();
-        buf
-                .append(eol + "[try to read from source object")
-                .append(eol + "  source obj class: " + (obj != null ? obj.getClass().getName() : null))
-                .append(eol + "  target field name: " + (aField != null ? aField.getName() : null))
-                .append(eol + "  target field type: " + (aField != null ? aField.getType() : null))
-                .append(eol + "  target field declared in: " + (aField != null ? aField.getDeclaringClass().getName() : null))
-                .append(eol + "]");
+        buf.append(eol).append("[try to read from source object");
+        buf.append(eol).append("  source obj class: ").append(obj != null ? obj.getClass().getName() : null);
+        buf.append(eol).append("  target field name: ").append(aField != null ? aField.getName() : null);
+        buf.append(eol).append("  target field type: ").append(aField != null ? aField.getType() : null);
+        buf.append(eol).append("  target field declared in: ").append(aField != null ? aField.getDeclaringClass().getName() : null);
+        buf.append(eol).append("]");
         return buf.toString();
     }
 
@@ -250,34 +248,30 @@
         String eol = SystemUtils.LINE_SEPARATOR;
         StringBuffer buf = new StringBuffer();
         String type = (isSetter ? "setter" : "getter");
-        buf
-            .append(eol + "["
-                    + (msg == null ? "try to handle " + type + " property call" : type + " property call: " + msg))
-            .append(eol + "  Declaring class [" + getDeclaringClass().getName() + "]")
-            .append(eol + "  Property Name [" + getName() + "]")
-            .append(eol + "  Property Type ["
-                    + (returnOrArgumentType != null ? returnOrArgumentType.getName() : "not available") + "]");
+        buf.append(eol).append("[").append(msg == null ? "try to handle " + type + " property call" : type + " property call: " + msg);
+        buf.append(eol).append("  Declaring class [").append(getDeclaringClass().getName()).append("]");
+        buf.append(eol).append("  Property Name [").append(getName()).append("]");
+        buf.append(eol).append("  Property Type [").append(returnOrArgumentType != null ? returnOrArgumentType.getName() : "not available").append("]");
 
         if (anObject != null)
         {
             //buf.append("the " + (isSetter ? "target" : "source") + " object was [" + anObject + "]");
-            buf.append(eol + "  the "
-                    + (isSetter ? "target" : "source") + " object type was [" + anObject.getClass().getName() + "]");
+            buf.append(eol).append("  the ").append(isSetter ? "target" : "source").append(" object type was [").append(anObject.getClass().getName()).append("]");
         }
         else
         {
-            buf.append(eol + "  the " + (isSetter ? "target" : "source") + " object was 'null'");
+            buf.append(eol).append("  the ").append(isSetter ? "target" : "source").append(" object was 'null'");
         }
         if(isSetter)
         {
             if (aValue != null)
             {
-                buf.append(eol + "  the value was [" + aValue + "]");
-                buf.append(eol + "  the value type was [" + aValue.getClass().getName() + "]");
+                buf.append(eol).append("  the value was [").append(aValue).append("]");
+                buf.append(eol).append("  the value type was [").append(aValue.getClass().getName()).append("]");
             }
             else
             {
-                buf.append(eol + "  the value was 'null'");
+                buf.append(eol).append("  the value was 'null'");
             }
         }
         return buf.toString();