Remove editable property from models

This patch move logic connected with editable property
to presenters

Change-Id: I529de8fa3c67a15e8733b6094ce36ddcaaaaaae7

diff --git a/src/model/AutoMoveIntervalModel.hpp b/src/model/AutoMoveIntervalModel.hpp
index 80fc5b484f1e693c7e29a75f641e5247ef63debf..531bed5b83467b7286c69520d1a868161bce527a 100644 (file)
--- a/src/model/AutoMoveIntervalModel.hpp
+++ b/src/model/AutoMoveIntervalModel.hpp
@@ -17,7 +17,6 @@ class AutoMoveIntervalModel
        const double step_ = 0.1;
        const double defaultValue_ = 1.0;
 
-       const bool editable_ = true;
        const bool defaultState_ = false;
 
        private:

diff --git a/src/model/AutoScanIntervalModel.hpp b/src/model/AutoScanIntervalModel.hpp
index 44bb1f014732613b55be109c7067c26ee0381596..85b531279097e8a92ff9d40e1ebc6a320e0c02b3 100644 (file)
--- a/src/model/AutoScanIntervalModel.hpp
+++ b/src/model/AutoScanIntervalModel.hpp
@@ -13,7 +13,6 @@ class AutoScanIntervalModel
 
        const utils::Range<double> range_ = {0.5, 9.9};
        const double step_ = 0.1;
-       const bool editable_ = true;
 
        private:
        VConfInterface::CallbackHandle autoScanIntervalValueHandle_;

diff --git a/src/model/AutoTapModel.hpp b/src/model/AutoTapModel.hpp
index 9ac0ba1698228e46c28193cb3c803ec40acdb90d..7d250b353b34453b4a7a3a5dc468c83d3081558e 100644 (file)
--- a/src/model/AutoTapModel.hpp
+++ b/src/model/AutoTapModel.hpp
@@ -17,7 +17,6 @@ class AutoTapModel
        const double step_ = 0.1;
        const double defaultValue_ = 1.0;
 
-       const bool editable_ = true;
        const bool defaultState_ = false;
 
        private:

diff --git a/src/model/NumberOfAutoScanLoopsModel.hpp b/src/model/NumberOfAutoScanLoopsModel.hpp
index 9314acef64ff4dcdf7412b9ce3d97d219b52863d..605451ddc1bdabd2ab0728cc434795348804e35f 100644 (file)
--- a/src/model/NumberOfAutoScanLoopsModel.hpp
+++ b/src/model/NumberOfAutoScanLoopsModel.hpp
@@ -2,9 +2,9 @@
 #define NUMBER_OF_AUTO_SCAN_LOOPS_MODEL_HPP_
 
 #include "ObservableProperty.hpp"
+#include "UniversalSwitchConstants.hpp"
 #include "VConf.hpp"
 #include "utils.hpp"
-#include "UniversalSwitchConstants.hpp"
 
 class NumberOfAutoScanLoopsModel
 {
@@ -15,11 +15,9 @@ class NumberOfAutoScanLoopsModel
        const utils::Range<double> range_ = {RANGE_MIN_SCANS_NUM, RANGE_MAX_SCANS_NUM};
        const double step_ = STEP_SCANS_NUM;
        const double defaultValue_ = DEFAULT_AUTO_SCAN_NUMBER;
-       const bool editable_ = true;
 
        private:
        VConfInterface::CallbackHandle valueHandle_;
-
 };
 
 #endif

diff --git a/src/model/PauseOnFirstModel.hpp b/src/model/PauseOnFirstModel.hpp
index 70445c80ddce44748358c378aebfd2ee1d433a14..49da5901678f2ed43d33b0aecf135977ecfdf4f2 100644 (file)
--- a/src/model/PauseOnFirstModel.hpp
+++ b/src/model/PauseOnFirstModel.hpp
@@ -14,7 +14,6 @@ class PauseOnFirstModel
 
        const utils::Range<double> range_ = {0.1, 5.0};
        const double step_ = 0.1;
-       const bool editable_ = true;
 
        private:
        VConfInterface::CallbackHandle stateHandle_;

