I forgot to mention productThumbnail is the alias for the media picker, and the images are if a folder within the media picker if that makes any difference.

Try this one out:

<xsl:if test="$currentPage/productThumbnail!=''">
        <xsl:variable name="media" select="umbraco.library:GetMedia($currentPage/productThumbnail, 0)" />
        <xsl:variable name="url" select="$media/umbracoFile" />
        <xsl:variable name="width" select="$media/umbracoWidth" />
        <xsl:variable name="height" select="$media/umbracoHeight" />
        <img src="{$url}" width="{$width}" height="{$height}" />
 </xsl:if>

The code that you had before tried to use the GetMedia-extension even if there where no image selected. This causes an error if no image is selected in one of the mediapickers. Other than that, you where using some parts of the old XML scheme in your $url, $width and $height.

Does the above code work?

/Kim A

Hi Ivor,

you definitely don't need the @alias part anymore in 4.5+ and you would need to get the image by appending /Image, please see here at the bottom.

Hope that helps,
Sascha

Actually you might want to put the first:

<a href="{umbraco.library:NiceUrl(@id)}" >
...
</a>

inside of the <xsl:if> as well, to make sure there's not rendered an empty a-tag if no image is selected in the media picker.

/Kim A

@Sascha:

In v.4.5.1 there is no more need of the /Image as far as I'm concerned :)

/Kim A

Oh, that's much better, didn't know that. :)

@Kim Thanks for the snippet I'm not getting the same error,However its not showing the image or the link now though

my codes looks like this now:

<ul class="all-products">

<xsl:for-each select="$currentPage/* [name() = $documentTypeAlias and string(umbracoNaviHide) != '1']">
  <li>
    
<xsl:if test="$currentPage/productThumbnail !=''">
  <a href="{umbraco.library:NiceUrl(@id)}" >
        <xsl:variable name="media" select="umbraco.library:GetMedia($currentPage/productThumbnail, 0)" />
        <xsl:variable name="url" select="$media/umbracoFile" />
        <xsl:variable name="width" select="$media/umbracoWidth" />
        <xsl:variable name="height" select="$media/umbracoHeight" />
        <img src="{$url}" width="{$width}" height="{$height}" />
    </a>
 </xsl:if>
  
<h2><a href="{umbraco.library:NiceUrl(@id)}" ><xsl:value-of select="@nodeName"/></a></h2>
    <p><xsl:value-of select="umbraco.library:TruncateString(umbraco.library:StripHtml(./productDescription), 100,'...')"  /></p>
   <!--<xsl:value-of
 
select="umbraco.library:TruncateString(umbraco.library:StripHtml(./data[@alias
 = 'productDesc']), 100,'...')"/>-->

<p><a href="{umbraco.library:NiceUrl(@id)}" title="" class="btn btnGreen">View product</a></p>
</li>
 </xsl:for-each>
</ul>

Cool that worked, the child-nodes have the mediaPicker on it.

is working on a reply...