Uize.Node.Classes reports incorrect state

[BenAtZazzle]

Uize.Node.Classes has a bug in with respect to how it tests if a string is a class w/in the className property. This is easily seen with Uize.Node.Classes.hasClass.

Let's say we have the following node:

<div id="foo" class="bar-bat baz">Test</div>

And we execute the following code:

Uize.Node.Classes.hasClass('foo', 'bar')

This will return true even tho bar is not a class of the foo div. Looking deep into the code, it appears that the regular expression used to determine if a string is in the className uses a word boundary (\b) which works for spaces or the beginning/end of the className, but also targets hyphens which are actually a valid CSS character.

[BenAtZazzle]

Ok, I think I fixed it by changing the following regular expression:

'\\b(?:' + _classes.join ('|').replace (/\|{2,}/g,'|').replace (/^\||\|$/g,'') + ')\\b'

to:

'(?:^| )(' + _classes.join ('|').replace (/\|{2,}/g,'|').replace (/^\||\|$/g,'') + ')(?: |$)'

There's also the corresponding change in getState to get the value at index 1 instead of index 0.

[BenAtZazzle]

Looks like my fix breaks the code that works for changing/setting state (due to the addition of 2 new regex groups), so I'm reverting my changes.

[BenAtZazzle]

Any updates to this? Using classList instead of className could help...