diff --git a/src/model/PointScanSpeedModel.hpp b/src/model/PointScanSpeedModel.hpp
index adaf1f14c3e52ccb8ec12604d1fc2804df36af00..27b2b23c15f64187451d7abfc63366fc1039a01a 100644 (file)
--- a/src/model/PointScanSpeedModel.hpp
+++ b/src/model/PointScanSpeedModel.hpp
@@ -2,9 +2,9 @@
 #define POINT_SCAN_SPEED_MODEL_HPP_
 
 #include "ObservableProperty.hpp"
+#include "UniversalSwitchConstants.hpp"
 #include "VConf.hpp"
 #include "utils.hpp"
-#include "UniversalSwitchConstants.hpp"
 
 class PointScanSpeedModel
 {
@@ -15,11 +15,9 @@ class PointScanSpeedModel
        const utils::Range<double> range_ = {RANGE_MIN_SCAN_SPEED, RANGE_MAX_SCAN_SPEED};
        const double step_ = STEP_SCAN_SPEED;
        const double defaultValue_ = DEFAULT_SCAN_PT_SPEED;
-       const bool editable_ = true;
 
        private:
        VConfInterface::CallbackHandle valueHandle_;
-
 };
 
 #endif

diff --git a/src/model/SingleInteractionIntervalModel.hpp b/src/model/SingleInteractionIntervalModel.hpp
index 0889429beab9b81c588c00bab4d7e13e0c6ff618..219e3cf307e2c23d160daeaadb1553ec3709fd69 100644 (file)
--- a/src/model/SingleInteractionIntervalModel.hpp
+++ b/src/model/SingleInteractionIntervalModel.hpp
@@ -17,7 +17,6 @@ class SingleInteractionIntervalModel
        const double step_ = 0.1;
        const double defaultValue_ = 1.0;
 
-       const bool editable_ = true;
        const bool defaultState_ = false;
 
        private:

diff --git a/src/model/TapDurationModel.hpp b/src/model/TapDurationModel.hpp
index 4465b00b002d786232c4d4ae8e92e97b308f19ca..9427f5526dd4183a00ca684b11b964f89c63560f 100644 (file)
--- a/src/model/TapDurationModel.hpp
+++ b/src/model/TapDurationModel.hpp
@@ -17,7 +17,6 @@ class TapDurationModel
        const double step_ = 0.1;
        const double defaultValue_ = 1.0;
 
-       const bool editable_ = true;
        const bool defaultState_ = false;
 
        private:

diff --git a/src/presenter/AutoMoveIntervalPresenter.cpp b/src/presenter/AutoMoveIntervalPresenter.cpp
index bf37a2886e975be85adc97485eb951d4a028d0bf..d8f3f5c09a308b10ba7847dbff48556f4e03372b 100644 (file)
--- a/src/presenter/AutoMoveIntervalPresenter.cpp
+++ b/src/presenter/AutoMoveIntervalPresenter.cpp
@@ -19,11 +19,6 @@ utils::Range<double> AutoMoveIntervalPresenter::getRange()
        return model_.range_;
 }
 
