LEN method returns wrong result with an empty values

[nikita-prtsr]

I have a problem:

const context = {
  name: null
}
expect(axel.compile('LEN(name)')(context)).toEqual(0) // error here with result - 4

I found that it is because of converting of nil values to "NULL" string instead of "" an empty string in case of excel functions which accepts string as an argument.

I've found that the excel-string function used in LEN func implementation and guessing that we should add an additional (if (nil? text) "" text) to avoid nil in excel-string function call.

What should I do?
It would be nice if you adjust the LEN function to accept nil values as an "" empty string instead of "NULL" string.

[zarkone]

sounds fare, I think.

I also found that LEN also works on numbers in gdocs:

when in axel-f it is always 4 for numbers because of the same reason

[nikita-prtsr]

@zarkone So would it be better to add more conditions for argument type of LEN implementation?

[zarkone]

I think it makes sense 👍