Fix bug on dicealias replacement (when several replacement must be done

on the same command).

2 files changed, 5 insertions, 3 deletions

diff --git a/dicealias.cpp b/dicealias.cpp
index a27919d..f514e9e 100644
--- a/dicealias.cpp
+++ b/dicealias.cpp
@@ -89,9 +89,11 @@ QString makeReplament(const QString& pattern, const QString& replacement, QStrin
                     patternPosList.push_back(pos);
                 pos+=1;
             }
-            for(auto pos : patternPosList)
+
+            // TODO to be replace by C++14 when it is ready
+            for (auto i = patternPosList.rbegin(); i != patternPosList.rend(); ++i)
             {
-                cmd.replace(pos,1,replacement);
+                cmd.replace(*i,1,replacement);
             }
         }
     }
diff --git a/dicealias.h b/dicealias.h
index a85c9a0..8320445 100644
--- a/dicealias.h
+++ b/dicealias.h
@@ -37,7 +37,7 @@ public:
      * @param key
      * @param isReplace
      */
-    DiceAlias(QString cmd, QString key, bool isReplace = true, bool isEnable = true);
+    DiceAlias(QString pattern, QString remplacement, bool isReplace = true, bool isEnable = true);
     /**
      * @brief ~DiceAlias
      */