-bool AutoMoveIntervalPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 double AutoMoveIntervalPresenter::getStep()
 {
        return model_.step_;

diff --git a/src/presenter/AutoMoveIntervalPresenter.hpp b/src/presenter/AutoMoveIntervalPresenter.hpp
index 0d6566eb1b0f0e90bf64b0c6fe185cb2e8118825..111fd8a1ebaccf711785fe93ad1372ba9f804786 100644 (file)
--- a/src/presenter/AutoMoveIntervalPresenter.hpp
+++ b/src/presenter/AutoMoveIntervalPresenter.hpp
@@ -1,8 +1,8 @@
 #ifndef AUTO_MOVE_INTERVAL_PRESENTER_HPP_
 #define AUTO_MOVE_INTERVAL_PRESENTER_HPP_
 
-#include "SpinnerPresenterWithToggle.hpp"
 #include "AutoMoveIntervalModel.hpp"
+#include "SpinnerPresenterWithToggle.hpp"
 
 class AutoMoveIntervalPresenter : public SpinnerPresenterWithToggle
 {
@@ -11,7 +11,6 @@ class AutoMoveIntervalPresenter : public SpinnerPresenterWithToggle
 
        utils::Range<double> getRange() override;
        double getStep() override;
-       bool isEditable() override;
        std::string getLabel() override;
        std::string getFormat() override;
 

diff --git a/src/presenter/AutoScanIntervalPresenter.cpp b/src/presenter/AutoScanIntervalPresenter.cpp
index 8164bfe744dbe5dc09f0a3196395294a2bec7428..20bebd6fb7d90b8fdfc30f57eabe870f2d0da2d8 100644 (file)
--- a/src/presenter/AutoScanIntervalPresenter.cpp
+++ b/src/presenter/AutoScanIntervalPresenter.cpp
@@ -19,11 +19,6 @@ double AutoScanIntervalPresenter::getStep()
        return model_.step_;
 }
 
-bool AutoScanIntervalPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 std::string AutoScanIntervalPresenter::getLabel()
 {
        return "IDS_ACCS_UNIVERSAL_SWITCH_SETTINGS_SET_VALUE_DESC_AUTO_SCAN_INT";

diff --git a/src/presenter/AutoScanIntervalPresenter.hpp b/src/presenter/AutoScanIntervalPresenter.hpp
index 89fa916848ca25be2c883f2a7409dada0c68d0d5..19ce5126bb76b742e9696d24946b7b6b888546a9 100644 (file)
--- a/src/presenter/AutoScanIntervalPresenter.hpp
+++ b/src/presenter/AutoScanIntervalPresenter.hpp
@@ -10,7 +10,6 @@ class AutoScanIntervalPresenter : public SpinnerPresenter
        AutoScanIntervalPresenter();
        utils::Range<double> getRange() override;
        double getStep() override;
-       bool isEditable() override;
        std::string getLabel() override;
        std::string getFormat() override;
 

diff --git a/src/presenter/AutoTapPresenter.cpp b/src/presenter/AutoTapPresenter.cpp
index c03727eec212973c47a42e9179e0288b87013583..781cbb6ae8679575f42de8eb5555e5223b580ded 100644 (file)
--- a/src/presenter/AutoTapPresenter.cpp
+++ b/src/presenter/AutoTapPresenter.cpp
@@ -19,11 +19,6 @@ utils::Range<double> AutoTapPresenter::getRange()
        return model_.range_;
 }
 
-bool AutoTapPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 double AutoTapPresenter::getStep()
 {
        return model_.step_;

diff --git a/src/presenter/AutoTapPresenter.hpp b/src/presenter/AutoTapPresenter.hpp
index c3b36d4ef27c3bbf71ca8c67ce5fb05b9f6ed31c..cb6df5903b9a6aa2a8a79f92197f8941fd409031 100644 (file)
--- a/src/presenter/AutoTapPresenter.hpp
+++ b/src/presenter/AutoTapPresenter.hpp
@@ -1,8 +1,8 @@
 #ifndef AUTO_TAP_PRESENTER_HPP_
 #define AUTO_TAP_PRESENTER_HPP_
 
-#include "SpinnerPresenterWithToggle.hpp"
 #include "AutoTapModel.hpp"
+#include "SpinnerPresenterWithToggle.hpp"
 
 class AutoTapPresenter : public SpinnerPresenterWithToggle
 {
@@ -11,7 +11,6 @@ class AutoTapPresenter : public SpinnerPresenterWithToggle
 
        utils::Range<double> getRange() override;
        double getStep() override;
-       bool isEditable() override;
        std::string getLabel() override;
        std::string getFormat() override;
 

diff --git a/src/presenter/NumberOfAutoScanLoopsPresenter.cpp b/src/presenter/NumberOfAutoScanLoopsPresenter.cpp
index 45a28b5b82bd3c82fc8ad7c34fa83384f6f23bca..b2a3200f3f7eb0711776a799e78013c75608a553 100644 (file)
--- a/src/presenter/NumberOfAutoScanLoopsPresenter.cpp
+++ b/src/presenter/NumberOfAutoScanLoopsPresenter.cpp
@@ -7,8 +7,6 @@ NumberOfAutoScanLoopsPresenter::NumberOfAutoScanLoopsPresenter()
        value_.attach([this](auto val) {
                model_.value_ = val;
        });
-
-
 }
 
 utils::Range<double> NumberOfAutoScanLoopsPresenter::getRange()
@@ -16,11 +14,6 @@ utils::Range<double> NumberOfAutoScanLoopsPresenter::getRange()
        return model_.range_;
 }
 
-bool NumberOfAutoScanLoopsPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 double NumberOfAutoScanLoopsPresenter::getStep()
 {
        return model_.step_;

diff --git a/src/presenter/NumberOfAutoScanLoopsPresenter.hpp b/src/presenter/NumberOfAutoScanLoopsPresenter.hpp
index 4b0df8df1040b4a90fcf4655067c88513ad00fa4..16ce8315d4719d790664c2dcc05ba1b0148e5141 100644 (file)
--- a/src/presenter/NumberOfAutoScanLoopsPresenter.hpp
+++ b/src/presenter/NumberOfAutoScanLoopsPresenter.hpp
@@ -10,7 +10,6 @@ class NumberOfAutoScanLoopsPresenter : public SpinnerPresenter
        NumberOfAutoScanLoopsPresenter();
 
        utils::Range<double> getRange();
-       bool isEditable();
        double getStep();
        std::string getFormat();
        std::string getLabel();

diff --git a/src/presenter/PauseOnFirstPresenter.cpp b/src/presenter/PauseOnFirstPresenter.cpp
index 2955044db300599f5d78b3784aafe13b8eb9d006..5e88c5d45599c1e03d0a75fd7a5e844e13a9d12f 100644 (file)
--- a/src/presenter/PauseOnFirstPresenter.cpp
+++ b/src/presenter/PauseOnFirstPresenter.cpp
@@ -24,11 +24,6 @@ double PauseOnFirstPresenter::getStep()
        return model_.step_;
 }
 
-bool PauseOnFirstPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 std::string PauseOnFirstPresenter::getLabel()
 {
        return "IDS_ACCS_UNIVERSAL_SWITCH_SETTINGS_GROUP_ADD_SETTINGS_PAUSE_ON_FIRST";

diff --git a/src/presenter/PauseOnFirstPresenter.hpp b/src/presenter/PauseOnFirstPresenter.hpp
index 4ae723be592676090d0fdefa43f7fc2a6f33fee6..2a57e5f8fa33fdcabba66cef2e91fc7d072720b0 100644 (file)
--- a/src/presenter/PauseOnFirstPresenter.hpp
+++ b/src/presenter/PauseOnFirstPresenter.hpp
@@ -11,7 +11,6 @@ class PauseOnFirstPresenter : public SpinnerPresenterWithToggle
 
        utils::Range<double> getRange() override;
        double getStep() override;
-       bool isEditable() override;
        std::string getLabel() override;
        std::string getFormat() override;
 

diff --git a/src/presenter/PointScanSpeedPresenter.cpp b/src/presenter/PointScanSpeedPresenter.cpp
index 006bd467f642836f0f847704d91dcd51db3d94dc..df73041dfb75bdbffa3498c7757a87fb0669a9de 100644 (file)
--- a/src/presenter/PointScanSpeedPresenter.cpp
+++ b/src/presenter/PointScanSpeedPresenter.cpp
@@ -7,8 +7,6 @@ PointScanSpeedPresenter::PointScanSpeedPresenter()
        value_.attach([this](auto val) {
                model_.value_ = val;
        });
-
-
 }
 
 utils::Range<double> PointScanSpeedPresenter::getRange()
@@ -16,11 +14,6 @@ utils::Range<double> PointScanSpeedPresenter::getRange()
        return model_.range_;
 }
 
