<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Maybe we could also add few references on <a href="http://make.wordpress.org/themes">make.wordpress.org/themes</a> and/or Theme Review page to:<br clear="all">
</font></font></font><div><div><div><font face="arial, helvetica, sans-serif" color="#333333"><br></font></div><div><font face="arial, helvetica, sans-serif" color="#333333"><a href="http://ottopress.com/2009/wordpress-settings-api-tutorial/">http://ottopress.com/2009/wordpress-settings-api-tutorial/</a></font></div>
<div><font face="arial, helvetica, sans-serif"><a href="http://themeshaper.com/2010/06/03/sample-theme-options/">http://themeshaper.com/2010/06/03/sample-theme-options/</a></font></div><div><font face="arial, helvetica, sans-serif"><a href="http://www.chipbennett.net/2011/02/17/incorporating-the-settings-api-in-wordpress-themes/">http://www.chipbennett.net/2011/02/17/incorporating-the-settings-api-in-wordpress-themes/</a></font></div>
<div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">Cheers,</font></div><div><font face="arial, helvetica, sans-serif">Emil</font></div><div><font face="arial, helvetica, sans-serif"><br>
</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">----</font></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br><div class="gmail_quote">On Sun, Jun 12, 2011 at 3:45 PM, Azizur Rahman <span dir="ltr"><<a href="mailto:prodevstudio%2Bwordpress@gmail.com">prodevstudio+wordpress@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Otto,<br><br>I take this could also apply to plugins!<br><br clear="all">Kind Regards,<br><font color="#888888">Azizur Rahman</font><div><div></div><div class="h5"><br><br><br><div class="gmail_quote">On Sun, Jun 12, 2011 at 9:26 PM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is just a note about a rule that I think should be added to the<br>
theme review process:<br>
<br>
- Themes should not do anything to the database merely by virtue of<br>
being activated.<br>
<br>
By that, I mean that having a theme check for a setting, like<br>
get_option or get_theme_mod and then calling set_option or<br>
set_theme_mod to set it to some default is wrong.<br>
<br>
Both get_option and get_theme_mod accept a second parameter of a<br>
default setting. If I call get_option('whatever',123) and there is no<br>
whatever option, then that will return 123.<br>
<br>
Use the defaults properly. Don't set them just because they're not there.<br>
<font color="#888888"><br>
-Otto<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
</font></blockquote></div><br><div style="padding:0px;margin-left:0px;margin-top:0px;overflow:hidden;word-wrap:break-word;color:black;font-size:10px;text-align:left;line-height:130%">
</div>
<div style="padding:0px;margin-left:0px;margin-top:0px;overflow:hidden;word-wrap:break-word;color:black;font-size:10px;text-align:left;line-height:130%">
</div>
</div></div><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></div>