squizlabs/HTML_CodeSniffer — pull request #35 by lwright-sq (2012-09-09 22:37:26+00:00)
Title: Fix disabled level switches and View Report button on the landing page of the Auditor

This relates to a situation where there were no instances of one or more levels of messages (eg. no errors).

Disabled switches could still be selected, and this resulted in multiple instances of the "disabled" class appearing on the View Report button. Only one of these was being removed each time a combination existed that could enable it.

Therefore if there was a situation where, eg. with no errors, if you turned off warnings, then errors, two disabled classes would be added. If you then enable notices, it would not remove all disabled classes until either other switch was switched back on.

Provide better feedback for disabled level switches (it now appears off and unswitchable) - which should avoid the possibility of doubled disabled classes in the first place - and also fix the regex that removes the disabled class.