Fix Wdeprecated-declarations QProcess::startDetached

QProcess::startDetached(const QString&)’ is deprecated
Use
QProcess::startDetached(const QString &program,
			const QStringList &arguments)
instead

elements/10_electric/91_en_60617/en_60617_11/qet_directory

@@ -3,5 +3,7 @@
         <name lang="fr">11- Schémas et plans d'installation, architecturaux et topographiques</name>
         <name lang="es">11- Dibujos y planos arquitectónicos y topográficos</name>
         <name lang="da">11- Bygningsmæssige og topografiske installationstegninger og -skemaer</name>
+        <name lang="en">11- Installation, architectural and topographical diagrams and plans</name>
+        <name lang="be">11- Installatie, bouwkundig en topografische schema's en plannen</name>
     </names>
 </qet-directory>

sources/borderproperties.cpp

@@ -27,7 +27,7 @@
 	- 17 columns of 60.0 px wide by 20.0px high
 	- 8    lines of 80.0 px high by 20.0px wide
 
-	Initialise un objet BorderProperties avec les proprietes par
+	\~French Initialise un objet BorderProperties avec les proprietes par
 	defaut suivantes :
 	- 17 colonnes affichees de 60.0 px de large pour 20.0px de haut
 	- 8    lignes affichees de 80.0 px de haut pour 20.0px de large
