4820

pending completion

Build # 4820

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

[bugfix] Properly fallback to base locales in array getter (#5459)

It looks like the existing implementation of this feature was pretty
well thought out and correct, the problem was that a compareArray
function was used which was comparing arrays in a rather bizzare
fashion that was not in line with what a "normal person" would expect.
So now the array comparator is correct, and the logic works properly.

The logic works as follows -- the next locale is picked and tried as-is.
If that fails, the code normally tries to chop the last bit (normally
the country designation) and try again. However, if the next array
element has the same or longer prefix as the one to be tried, the
iteration continues. So for example if the array has the sequence

    "AA-BB", "AA-CC", "XX-YY"

then first "AA-BB" is tried, then a naive solution would try "AA", but
this one instead checks to see that "AA-CC" is actually more concrete
than "AA", so it tries "AA-CC" next, and only after it fails (if it
fails) it tries "AA", because "XX-YY" does not have "AA" as prefix. So
in the end the following locales are tried in this order (assuming all
fail so the next one is tried):

    "AA-BB", "AA-CC", "AA", "XX-YY", "XX"

Fixes #4780

Run Details

7387 of 14519 branches covered (50.88%)

45205 of 51145 relevant lines covered (88.39%)

9002.45 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

93.62

/build/umd/moment.js

Source Not Available

The file "build/umd/moment.js" isn't available on github. Either it's been removed, or the repo root directory needs to be updated.

moment / moment / 4820

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available

Source File
Press 'n' to go to next uncovered line, 'b' for previous