From e132d0d832de265282323c43bc1920e02b6e4662 Mon Sep 17 00:00:00 2001
From: Kevin Roast <kevin.roast@alfresco.com>
Date: Wed, 14 Jan 2009 14:05:43 +0000
Subject: [PATCH] Merged V3.0 to HEAD    12734: Fix for ETHREEOH-1007    12735:
 Fix for ALFCOM-2326 (applied to 3.0E original code location)

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@12739 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../web/bean/coci/UploadNewVersionDialog.java   | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/source/java/org/alfresco/web/bean/coci/UploadNewVersionDialog.java b/source/java/org/alfresco/web/bean/coci/UploadNewVersionDialog.java
index c7d2547f6d..6b61dab07d 100644
--- a/source/java/org/alfresco/web/bean/coci/UploadNewVersionDialog.java
+++ b/source/java/org/alfresco/web/bean/coci/UploadNewVersionDialog.java
@@ -28,9 +28,11 @@ import javax.faces.context.FacesContext;
 
 import org.alfresco.web.app.Application;
 
+/**
+ * Dialog supporting the Upload New Version action on a working copy node.
+ */
 public class UploadNewVersionDialog extends DoneEditingDialog
 {
-
     private final static String MSG_UPLOAD_NEW_VERSION = "upload_new_version";
     private final static String MSG_OF = "of";
 
@@ -52,7 +54,6 @@ public class UploadNewVersionDialog extends DoneEditingDialog
         return property.getFile() == null;
     }
 
-
     @Override
     public String getContainerTitle()
     {
@@ -63,8 +64,15 @@ public class UploadNewVersionDialog extends DoneEditingDialog
     @Override
     protected String finishImpl(FacesContext context, String outcome) throws Exception
     {
-        property.setKeepCheckedOut(!finishedEditing);
-        return checkinFileOK(context, outcome);
+        if (finishedEditing)
+        {
+            property.setKeepCheckedOut(false);
+            return checkinFileOK(context, outcome);
+        }
+        else
+        {
+            return updateFileOK(context, outcome);
+        }
     }
     
     @Override
@@ -73,5 +81,4 @@ public class UploadNewVersionDialog extends DoneEditingDialog
         super.resetState();
         finishedEditing = false;
     }
-
 }