Sankore-958 recursive fix

src/domain/UBGraphicsItemDelegate.cpp

@@ -471,24 +471,18 @@ void UBGraphicsItemDelegate::lock(bool locked)
     mFrame->positionHandles();
 }
 
+void UBGraphicsItemDelegate::showHideRecurs(const QVariant &pShow, QGraphicsItem *pItem)
+{
+    pItem->setData(UBGraphicsItemData::ItemLayerType, pShow);
+    foreach (QGraphicsItem *insideItem, pItem->childItems()) {
+        showHideRecurs(pShow, insideItem);
+    }
+}
 
 void UBGraphicsItemDelegate::showHide(bool show)
 {
-    if (show) {
+    QVariant showFlag = QVariant(show ? UBItemLayerType::Object : UBItemLayerType::Control);
-        mDelegated->setData(UBGraphicsItemData::ItemLayerType, QVariant(UBItemLayerType::Object));
+    showHideRecurs(showFlag, mDelegated);
-        if (mDelegated->childItems().count()) {
-            foreach (QGraphicsItem *item, mDelegated->childItems()) {
-                item->setData(UBGraphicsItemData::ItemLayerType, QVariant(UBItemLayerType::Object));
-            }
-        }
-    } else {
-        mDelegated->setData(UBGraphicsItemData::ItemLayerType, QVariant(UBItemLayerType::Control));
-        if (mDelegated->childItems().count()) {
-            foreach (QGraphicsItem *item, mDelegated->childItems()) {
-                item->setData(UBGraphicsItemData::ItemLayerType, QVariant(UBItemLayerType::Control));
-            }
-        }
-    }
     mDelegated->update();
 
     emit showOnDisplayChanged(show);

src/domain/UBGraphicsItemDelegate.h

@@ -297,6 +297,7 @@ protected slots:
 private:
         void updateFrame();
         void updateButtons(bool showUpdated = false);
+        inline void showHideRecurs(const QVariant &pShow, QGraphicsItem *pItem);
 
         QPointF mOffset;
         QTransform mPreviousTransform;