-bool PointScanSpeedPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 double PointScanSpeedPresenter::getStep()
 {
        return model_.step_;

diff --git a/src/presenter/PointScanSpeedPresenter.hpp b/src/presenter/PointScanSpeedPresenter.hpp
index 0c82529a18b990e59c7d909fd9ef49139544646b..42352e22d72a13ff9a55b1eec2cccf1a2d8a0b93 100644 (file)
--- a/src/presenter/PointScanSpeedPresenter.hpp
+++ b/src/presenter/PointScanSpeedPresenter.hpp
@@ -9,11 +9,10 @@ class PointScanSpeedPresenter : public SpinnerPresenter
        public:
        PointScanSpeedPresenter();
 
-       utils::Range<double> getRange();
-       bool isEditable();
-       double getStep();
-       std::string getFormat();
-       std::string getLabel();
+       utils::Range<double> getRange() override;
+       double getStep() override;
+       std::string getFormat() override;
+       std::string getLabel() override;
 
        private:
        PointScanSpeedModel model_;

diff --git a/src/presenter/SingleInteractionIntervalPresenter.cpp b/src/presenter/SingleInteractionIntervalPresenter.cpp
index 5edc58375a60a3748f2545cbd98b9dd1017ee458..bc7ce71bec34ea63b4316e4a48d090b4e81eb8cb 100644 (file)
--- a/src/presenter/SingleInteractionIntervalPresenter.cpp
+++ b/src/presenter/SingleInteractionIntervalPresenter.cpp
@@ -18,12 +18,6 @@ utils::Range<double> SingleInteractionIntervalPresenter::getRange()
 {
        return model_.range_;
 }
-
-bool SingleInteractionIntervalPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 double SingleInteractionIntervalPresenter::getStep()
 {
        return model_.step_;

diff --git a/src/presenter/SingleInteractionIntervalPresenter.hpp b/src/presenter/SingleInteractionIntervalPresenter.hpp
index 11a9f8a9647576bab24e71070d7c6292600c8642..8257b17e1897633c8110c703fcf49db409683e32 100644 (file)
--- a/src/presenter/SingleInteractionIntervalPresenter.hpp
+++ b/src/presenter/SingleInteractionIntervalPresenter.hpp
@@ -11,7 +11,6 @@ class SingleInteractionIntervalPresenter : public SpinnerPresenterWithToggle
 
        utils::Range<double> getRange() override;
        double getStep() override;
-       bool isEditable() override;
        std::string getLabel() override;
        std::string getFormat() override;
 

diff --git a/src/presenter/SpinnerPresenter.cpp b/src/presenter/SpinnerPresenter.cpp
index bfbe067845f46392cf4af5033a0bbcb622ee38fa..e1c0475af29ec0777673bdd1acadc7cf38c3d4e1 100644 (file)
--- a/src/presenter/SpinnerPresenter.cpp
+++ b/src/presenter/SpinnerPresenter.cpp
@@ -18,3 +18,8 @@ std::string SpinnerPresenter::getRangeDescription()
 
        return out.str();
 }
+
+bool SpinnerPresenter::isEditable()
+{
+       return true;
+}
\ No newline at end of file

diff --git a/src/presenter/SpinnerPresenter.hpp b/src/presenter/SpinnerPresenter.hpp
index 78ed3aefd543781c6ccd1625efe717dea87acda9..75c087e3e01162dfbdc348a411b59d0c6230f33b 100644 (file)
--- a/src/presenter/SpinnerPresenter.hpp
+++ b/src/presenter/SpinnerPresenter.hpp
@@ -16,7 +16,7 @@ class SpinnerPresenter : public Presenter
 
        virtual utils::Range<double> getRange() = 0;
        virtual double getStep() = 0;
-       virtual bool isEditable() = 0;
+       virtual bool isEditable();
        virtual std::string getLabel() = 0;
        virtual std::string getFormat() = 0;
 

diff --git a/src/presenter/TapDurationPresenter.cpp b/src/presenter/TapDurationPresenter.cpp
index 16303b0e563d7117e3544453c5fb69f62149b176..f1bea7572f46640f2be45f2eec4ab72cfa4c0dac 100644 (file)
--- a/src/presenter/TapDurationPresenter.cpp
+++ b/src/presenter/TapDurationPresenter.cpp
@@ -19,11 +19,6 @@ utils::Range<double> TapDurationPresenter::getRange()
        return model_.range_;
 }
 
-bool TapDurationPresenter::isEditable()
-{
-       return model_.editable_;
-}
-
 double TapDurationPresenter::getStep()
 {
        return model_.step_;

diff --git a/src/presenter/TapDurationPresenter.hpp b/src/presenter/TapDurationPresenter.hpp
index d8698eea5ea28e414981c8edd4a7411e28ff9d7a..4be31cf000fabab37c16a541402fd68a9a52cbe2 100644 (file)
--- a/src/presenter/TapDurationPresenter.hpp
+++ b/src/presenter/TapDurationPresenter.hpp
@@ -11,7 +11,6 @@ class TapDurationPresenter : public SpinnerPresenterWithToggle
 
        utils::Range<double> getRange() override;
        double getStep() override;
-       bool isEditable() override;
        std::string getLabel() override;
        std::string getFormat() override;