[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - svx/source

[Tomaž Vajngerl (via logerrit)]

 svx/source/svdraw/svdmrkv.cxx        |   16 +++-------------
 svx/source/table/tablecontroller.cxx |    3 ---
 2 files changed, 3 insertions(+), 16 deletions(-)

New commits:
commit e71c70f4c60731bca441f9020176d0a819befd83
Author:     Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
AuthorDate: Wed Oct 2 14:32:07 2019 +0200
Commit:     Tomaž Vajngerl <quikee at gmail.com>
CommitDate: Thu Oct 3 10:41:52 2019 +0200

    tdf#125565 always send the graphic selection for the table
    
    Reviewed-on: https://gerrit.libreoffice.org/80034
    Tested-by: Jenkins
    Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>
    (cherry picked from commit c9336bfb6bbf6d73d3f23c124262ade30133448d)
    
    Change-Id: I7085b2f1ddcfd83eccbbcaba4892cde95b6bd85c
    Reviewed-on: https://gerrit.libreoffice.org/80092
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>

diff --git a/svx/source/svdraw/svdmrkv.cxx b/svx/source/svdraw/svdmrkv.cxx
index 443a4b59e08f..b66bc9d69e45 100644
--- a/svx/source/svdraw/svdmrkv.cxx
+++ b/svx/source/svdraw/svdmrkv.cxx
@@ -746,7 +746,7 @@ void SdrMarkView::SetMarkHandles(SfxViewShell* pOtherShell)
     }
 
     SfxViewShell* pViewShell = GetSfxViewShell();
-    bool bIsInTextEditMode = false;
+
     // check if text edit or ole is active and handles need to be suppressed. This may be the case
     // when a single object is selected
     // Using a strict return statement is okay here; no handles means *no* handles.
@@ -764,9 +764,7 @@ void SdrMarkView::SetMarkHandles(SfxViewShell* pOtherShell)
 
             if (pSdrTextObj && pSdrTextObj->IsInEditMode())
             {
-                if (bTiledRendering)
-                    bIsInTextEditMode = true;
-                else
+                if (!bTiledRendering)
                     return;
             }
         }
@@ -832,19 +830,11 @@ void SdrMarkView::SetMarkHandles(SfxViewShell* pOtherShell)
             OString sSelectionText;
             boost::property_tree::ptree aTableJsonTree;
             bool bTableSelection = false;
-            bool bCellsAreSelected = false;
 
             if (mpMarkedObj && mpMarkedObj->GetObjIdentifier() == OBJ_TABLE)
             {
                 auto* pTableObject = dynamic_cast<sdr::table::SdrTableObj*>(mpMarkedObj);
                 bTableSelection = pTableObject->createTableEdgesJson(aTableJsonTree);
-
-                rtl::Reference<sdr::SelectionController> xController = static_cast<SdrView*>(this)->getSelectionController();
-                if (xController.is() && xController->hasSelectedCells())
-                {
-                    // The table shape has selected cells, which provide text selection already -> no graphic selection.
-                    bCellsAreSelected = true;
-                }
             }
             if (GetMarkedObjectCount())
             {
@@ -1002,7 +992,7 @@ void SdrMarkView::SetMarkHandles(SfxViewShell* pOtherShell)
                 }
             }
 
-            if (sSelectionText.isEmpty() || bCellsAreSelected || bIsInTextEditMode)
+            if (sSelectionText.isEmpty())
                 sSelectionText = "EMPTY";
 
             if (bTableSelection)
diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx
index 5b5116c7877a..a07a90ac1bb5 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -2296,9 +2296,6 @@ void SvxTableController::updateSelectionOverlay()
                 {
                     pViewShell->libreOfficeKitViewCallback(LOK_CALLBACK_CELL_SELECTION_AREA, aSelection.toString().getStr());
                     pViewShell->libreOfficeKitViewCallback(LOK_CALLBACK_TEXT_SELECTION, aSelection.toString().getStr());
-                    OString sSelection("EMPTY");
-                    pViewShell->libreOfficeKitViewCallback(LOK_CALLBACK_GRAPHIC_SELECTION, sSelection.getStr());
-                    SfxLokHelper::notifyOtherViews(pViewShell, LOK_CALLBACK_GRAPHIC_VIEW_SELECTION, "selection", sSelection);
                 }
             }
         }