[theme-reviewers] Undefined index error in themes

[Chip Bennett]

As a matter of fact, that very error was pointed out in that very Theme, in
its review ticket <https://themes.trac.wordpress.org/ticket/164>.

Chip

On Mon, Sep 27, 2010 at 2:28 AM, Simrandeep Singh <
simrankalsi2004 at yahoo.co.in> wrote:

> Do you consider the following error while reviewing the themes ?
>
> Undefined index: page in \themes\theme-name\includes\theme-options.php on
> line 24
>
> I downloaded few themes from directory and they were getting the similar
> error including the featured theme Titan.
>
> Regards
> --
> Simrandeep Singh | http://www.tricksdaddy.com
>
>
> --- On *Mon, 27/9/10, Philip M. Hofer (Frumph) <philip at frumph.net>* wrote:
>
>
> From: Philip M. Hofer (Frumph) <philip at frumph.net>
> Subject: Re: [theme-reviewers] Theme Uploader and CSS Classes
> To: theme-reviewers at lists.wordpress.org
> Date: Monday, 27 September, 2010, 9:55 AM
>
> I look at it more like Otto just snagging the regexp stuff and ignoring the
> underlying reason to use it, the reporting back of information.
>
> 'steal bits' is pretty much just stripping out stuff to put into the
> wordpress one, otto can create his own code for that, the functionality of
> pross's is what would be good to use for the wordpress site.  If that's not
> going to be done, then just leave it on pross's.
>
>
>
> ----- Original Message -----
> *From:* Edward Caissie <http://mc/compose?to=edward.caissie@gmail.com>
> *To:* theme-reviewers at lists.wordpress.org<http://mc/compose?to=theme-reviewers@lists.wordpress.org>
> *Sent:* Sunday, September 26, 2010 8:36 PM
> *Subject:* Re: [theme-reviewers] Theme Uploader and CSS Classes
>
> I'm sure there's a way to merge the best of both so it can be used in both
> places ...
>
> Pross' has more of the Theme Review items the Theme Review Team wants added
> to the uploader checks while the uploader (I imagine) has more script
> checking items than Pross'
>
> ... at least that's my view on it.
>
>
> Cais.
>
> On Sun, Sep 26, 2010 at 11:08 PM, Philip M. Hofer (Frumph) <
> philip at frumph.net <http://mc/compose?to=philip@frumph.net>> wrote:
>
> Steal bits? hrm.   Naw, let's just leave it up on Pross's website if thats
> the attitude your going to take on it.   Either use it or don't.
>
>
>
>
>
> ----- Original Message ----- From: "Otto" <otto at ottodestruct.com<http://mc/compose?to=otto@ottodestruct.com>
> >
> To: <theme-reviewers at lists.wordpress.org<http://mc/compose?to=theme-reviewers@lists.wordpress.org>
> >
> Sent: Sunday, September 26, 2010 2:31 PM
>
> Subject: Re: [theme-reviewers] Theme Uploader and CSS Classes
>
>
>   If you feel like emailing me your code, I'll be happy to take a look
> at it and steal bits for the image uploader. Save us some time. ;)
>
> -Otto
>
>
>
> On Sun, Sep 26, 2010 at 11:52 AM, Simon Prosser <pross at pross.org.uk<http://mc/compose?to=pross@pross.org.uk>>
> wrote:
>
> On 09/25/2010 11:34 PM, Edward Caissie wrote:
>
> Yes, that would be very good!
>
> That way as items change their importance we can simply change the
> recommended to required or vice-versa.
>
> ok new version uploaded, now checks for php shortags, added recommended
> and required to functions. Also checks style.css for the alignment css
>
> --
> My Blog: http://www.pross.org.uk/
> Plugins : http://www.pross.org.uk/plugins/
> Themes: http://wordpress.org/extend/themes/profile/pross
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org<http://mc/compose?to=theme-reviewers@lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org<http://mc/compose?to=theme-reviewers@lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org<http://mc/compose?to=theme-reviewers@lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>  ------------------------------
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
> -----Inline Attachment Follows-----
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org<http://mc/compose?to=theme-reviewers@lists.wordpress.org>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20100927/8a343740/attachment.htm>