Topic: BUG ReadyToUse : 8275 <version <8313
Bonjour,
un bug est apparut entre la 8275 sur la 8313: les variables de projet qui n'ont pas de valeur ou un espace en valeur disparaissent après avoir enregistré, puis réouvert le projet.
Merci.
Re: BUG ReadyToUse : 8275 <version <8313
La 8290 n'a pas le problème
- scorpio810
- QElectroTech Team Manager, Developer, Packager
- Offline
Re: BUG ReadyToUse : 8275 <version <8313
Bonjour Alexis,
merci du retour,
peux tu être plus précis en donnant le sha5 des commits?
galexis wrote:
La 8290 n'a pas le problème
Installer_QElectroTech-0.100.0_x86_64-win64+git8290-1.exe 2025-01-30 10:29 23M
"Le jour où tu découvres le Libre, tu sais que tu ne pourras jamais plus revenir en arrière..."Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
Re: BUG ReadyToUse : 8275 <version <8313
galexis wrote:
(...) les variables de projet qui n'ont pas de valeur ou un espace en valeur disparaissent après avoir enregistré, puis réouvert le projet.
To avoid misunderstandings:
What do you mean with "variables de projet"?
Maybe the following?
In this commit there was a change with element-information:
In the entries leading and trailing whitespace is removed from content and entries without content are not saved anymore. Why should we save overhead without additional information?
Fragen zu QET gehören in dieses Forum und werden nicht per PM beantwortet! – Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
Re: BUG ReadyToUse : 8275 <version <8313
O.k. ... guess I found what you mean.
Is it about such entries in project-file?
<properties>
<property name="creator" show="1"></property>
<property name="plant" show="1"></property>
<property name="author" show="1"></property>
<property name="folio" show="1"></property>
(...)That piece of code changed in the commit is used more than once:
Not just for element-information but also for properties in project-file.
Guess what you mean:
These entries can be used in a "template" for use with new projects.
Then we could differentiate:
- do not save empty information in elements (these are fixed in sourcecode)
- keep property-entries in project-files (entries can be added during runtime)
What do you think?
Fragen zu QET gehören in dieses Forum und werden nicht per PM beantwortet! – Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
- scorpio810
- QElectroTech Team Manager, Developer, Packager
- Offline
Re: BUG ReadyToUse : 8275 <version <8313
Original
No need to edit old variables, empty variables was affiched! just saving with new 0.100-dev versions!
"Le jour où tu découvres le Libre, tu sais que tu ne pourras jamais plus revenir en arrière..."Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
- scorpio810
- QElectroTech Team Manager, Developer, Packager
- Offline
Re: BUG ReadyToUse : 8275 <version <8313
We should wait for a response from Galexis before correcting the code.
As far as I'm concerned, if the variables are empty, there's no need to display them in the project!
"Le jour où tu découvres le Libre, tu sais que tu ne pourras jamais plus revenir en arrière..."Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
Re: BUG ReadyToUse : 8275 <version <8313
That would mean, we only remove empty entries with element-information in element-files, but keep all other properties with empty content.
I would like to remove empty entries in elementInformation from element-files, because it would be a massive overhead and useless information, if an element like an ordinary terminal would have the empty information about up to four auxiliary parts...
Already prepared this diff:
diff --git a/sources/diagramcontext.cpp b/sources/diagramcontext.cpp
index 864ad9b3a..681fed49e 100644
--- a/sources/diagramcontext.cpp
+++ b/sources/diagramcontext.cpp
@@ -151,7 +151,10 @@ bool DiagramContext::operator!=(const DiagramContext &dc) const
void DiagramContext::toXml(QDomElement &e, const QString &tag_name) const
{
foreach (QString key, keys()) {
- if (m_content[key].toString().trimmed().isEmpty()) { continue; }
+ if ((tag_name == "elementInformation") &&
+ (m_content[key].toString().trimmed().isEmpty())) {
+ continue;
+ }
QDomElement property = e.ownerDocument().createElement(tag_name);
// try to sort attributes by removing and re-adding
property.removeAttribute("show");What do you think, Galexis and Laurent?
Fragen zu QET gehören in dieses Forum und werden nicht per PM beantwortet! – Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
Re: BUG ReadyToUse : 8275 <version <8313
scorpio810 wrote:
Original
No need to edit old variables, empty variables was affiched! just saving with new 0.100-dev versions!
Je regarderais les sha5 lundi. Depuis toujours les variables de projet même vides sont sauvegardées. Sinon on se retrouve avec des %myvariable dans les cartouches comme dans ta capture d'écran. Pour la version qui fonctionne, voici la capture de téléchargement. Je n'ai pas eu le temps de tester les versions suivantes jusqu'à la 8313 pour voir quand le problème arrive, mon chef était à mon bureau !
Post's attachments
Capture dâécran du 2025-02-07 20-16-40.png 49.1 kb, 13 downloads since 2025-02-07
Re: BUG ReadyToUse : 8275 <version <8313
Sous linuxmint : Built with Qt 5.15.13 - x86_64 - Date : Feb 6 2025 : 15:42:39 , même problème
Re: BUG ReadyToUse : 8275 <version <8313
Salut Galexis !
Did you read my posts?
especially this one: https://qelectrotech.org/forum/viewtopi … 021#p21021
When the problem exists, it exists in all current flavors of QET!
But as I wrote:
I think I found what you mean and can fix it with the patch in this post:
https://qelectrotech.org/forum/viewtopi … 024#p21024
In the meantime I created a project with empty properties, saved the file and re-opened it again – et voila: see attachment!
Empty properties are saved and reloaded!
Post's attachments
Bildschirmfoto_2025-02-07_user-properties_of_folio.png 45.43 kb, 11 downloads since 2025-02-07
Fragen zu QET gehören in dieses Forum und werden nicht per PM beantwortet! – Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
Re: BUG ReadyToUse : 8275 <version <8313
plc-user wrote:
Salut Galexis !
Did you read my posts?
especially this one: https://qelectrotech.org/forum/viewtopi … 021#p21021When the problem exists, it exists in all current flavors of QET!
But as I wrote:
I think I found what you mean and can fix it with the patch in this post:
https://qelectrotech.org/forum/viewtopi … 024#p21024In the meantime I created a project with empty properties, saved the file and re-opened it again – et voila: see attachment!
Empty properties are saved and reloaded!
It's should be perfect, like before.
13 2025-02-07 22:38:04 (edited by plc-user 2025-02-08 08:35:10)
Re: BUG ReadyToUse : 8275 <version <8313
Edited post!
-------------------------------------------------
Hallo Galexis!
Kannst Du kurz erklären, wofür du diese leeren Eigenschaftsfelder im Projekt benutzt?
(...)
-------------------------------------------------
Salut Galexis !
Peux-tu expliquer brièvement à quoi tu utilises ces champs de propriétés vides dans le projet ?
(...)
-------------------------------------------------
withdraw the question:
If not all the information from the template is entered when a folio is created, the fields should still remain so that something can be entered later.
I will put the diff above in a PR.
Fragen zu QET gehören in dieses Forum und werden nicht per PM beantwortet! – Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
- scorpio810
- QElectroTech Team Manager, Developer, Packager
- Offline
Re: BUG ReadyToUse : 8275 <version <8313
Hallo plc-user,
see this project in examples directory: Projet_vierge.qet
"Le jour où tu découvres le Libre, tu sais que tu ne pourras jamais plus revenir en arrière..."Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !
- scorpio810
- QElectroTech Team Manager, Developer, Packager
- Offline
Re: BUG ReadyToUse : 8275 <version <8313
Your diff seem's to be fixing this issue.
"Le jour où tu découvres le Libre, tu sais que tu ne pourras jamais plus revenir en arrière..."Questions regarding QET belong in this forum and will NOT be answered via PM! – Les questions concernant QET doivent être posées sur ce forum et ne seront pas traitées par MP !