Menu ▾ ▴

#416 Multiline headlines are not supported by 'stats'

v1.1.x
fixed
Erik Hänel
None
v1.1.1
Bug Fix
2019-01-26
2018-12-01
Erik Hänel
No

Multiline headlines are not supported by the command stats, neither printed to the terminal nor printed to a file.

Analysis:

This may be fixed by adding a new method to the Memory class, which will return the number of headlines for the current selected table. The result may be used to detect the correct number of headlines, which will be used to display the output correctly.

Implementation:

  • Implementation: The fix was processed as indicated by the analysis.
  • Revision: [r441]
  • Implementation test: Statistics of a table with multiple headline lines were created and showed no deviations.

Documentation:

  • ChangesLog: updated
  • Comments: New member functions were commented
  • Documentation articles: Not needed
  • Language files: Not needed

Tests:

Fix was added to the automatic tests. No deviations occured. Bug fix implemented successfully.

Related

Commit: [r441]

Discussion

  • Erik Hänel

    Erik Hänel - 2018-12-01
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2018-12-02
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1 +1,18 @@
 Multiline headlines are not supported by the command `stats`, neither printed to the terminal nor printed to a file.
+
+###Analysis:
+(*Describe, what's the issue and which changes have to be made*)
+
+###Implementation:
+* Implementation: (*Describe, what you've changed*) 
+* Revision: [rXXX]
+* Implementation test: (*Describe the type of test, which you performed, and if it was successful*)
+
+###Documentation:
+* ChangesLog: (*Have you updated the changes log?*)
+* Comments: (*Have you written comments in the code, which describe your change?*)
+* Documentation articles: (*Have you updated the documentation articles?*)
+* Language files: (*Have you updated the language files?*)
+
+###Tests:
+(*Describe, which tests you performed and their outcome*)
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2018-12-03
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1,7 +1,7 @@
 Multiline headlines are not supported by the command `stats`, neither printed to the terminal nor printed to a file.

 ###Analysis:
-(*Describe, what's the issue and which changes have to be made*)
+This may be fixed by adding a new method to the `Memory` class, which will return the number of headlines for the current selected table. The result may be used to detect the correct number of headlines, which will be used to display the output correctly.

 ###Implementation:

 * Implementation: (*Describe, what you've changed*)
    • status: analyzing --> implementing
     

  • Erik Hänel

    Erik Hänel - 2018-12-03
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -4,15 +4,15 @@
 This may be fixed by adding a new method to the `Memory` class, which will return the number of headlines for the current selected table. The result may be used to detect the correct number of headlines, which will be used to display the output correctly.

 ###Implementation:
-* Implementation: (*Describe, what you've changed*) 
-* Revision: [rXXX]
-* Implementation test: (*Describe the type of test, which you performed, and if it was successful*)
+* Implementation: The fix was processed as indicated by the analysis.
+* Revision: [r441]
+* Implementation test: Statistics of a table with multiple headline lines were created and showed no deviations.

 ###Documentation:
-* ChangesLog: (*Have you updated the changes log?*)
-* Comments: (*Have you written comments in the code, which describe your change?*)
-* Documentation articles: (*Have you updated the documentation articles?*)
-* Language files: (*Have you updated the language files?*)
+* ChangesLog: updated
+* Comments: New member functions were commented
+* Documentation articles: Not needed
+* Language files: Not needed

 ###Tests:
 (*Describe, which tests you performed and their outcome*)
    • status: implementing --> testing
     

    Related

    Commit: [r441]

  • Erik Hänel

    Erik Hänel - 2019-01-26
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -15,4 +15,4 @@

 * Language files: Not needed

 ###Tests:
-(*Describe, which tests you performed and their outcome*)
+Fix was added to the automatic tests. No deviations occured. Bug fix implemented successfully.
    • status: testing --> fixed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB