Merge branch 'develop' of github.com:Sankore/Sankore-3.1 into develop

src/board/UBBoardController.cpp

@@ -611,18 +611,28 @@ void UBBoardController::duplicateItem(UBItem *item)
     {
     	UBGraphicsGroupContainerItem* groupItem = dynamic_cast<UBGraphicsGroupContainerItem*>(item);
     	if(groupItem){
+    		QTransform groupTransform = groupItem->transform();
+    		groupItem->resetTransform();
+
     		QList<QGraphicsItem*> children = groupItem->childItems();
     		foreach(QGraphicsItem* pIt, children){
     			UBItem* pItem = dynamic_cast<UBItem*>(pIt);
     			if(NULL != pItem){
-    				duplicateItem(pItem);	// The duplication already copies the item parameters
-    				if(NULL != mLastCreatedItem){
-    					mLastCreatedItem->setSelected(true);
-    				}
+    				duplicateItem(pItem);
     			}
     		}
+    		groupItem->setTransform(groupTransform);
     		groupItem->setSelected(false);
     		UBApplication::mainWindow->actionGroupItems->trigger();
+    		QList<QGraphicsItem*> selItems =  mActiveScene->selectedItems();
+    		if(!selItems.empty()){
+    			// I don't like this solution but for now this is the only way I found.
+    			// Normally, at this state, only the duplicated group should be selected
+    			UBGraphicsGroupContainerItem* duplicatedGroup = dynamic_cast<UBGraphicsGroupContainerItem*>(selItems.at(0));
+    			if(NULL != duplicatedGroup){
+    				duplicatedGroup->setTransform(groupTransform);
+    			}
+    		}
     	}
     	return;
     	break;
@@ -652,6 +662,7 @@ void UBBoardController::duplicateItem(UBItem *item)
         if (createdGitem)
             createdGitem->setPos(itemPos);
         mLastCreatedItem = dynamic_cast<QGraphicsItem*>(createdItem);
+        mLastCreatedItem->setSelected(true);
     }
 }
 
@@ -2176,8 +2187,11 @@ void UBBoardController::processMimeData(const QMimeData* pMimeData, const QPoint
         {
             foreach(UBItem* item, mimeData->items())
             {
+            	QGraphicsItem* pItem = dynamic_cast<QGraphicsItem*>(item);
+            	if(NULL != pItem){
             		duplicateItem(item);
             	}
+            }
 
             return;
         }

src/domain/UBGraphicsPolygonItem.cpp

@@ -171,6 +171,7 @@ void UBGraphicsPolygonItem::copyItemParameters(UBItem *copy) const
 
         cp->setColorOnDarkBackground(this->colorOnDarkBackground());
         cp->setColorOnLightBackground(this->colorOnLightBackground());
+        //cp->setTransform(transform());
 
         cp->setData(UBGraphicsItemData::ItemLayerType, this->data(UBGraphicsItemData::ItemLayerType));
     }

src/domain/UBGraphicsStrokesGroup.cpp

@@ -115,17 +115,22 @@ UBItem* UBGraphicsStrokesGroup::deepCopy() const
 {
 	UBGraphicsStrokesGroup* copy = new UBGraphicsStrokesGroup();
 
+	QTransform groupTransform = transform();
+	const_cast<UBGraphicsStrokesGroup*>(this)->resetTransform();
+
 	QList<QGraphicsItem*> chl = childItems();
 
 	foreach(QGraphicsItem *child, chl)
 	{
 		UBGraphicsPolygonItem *polygon = dynamic_cast<UBGraphicsPolygonItem*>(child);
-        if (polygon)
-        {
-            copy->addToGroup(dynamic_cast<QGraphicsItem*>(polygon->deepCopy()));
-            polygon->setStrokesGroup(copy);
+
+		if (polygon){
+			QGraphicsItem* pItem = dynamic_cast<QGraphicsItem*>(polygon->deepCopy());
+			copy->addToGroup(pItem);
 		}
+
 	}
+	const_cast<UBGraphicsStrokesGroup*>(this)->setTransform(groupTransform);
 	copyItemParameters(copy);
 
 	return copy;
@@ -133,7 +138,8 @@ UBItem* UBGraphicsStrokesGroup::deepCopy() const
 
 void UBGraphicsStrokesGroup::copyItemParameters(UBItem *copy) const
 {
-    UBGraphicsStrokesGroup *cp = dynamic_cast<UBGraphicsStrokesGroup*>(copy);
+	QGraphicsItem *cp = dynamic_cast<QGraphicsItem*>(copy);
+    if(NULL != cp)
     {
         cp->setTransform(transform());