realAscot/qelectrotech-source-mirror
realAscot/qelectrotech-source-mirror
1
0
Fork 0
mirror of https://github.com/qelectrotech/qelectrotech-source-mirror.git synced 2025-09-13 20:23:04 +02:00
Code Packages Releases 1 Activity

Bug fix : in some condition element are not loaded when open a project

Browse Source 
If an element are overwrite by a modified element and the terminals of
the modified element are moved, the old element is not loaded because
some terminal are not found.
This commit remove the checking of not found terminal because it's
useless now.
This commit is contained in:
joshua 2021-09-07 22:23:43 +02:00
parent 54b53b4086
commit 6775930e81
1 changed files with 24 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

58
sources/qetgraphicsitem/element.cpp
View File

@ -736,50 +736,40 @@ bool Element::fromXml(
} }
QHash<int, Terminal *> priv_id_adr; QHash<int, Terminal *> priv_id_adr;
int terminals_non_trouvees = 0;
foreach(QGraphicsItem *qgi, childItems()) { for (auto *qgi : childItems())
if (Terminal *p = qgraphicsitem_cast<Terminal *>(qgi)) { {
bool terminal_trouvee = false; if (auto terminal_ = qgraphicsitem_cast<Terminal *>(qgi))
foreach(QDomElement qde, liste_terminals) { {
if (p -> fromXml(qde)) { for(auto qde : liste_terminals)
priv_id_adr.insert( {
qde.attribute( if (terminal_ -> fromXml(qde))
"id").toInt(), {
p); priv_id_adr.insert(qde.attribute(QStringLiteral("id")).toInt(),
terminal_trouvee = true; terminal_);
// We used to break here, because we did not expect
// several terminals to share the same position.
// Of course, it finally happened.
} }
} }
if (!terminal_trouvee) ++ terminals_non_trouvees;
} }
} }
if (terminals_non_trouvees > 0)
//Check that associated id/adress doesn't conflict with table_id_adr
for(auto found_id : priv_id_adr.keys())
{ {
m_state = QET::GIOK; if (table_id_adr.contains(found_id))
return(false);
}
else
{
// verifie que les associations id / adr n'entrent pas en conflit avec table_id_adr
foreach(int id_trouve, priv_id_adr.keys())
{ {
if (table_id_adr.contains(id_trouve)) //This element got an id who is already referenced (= conflict)
{ m_state = QET::GIOK;
// cet element possede un id qui est deja reference (= conflit) return(false);
m_state = QET::GIOK;
return(false);
}
}
// copie des associations id / adr
foreach(int id_trouve, priv_id_adr.keys()) {
table_id_adr.insert(id_trouve,
priv_id_adr.value(id_trouve));
} }
} }
//Copie the association id/adress
for(auto found_id : priv_id_adr.keys()) {
table_id_adr.insert(found_id,
priv_id_adr.value(found_id));
}
//load uuid of connected elements //load uuid of connected elements
QList <QDomElement> uuid_list = QET::findInDomElement(e, QList <QDomElement> uuid_list = QET::findInDomElement(e,
"links_uuids", "links_uuids",