Menu ▾ ▴

#71 Investigate progress issue

v1.0.x
fixed
Erik Hänel
None
v1.0.9
Bug Fix
2016-07-14
2016-06-30
Erik Hänel
No

Investigate the issue with progress occured at

progress i -set last=num(string(:))

Analysis:
Bug results from failing function getStringArgument() in BI_evalParamString(). This is a globale issue not only relevant vor progress. Fix includes the replacement of failing function.

Second issue bases on the passing of the obtion string, if the option type=TYPE is missing. Can be fixed by adding an else-clause with a default argument.

Implementation:
Function was corrected. Maybe there's a second issue with the order of the options.

Second issue was located in two files:

  • built-in.cpp
  • loop.cpp

Implementation tests were successful.

Tests:
Test were passed successful. Bug fix accepted.

Discussion

  • Erik Hänel

    Erik Hänel - 2016-07-01
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2016-07-01
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -2,3 +2,9 @@
 ~~~
 progress i -set last=num(string(:))
 ~~~
+
+**Analysis:**
+
+**Implementation:**
+
+**Tests:**
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2016-07-01
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -4,6 +4,7 @@
 ~~~

 **Analysis:**
+Bug results from failing function `getStringArgument()` in `BI_evalParamString()`. This is a globale issue not only relevant vor `progress`. Fix includes the replacement of failing function.

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

  • Erik Hänel

    Erik Hänel - 2016-07-01
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -7,5 +7,6 @@
 Bug results from failing function `getStringArgument()` in `BI_evalParamString()`. This is a globale issue not only relevant vor `progress`. Fix includes the replacement of failing function.

 **Implementation:**
+Function was corrected. Maybe there's a second issue with the order of the options.

 **Tests:**
     

  • Erik Hänel

    Erik Hänel - 2016-07-03
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -6,6 +6,8 @@
 **Analysis:**
 Bug results from failing function `getStringArgument()` in `BI_evalParamString()`. This is a globale issue not only relevant vor `progress`. Fix includes the replacement of failing function.

+Second issue bases on the passing of the obtion string, if the option `type=TYPE` is missing. Can be fixed by adding an `else`-clause with a default argument.
+
 **Implementation:**
 Function was corrected. Maybe there's a second issue with the order of the options.
     

  • Erik Hänel

    Erik Hänel - 2016-07-03
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -11,4 +11,10 @@
 **Implementation:**
 Function was corrected. Maybe there's a second issue with the order of the options.

+Second issue was located in two files:
+- built-in.cpp
+- loop.cpp
+
+Implementation tests were successful.
+
 **Tests:**
    • status: implementing --> testing
     

  • Erik Hänel

    Erik Hänel - 2016-07-03
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -12,6 +12,7 @@
 Function was corrected. Maybe there's a second issue with the order of the options.

 Second issue was located in two files:
+

 - built-in.cpp
 - loop.cpp
     

  • Erik Hänel

    Erik Hänel - 2016-07-14
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -19,3 +19,4 @@
 Implementation tests were successful.

 **Tests:**
+Test were passed successful. Bug fix accepted.
    • status: testing --> fixed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB