On Aug 30, 2007, at 12:50 AM, Travis Snoozy wrote:

IMHO, the change is more "subtle and insidious" than "low impact" ;).
Adding values to enums means hitting "default" in any switch statements
for those enums, and that either leads to (a) incorrect behavior
(e.g., default == everything else == published), (b) an error (i.e.,
default == unknown == WTF? Error!), or (c) incorrect behavior that
looks correct (e.g., default == everything else == unpublished; correct
in the case of "pending").

I can guarantee that this will be a problem for In Series, at least.

It's good to keep these things in mind as you code plugins. Consider using multiple OR'd conditions instead of a negated condition that may in the future include unwanted results.

--
Mark Jaquith
http://markjaquith.com/

Covered Web Services
http://coveredwebservices.com/

WordPress Ninja @ b5media Inc
http://b5media.com/


_______________________________________________
wp-testers mailing list
[email protected]
http://lists.automattic.com/mailman/listinfo/wp-testers

Reply via email to