@@ -53,12 +53,11 @@ BorderProperties::~BorderProperties() {
 
 /**
 	@brief BorderProperties::operator ==
-	@param bp :
-	- other BorderProperties container/class
-	- autre conteneur BorderProperties
-	@return
-	- true if it and this container are identical, false otherwise
-	- true si ip et ce conteneur sont identiques, false sinon
+
+	\~ @param bp : Other BorderProperties container/class.
+	\~French Autre conteneur BorderProperties
+	\~ @return True if it and this container are identical, false otherwise.
+	\~French True si ip et ce conteneur sont identiques, false sinon
 */
 bool BorderProperties::operator==(const BorderProperties &bp) {
 	return(
@@ -75,12 +74,13 @@ bool BorderProperties::operator==(const BorderProperties &bp) {
 
 /**
 	@brief BorderProperties::operator !=
-	@param bp :
-	- other BorderProperties container/class
-	- autre conteneur BorderProperties
-	@return
-	- false if it and this container are identical, true otherwise
-	- false si bp et ce conteneur sont identiques, true sinon
+
+	\~ @param bp :
+	Other BorderProperties container/class.
+	\~French Autre conteneur BorderProperties
+	\~ @return
+	False if it and this container are identical, true otherwise.
+	\~French False si bp et ce conteneur sont identiques, true sinon
 */
 bool BorderProperties::operator!=(const BorderProperties &bp) {
 	return(!(*this == bp));
@@ -88,11 +88,12 @@ bool BorderProperties::operator!=(const BorderProperties &bp) {
 
 /**
 	@brief BorderProperties::toXml
-	- Exports dimensions as XML attributes added to element e.
-	- Exporte les dimensions sous formes d'attributs XML ajoutes a l'element e.
-	@param e :
-	- XML element to which attributes will be added
-	- Element XML auquel seront ajoutes des attributs
+	Exports dimensions as XML attributes added to element e.
+	\~French Exporte les dimensions sous formes d'attributs XML ajoutes a l'element e.
+
+	\~ @param e :
+	XML element to which attributes will be added
+	\~French Element XML auquel seront ajoutes des attributs
 */
 void BorderProperties::toXml(QDomElement &e) const {
 	e.setAttribute("cols",        columns_count);
@@ -105,11 +106,12 @@ void BorderProperties::toXml(QDomElement &e) const {
 
 /**
 	@brief BorderProperties::fromXml
-	- Import dimensions from XML attributes of element e
-	- Importe les dimensions a partir des attributs XML de l'element e
-	@param e :
-	- XML element whose attributes will be read
-	- Element XML dont les attributs seront lus
+	Import dimensions from XML attributes of element e
+	\~French Importe les dimensions a partir des attributs XML de l'element e
+
+	\~ @param e :
+	XML element whose attributes will be read
+	\~French Element XML dont les attributs seront lus
 */
 void BorderProperties::fromXml(QDomElement &e) {
 	if (e.hasAttribute("cols"))        columns_count   = e.attribute("cols").toInt();
@@ -122,14 +124,15 @@ void BorderProperties::fromXml(QDomElement &e) {
 
 /**
 	@brief BorderProperties::toSettings
-	- Export dimensions in a QSettings object.
-	- Exporte les dimensions dans une configuration.
-	@param settings :
-	- QSettings object to write
-	- Parametres a ecrire
-	@param prefix :
-	- prefix to be added before the names of the parameters
-	- prefixe a ajouter devant les noms des parametres
+	Export dimensions in a QSettings object.
+	\~French Exporte les dimensions dans une configuration.
+
+	\~ @param settings :
+	QSettings object to write
+	\~French Parametres a ecrire
+	\~ @param prefix :
+	prefix to be added before the names of the parameters
+	\~French prefixe a ajouter devant les noms des parametres
 */
 void BorderProperties::toSettings(QSettings &settings, const QString &prefix) const {
 	settings.setValue(prefix + "cols",        columns_count);
@@ -142,14 +145,12 @@ void BorderProperties::toSettings(QSettings &settings, const QString &prefix) co
 
 /**
 	@brief BorderProperties::fromSettings
-	- Import dimensions from a QSettings object.
-	- Importe les dimensions depuis une configuration.
-	@param settings :
-	- QSettings object to read
-	- Parametres a lire
-	@param prefix :
-	- prefix to be added before the names of the parameters
-	- prefixe a ajouter devant les noms des parametres
+	Import dimensions from a QSettings object.
+	\~French Importe les dimensions depuis une configuration.
+	\~ @param settings : QSettings object to read
+	\~French Parametres a lire
+	\~ @param prefix : prefix to be added before the names of the parameters
+	\~French prefixe a ajouter devant les noms des parametres
 */
 void BorderProperties::fromSettings(QSettings &settings, const QString &prefix) {
 	columns_count   = settings.value(prefix + "cols",            columns_count).toInt();

sources/bordertitleblock.h

@@ -29,6 +29,7 @@ class DiagramPosition;
 class TitleBlockTemplate;
 class TitleBlockTemplateRenderer;
 /**
+	@brief The BorderTitleBlock class
 	This class represents the border and the titleblock which frame a
 	particular electric diagram.
 */
@@ -50,30 +51,43 @@ class BorderTitleBlock : public QObject
 	
 		//METHODS TO GET DIMENSION
 		//COLUMNS
-			/// @return the number of columns
+		/// @return the number of columns
 		int   columnsCount() const { return(columns_count_); }
-			/// @return the columns width, in pixels
+		/// @return the columns width, in pixels
 		qreal columnsWidth() const { return(columns_width_); }
-			/// @return the total width of all columns, headers excluded
-		qreal columnsTotalWidth() const { return(columns_count_ * columns_width_); }
-			/// @return the column headers height, in pixels
-		qreal columnsHeaderHeight() const { return(columns_header_height_); }
+		/// @return the total width of all columns, headers excluded
+		qreal columnsTotalWidth() const {
+			return(columns_count_ * columns_width_); }
+		/// @return the column headers height, in pixels
+		qreal columnsHeaderHeight() const {
+			return(columns_header_height_); }
 	
 		//ROWS
-			/// @return the number of rows
+		/// @return the number of rows
 		int rowsCount() const { return(rows_count_); }
-			/// @return the rows height, in pixels
+		/// @return the rows height, in pixels
 		qreal rowsHeight() const { return(rows_height_); }
-			/// @return the total height of all rows, headers excluded
-		qreal rowsTotalHeight() const { return(rows_count_ * rows_height_); }
-			/// @return la rows header width, in pixels
+		/// @return the total height of all rows, headers excluded
+		qreal rowsTotalHeight() const {
+			return(rows_count_ * rows_height_); }
+		/// @return la rows header width, in pixels
 		qreal rowsHeaderWidth() const { return(rows_header_width_); }
 	
 		// border - title block = diagram
-			/// @return the diagram width, i.e. the width of the border without title block
-		qreal diagramWidth() const { return(columnsTotalWidth() + rowsHeaderWidth()); }
-			/// @return the diagram height, i.e. the height of the border without title block
-		qreal diagramHeight() const { return(rowsTotalHeight() + columnsHeaderHeight()); }
+		/**
+			@brief diagramWidth
+			@return the diagram width,
+			i.e. the width of the border without title block
+		*/
+		qreal diagramWidth() const {
+			return(columnsTotalWidth() + rowsHeaderWidth()); }
+		/**
+			@brief diagramHeight
+			@return the diagram height,
+			i.e. the height of the border without title block
+		*/
+		qreal diagramHeight() const {
+			return(rowsTotalHeight() + columnsHeaderHeight()); }
 
 		QRectF titleBlockRect () const;
 
@@ -88,45 +102,49 @@ class BorderTitleBlock : public QObject
 		QRectF outsideBorderRect() const;
 		QRectF insideBorderRect() const;
 	
-			// methods to get title block basic data
-			/// @return the value of the title block "Author" field
+		// methods to get title block basic data
+		/// @return the value of the title block "Author" field
 		QString author() const { return(btb_author_); }
-			/// @return the value of the title block "Date" field
+		/// @return the value of the title block "Date" field
 		QDate date() const { return(btb_date_); }
-			/// @return the value of the title block "Title" field
+		/// @return the value of the title block "Title" field
 		QString title() const { return(btb_title_); }
-			/// @return the value of the title block "Folio" field
+		/// @return the value of the title block "Folio" field
 		QString folio() const { return(btb_folio_); }
-			/// @return the value of the title block "Folio" field as displayed
+		/// @return the value of the title block "Folio" field as displayed
 		QString finalfolio() const { return(btb_final_folio_); }
-			/// @return the value of the title block "Plant" field
+		/// @return the value of the title block "Plant" field
 		QString plant() const { return(btb_plant_); }
-			/// @return the value of the title block "Locmach" field
+		/// @return the value of the title block "Locmach" field
 		QString locmach() const { return(btb_locmach_); }
-			/// @return the value of the revision index block "Folio" field
+		/// @return the value of the revision index block "Folio" field
 		QString indexrev() const { return(btb_indexrev_); }
-			/// @return the value of the title block "File" field
+		/// @return the value of the title block "File" field
 		QString fileName() const { return(btb_filename_); }
-			/// @return the value of the title block Additional Fields
+		/// @return the value of the title block Additional Fields
 		QString version() const { return(btb_version_); }
-			/// @return the value of the title block Additional Fields
-		DiagramContext additionalFields() const { return (additional_fields_); }
-			/// @return the value of the title block
+		/// @return the value of the title block Additional Fields
+		DiagramContext additionalFields() const {
+			return (additional_fields_); }
+		/// @return the value of the title block
 		QString autoPageNum() const { return(btb_auto_page_num_); }
-			/// @return the value of the total number of folios
+		/// @return the value of the total number of folios
 		int folioTotal() const { return(folio_total_);}
 	
-			// methods to get display options
-			/// @return true si le cartouche est affiche, false sinon
-		bool titleBlockIsDisplayed() const { return(display_titleblock_); }
-			/// @return true si les entetes des colonnes sont affiches, false sinon
+		// methods to get display options
+		/// @return true si le cartouche est affiche, false sinon
+		bool titleBlockIsDisplayed() const {
+			return(display_titleblock_); }
+		/// @return true si les entetes des colonnes sont affiches,
+		/// false sinon
 		bool columnsAreDisplayed() const { return(display_columns_); }
-			/// @return true si les entetes des lignes sont affiches, false sinon
+		/// @return true si les entetes des lignes sont affiches,
+		/// false sinon
 		bool rowsAreDisplayed() const { return(display_rows_); }
-			/// @return true si la bordure est affichee, false sinon
+		/// @return true si la bordure est affichee, false sinon
 		bool borderIsDisplayed() const { return(display_border_); }
 		
-			// methods to set dimensions
+		// methods to set dimensions
 		void setColumnsCount(int);
 		void setRowsCount(int);
 		void setColumnsWidth(const qreal &);
@@ -137,12 +155,13 @@ class BorderTitleBlock : public QObject
 		
 		DiagramPosition convertPosition(const QPointF &);
 		
-			// methods to set title block basic data
+		// methods to set title block basic data
 		void setAuthor(const QString &author);
 		void setDate(const QDate &date);
 		void setTitle(const QString &title);
 		void setFolio(const QString &folio);
-		void setFolioData(int, int, const QString& = nullptr, const DiagramContext & = DiagramContext());
+		void setFolioData(int, int, const QString& = nullptr,
+				  const DiagramContext & = DiagramContext());
 		void setPlant(const QString &plant);
 		void setLocMach(const QString &locmach);
 		void setIndicerev(const QString &indexrev);
@@ -168,9 +187,11 @@ class BorderTitleBlock : public QObject
 		
 	public slots:
 		void titleBlockTemplateChanged(const QString &);
-		void titleBlockTemplateRemoved(const QString &, const TitleBlockTemplate * = nullptr);
+		void titleBlockTemplateRemoved(
+				const QString &,
+				const TitleBlockTemplate * = nullptr);
 		
-			// methods to set display options
+		// methods to set display options
 		void displayTitleBlock(bool);
 		void displayColumns(bool);
 		void displayRows(bool);
@@ -179,77 +200,86 @@ class BorderTitleBlock : public QObject
 	
 	private:
 		void updateRectangles();
-		void updateDiagramContextForTitleBlock(const DiagramContext & = DiagramContext());
+		void updateDiagramContextForTitleBlock(
+				const DiagramContext & = DiagramContext());
 		QString incrementLetters(const QString &);
 	
 		signals:
-			/**
-				Signal emitted after the border has changed
-				@param old_border Former border
-				@param new_border New border
-			*/
+		/**
+			@brief borderChanged
+			Signal emitted after the border has changed
+			@param old_border Former border
+			@param new_border New border
+		*/
 		void borderChanged(QRectF old_border, QRectF new_border);
-			/**
-				Signal emitted after display options have changed
-			*/
+		/**
+			@brief displayChanged
+			Signal emitted after display options have changed
+		*/
 		void displayChanged();
-			/**
-				Signal emitted after the title has changed
-			*/
+		/**
+			@brief diagramTitleChanged
+			Signal emitted after the title has changed
+		*/
 		void diagramTitleChanged(const QString &);
-			/**
-			 @brief titleBlockFolioChanged
-				Signal emitted after Folio has changed
-			*/
+		/**
+			@brief titleBlockFolioChanged
+			Signal emitted after Folio has changed
+		*/
 		void titleBlockFolioChanged(const QString &);
-			/**
-				Signal emitted when the title block requires its data to be updated in order
-				to generate the folio field.
-			*/
+		/**
+			@brief needFolioData
+			Signal emitted when the title block
+			requires its data to be updated
+			in order to generate the folio field.
+		*/
 		void needFolioData();
-			/**
-				Signal emitted when this object needs to set a specific title block
-				template. This object cannot handle the job since it does not know of
-				its parent project.
-			*/
+		/**
+			@brief needTitleBlockTemplate
+			Signal emitted when this object needs to
+			set a specific title block template.
+			This object cannot handle the job
+			since it does not know of its parent project.
+		*/
 		void needTitleBlockTemplate(const QString &);
 	
-		// attributes
+	// attributes
 	private:
-			// titleblock basic data
-		QString btb_author_;
-		QDate   btb_date_;
-		QString btb_title_;
-		QString btb_folio_;
-		QString btb_plant_;
-		QString btb_locmach_;
-		QString btb_indexrev_;
-		QString btb_final_folio_;
-		QString btb_auto_page_num_;
-		int folio_index_;
-		int folio_total_;
-		QString btb_filename_;
-		QString btb_version_;
+		// titleblock basic data
+		QString btb_author_;        ///< titleblock author
+		QDate   btb_date_;          ///< titleblock date
+		QString btb_title_;         ///< titleblock title
+		QString btb_folio_;         ///< titleblock folio
+		QString btb_plant_;         ///< titleblock plant
+		QString btb_locmach_;       ///< titleblock locmach
+		QString btb_indexrev_;      ///< titleblock index rev
+		QString btb_final_folio_;   ///< titleblock final folio
+		QString btb_auto_page_num_; ///< titleblock auto page num
+		int folio_index_;           ///< titleblock index
+		int folio_total_;           ///< titleblock total
+		QString btb_filename_;      ///< titleblock filename
+		QString btb_version_;       ///< titleblock version
+		/// titleblock additional fields
 		DiagramContext additional_fields_;
-		Qt::Edge m_edge;
-		QString m_next_folio_num,
-				m_previous_folio_num;
+		Qt::Edge m_edge;              ///< titleblock edge
+		QString m_next_folio_num;     ///< titleblock next folio num
+		QString m_previous_folio_num; ///< titleblock previous folio num
 	
-			// border dimensions (rows and columns)
-			// columns: number and dimensions
-		int columns_count_;
-		qreal columns_width_;
-		qreal columns_header_height_;
+		// border dimensions (rows and columns)
+		// columns: number and dimensions
+		int columns_count_;           ///< columns count
+		qreal columns_width_;         ///< columns width
+		qreal columns_header_height_; ///< columns header height
 	
-			// rows: number and dimensions
-		int rows_count_;
-		qreal rows_height_;
-		qreal rows_header_width_;
+		// rows: number and dimensions
+		int rows_count_;          ///< rows count
+		qreal rows_height_;       ///< rows height
+		qreal rows_header_width_; ///< rows header width
 	
-			// title block dimensions
+		// title block dimensions
 		qreal titleblock_height_;
 	
-			// rectangles used for drawing operations
+		// rectangles used for drawing operations
 		QRectF diagram_rect_;
 	
 			// display options

sources/conductorautonumerotation.cpp

@@ -1,5 +1,5 @@
 /*
-	Copyright 2006-2014 The QElectroTech team
+	Copyright 2006-2020 The QElectroTech team
 	This file is part of QElectroTech.
 
 	QElectroTech is free software: you can redistribute it and/or modify
@@ -26,16 +26,23 @@
 #include "assignvariables.h"
 
 /**
- * @brief ConductorAutoNumerotation::ConductorAutoNumerotation
- * Constructor of autonum, after create a class, call numerate to apply the autonum.
- * When autonum is applyed, they do with an undo command added to the stack of diagram.
- * If you give a parent_undo at constructor, the undo command create in this class have parent_undo for parent,
- * and wasn't added to the stack of diagram (it's the responsabillty of the parent_undo)
- * @param conductor   : the conductor to apply automatic numerotation
- * @param diagram     : the diagram of conductor
- * @param parent_undo : parent undo command
- */
-ConductorAutoNumerotation::ConductorAutoNumerotation(Conductor *conductor, Diagram *diagram, QUndoCommand *parent_undo) :
+	@brief ConductorAutoNumerotation::ConductorAutoNumerotation
+	Constructor of autonum, after create a class,
+	call numerate to apply the autonum.
+	When autonum is applyed,
+	they do with an undo command added to the stack of diagram.
+	If you give a parent_undo at constructor,
+	the undo command create in this class have parent_undo for parent,
+	and wasn't added to the stack of diagram
+	(it's the responsabillty of the parent_undo)
+	@param conductor   : the conductor to apply automatic numerotation
+	@param diagram     : the diagram of conductor
+	@param parent_undo : parent undo command
+*/
+ConductorAutoNumerotation::ConductorAutoNumerotation(
+		Conductor *conductor,
+		Diagram *diagram,
+		QUndoCommand *parent_undo) :
 	m_diagram      (diagram),
 	m_conductor     (conductor),
 	conductor_list (conductor -> relatedPotentialConductors().values()),
@@ -43,19 +50,22 @@ ConductorAutoNumerotation::ConductorAutoNumerotation(Conductor *conductor, Diagr
 {}
 
 /**
- * @brief ConductorAutoNumerotation::numerate
- * execute the automatic numerotation
- */
+	@brief ConductorAutoNumerotation::numerate
+	execute the automatic numerotation
+*/
 void ConductorAutoNumerotation::numerate() {
 	if (!m_conductor) return;
 	if (conductor_list.size() >= 1 ) numeratePotential();
-	else if (m_conductor -> properties().type == ConductorProperties::Multi) numerateNewConductor();
+	else if (m_conductor -> properties().type == ConductorProperties::Multi)
+		numerateNewConductor();
 }
 
 /**
- * @brief ConductorAutoNumerotation::applyText
- * apply the text @t to @conductor_ and all conductors at the same potential
- */
+	@brief ConductorAutoNumerotation::applyText
+	apply the text @t to @conductor_
+	and all conductors at the same potential
+	@param t : Conductor text
+*/
 void ConductorAutoNumerotation::applyText(const QString& t)
 {
 	if (!m_conductor) return;
@@ -70,20 +80,36 @@ void ConductorAutoNumerotation::applyText(const QString& t)
 
 	if (m_parent_undo)
 	{
-		new QPropertyUndoCommand(m_conductor, "properties", old_value, new_value, m_parent_undo);
+		new QPropertyUndoCommand(
+					m_conductor,
+					"properties",
+					old_value,
+					new_value,
+					m_parent_undo);
 		undo = m_parent_undo;
 	}
 	else
 	{
 		undo = new QUndoCommand();
-		new QPropertyUndoCommand(m_conductor, "properties", old_value, new_value, undo);
-		undo->setText(QObject::tr("Modifier les propriétés d'un conducteur", "undo caption"));
+		new QPropertyUndoCommand(
+					m_conductor,
+					"properties",
+					old_value,
+					new_value,
+					undo);
+		undo->setText(
+			QObject::tr(
+				"Modifier les propriétés d'un conducteur",
+				"undo caption"));
 	}
 
 	if (!conductor_list.isEmpty())
 	{
 		if (!m_parent_undo)
-			undo->setText(QObject::tr("Modifier les propriétés de plusieurs conducteurs", "undo caption"));
+			undo->setText(
+				QObject::tr(
+					"Modifier les propriétés de plusieurs conducteurs",
+					"undo caption"));
 
 		foreach (Conductor *cond, conductor_list)
 		{
@@ -91,7 +117,12 @@ void ConductorAutoNumerotation::applyText(const QString& t)
 			old_value.setValue(cp2);
 			cp2.text = t;
 			new_value.setValue(cp2);
-			new QPropertyUndoCommand(cond, "properties", old_value, new_value, undo);
+			new QPropertyUndoCommand(
+						cond,
+						"properties",
+						old_value,
+						new_value,
+						undo);
 		}
 	}
 
@@ -100,15 +131,19 @@ void ConductorAutoNumerotation::applyText(const QString& t)
 }
 
 /**
- * @brief ConductorAutoNumerotation::newProperties
- * Create a new properties according to the current autonum rule of diagram
- * @param d
- * @param cp
- * @param seq
- */
-void ConductorAutoNumerotation::newProperties(Diagram *diagram, ConductorProperties &cp, autonum::sequentialNumbers &seq)
+	@brief ConductorAutoNumerotation::newProperties
+	Create a new properties according to the current autonum rule of diagram
+	@param diagram : Diagram class
+	@param cp : ConductorProperties
+	@param seq : sequentialNumbers
+*/
+void ConductorAutoNumerotation::newProperties(
+		Diagram *diagram,
+		ConductorProperties &cp,
+		autonum::sequentialNumbers &seq)
 {	
-	NumerotationContext context = diagram->project()->conductorAutoNum(diagram->conductorsAutonumName());
+	NumerotationContext context = diagram->project()->conductorAutoNum(
+				diagram->conductorsAutonumName());
 	if (context.isEmpty()) {
 		return;
 	}
@@ -124,9 +159,9 @@ void ConductorAutoNumerotation::newProperties(Diagram *diagram, ConductorPropert
 }
 
 /**
- * @brief ConductorAutoNumerotation::numeratePotential
- * Numerate a conductor on an existing potential
- */
+	@brief ConductorAutoNumerotation::numeratePotential
+	Numerate a conductor on an existing potential
+*/
 void ConductorAutoNumerotation::numeratePotential()
 {
 	ConductorProperties cp = conductor_list.first()->properties();
@@ -136,7 +171,8 @@ void ConductorAutoNumerotation::numeratePotential()
 		if (conductor->properties() != cp)
 			properties_equal = false;
 	}
-		//Every properties of the potential is equal, so we apply it to m_conductor
+	// Every properties of the potential is equal,
+	// so we apply it to m_conductor
 	if (properties_equal)
 	{
 		m_conductor->setProperties(cp);
@@ -169,21 +205,25 @@ void ConductorAutoNumerotation::numeratePotential()
 		//the texts isn't identicals
 	else
 	{
-		PotentialSelectorDialog psd(m_conductor, m_parent_undo, m_conductor->diagramEditor());
+		PotentialSelectorDialog psd(
+					m_conductor,
+					m_parent_undo,
+					m_conductor->diagramEditor());
 		psd.exec();
 	}
 }
 
 /**
- * @brief ConductorAutoNumerotation::numerateNewConductor
- * create and apply a new numerotation to @m_conductor
- */
+	@brief ConductorAutoNumerotation::numerateNewConductor
+	create and apply a new numerotation to @m_conductor
+*/
 void ConductorAutoNumerotation::numerateNewConductor()
 {
 	if (!m_conductor || m_diagram->conductorsAutonumName().isEmpty())
 		return;
 
-	NumerotationContext context = m_diagram->project()->conductorAutoNum(m_diagram -> conductorsAutonumName());
+	NumerotationContext context = m_diagram->project()->conductorAutoNum(
+				m_diagram -> conductorsAutonumName());
 	if (context.isEmpty())
 		return;
 
@@ -194,10 +234,17 @@ void ConductorAutoNumerotation::numerateNewConductor()
 	cp.m_formula = formula;
 	m_conductor->setProperties(cp);
 
-	autonum::setSequential(formula, m_conductor->rSequenceNum(), context, m_diagram, autoNum_name);
+	autonum::setSequential(formula,
+			       m_conductor->rSequenceNum(),
+			       context,
+			       m_diagram,
+			       autoNum_name);
 
 	NumerotationContextCommands ncc (context, m_diagram);
 	m_diagram->project()->addConductorAutoNum(autoNum_name, ncc.next());
 
-	applyText(autonum::AssignVariables::formulaToLabel(formula, m_conductor->rSequenceNum(), m_diagram));
+	applyText(autonum::AssignVariables::formulaToLabel(
+			  formula,
+			  m_conductor->rSequenceNum(),
+			  m_diagram));
 }

sources/conductorautonumerotation.h

@@ -1,5 +1,5 @@
 /*
-	Copyright 2006-2014 The QElectroTech team
+	Copyright 2006-2020 The QElectroTech team
 	This file is part of QElectroTech.
 
 	QElectroTech is free software: you can redistribute it and/or modify
@@ -26,19 +26,26 @@ class Conductor;
 class QUndoCommand;
 class ConductorProperties;
 
+/**
+	@brief The ConductorAutoNumerotation class
+*/
 class ConductorAutoNumerotation
 {
 	public:
 		//constructors & destructor
-		ConductorAutoNumerotation (Conductor *conductor, Diagram *diagram, QUndoCommand *undo_parent = nullptr);
+		ConductorAutoNumerotation (Conductor *conductor,
+					   Diagram *diagram,
+					   QUndoCommand *undo_parent = nullptr);
 
-			//methods
+		//methods
 		void        numerate       ();
 		void        applyText      (const QString&);
-		static void newProperties(Diagram *diagram, ConductorProperties &cp, autonum::sequentialNumbers &seq);
+		static void newProperties(Diagram *diagram,
+					  ConductorProperties &cp,
+					  autonum::sequentialNumbers &seq);
 
 	private:
-			//methods
+		//methods
 		void numeratePotential ();
 		void numerateNewConductor ();
 

sources/diagram.cpp

@@ -927,8 +927,6 @@ bool Diagram::fromXml(QDomElement &document, QPointF position, bool consider_inf
 	const QDomElement& root = document;
 	// The first element must be a diagram
 	if (root.tagName() != "diagram") return(false);
-
-    qDebug() << "Diagram: " << root.attribute("title");
 	
 	// Read attributes of this diagram
 	if (consider_informations) {
@@ -1025,7 +1023,6 @@ bool Diagram::fromXml(QDomElement &document, QPointF position, bool consider_inf
 		}
 		
 		int state = 0;
-        qDebug() << "Element from collection: " << element_location;
 		Element *nvel_elmt = ElementFactory::Instance() -> createElement(element_location, nullptr, &state);
 		if (state)
 		{

sources/diagramcontext.cpp

@@ -118,7 +118,7 @@ int DiagramContext::count() {
 
 /**
  * @brief DiagramContext::keyMustShow
- * @return the value pairs with key, if key not found, return false
+ * @return the value pairs with key, if key no found, return false
  */
 bool DiagramContext::keyMustShow(const QString &key) const {
 	if (m_content_show.contains(key))

sources/qetdiagrameditor.cpp

@@ -2220,19 +2220,19 @@ void QETDiagramEditor::generateTerminalBlock()
 		success = process->startDetached("qet_tb_generator", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached("qet_tb_generator");
+		success = process->startDetached("qet_tb_generator", {("")});
 	}
 	if (openedProjects().count()){
 		success = process->startDetached(QDir::homePath() + "/Application Data/qet/qet_tb_generator.exe", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached(QDir::homePath() + "/Application Data/qet/qet_tb_generator.exe");
+		success = process->startDetached(QDir::homePath() + "/Application Data/qet/qet_tb_generator.exe", {("")});
 	}
 	if (openedProjects().count()){
 		success = process->startDetached(QDir::homePath() + "/qet_tb_generator.exe", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached(QDir::homePath() + "/qet_tb_generator.exe");
+		success = process->startDetached(QDir::homePath() + "/qet_tb_generator.exe", {("")});
 	}
 
 #elif  defined(Q_OS_MAC)
@@ -2240,13 +2240,13 @@ void QETDiagramEditor::generateTerminalBlock()
 		success = process->startDetached("/Library/Frameworks/Python.framework/Versions/3.8/bin/qet_tb_generator", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached("/Library/Frameworks/Python.framework/Versions/3.8/bin/qet_tb_generator");
+		success = process->startDetached("/Library/Frameworks/Python.framework/Versions/3.8/bin/qet_tb_generator", {("")});
 	}
 		if (openedProjects().count()){
 		success = process->startDetached(QDir::homePath() + "/.qet/qet_tb_generator.app", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached(QDir::homePath() + "/.qet/qet_tb_generator.app");
+		success = process->startDetached(QDir::homePath() + "/.qet/qet_tb_generator.app", {("")});
 	}
 	
 #else
@@ -2254,13 +2254,13 @@ void QETDiagramEditor::generateTerminalBlock()
 		success = process->startDetached("qet_tb_generator", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached("qet_tb_generator");
+		success = process->startDetached("qet_tb_generator", {("")});
 	}
 	if (openedProjects().count()){
 		success = process->startDetached(QDir::homePath() + "/.qet/qet_tb_generator", {(QETDiagramEditor::currentProjectView()->project()->filePath())});
 	}
 	else  {
-		success = process->startDetached(QDir::homePath() + "/.qet/qet_tb_generator");
+		success = process->startDetached(QDir::homePath() + "/.qet/qet_tb_generator", {("")});
 	}
 	
 #endif

sources/qetgraphicsitem/element.cpp

@@ -418,7 +418,6 @@ bool Element::buildFromXml(const QDomElement &xml_def_elmt, int *state)
 		{
 				//Minor workaround to find if there is a "input" tagg as label.
 				//If not, we set the tagg "label" to the first "input.
-            // TODO: Remove in later version begin
 			QList <QDomElement> input_field;
 			bool have_label = false;
 			for (QDomElement input_node = node.firstChildElement("input") ; !input_node.isNull() ; input_node = input_node.nextSiblingElement("input"))
@@ -430,11 +429,8 @@ bool Element::buildFromXml(const QDomElement &xml_def_elmt, int *state)
 						have_label = true;
 				}
 			}
-            if(!have_label && !input_field.isEmpty()) {
-                qDebug() << input_field.first().attribute("text");
+			if(!have_label && !input_field.isEmpty())
 				input_field.first().setAttribute("tagg", "label");
-            }
-            // TODO: Remove in later version end
 			
 				//Parse the definition
 			for (QDomNode n = node.firstChild() ; !n.isNull() ; n = n.nextSibling())
@@ -720,21 +716,16 @@ bool Element::fromXml(QDomElement &e, QHash<int, Terminal *> &table_id_adr, bool
 	} else {
 		setRotation(90*read_ori);
 	}
-
-    // TODO: elementInformations are not read from the diagram section or? Yes see below
 	
-//#######################################################################################################################
-//####    START   #######################################################################################################
-//#######################################################################################################################
-        //Before load the dynamic text field,
-        //we remove the dynamic text field created from the description of this element, to avoid doublons.
+		//Befor load the dynamic text field,
+		//we remove the dynamic text field created from the description of this element, to avoid doublons.
 	for(DynamicElementTextItem *deti : m_dynamic_text_list)
 		delete deti;
 	m_dynamic_text_list.clear();
+    
 		//************************//
 		//***Dynamic texts item***//
 		//************************//
-        // Texts in diagram section
 	for (const QDomElement& qde : QET::findInDomElement(e, "dynamic_texts", DynamicElementTextItem::xmlTagName()))
     {
         DynamicElementTextItem *deti = new DynamicElementTextItem(this);
@@ -742,24 +733,20 @@ bool Element::fromXml(QDomElement &e, QHash<int, Terminal *> &table_id_adr, bool
         deti->fromXml(qde);
     }
 	
-    // TODO: remove in later version begin
 
 		//************************//
 		//***Element texts item***//
 		//************************//
-        // Inputs in diagram section
-    QList<QDomElement> inputs = QET::findInDomElement(e, "inputs", "input"); // Inputs in diagram section
+	QList<QDomElement> inputs = QET::findInDomElement(e, "inputs", "input");
 	
 		//First case, we check for the text item converted to dynamic text item
-    const QList <DynamicElementTextItem *> conv_deti_list = m_converted_text_from_xml_description.keys(); // Texts in element definition (elements collection)
+	const QList <DynamicElementTextItem *> conv_deti_list = m_converted_text_from_xml_description.keys();
 	QList <DynamicElementTextItem *> successfully_converted; 
 	const QList <QDomElement> dom_inputs = inputs;
 	
-
-
-    for (DynamicElementTextItem *deti : conv_deti_list) // Texts from element definition (elements collection)
+	for (DynamicElementTextItem *deti : conv_deti_list)
 	{
-        for(const QDomElement& dom_input : dom_inputs) // Inputs in diagram section
+		for(const QDomElement& dom_input : dom_inputs)
 		{
 				//we use the same method used in ElementTextItem::fromXml to compar and know if the input dom element is for one of the text stored.
 				//The comparaison is made from the text position : if the position of the text is the same as the position stored in 'input' dom element
@@ -830,7 +817,7 @@ bool Element::fromXml(QDomElement &e, QHash<int, Terminal *> &table_id_adr, bool
 		group->fromXml(qde);
 	}
 	
-        //load elementInformations from diagram section
+		//load informations
 	DiagramContext dc;
 	dc.fromXml(e.firstChildElement("elementInformations"), "elementInformation");
 		/**
@@ -1004,12 +991,6 @@ bool Element::fromXml(QDomElement &e, QHash<int, Terminal *> &table_id_adr, bool
 			}
 		}
 	}
-
-    // TODO: remove in later version end
-
-//#######################################################################################################################
-//####    END     #######################################################################################################
-//#######################################################################################################################
     m_state = QET::GIOK;
 	return(true);
 }

sources/ui/dynamicelementtextitemeditor.cpp

@@ -186,10 +186,6 @@ QUndoCommand *DynamicElementTextItemEditor::associatedUndo() const
 		return nullptr;
 }
 
-/*!
- * \brief DynamicElementTextItemEditor::dataEdited
- * Apply changes to the text item. It is called when dataChanged() signal is called
- */
 void DynamicElementTextItemEditor::dataEdited()
 {
 	if (m_live_edit)

sources/ui/dynamicelementtextitemeditor.h

@@ -29,10 +29,6 @@ namespace Ui {
 	class DynamicElementTextItemEditor;
 }
 
-/*!
- * \brief The DynamicElementTextItemEditor class
- * Widget used to modify the properties of DynamicElementTextItem.
- */
 class DynamicElementTextItemEditor : public AbstractElementPropertiesEditorWidget
 {
     Q_OBJECT

sources/ui/dynamicelementtextmodel.cpp

@@ -57,10 +57,6 @@ static int adjust_grp_row         = 4;
 static int frame_grp_row          = 5;
 static int hold_to_bottom_grp_row = 6;
 
-const char* user_text = "Texte utilisateur";
-const char* element_information = "Information de l'élément";
-const char* composite_text = "Texte composé";
-
 DynamicElementTextModel::DynamicElementTextModel(Element *element, QObject *parent) :
 	QStandardItemModel(parent),
 	m_element(element)
@@ -140,9 +136,9 @@ QList<QStandardItem *> DynamicElementTextModel::itemsForText(DynamicElementTextI
     src->setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled);
     
 	QString title;
-    if (deti->textFrom() == DynamicElementTextItem::UserText) title = tr(user_text);
-    else if (deti->textFrom() == DynamicElementTextItem::ElementInfo) title = tr(element_information);
-    else title =  tr(composite_text);
+	if (deti->textFrom() == DynamicElementTextItem::UserText) title = tr("Texte utilisateur");
+	else if (deti->textFrom() == DynamicElementTextItem::ElementInfo) title = tr("Information de l'élément");
+	else title =  tr("Texte composé");
     QStandardItem *srca = new QStandardItem(title);
     srca->setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsEditable);
     srca->setData(textFrom, Qt::UserRole+1);
@@ -176,7 +172,7 @@ QList<QStandardItem *> DynamicElementTextModel::itemsForText(DynamicElementTextI
     qsi->appendRow(qsi_list);
 	
 		//Composite text
-    QStandardItem *composite = new QStandardItem(tr(composite_text));
+	QStandardItem *composite = new QStandardItem(tr("Texte composé"));
 	composite->setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled);
 	
 	DiagramContext dc;
@@ -401,7 +397,7 @@ DynamicElementTextItem *DynamicElementTextModel::textFromItem(QStandardItem *ite
 	
 	
 	QStandardItem *text_item = item;
-    while (text_item->parent()) // recursive search for the highest parent
+	while (text_item->parent())
 		text_item = text_item->parent();
 	
 	if (m_texts_list.values().contains(text_item)) //The item is a text
@@ -462,26 +458,26 @@ QUndoCommand *DynamicElementTextModel::undoForEditedText(DynamicElementTextItem
 	QStandardItem *text_qsi = m_texts_list.value(deti);
 	
 	QString from = text_qsi->child(src_txt_row,1)->data(Qt::DisplayRole).toString();
-    if ((from == tr(user_text)) && (deti->textFrom() != DynamicElementTextItem::UserText))
+	if ((from == tr("Texte utilisateur")) && (deti->textFrom() != DynamicElementTextItem::UserText))
 		new QPropertyUndoCommand(deti, "textFrom", QVariant(deti->textFrom()), QVariant(DynamicElementTextItem::UserText), undo);
-    else if ((from == tr(element_information)) && (deti->textFrom() != DynamicElementTextItem::ElementInfo))
+	else if ((from == tr("Information de l'élément")) && (deti->textFrom() != DynamicElementTextItem::ElementInfo))
 		new QPropertyUndoCommand(deti, "textFrom", QVariant(deti->textFrom()), QVariant(DynamicElementTextItem::ElementInfo), undo);
-    else if ((from == tr(composite_text)) && (deti->textFrom() != DynamicElementTextItem::CompositeText))
+	else if ((from == tr("Texte composé")) && (deti->textFrom() != DynamicElementTextItem::CompositeText))
 		new QPropertyUndoCommand(deti, "textFrom", QVariant(deti->textFrom()), QVariant(DynamicElementTextItem::CompositeText), undo);
 	
-    if(from == tr(user_text))
+	if(from == tr("Texte utilisateur"))
 	{
 		QString text = text_qsi->child(usr_txt_row,1)->data(Qt::DisplayRole).toString();
 		if (text != deti->text())
 			new QPropertyUndoCommand(deti, "text", QVariant(deti->text()), QVariant(text), undo);
 	}
-    else if (from == tr(element_information))
+	else if (from == tr("Information de l'élément"))
 	{
 		QString info_name = text_qsi->child(info_txt_row,1)->data(Qt::UserRole+2).toString();
 		if(info_name != deti->infoName())
 			new QPropertyUndoCommand(deti, "infoName", QVariant(deti->infoName()), QVariant(info_name), undo);
 	}
-    else if (from == tr(composite_text))
+	else if (from == tr("Texte composé"))
 	{
 		QString composite_text = text_qsi->child(compo_txt_row,1)->data(Qt::UserRole+2).toString();
 		if(composite_text != deti->compositeText())
@@ -1116,7 +1112,6 @@ void DynamicElementTextModel::enableSourceText(DynamicElementTextItem *deti, Dyn
 		case DynamicElementTextItem::CompositeText: compo = true;break;
 	}
 
-    m_ignore_item_change = true;
 		//User text
 	qsi->child(usr_txt_row,0)->setEnabled(usr);
 	qsi->child(usr_txt_row,1)->setEnabled(usr);
@@ -1125,7 +1120,6 @@ void DynamicElementTextModel::enableSourceText(DynamicElementTextItem *deti, Dyn
 	qsi->child(info_txt_row,1)->setEnabled(info);
 		//Composite text
 	qsi->child(compo_txt_row,0)->setEnabled(compo);
-    m_ignore_item_change = false;
 	qsi->child(compo_txt_row,1)->setEnabled(compo);
 }
 
@@ -1163,10 +1157,6 @@ void DynamicElementTextModel::enableGroupRotationAndPos(ElementTextItemGroup *gr
 
 void DynamicElementTextModel::itemDataChanged(QStandardItem *qsi)
 {
-
-    if (m_ignore_item_change)
-        return;
-
 	DynamicElementTextItem *deti = textFromItem(qsi);
 	ElementTextItemGroup *etig = groupFromItem(qsi);
 	if (!deti && !etig)
@@ -1180,15 +1170,15 @@ void DynamicElementTextModel::itemDataChanged(QStandardItem *qsi)
 			dc = deti->elementUseForInfo()->elementInformations();
 		
 		if (qsi->data().toInt() == textFrom)
-        { // change textFrom
+		{
 			QString from = qsi->data(Qt::DisplayRole).toString();
 			
-            if (from == tr(user_text))
+			if (from == tr("Texte utilisateur"))
 			{
 				enableSourceText(deti, DynamicElementTextItem::UserText);
 				text_qsi->setData(text_qsi->child(usr_txt_row,1)->data(Qt::DisplayRole).toString());
 			}
-            else if (from == tr(element_information))
+			else if (from == tr("Information de l'élément"))
 			{
 				enableSourceText(deti, DynamicElementTextItem::ElementInfo);
 				QString info = text_qsi->child(info_txt_row,1)->data(Qt::UserRole+2).toString();
@@ -1223,7 +1213,7 @@ void DynamicElementTextModel::itemDataChanged(QStandardItem *qsi)
 		//We emit the signal only if @qsi is in the second column, because the data are stored on this column
 		//the first column is use only for display the title of the property, except for the name of texts group
 	if((m_groups_list.values().contains(qsi) || qsi->column() == 1) && !m_block_dataChanged)
-        emit dataChanged(); // TODO: where it goes?
+		emit dataChanged();
     if(deti) deti->updateXref();
 }
 
@@ -1319,9 +1309,9 @@ void DynamicElementTextModel::updateDataFromText(DynamicElementTextItem *deti, V
 		{
 			switch (deti->textFrom())
 			{
-                case DynamicElementTextItem::UserText: qsi->child(0,1)->setData(tr(user_text), Qt::DisplayRole); break;
-                case DynamicElementTextItem::ElementInfo : qsi->child(0,1)->setData(tr(element_information), Qt::DisplayRole); break;
-                case DynamicElementTextItem::CompositeText : qsi->child(0,1)->setData(tr(composite_text), Qt::DisplayRole); break;
+				case DynamicElementTextItem::UserText: qsi->child(0,1)->setData(tr("Texte utilisateur"), Qt::DisplayRole); break;
+				case DynamicElementTextItem::ElementInfo : qsi->child(0,1)->setData(tr("Information de l'élément"), Qt::DisplayRole); break;
+				case DynamicElementTextItem::CompositeText : qsi->child(0,1)->setData(tr("Texte composé"), Qt::DisplayRole); break;
 			}
 			enableSourceText(deti, deti->textFrom());
 			qsi->setData(deti->toPlainText(), Qt::DisplayRole);
@@ -1458,15 +1448,6 @@ DynamicTextItemDelegate::DynamicTextItemDelegate(QObject *parent) :
     QStyledItemDelegate(parent)
 {}
 
-/*!
- * \brief DynamicTextItemDelegate::createEditor
- * Different widgets are used depending of the data which should be shown in the view.
- * For example for choosing color, a color chooser is used.
- * \param parent
- * \param option
- * \param index
- * \return
- */
 QWidget *DynamicTextItemDelegate::createEditor(QWidget *parent, const QStyleOptionViewItem &option, const QModelIndex &index) const
 {
 	switch (index.data(Qt::UserRole+1).toInt())
@@ -1475,9 +1456,9 @@ QWidget *DynamicTextItemDelegate::createEditor(QWidget *parent, const QStyleOpti
 		{
 			QComboBox *qcb = new QComboBox(parent);
 			qcb->setObjectName("text_from");
-            qcb->addItem(tr(user_text));
-            qcb->addItem(tr(element_information));
-            qcb->addItem(tr(composite_text));
+			qcb->addItem(tr("Texte utilisateur"));
+			qcb->addItem(tr("Information de l'élément"));
+			qcb->addItem(tr("Texte composé"));
 			return qcb;
 		}
 		case DynamicElementTextModel::infoText:

sources/ui/dynamicelementtextmodel.h

@@ -106,7 +106,6 @@ class DynamicElementTextModel : public QStandardItemModel
 	QHash <DynamicElementTextItem *, QList<QMetaObject::Connection>> m_hash_text_connect;
 	QHash <ElementTextItemGroup *, QList<QMetaObject::Connection>> m_hash_group_connect;
 	bool m_block_dataChanged = false;
-    bool m_ignore_item_change{false};
 };
 
 class DynamicTextItemDelegate : public QStyledItemDelegate