Issues in documentation for parsing

[David Contreras]

Hi,

Documentation states that in a parser getAll() retrieves a map, but it's an object.

mathjs/docs/expressions/parsing.md

Lines 169 to 170 in cee9deb

 - `getAll()` 
   Retrieve a map with all defined a variables from the parser's scope. 

The one that retrieves a map is getAllAsMap but it's not documented, fixing the grammatical error of "defined a variables" probably should be:

• getAll()
  Retrieve an object with all defined variables in the parser's scope.
• getAllAsMap()
  Retrieve a map with all defined variables in the parser's scope.

Reference

mathjs/src/expression/Parser.js

Lines 99 to 113 in cee9deb

   /** 
    * Get a map with all defined variables 
    * @return {Object} values 
    */ 
   Parser.prototype.getAll = function () { 
     return toObject(this.scope) 
   } 
  
   /** 
    * Get a map with all defined variables 
    * @return {Map} values 
    */ 
   Parser.prototype.getAllAsMap = function () { 
     return this.scope 
   } 

[Jos de Jong]

Good find, we should update and clarify the docs!

[Jos de Jong]

Fixed via #3158

[Jos de Jong]

The fix is now published in v12.4.0