[theme-reviewers] Distilled Review

[Tom Lany]

Have you tried <?php echo get_avatar( get_the_author_meta( 'user_email' 
)); ?> , or something similar for pulling the Avatar?  Look on line 37 
of 
http://core.trac.wordpress.org/browser/trunk/wp-content/themes/twentyten/single.php, 
as I suggested below for the code used in the default  (in 3.0) 
WordPress theme.  They add a gravatar to single.php without a DB query.

Tom Lany
http://tomlany.net

On 6/16/10 1:42 AM, randall wrote:
> 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 ;)
> _______________________________________________
> 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/mailman/private/theme-reviewers/attachments/20100616/46b94b0c/attachment.htm>