[theme-reviewers] Another Artiseer Theme

[Chip Bennett]

Darn it, the Theme Check output was so long, my message got flagged for
moderation... here it is again, with just the "REQUIRED" output.

Nope; Theme Check didn't catch it:


   - REQUIRED: No content width has been defined. Example:

   if ( ! isset( $content_width ) ) $content_width = 900;

   - REQUIRED: *License:* is missing from your style.css header.
   - REQUIRED: *License URI:* is missing from your style.css header.
   - REQUIRED: *wp_specialchars* found in the file *library/legacy.php*.
   Deprecated since version *2.8*. Use *esc_html()* instead.

   Line 91: return wp_specialchars($text);

   - REQUIRED: *register_sidebar_widget* found in the file *
   library/legacy.php*. Deprecated since version *2.8*. Use *
   wp_register_sidebar_widget()* instead.
   - REQUIRED: *get_option('home')* was found in the file *
   library/navigation.php*. Use *home_url()* instead.

   Line 163: 'attr' => array('class' => ($active ? 'active' : ''),
'href' => get_option('home'), 'title' => $title),

   Line 171: $href = (($frontID && $frontID == $id) ?
get_option('home') : get_page_link($id));

   - REQUIRED: *get_option('home')* was found in the file *header.php*. Use
   *home_url()* instead.

   Line 69: <h1 id='name-text' class='edelBlau-logo-name'><a
href='<?php echo get_option('home'); ?>/'><?php bloginfo('name');
?></a></h1>

   - REQUIRED: *attribute_escape* found in the file *library/legacy.php*.
   Deprecated since version *2.8*. Use *esc_attr()* instead.

   Line 85: return attribute_escape($text);



Strange.

By the way, why not check for the string "artiseer", or function names/CSS
classes that start with "art_"?

Chip

On Mon, Jan 17, 2011 at 3:28 PM, Simon Prosser <pross at pross.org.uk> wrote:

> On 01/17/11 21:27, Chip Bennett wrote:
> > Don't know. Didn't get that far. Let me download, install, and check
> (unless
> > you can do it through the IRC channel real quick?).
> you go ahaed
>
> --
> 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://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110117/d54e488e/attachment-0001.htm>