You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chemistry.apache.org by fm...@apache.org on 2012/08/10 12:48:41 UTC

svn commit: r1371646 [2/2] - in /chemistry/opencmis/trunk: chemistry-opencmis-bridge/chemistry-opencmis-bridge/src/main/java/org/apache/chemistry/opencmis/bridge/client/ chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apa...

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/UpdateSmokeTest.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/UpdateSmokeTest.java?rev=1371646&r1=1371645&r2=1371646&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/UpdateSmokeTest.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/crud/UpdateSmokeTest.java Fri Aug 10 10:48:40 2012
@@ -55,8 +55,6 @@ public class UpdateSmokeTest extends Abs
 
     @Override
     public void run(Session session) {
-        CmisTestResult f;
-
         try {
             // create test folder
             Folder testFolder = createTestFolder(session);

Modified: chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java?rev=1371646&r1=1371645&r2=1371646&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-test/chemistry-opencmis-test-tck/src/main/java/org/apache/chemistry/opencmis/tck/tests/versioning/CheckedOutTest.java Fri Aug 10 10:48:40 2012
@@ -108,7 +108,7 @@ public class CheckedOutTest extends Abst
 
         for (Document pwc : pwcs) {
             String[] propertiesToCheck = getAllProperties(pwc);
-            addResult(checkObject(session, pwc, propertiesToCheck, "PWC check: " + pwc.getId()));
+            addResult(checkObject(session, pwc, propertiesToCheck, "PWC check: " + (pwc == null ? "?" : pwc.getId())));
 
             if (pwc != null) {
                 f = createResult(WARNING, "PWC is not the latest version! Id: " + pwc.getId()

Modified: chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java?rev=1371646&r1=1371645&r2=1371646&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java Fri Aug 10 10:48:40 2012
@@ -89,25 +89,25 @@ public class ClientHelper {
     public static final Color LINK_COLOR = new Color(105, 29, 21);
     public static final Color LINK_SELECTED_COLOR = new Color(255, 255, 255);
 
-    private static final Logger log = LoggerFactory.getLogger(ClientHelper.class);
+    private static final Logger LOG = LoggerFactory.getLogger(ClientHelper.class);
     private static final int BUFFER_SIZE = 64 * 1024;
 
     private ClientHelper() {
     }
 
     public static void showError(Component parent, Exception ex) {
-        if (log.isErrorEnabled()) {
-            log.error(ex.getClass().getSimpleName() + ": " + ex.getMessage(), ex);
+        if (LOG.isErrorEnabled()) {
+            LOG.error(ex.getClass().getSimpleName() + ": " + ex.getMessage(), ex);
 
             if (ex instanceof CmisBaseException) {
                 CmisBaseException cex = (CmisBaseException) ex;
 
                 if (cex.getCode() != null) {
-                    log.error("Error code: " + cex.getCode());
+                    LOG.error("Error code: " + cex.getCode());
                 }
 
                 if (cex.getErrorContent() != null) {
-                    log.error("Error content: " + cex.getErrorContent());
+                    LOG.error("Error content: " + cex.getErrorContent());
                 }
             }
         }

Modified: chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/checks/SwingReport.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/checks/SwingReport.java?rev=1371646&r1=1371645&r2=1371646&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/checks/SwingReport.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/checks/SwingReport.java Fri Aug 10 10:48:40 2012
@@ -151,10 +151,6 @@ public class SwingReport extends Abstrac
                         CmisTestReport report = null;
 
                         switch (reportType.getSelectedIndex()) {
-                        case 0:
-                            tempReportFile = File.createTempFile("cmistck", ".html");
-                            report = new HtmlReport();
-                            break;
                         case 1:
                             tempReportFile = File.createTempFile("cmistck", ".txt");
                             report = new TextReport();
@@ -163,6 +159,10 @@ public class SwingReport extends Abstrac
                             tempReportFile = File.createTempFile("cmistck", ".xml");
                             report = new XmlReport();
                             break;
+                        default:
+                            tempReportFile = File.createTempFile("cmistck", ".html");
+                            report = new HtmlReport();
+                            break;
                         }
 
                         tempReportFile.deleteOnExit();