[theme-reviewers] Distilled Review

[randall]

On 06/15/2010 11:37 PM, Tom Lany wrote:
> Hi Randall,
>
> Thanks for taking a look at some of these issues.
>
> Regarding the DB query, you should be able to pull an avatar without a 
> direct database query.  See line 37 of: 
> http://core.trac.wordpress.org/browser/trunk/wp-content/themes/twentyten/single.php 
> for an example of how this is done in Twentyten (the new default theme).
>
> Regarding the "image is displayed twice" issue, when I tested the 
> theme, I saw two copies of the same image on an attachment image 
> page.  To reproduce, upload an image on a post page.  For the link 
> option in the image option dialog that appears once the image has been 
> uploaded, choose "Post URL", and then insert the image into the post.  
> Click on the link, and you should see the double inserted image.
>
> Thanks!
>
> Tom Lany
> http://tomlany.net
thanks Tom,

have added an attachment.php and this seems to fix the issue of the 
double images.

leaves the db query, pulling an avatar like on line 37 of the link is 
what i have in my single.php as well, this pulls the data + avatar of 
the post author,
but, what i cant figure out is to have a list of all the blogs authors 
with there avatars, when hitting the search engines for this it seems to 
confirm that it is not possible without a direct query as described here 
http://codex.wordpress.org/Function_Reference/wpdb_Class.
but maybe i'm overlooking something here....would love to be proven wrong.

in case anybody knows some objection regards to safety/performance etc.. 
, it would be simple to remove the query and have it replaced with a 
list without any avatars, most bloggers are ugly anyway ;)