<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Or like in Coraline Theme?<br clear="all"></font></font></font><div><div><div><font face="arial, helvetica, sans-serif" color="#333333"><br></font></div>

<div><font class="Apple-style-span" color="#333333" face="arial, helvetica, sans-serif"><div>/**</div><div> * Set default options</div><div> */</div><div>function coraline_default_options() {</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>$options = get_option( &#39;coraline_theme_options&#39; );</div>

<div><br></div><div><span class="Apple-tab-span" style="white-space:pre">        </span>if ( ! isset( $options[&#39;color_scheme&#39;] ) ) {</div><div><span class="Apple-tab-span" style="white-space:pre">                </span>$options[&#39;color_scheme&#39;] = &#39;light&#39;;</div>

<div><span class="Apple-tab-span" style="white-space:pre">                </span>update_option( &#39;coraline_theme_options&#39;, $options );</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>}</div><div><br></div>

<div><span class="Apple-tab-span" style="white-space:pre">        </span>if ( ! isset( $options[&#39;theme_layout&#39;] ) ) {</div><div><span class="Apple-tab-span" style="white-space:pre">                </span>$options[&#39;theme_layout&#39;] = &#39;content-sidebar&#39;;</div>

<div><span class="Apple-tab-span" style="white-space:pre">                </span>update_option( &#39;coraline_theme_options&#39;, $options );</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>}</div><div>}</div><div>

add_action( &#39;init&#39;, &#39;coraline_default_options&#39; );</div></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"><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 4:35 PM, Philip M. Hofer (Frumph) <span dir="ltr">&lt;<a href="mailto:philip@frumph.net">philip@frumph.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

I am not having an issue with the that being a &#39;best practice&#39; rule., maybe not a requirement [at this time] but definitely best practice.<br>
<br>
Are you saying this?<br>
<br>
$default_theme_options = array(&#39;someoption&#39; =&gt; true, &#39;someotheroption&#39; =&gt; false);<br>
// if &#39;theme_options&#39; returns false use all of the parameters from $default_theme_options<br>
$theme_options = get_option(&#39;theme_options&#39;, $default_theme_options);<br>
<br>
<br>
<br>
<br>
-----Original Message----- From: Otto<br>
Sent: Sunday, June 12, 2011 1:26 PM<br>
To: WP Theme Reviewers Mailing List<br>
Subject: [theme-reviewers] Note to theme reviewers about settings in themes<div><div></div><div class="h5"><br>
<br>
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(&#39;whatever&#39;,123) and there is no<br>
whatever option, then that will return 123.<br>
<br>
Use the defaults properly. Don&#39;t set them just because they&#39;re not there.<br>
<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>
_______________________________________________<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>
</div></div></blockquote></div><br></div></div>