Menu ▾ ▴

#106 workpath doesn't work with "move"

v1.0.x
fixed
Erik Hänel
critical (96)
v1.0.9
Bug Fix
2016-10-08
2016-09-10
Erik Hänel
No

Something like this

move <wp>/ekli* -target=<wp>/_archive/ -all

doesn't work.

Analysis:
In the function vector<string> getFileList(const string&, const Settings&, int), which is called in this command, the path placeholders are replaced independently and <wp> is missing.

Implementation:
Fix was implemented as indicated by the analysis. Implementation tests were passed successfully.

Documentation:
Not needed - bug fix

Tests:
Workpath now works with move. Bug fix accepted.

Discussion

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -3,3 +3,11 @@
 move <wp>/ekli* -target=<wp>/_archive/ -all
 ~~~
 doesn't work.
+
+**Analysis:**
+
+**Implementation:**
+
+**Documentation:**
+
+**Tests:**
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -5,6 +5,7 @@
 doesn't work.

 **Analysis:**
+In the function `vector<string> getFileList(const string&, const Settings&, int)`, which is called in this command, the path placeholders are replaced independently and `<wp>` is missing.

 **Implementation:**
    • status: analyzing --> implementing
     

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -8,7 +8,9 @@
 In the function `vector<string> getFileList(const string&, const Settings&, int)`, which is called in this command, the path placeholders are replaced independently and `<wp>` is missing.

 **Implementation:**
+Fix was implemented was indicated by the analysis. Implementation tests were passed successfully.

 **Documentation:**
+Not needed - bug fix

 **Tests:**
    • status: implementing --> testing
     

  • Erik Hänel

    Erik Hänel - 2016-09-26
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -8,9 +8,10 @@
 In the function `vector<string> getFileList(const string&, const Settings&, int)`, which is called in this command, the path placeholders are replaced independently and `<wp>` is missing.

 **Implementation:**
-Fix was implemented was indicated by the analysis. Implementation tests were passed successfully.
+Fix was implemented as indicated by the analysis. Implementation tests were passed successfully.

 **Documentation:**
 Not needed - bug fix

 **Tests:**
+Workpath now works with `move`. Bug fix accepted.
    • status: testing --> fixed
     

  • Erik Hänel

    Erik Hänel - 2016-10-08
    • Labels: --> critical
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB