Simon,<br>Thanks for the response. While I will fix get_links to get_bookmarks (though the remarks clearly state that deprecated APIs are okay), the information provided about the "undefined index" notices is just wrong and there is no way I can fix more than 90% of those. <br>
<br>Pick the very first one for example, which says that there is no index called "#horizontal-outer-widgets-1 a:hover/color" in theme-definitions.php. That is incorrect information and you really need to see the code to see how it works. This particular index has been defined multiple times, and there is an inheritance array defined, so if the variable is not defined in one place it picks it from the parent etc. <br>
<br>Now pick the second message that talks about a missing index called "parent" in the same file. Same concept there - there is an inheritance defined and that segment of the loop is executed only if a particular index is missing.<br>
<br>Ditto for ALL messages in that dump that talk about missing indexes - they completely miss the point about how the coding has been done. I could list out each one of them and explain why it is not an error and how they are encased in conditions that will only execute if all related variables are set. Without someone actually verifying what is happening, it is very irritating to see the entire dump of notices (most of which are not even close to being an error) thrown at me without context. An analogous situation would be the debugger reporting a missing function even if that function is enclosed in a "function_exists()" check, and you asking me to fix it. It is more a problem of the debugger getting over-zealous than of the code being problematic - that is the point I am trying to make.<br>
<br>Sayontan.<br><br><div class="gmail_quote">On Sat, Jul 3, 2010 at 5:46 AM, Simon Prosser <span dir="ltr"><<a href="mailto:pross@pross.org.uk">pross@pross.org.uk</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;">
Im sorry the queue is so long, there is a very small team wading through this long list.<br><br>I understand that the debug info is overwhelming but it isnt just variables not being set.<br><br>for example:<br><br>Notice: get_links is <b>deprecated</b> since version 2.1! Use
get_bookmarks() instead.<br><br>Notice: automatic_feed_links is <b>deprecated</b> since
version 3.0! Use add_theme_support( 'automatic-feed-links' ) instead. (this is optional at this time)<br><br>Most of the undefined variables are the same repeated over and over so it wouldnt be too hard to fix later. but the get_links has to be fixed.<br>
<br>I understand you have previous versions approved, so do a lot of other themes. I spoke to Joseph about this ticket before i closed it.<br><br>Please re-upload a new version and i will prioritise it. this weekend i am off work, and here in ireland there is no wierd holidays going on ;)<div>
<div></div><div class="h5"><br>
<br> <br><br><div class="gmail_quote">On 3 July 2010 12:08, Sayontan Sinha <span dir="ltr"><<a href="mailto:sayontan@gmail.com" target="_blank">sayontan@gmail.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;">
Simon,<div>I have no idea what your comments mean. They give me no indication of what you are asking me to fix.<br><ol><li>Your comment states <br>"Please review the theme with the test data found in the Theme
Development Checklist (TDC); and with <b>define('WP_DEBUG', true);</b>
included in wp-config.php<br>
(NB: functions deprecated in 3.0 should be addressed but are not
show-stoppers at this time.)" <br>When I actually look at the log, ALL the points you have listed are debug "<u><b>notices</b></u>", not "<u><b>errors</b></u>" as you state, and they have no bearing on what shows up on a user's screen. Also, I hope you understand that running something with debug turned on will only present debug information at the best - not bonafide error information.<br>
</li><li>Also, the theme has been checked against the TDC multiple times and there is nothing on that list which the theme fails.</li><li>Coming to the individual "errors" you listed, I don't think you did a diff with the previous version. Of all the files you listed, I haven't touched most of them since the past 5-10 releases, if not more (e.g. theme-definitions.php, most of the files in the widgets folder etc). Hence if they are truly errors, none of the last several versions should have been approved.<br>
</li><li>Also, can you please do a proper review? Your comments seem to apply to version 3.5.3, while the version I submitted is version 3.5.4. A simple check. I am being showed a line in actions.php, line 1414 which apparently has an undefined variable called "ret". <br>
</li></ol>This theme review process is getting quite absurd. I am being provided comments for a version that is <b>already live (3.5.3) </b>and being told that it has innumerable errors (which are really debug notices that don't impact the theme), all while version 3.5.3 has had 15,000+ downloads so far that seem to be going fine for the users. Wonder how it got approved, then, in the first place?<br>
<br>Can someone please do a proper review and not just provide me with a list of debug notices? I understand that you guys have a backlog of reviews, but frankly, before the theme review experiment started (<a href="http://wordpress.org/development/2010/06/expanding-the-theme-review-experiment/" target="_blank">http://wordpress.org/development/2010/06/expanding-the-theme-review-experiment/</a>) reviews always seemed a much more streamlined and enjoyable process. The feedback I received from Joseph was timely, more precise and to the point, and always helped improve the theme, quite unlike the interminable delays and incomplete feedback that I am seeing now.<br>
<br>Sayontan.<div><div></div><div><br><br><div class="gmail_quote">On Fri, Jul 2, 2010 at 9:06 PM, Simon Prosser <span dir="ltr"><<a href="mailto:pross@pross.org.uk" target="_blank">pross@pross.org.uk</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;">Feedback posted at <a href="http://themes.trac.wordpress.org/ticket/49" target="_blank">http://themes.trac.wordpress.org/ticket/49</a><br clear="all">
<font color="#888888"><br>-- <br>My Blog: <a href="http://www.pross.org.uk/" target="_blank">http://www.pross.org.uk/</a><br>Plugins : <a href="http://www.pross.org.uk/plugins/" target="_blank">http://www.pross.org.uk/plugins/</a><br>
Themes: <a href="http://wordpress.org/extend/themes/profile/pross" target="_blank">http://wordpress.org/extend/themes/profile/pross</a><br>
</font></blockquote></div><br><br clear="all"><br></div></div>-- <br>Sayontan Sinha<br><a href="http://mynethome.net" target="_blank">http://mynethome.net</a> | <a href="http://mynethome.net/blog" target="_blank">http://mynethome.net/blog</a><br>
--<br>Lake Chargoggagoggmanchauggagoggchaubunagungamaugg - 45-letter Native American name for Lake Webster, Webster, Massachusetts, meaning "You fish on your side; I fish on my side; nobody fish in the middle."<br>
</div>
</blockquote></div><br><br clear="all"><br>-- <br>My Blog: <a href="http://www.pross.org.uk/" target="_blank">http://www.pross.org.uk/</a><br>Plugins : <a href="http://www.pross.org.uk/plugins/" target="_blank">http://www.pross.org.uk/plugins/</a><br>
Themes: <a href="http://wordpress.org/extend/themes/profile/pross" target="_blank">http://wordpress.org/extend/themes/profile/pross</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Sayontan Sinha<br><a href="http://mynethome.net">http://mynethome.net</a> | <a href="http://mynethome.net/blog">http://mynethome.net/blog</a><br>--<br>Lake Chargoggagoggmanchauggagoggchaubunagungamaugg - 45-letter Native American name for Lake Webster, Webster, Massachusetts, meaning "You fish on your side; I fish on my side; nobody fish in the middle."<br>