Menu ▾ ▴

#119 findfile() doesn't understand wildcards

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

The function findfile("Vers*") will throw a standard exception.

Analysis:
A quite early change in v1.0.9rc2 added the custom file extensions and changed the base object from Datafile to FileSystem, which means that the base file path now is the root folder instead of the base load path. The only consistent fi would be adding an optional argument to findfile(), which defines the base path for this call if findfile().
The standard exception results from a untested call to string::substr(string::find()).

Implementation:
An additional argument was added to findfile(STR1,STR2). In this case STR1 identifies the filename and STR2 the standard search path, which defaults to the NumeRe root directory. However, the search path may also be passed explicitely in STR1.

Documentation:
New feature of findfile() was added to the language files and to the search data bases.

Tests:
Final tests were passed successfully. Bug fix accepted.

Discussion

  • Erik Hänel

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

  • Erik Hänel

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

    Diff:

    --- old
+++ new
@@ -1 +1,9 @@
 The function `findfile("Vers*")` will throw a standard exception.
+
+**Analysis:**
+
+**Implementation:**
+
+**Documentation:**
+
+**Tests:**
    • status: accepted --> analyzing
     

  • Erik Hänel

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

    Diff:

    --- old
+++ new
@@ -1,6 +1,7 @@
 The function `findfile("Vers*")` will throw a standard exception.

 **Analysis:**
+A quite early change in v1.0.9rc2 added the custom file extensions and changed the base object from `Datafile` to `FileSystem`, which means that the base file path now is the root folder instead of the base load path. The only consistent fi would be adding an optional argument to `findfile()`, which defines the base path for this call if `findfile()`.

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

  • Erik Hänel

    Erik Hänel - 2016-10-08
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -2,9 +2,12 @@

 **Analysis:**
 A quite early change in v1.0.9rc2 added the custom file extensions and changed the base object from `Datafile` to `FileSystem`, which means that the base file path now is the root folder instead of the base load path. The only consistent fi would be adding an optional argument to `findfile()`, which defines the base path for this call if `findfile()`.
+The standard exception results from a untested call to `string::substr(string::find())`.

 **Implementation:**
+An additional argument was added to `findfile(STR1,STR2)`. In this case `STR1` identifies the filename and `STR2` the standard search path, which defaults to the NumeRe root directory. However, the search path may also be passed explicitely in `STR1`.

 **Documentation:**
+New feature of `findfile()` was added to the language files and to the search data bases.

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

  • Erik Hänel

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

  • Erik Hänel

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

    Diff:

    --- old
+++ new
@@ -11,3 +11,4 @@
 New feature of `findfile()` was added to the language files and to the search data bases.

 **Tests:**
+Final tests were passed successfully. Bug fix accepted.
    • status: testing --> fixed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB