<div dir="ltr">Well I would definitely be interested :) - Although my time seems to be split into a million different ways at the moment</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 25 June 2014 17:14, Dylan Scott <span dir="ltr"><<a href="mailto:dylanrhysscott@gmail.com" target="_blank">dylanrhysscott@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 dir="ltr">Well I would definitely be interested :) - Although my time seems to be split into a million different ways at the moment<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div><div dir="ltr">Dylan<br></div>
</div></font></span><div><div class="h5">
<br><br><div class="gmail_quote">On 25 June 2014 17:09, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">That's part of why I thought this exercise might be helpful. :)</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 25, 2014 at 11:01 AM, Dylan Scott <span dir="ltr"><<a href="mailto:dylan.scott@webunitydesign.com" target="_blank">dylan.scott@webunitydesign.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Chip,<br><br></div>After some of the reviews I've done I've found options frameworks a bit infuriating to review as everything seems to be buried and difficult to find. Has anyone else found this? Or I am just new to the process and way of thinking with a framework?<br>



<br></div>Kind Regards<br><br></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div>On 25 June 2014 15:28, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br>



</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr"><div>I think the ThemeForest-sponsored Options Tree framework ( <a href="https://github.com/valendesigns/option-tree" target="_blank">https://github.com/valendesigns/option-tree</a> ) needs a detailed vetting, as I'm seeing it used in several Themes, and note at least two issues:<br>




</div><div><br></div><div>1. Saving default settings to the database (not using sane defaults)</div><div>2. Multiple wp_options entries (multiple register_setting() and update_option() calls, based on single-setting ID)</div>




<div><br></div><div>I don't have time at the moment to go through the framework at the level of detail needed (doing the best I can, just to keep the approval queue in reasonable shape), so I thought it might be a good idea to make a group project out of it. Anyone who wants to practice reviewing an options framework, and who has time to contribute, please dig through it, and post any comments here.</div>




<div><br></div><div>The purpose here is two-fold: to review this specific framework, and also to provide an opportunity for new reviewers for whom review of such code might seem daunting. So, let's see how it goes!</div>




</div>
<br></div></div>_______________________________________________<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></blockquote></div><br></div>
<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></blockquote></div><br></div>
</div></div><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></blockquote></div><br></div></div></div>
</blockquote></div><br></div>