Automattic/Ad-Code-Manager — pull request #83 by harvitronix (2013-07-31 16:13:20+00:00)
Title: Change not "trick" on lines 945-949

On lines 945-949, there's a "special trick" to allow conditionals to be notted with a ! in front of the function name. For instance, it should allow you to do !is_home to display an ad unit on every page other than home.

However, in the Tools->ACM dashboard panel, selecting a !is_home conditional will not save, instead reverting back to is_home.

I propose changing the not trick to use not_ in front of the conditional. This way, it allows you to actually set a not conditional without having to change the save sanitation logic. As an added bonus, the select box displays "Not is home" instead of "!is home", which is easier on the eyes.