[Lynne]

Look at the difference between the two templates in just the call for the avatar.

Original:

<div class="cms_widget_post_useravatar widget_post_useravatar">
						<a class="comments_member_avatar_link" href="{vb:link member, {vb:raw post}}">
						<vb:if condition="$post['avatarurl']" >
							<img src="{vb:raw post.avatarurl}" alt="{vb:raw post.username}" />
						<vb:else />
							<img src="{vb:stylevar imgdir_misc}/unknown.gif" alt="{vb:raw post.username}" />
						</vb:if>
						</a>
					</div>

The one you replaced it with:

<a class="comments_member_avatar_link" href="{vb:link member, {vb:raw userinfo}}">
			<vb:if condition="$avatar">
				<img src="{vb:raw avatar.0}" {vb:raw avatar.1} alt="{vb:raw post.username}" />
			<vb:else />
				<img src="{vb:stylevar imgdir_misc}/unknown.gif" alt="{vb:raw post.username}" />
			</vb:if>
			</a>

One uses the variable $post[avatarurl], the other uses just $avatar. You need to use the variables used in the original template or it's not going to work.