Perhaps if, as Nacin indicates, we let the uploader script fail Themes that use deprecated functions (regardless of whether or not that use is part of backward-compatibility code), we can avoid any real issues on the subject?<div>
<br></div><div>I agree that one of my primary concerns is the use of deprecated functions. But I'm equally concerned about the Repository giving the appearance of facilitating the use of outdated versions of WordPress.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Wed, Oct 20, 2010 at 11:21 AM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com">edward.caissie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><div class="gmail_quote">On Wed, Oct 20, 2010 at 11:37 AM, Andrew Nacin <span dir="ltr"><<a href="mailto:wp@andrewnacin.com" target="_blank">wp@andrewnacin.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">
If this is about deprecated functions, then they should be surrounding things in function_exists checks, so proper functions get run when they are available. If the upload script catches such functions, then it's still not our problem. That's the solution, really... Don't worry about it, and let the upload script bark at them.</blockquote>
</div><br></div>I would say my only real concern with backward compatibility is the use of deprecated functions, and as far as that goes, I would agree to let the theme uploader script take the heat for not allowing a Theme into the review queue versus it being "not-approved" for using them ... with the proviso the most current and correct function is used first as suggested with 'function_exists' wrappers.<br>
<br>But does this alleviate the Theme Reviewer from having to check the deprecated function being called?<br><br><br>Cais.<br>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>