wsd/ClientSession.cpp |    4 ++--
 wsd/reference.txt     |    8 +++-----
 2 files changed, 5 insertions(+), 7 deletions(-)

New commits:
commit 1a1ab5fd5751f9d6353bb91be918e96cf9c53b90
Author: Pranav Kant <pran...@collabora.co.uk>
Date:   Thu May 25 14:11:33 2017 +0530

    wsd: Only disable copying when DisableCopy is mentioned
    
    Earlier, DisableCopy was a misnomer disabling both copy and paste. Now
    it only disables copy from the document but allows pasting into the
    document.
    
    Change-Id: I8ddfdd493918331276f0656468d3b94c4283fa4d

diff --git a/wsd/ClientSession.cpp b/wsd/ClientSession.cpp
index 387f7629..4a886a43 100644
--- a/wsd/ClientSession.cpp
+++ b/wsd/ClientSession.cpp
@@ -420,12 +420,12 @@ bool ClientSession::filterMessage(const std::string& 
message) const
                 LOG_WRN("No value of id in downloadas message");
         }
     }
-    else if (tokens[0] == "gettextselection" || tokens[0] == "paste" || 
tokens[0] == "insertfile")
+    else if (tokens[0] == "gettextselection")
     {
         if (_wopiFileInfo && _wopiFileInfo->_disableCopy)
         {
             allowed = false;
-            LOG_WRN("WOPI host has disabled copying to/from the document");
+            LOG_WRN("WOPI host has disabled copying from the document");
         }
     }
     else if (isReadOnly())
diff --git a/wsd/reference.txt b/wsd/reference.txt
index e1af6d4b..5c706621 100644
--- a/wsd/reference.txt
+++ b/wsd/reference.txt
@@ -41,11 +41,9 @@ DisableExport
        HideExportOption is assumed to be true
 
 DisableCopy
-       Disables copy/paste from/to the document in libreoffice online backend.
-       However, it is still possible to do an "internal" cut/copy/paste i.e
-       copy from the document and paste to the same document view. The context
-       menu options pertaining to cut/copy/paste would be renamed to 'Internal
-       Cut/Copy/Paste' when this option is mentioned.
+       Disables copying from the document in libreoffice online
+       backend. Pasting into the document would still be possible.
+       However, it is still possible to do an "internal" cut/copy/paste.
 
 EnableOwnerTermination
        If set to true, it allows the document owner (the one with OwnerId =
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to