diff --git a/sources/autoNum/assignvariables.cpp b/sources/autoNum/assignvariables.cpp index f22449de8..2cafd7b12 100644 --- a/sources/autoNum/assignvariables.cpp +++ b/sources/autoNum/assignvariables.cpp @@ -27,24 +27,13 @@ namespace autonum { + /** * @brief sequentialNumbers::sequentialNumbers */ sequentialNumbers::sequentialNumbers() {} - sequentialNumbers::sequentialNumbers(const sequentialNumbers &other) - { - unit = other.unit; - unit_folio = other.unit_folio; - ten = other.ten; - ten_folio = other.ten_folio; - hundred = other.hundred; - hundred_folio = other.hundred_folio; - } - - sequentialNumbers::~sequentialNumbers() {} - sequentialNumbers &sequentialNumbers::operator=(const sequentialNumbers &other) { if (&other == this || other == *this) @@ -227,22 +216,22 @@ namespace autonum for (int i=1; i<=max ; i++) { - if (m_assigned_label.contains("%sequ_" + QString::number(i)) && m_seq_struct.unit.size() >= i) { + if (m_assigned_label.contains("%sequ_" + QString::number(i)) && !m_seq_struct.unit.isEmpty()) { m_assigned_label.replace("%sequ_" + QString::number(i),m_seq_struct.unit.at(i-1)); } - if (m_assigned_label.contains("%seqt_" + QString::number(i)) && m_seq_struct.ten.size() >= i) { + if (m_assigned_label.contains("%seqt_" + QString::number(i)) && !m_seq_struct.ten.isEmpty()) { m_assigned_label.replace("%seqt_" + QString::number(i),m_seq_struct.ten.at(i-1)); } - if (m_assigned_label.contains("%seqh_" + QString::number(i)) && m_seq_struct.hundred.size() >= i) { + if (m_assigned_label.contains("%seqh_" + QString::number(i)) && !m_seq_struct.hundred.isEmpty()) { m_assigned_label.replace("%seqh_" + QString::number(i),m_seq_struct.hundred.at(i-1)); } - if (m_assigned_label.contains("%sequf_" + QString::number(i)) && m_seq_struct.unit_folio.size() >= i) { + if (m_assigned_label.contains("%sequf_" + QString::number(i)) && !m_seq_struct.unit_folio.isEmpty()) { m_assigned_label.replace("%sequf_" + QString::number(i),m_seq_struct.unit_folio.at(i-1)); } - if (m_assigned_label.contains("%seqtf_" + QString::number(i)) && m_seq_struct.ten_folio.size() >= i) { + if (m_assigned_label.contains("%seqtf_" + QString::number(i)) && !m_seq_struct.ten_folio.isEmpty()) { m_assigned_label.replace("%seqtf_" + QString::number(i),m_seq_struct.ten_folio.at(i-1)); } - if (m_assigned_label.contains("%seqhf_" + QString::number(i)) && m_seq_struct.hundred_folio.size() >= i) { + if (m_assigned_label.contains("%seqhf_" + QString::number(i)) && !m_seq_struct.hundred_folio.isEmpty()) { m_assigned_label.replace("%seqhf_" + QString::number(i),m_seq_struct.hundred_folio.at(i-1)); } } diff --git a/sources/autoNum/assignvariables.h b/sources/autoNum/assignvariables.h index 712f215c6..86d936cf7 100644 --- a/sources/autoNum/assignvariables.h +++ b/sources/autoNum/assignvariables.h @@ -34,8 +34,6 @@ namespace autonum { public: sequentialNumbers(); - sequentialNumbers(const sequentialNumbers &other); - ~sequentialNumbers(); sequentialNumbers &operator= (const sequentialNumbers &other); bool operator== (const sequentialNumbers &other) const; @@ -82,6 +80,4 @@ namespace autonum QString elementPrefixForLocation(const ElementsLocation &location); } -Q_DECLARE_METATYPE(autonum::sequentialNumbers) - #endif // ASSIGNVARIABLES_H diff --git a/sources/conductorautonumerotation.cpp b/sources/conductorautonumerotation.cpp index 8c2fd72d5..de1172df8 100644 --- a/sources/conductorautonumerotation.cpp +++ b/sources/conductorautonumerotation.cpp @@ -121,8 +121,9 @@ void ConductorAutoNumerotation::numeratePotential() ConductorProperties cp = m_conductor -> properties(); cp.text = text_list.first(); cp.m_formula = formula_list.first(); - m_conductor->rSequenceNum() = conductor_list.first()->sequenceNum(); m_conductor->setProperties(cp); + m_conductor->rSequenceNum() = conductor_list.first()->sequenceNum(); + m_conductor->setText(text_list.first()); } //the texts isn't identicals else diff --git a/sources/diagram.cpp b/sources/diagram.cpp index d4808ac95..ce6abd2c5 100644 --- a/sources/diagram.cpp +++ b/sources/diagram.cpp @@ -1158,7 +1158,8 @@ void Diagram::updateLabels() { elmt->updateLabel(); } foreach (Conductor *cnd, content().conductors()) { - cnd->refreshText(); + if (cnd->properties().text.contains("%F")) + cnd->setText(cnd->properties().text); } } diff --git a/sources/qetgraphicsitem/conductor.cpp b/sources/qetgraphicsitem/conductor.cpp index 8b96d5d74..3f43d63c0 100644 --- a/sources/qetgraphicsitem/conductor.cpp +++ b/sources/qetgraphicsitem/conductor.cpp @@ -79,7 +79,7 @@ Conductor::Conductor(Terminal *p1, Terminal* p2) : terminal2(p2), m_mouse_over(false), m_handler(10), - m_text_item(0), + text_item(0), segments(NULL), moving_segment(false), modified_path(false), @@ -127,8 +127,8 @@ Conductor::Conductor(Terminal *p1, Terminal* p2) : setAcceptHoverEvents(true); // Add the text field - m_text_item = new ConductorTextItem(m_properties.text, this); - connect(m_text_item, &ConductorTextItem::diagramTextChanged, this, &Conductor::displayedTextChanged); + text_item = new ConductorTextItem(m_properties.text, this); + connect(text_item, &ConductorTextItem::diagramTextChanged, this, &Conductor::displayedTextChanged); } /** @@ -543,7 +543,7 @@ Diagram *Conductor::diagram() const { @return le champ de texte associe a ce conducteur */ ConductorTextItem *Conductor::textItem() const { - return(m_text_item); + return(text_item); } /** @@ -597,7 +597,7 @@ void Conductor::mousePressEvent(QGraphicsSceneMouseEvent *event) { moving_segment = true; moved_segment = segmentsList().at(index+1); - before_mov_text_pos_ = m_text_item -> pos(); + before_mov_text_pos_ = text_item -> pos(); } } @@ -855,7 +855,7 @@ bool Conductor::fromXml(QDomElement &dom_element) bool return_ = pathFromXml(dom_element); - m_text_item -> fromXml(dom_element); + text_item -> fromXml(dom_element); ConductorProperties pr; pr.fromXml(dom_element); @@ -909,7 +909,7 @@ QDomElement Conductor::toXml(QDomDocument &dom_document, QHash // Export the properties and text m_properties. toXml(dom_element); - m_text_item -> toXml(dom_element); + text_item -> toXml(dom_element); return(dom_element); } @@ -1130,7 +1130,7 @@ QPointF Conductor::posForText(Qt::Orientations &flag) { * If text was moved by user, this function do nothing, except check if text is near conductor. */ void Conductor::calculateTextItemPosition() { - if (!m_text_item || !diagram() || m_properties.type != ConductorProperties::Multi) return; + if (!text_item || !diagram() || m_properties.type != ConductorProperties::Multi) return; if (diagram() -> defaultConductorProperties.m_one_text_per_folio == true && relatedPotentialConductors(false).size() > 0) { @@ -1149,37 +1149,37 @@ void Conductor::calculateTextItemPosition() { c -> textItem() -> setVisible(false); } //Make sure text item is visible - m_text_item -> setVisible(true); + text_item -> setVisible(true); } //position - if (m_text_item -> wasMovedByUser()) { + if (text_item -> wasMovedByUser()) { //Text field was moved by user : //we check if text field is yet near the conductor - QPointF text_item_pos = m_text_item -> pos(); + QPointF text_item_pos = text_item -> pos(); QPainterPath near_shape = nearShape(); if (!near_shape.contains(text_item_pos)) { - m_text_item -> setPos(movePointIntoPolygon(text_item_pos, near_shape)); + text_item -> setPos(movePointIntoPolygon(text_item_pos, near_shape)); } } else { //Position and rotation of text is calculated. Qt::Orientations rotation; QPointF text_pos = posForText(rotation); - if (!m_text_item -> wasRotateByUser()) { - rotation == Qt::Vertical ? m_text_item -> setRotationAngle(m_properties.verti_rotate_text): - m_text_item -> setRotationAngle(m_properties.horiz_rotate_text); + if (!text_item -> wasRotateByUser()) { + rotation == Qt::Vertical ? text_item -> setRotationAngle(m_properties.verti_rotate_text): + text_item -> setRotationAngle(m_properties.horiz_rotate_text); } //Adjust the position of text if his rotation //is 0° or 270°, to be exactly centered to the conductor - if (m_text_item -> rotation() == 0) - text_pos.rx() -= m_text_item -> boundingRect().width()/2; - else if (m_text_item -> rotation() == 270) - text_pos.ry() += m_text_item -> boundingRect().width()/2; + if (text_item -> rotation() == 0) + text_pos.rx() -= text_item -> boundingRect().width()/2; + else if (text_item -> rotation() == 270) + text_pos.ry() += text_item -> boundingRect().width()/2; //Finaly set the position of text - m_text_item -> setPos(text_pos); + text_item -> setPos(text_pos); } } @@ -1199,7 +1199,7 @@ void Conductor::saveProfile(bool undo) { conductor_profiles[current_path_type], current_path_type ); - undo_object -> setConductorTextItemMove(before_mov_text_pos_, m_text_item -> pos()); + undo_object -> setConductorTextItemMove(before_mov_text_pos_, text_item -> pos()); dia -> undoStack().push(undo_object); } } @@ -1248,81 +1248,61 @@ ConductorProfile Conductor::profile(Qt::Corner path_type) const { return(conductor_profiles[path_type]); } +/// @return le texte du conducteur +QString Conductor::text() const { + QString label = text_item->toPlainText(); + return(label); +} + +/** + * @brief Conductor::setText + * The text of this conductor + * @param t + */ +void Conductor::setText(const QString &t) +{ + text_item->setPlainText(t); +} + /** * @brief Conductor::refreshText * Refresh the text of this conductor. * recalcule and set the text according to the formula. */ -void Conductor::refreshText() -{ - if (!m_freeze_label && diagram()) - { - QString text = autonum::AssignVariables::formulaToLabel(m_properties.m_formula, m_autoNum_seq, diagram()); - m_properties.text = text; - m_text_item->setPlainText(text); - } -} - -/** - * @brief Conductor::setPropertiesToPotential - * @param properties - * @param only_text - * Set @propertie to conductor and every conductors in the same potential of @conductor. - * If @only_text is true only formula, text, function and tension/protocol is set - * to other conductor in the same potential, the other values of property stay unmodified - */ -void Conductor::setPropertyToPotential(const ConductorProperties &property, bool only_text) -{ - setProperties(property); - QSet potential_list = relatedPotentialConductors(); - - foreach(Conductor *other_conductor, potential_list) - { - if (only_text) - { - ConductorProperties other_properties = other_conductor->properties(); - other_properties.m_formula = m_properties.m_formula; - other_properties.text = m_properties.text; - other_properties.m_function = m_properties.m_function; - other_properties.m_tension_protocol = m_properties.m_tension_protocol; - other_conductor->setProperties(other_properties); - } - else - { - other_conductor->setProperties(property); - } - } +void Conductor::refreshText() { + setText(m_freeze_label? text_item->toPlainText() : properties().text); } /** * @brief Conductor::setProperties - * Set @property as current property of conductor + * Set new properties for this conductor + * Also change the common properties for every conductors at the same potential. + * (text, function and tension/protocol) other value of properties isn't changed. * @param properties : properties */ -void Conductor::setProperties(const ConductorProperties &property) +void Conductor::setProperties(const ConductorProperties &properties) { - if (m_properties == property) return; + if (m_properties == properties) return; - m_properties = property; + m_properties = properties; - if (!m_properties.m_formula.isEmpty()) + foreach(Conductor *other_conductor, relatedPotentialConductors()) { - if (diagram()) { - QString text = autonum::AssignVariables::formulaToLabel(m_properties.m_formula, m_autoNum_seq, diagram()); - m_properties.text = text; - } - else { - m_properties.text = m_properties.m_formula; - } + ConductorProperties other_properties = other_conductor->properties(); + other_properties.text = m_properties.text; + other_properties.color = m_properties.color; + other_properties.cond_size = m_properties.cond_size; + other_properties.m_function = m_properties.m_function; + other_properties.m_tension_protocol = m_properties.m_tension_protocol; + other_conductor->setProperties(other_properties); } - - m_text_item->setPlainText(m_properties.text); - m_text_item->setFontSize(m_properties.text_size); + setText(m_properties.text); + text_item -> setFontSize(m_properties.text_size); if (m_properties.type != ConductorProperties::Multi) - m_text_item->setVisible(false); + text_item -> setVisible(false); else - m_text_item->setVisible(m_properties.m_show_text); + text_item -> setVisible(m_properties.m_show_text); calculateTextItemPosition(); update(); @@ -1361,16 +1341,16 @@ void Conductor::setHighlighted(Conductor::Highlight hl) { */ void Conductor::displayedTextChanged() { + if ((text_item->toPlainText() == autonum::AssignVariables::formulaToLabel(m_properties.text, m_autoNum_seq, diagram())) || !diagram()) return; + QVariant old_value, new_value; old_value.setValue(m_properties); ConductorProperties new_properties(m_properties); - new_properties.m_formula = m_text_item->toPlainText(); - new_properties.text = m_text_item->toPlainText(); + new_properties.text = text_item -> toPlainText(); new_value.setValue(new_properties); - - QUndoCommand *undo = new QUndoCommand(tr("Modifier les propriétés d'un conducteur", "undo caption")); - new QPropertyUndoCommand(this, "properties", old_value, new_value, undo); + QPropertyUndoCommand *undo = new QPropertyUndoCommand(this, "properties", old_value, new_value); + undo->setText(tr("Modifier les propriétés d'un conducteur", "undo caption")); if (!relatedPotentialConductors().isEmpty()) { @@ -1380,8 +1360,7 @@ void Conductor::displayedTextChanged() { old_value.setValue(potential_conductor->properties()); ConductorProperties new_properties = potential_conductor->properties(); - new_properties.m_formula = m_text_item->toPlainText(); - new_properties.text = m_text_item->toPlainText(); + new_properties.text = text_item->toPlainText(); new_value.setValue(new_properties); new QPropertyUndoCommand (potential_conductor, "properties", old_value, new_value, undo); } @@ -1461,12 +1440,6 @@ void Conductor::editProperty() { ConductorPropertiesDialog::PropertiesDialog(this, diagramEditor()); } -void Conductor::setSequenceNum(autonum::sequentialNumbers sn) -{ - m_autoNum_seq = sn; - refreshText(); -} - /** @param a point @param b point diff --git a/sources/qetgraphicsitem/conductor.h b/sources/qetgraphicsitem/conductor.h index aafec5d30..da060557a 100644 --- a/sources/qetgraphicsitem/conductor.h +++ b/sources/qetgraphicsitem/conductor.h @@ -45,7 +45,6 @@ class Conductor : public QObject, public QGraphicsPathItem Q_PROPERTY(QPointF pos READ pos WRITE setPos) Q_PROPERTY(int animPath READ fakePath WRITE updatePathAnimate) Q_PROPERTY(ConductorProperties properties READ properties WRITE setProperties) - Q_PROPERTY(autonum::sequentialNumbers sequenceNum READ sequenceNum WRITE setSequenceNum) signals: void propertiesChange(); @@ -63,7 +62,9 @@ class Conductor : public QObject, public QGraphicsPathItem enum { Type = UserType + 1001 }; enum Highlight { None, Normal, Alert }; + /// First terminal the wire is attached to Terminal *terminal1; + /// Second terminal the wire is attached to Terminal *terminal2; public: @@ -88,6 +89,8 @@ class Conductor : public QObject, public QGraphicsPathItem qreal length() const; ConductorSegment *middleSegment(); QPointF posForText(Qt::Orientations &flag); + QString text() const; + void setText(const QString &); void refreshText(); public: @@ -100,11 +103,8 @@ class Conductor : public QObject, public QGraphicsPathItem public: QVector handlerPoints() const; const QList segmentsList() const; - - void setPropertyToPotential(const ConductorProperties &property, bool only_text = false); - void setProperties(const ConductorProperties &property); + void setProperties(const ConductorProperties &properties); ConductorProperties properties() const; - void setProfile(const ConductorProfile &, Qt::Corner); ConductorProfile profile(Qt::Corner) const; void setProfiles(const ConductorProfilesGroup &); @@ -118,7 +118,6 @@ class Conductor : public QObject, public QGraphicsPathItem autonum::sequentialNumbers sequenceNum () const {return m_autoNum_seq;} autonum::sequentialNumbers& rSequenceNum() {return m_autoNum_seq;} - void setSequenceNum(autonum::sequentialNumbers sn); private: autonum::sequentialNumbers m_autoNum_seq; @@ -144,7 +143,7 @@ class Conductor : public QObject, public QGraphicsPathItem /// Functional properties ConductorProperties m_properties; /// Text input for non simple, non-singleline conductors - ConductorTextItem *m_text_item; + ConductorTextItem *text_item; /// Segments composing the conductor ConductorSegment *segments; /// Attributs related to mouse interaction @@ -160,15 +159,14 @@ class Conductor : public QObject, public QGraphicsPathItem /// conductor profile: "photography" of what the conductor is supposed to look /// like - there is one profile per kind of traject ConductorProfilesGroup conductor_profiles; + /// QPen et QBrush objects used to draw conductors + static QPen conductor_pen; + static QBrush conductor_brush; + static bool pen_and_brush_initialized; /// Define whether and how the conductor should be highlighted Highlight must_highlight_; bool m_valid; bool m_freeze_label = false; - - /// QPen et QBrush objects used to draw conductors - static QPen conductor_pen; - static QBrush conductor_brush; - static bool pen_and_brush_initialized; private: void segmentsToPath(); diff --git a/sources/qetgraphicsitem/terminal.cpp b/sources/qetgraphicsitem/terminal.cpp index 7c468fa68..bfcd988bb 100644 --- a/sources/qetgraphicsitem/terminal.cpp +++ b/sources/qetgraphicsitem/terminal.cpp @@ -623,7 +623,8 @@ void Terminal::mouseReleaseEvent(QGraphicsSceneMouseEvent *e) if (use_properties) { Conductor *other = conductors_list.toList().first(); - new_conductor->setProperties(other->properties()); + new_conductor->setText(""); + new_conductor->setText(other->properties().text); } } diff --git a/sources/ui/conductorpropertieswidget.cpp b/sources/ui/conductorpropertieswidget.cpp index 5fd88c68d..68ec9a176 100644 --- a/sources/ui/conductorpropertieswidget.cpp +++ b/sources/ui/conductorpropertieswidget.cpp @@ -135,7 +135,7 @@ ConductorProperties ConductorPropertiesWidget::properties() const bool ConductorPropertiesWidget::event(QEvent *event) { if (event->type() == QEvent::WindowActivate) { - ui -> m_formula_le -> setFocus(); + ui -> m_text_le -> setFocus(); } return(QWidget::event(event)); } diff --git a/sources/ui/potentialselectordialog.cpp b/sources/ui/potentialselectordialog.cpp index 99f13d7c3..48a419b34 100644 --- a/sources/ui/potentialselectordialog.cpp +++ b/sources/ui/potentialselectordialog.cpp @@ -24,7 +24,6 @@ #include "diagram.h" #include "element.h" #include "reportelement.h" -#include "assignvariables.h" //### BEGIN PRIVATE CLASS ###// @@ -44,8 +43,8 @@ class NewConductorPotentialSelector : public AbstractPotentialSelector terminal_1->removeConductor(conductor); terminal_2->removeConductor(conductor); - getPotential(terminal_1, m_properties_1, m_seq_num_1, m_conductor_number_1); - getPotential(terminal_2, m_properties_2, m_seq_num_2, m_conductor_number_2); + getPotential(terminal_1, m_properties_1, m_conductor_number_1); + getPotential(terminal_2, m_properties_2, m_conductor_number_2); //There isn't a potential at terminal 1 or 2. if (m_conductor_number_1 == 0 && m_conductor_number_2 == 0) return; @@ -65,7 +64,7 @@ class NewConductorPotentialSelector : public AbstractPotentialSelector * @param properties * @param number */ - void getPotential(Terminal *terminal, ConductorProperties &properties, autonum::sequentialNumbers &seq_num , int &number) + void getPotential(Terminal *terminal, ConductorProperties &properties, int &number) { Conductor *conductor_in_potential = nullptr; @@ -100,7 +99,6 @@ class NewConductorPotentialSelector : public AbstractPotentialSelector if (!conductor_in_potential) return; properties = conductor_in_potential->properties(); - seq_num = conductor_in_potential->sequenceNum(); number = conductor_in_potential->relatedPotentialConductors().size()+1; //We add +1 because conductor_in_potential isn't count by relatedPotentialConductors } @@ -173,7 +171,7 @@ PotentialSelectorDialog::PotentialSelectorDialog(Conductor *conductor, QUndoComm * Constructor when we link two potentiels together, with a folio report. * @param report : one of the report used to link the potentials (report must be linked to another report) * @param parent_undo : undo parent to use - * @param parent : parent widget + * @param parent : paren widget */ PotentialSelectorDialog::PotentialSelectorDialog(Element *report, QUndoCommand *parent_undo, QWidget *parent) : QDialog(parent), @@ -202,22 +200,8 @@ void PotentialSelectorDialog::buildWidget() QRadioButton *rb1 = new QRadioButton(tr("Le potentiel avec numero de fil %1 est présent %2 fois").arg(m_potential_selector->m_properties_1.text).arg(m_potential_selector->m_conductor_number_1), this); QRadioButton *rb2 = new QRadioButton(tr("Le potentiel avec numero de fil %1 est présent %2 fois").arg(m_potential_selector->m_properties_2.text).arg(m_potential_selector->m_conductor_number_2), this); - connect(rb1, &QRadioButton::toggled, [this](bool t) - { - if(t) - { - this->m_selected_properties = this->m_potential_selector->m_properties_1; - this->m_sequential_num = this->m_potential_selector->m_seq_num_1; - } - }); - connect(rb2, &QRadioButton::toggled, [this](bool t) - { - if(t) - { - this->m_selected_properties = this->m_potential_selector->m_properties_2; - this->m_sequential_num = this->m_potential_selector->m_seq_num_2; - } - }); + connect(rb1, &QRadioButton::toggled, [this](bool t){if(t) this->m_selected_properties = this->m_potential_selector->m_properties_1;}); + connect(rb2, &QRadioButton::toggled, [this](bool t){if(t) this->m_selected_properties = this->m_potential_selector->m_properties_2;}); //Set the radio button of potential with the bigger number of conductors, //at first position, and check it @@ -248,7 +232,6 @@ void PotentialSelectorDialog::on_buttonBox_accepted() ConductorProperties new_properties = m_conductor->properties(); new_properties.text = m_selected_properties.text; - new_properties.m_formula = m_selected_properties.m_formula; new_properties.m_function = m_selected_properties.m_function; new_properties.m_tension_protocol = m_selected_properties.m_tension_protocol; @@ -256,32 +239,19 @@ void PotentialSelectorDialog::on_buttonBox_accepted() old_value.setValue(m_conductor->properties()); new_value.setValue(new_properties); - - QUndoCommand *undo = nullptr; - if (m_parent_undo) - undo = m_parent_undo; - else - undo = new QUndoCommand(tr("Modifier les propriétés de plusieurs conducteurs", "undo caption")); - //Set the properties for the new conductor - QVariant old_seq, new_seq; - old_seq.setValue(m_conductor->sequenceNum()); - new_seq.setValue(m_sequential_num); - new QPropertyUndoCommand(m_conductor, "sequenceNum", old_seq, new_seq, undo); - new QPropertyUndoCommand(m_conductor, "properties", old_value, new_value, undo); + QPropertyUndoCommand *undo = new QPropertyUndoCommand(m_conductor, "properties", old_value, new_value, m_parent_undo); + undo->setText(tr("Modifier les propriétés de plusieurs conducteurs", "undo caption")); //Set the new properties for each conductors of the new potential foreach(Conductor *cond, m_conductor->relatedPotentialConductors()) { new_properties = cond->properties(); new_properties.text = m_selected_properties.text; - new_properties.m_formula = m_selected_properties.m_formula; new_properties.m_function = m_selected_properties.m_function; new_properties.m_tension_protocol = m_selected_properties.m_tension_protocol; old_value.setValue(cond->properties()); new_value.setValue(new_properties); - old_seq.setValue(cond->sequenceNum()); - new QPropertyUndoCommand(cond, "sequenceNum", old_seq, new_seq, undo); new QPropertyUndoCommand(cond, "properties", old_value, new_value, undo); } @@ -293,7 +263,6 @@ void PotentialSelectorDialog::on_buttonBox_accepted() else { delete undo; - m_conductor->setSequenceNum(m_sequential_num); m_conductor->setProperties(new_properties); } } diff --git a/sources/ui/potentialselectordialog.h b/sources/ui/potentialselectordialog.h index bab18ba6e..ac2565f24 100644 --- a/sources/ui/potentialselectordialog.h +++ b/sources/ui/potentialselectordialog.h @@ -20,7 +20,6 @@ #include #include "conductorproperties.h" -#include "assignvariables.h" class Conductor; class QUndoCommand; class Element; @@ -33,7 +32,6 @@ class AbstractPotentialSelector virtual bool isValid() const = 0; ConductorProperties m_properties_1, m_properties_2; - autonum::sequentialNumbers m_seq_num_1, m_seq_num_2; int m_conductor_number_1, m_conductor_number_2; }; @@ -74,7 +72,6 @@ class PotentialSelectorDialog : public QDialog Element *m_report; QUndoCommand *m_parent_undo; ConductorProperties m_selected_properties; - autonum::sequentialNumbers m_sequential_num; AbstractPotentialSelector *m_potential_selector; }; #endif // POTENTIALSELECTORDIALOG_H