From fa5ec6f597249eac30749637cfad962389107e82 Mon Sep 17 00:00:00 2001
From: xavier
 <xavier@r991  xavier  2010-05-09 20:40:08 +0200 (dim. 09 mai 2010)>
Date: Sat, 12 May 2012 22:03:07 +0000
Subject: [PATCH] Element editor: fixed yet another bug in the zValue choice
 for newly added parts.

git-svn-id: svn+ssh://svn.tuxfamily.org/svnroot/qet/qet/trunk@1827 bfdf4180-ca20-0410-9c96-a3a8aa849046
---
 sources/editor/editorcommands.cpp | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/sources/editor/editorcommands.cpp b/sources/editor/editorcommands.cpp
index 1f081858f..6e6d8722e 100644
--- a/sources/editor/editorcommands.cpp
+++ b/sources/editor/editorcommands.cpp
@@ -236,9 +236,13 @@ void AddPartCommand::undo() {
 void AddPartCommand::redo() {
 	// le premier appel a redo, lors de la construction de l'objet, ne doit pas se faire
 	if (first_redo) {
-		QList<QGraphicsItem *> existing_items = editor_scene -> zItems();
-		qreal z = existing_items.count() ? existing_items.last() -> zValue() + 1 : 1;
-		part -> setZValue(z);
+		if (!part -> zValue()) {
+			// the added part has no specific zValue already defined, we put it
+			// above existing items (but still under terminals)
+			QList<QGraphicsItem *> existing_items = editor_scene -> zItems();
+			qreal z = existing_items.count() ? existing_items.last() -> zValue() + 1 : 1;
+			part -> setZValue(z);
+		}
 		editor_scene -> clearSelection();
 		part -> setSelected(true);
 		first_redo = false;