[he.encode] Avoid error if arg string is null about he HOT 5 CLOSED

Such a nice library and it falls on null? Uauuu, this should not happen... Now I have to have multiple times in my code if(... null) instead of one if in the library...
Please do it...
Thank you!

from he.

I think it's reasonable for a library to fail when given an input without an "obviously correct" way to handle it. Encoding null → "null" seems a sensible interpretation too.

Numbers are also rejected, likely for similar reasons.

To specify your own logic for the null case and avoid repeating the check in your other code, you could create a simple wrapper function in your code:

var he = function(string, opt) { return string === null ? "" : he(string, opt) };

Then just use he as you would otherwise.

from he.

Agreed with @anko (although you’d use he.encode or he.decode instead of he directly). null → "null" is a reasonable expectation as well. Use he as documented and create a wrapper as needed.

from he.

Okay,
After all, encoding null string should fail...
I'll use a wrapper, Thx!

from he.

This tip helped me out. I wrote my wrapper as such. I'm sure it could be consolidated but thought it might help someone else as is:

function isInt(value) {
  return !isNaN(value) &&
         parseInt(Number(value)) == value &&
         !isNaN(parseInt(value, 10));
}

function isFloat(n) {
    return Number(n) === n && n % 1 !== 0;
}

// Add encodeall method to avoid potential issues
he.encodeall = function(string, opt) {
  if (string === null) {
    return "";
  } else if (isInt(string) || isFloat(string)) {
    return string;
  } else {
    return he.encode(string, opt)
  }
};

now I just use he as such: he.encodeall(null)

from he.