Simple date in spanish not working: "Agosto 23"

[acrodrig]

The following program:

const chrono = require("chrono-node");
console.log(chrono.parse("Agosto 23"));
console.log(chrono.parse("August 23"));

Outputs:

[]
[ ParsedResult {
    ref: 2017-06-07T00:32:53.896Z,
    index: 0,
    text: 'August 23',
    tags: { ENMonthNameMiddleEndianParser: true },
    start: ParsedComponents { knownValues: [Object], impliedValues: [Object] } } ]

The second (english date) was parsed correctly, but the exact translation does not work.

Can you help?

[wanasit]

I don't speak Spanish, but look like Spanish date format is "little-endian". It should be "23 Agosto", not "August 23" like US English.

I guess that why @gimenete, the contributor, didn't add the middle-endian parser for Spanish.

[gimenete]

That's right. In Spanish you say "23 Agosto", but never "Agosto 23"

[acrodrig]

I beg to differ. There are a lot of circumstances where this date pattern Is useful. Even if it is true that the date first/month second is more common I do not see any downside in chrono recognizing it.

[gimenete]

The idea is to avoid false positives such as "He trabajado en agosto 23 días" (I've worked in August for 23 days)

[acrodrig]

@gimenete this is hitting us on a production software. If a user responds "Agosto 23" without any other words around, I do not think there are any false positives. Maybe we can have a strict flag? Or I can fork if you point me to the right field where to fix this. Thanks.

[gimenete]

@acrodrig I like the idea of an strict flag. Nevertheless I'm not a maintainer of this project. I made a few contributions and I give my opinion in a few issues, but that's all. You can make a PR if you want, instead of forking it.

[benaubin]

Doesn't Chrono already have a strict mode (instead of casual config). Might be mistaken, but I think we have something like it.

[wanasit]

@acrodrig For your use case, you can create the middle-endian as a custom parser for ES (see. how the little-endian parser work).

However, it would be nicer if you can implement the parser behind casual mode and submit make a PR. I don't have much time recently, but I think we can help reviewing the change.

[acrodrig]

Thanks @wanasit. I will do that.