FieldClassMatcher: Always yield integer indexes

* src/field/FieldClassMatcher.js (match): Cast `vis' to number. * test/field/FieldClassMatcherTest.js: Remove FIXME and change assertion.
2018-06-12 15:11:24 -04:00 · 2018-06-12 15:11:24 -04:00 · 5164c9dcbb
parent 6d6e80aac7
commit 5164c9dcbb
2 changed files with 2 additions and 3 deletions
--- a/src/field/FieldClassMatcher.js
+++ b/src/field/FieldClassMatcher.js
@ -102,7 +102,7 @@ module.exports = Class( 'FieldClassMatcher',
                    // default to visible; note that, if we've encountered any
                    // "all index" situations (scalars), then we must only be
                    // true if the scalar value was true
-                    vis[ i ] = (
+                    vis[ i ] = +(
                        ( !hasall || all )
                        && ( ( vis[ i ] === undefined )
                            ? 1
--- a/test/field/FieldClassMatcherTest.js
+++ b/test/field/FieldClassMatcherTest.js
@ -260,8 +260,7 @@ describe( 'FieldClassMatcher', () =>
                foo: {
                    all:     false,
                    any:     false,
-                    // FIXME: should be [ 0, 0 ]
-                    indexes: [ false, false ],
+                    indexes: [ 0, 0 ],
                },
            },
        },