[ 
https://issues.apache.org/jira/browse/TAP5-1425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13142279#comment-13142279
 ] 

Hudson commented on TAP5-1425:
------------------------------

Integrated in tapestry-trunk-freestyle #622 (See 
[https://builds.apache.org/job/tapestry-trunk-freestyle/622/])
    TAP5-1425 Throw an exception if asset is found but does not actually exists

mlusetti : 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1196608
Files : 
* 
/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/AssetSourceImpl.java

                
> AssetSource.getUnlocalizedAsset() should throw an exception if asset is found 
> but does not actually exist
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: TAP5-1425
>                 URL: https://issues.apache.org/jira/browse/TAP5-1425
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.3, 5.2
>            Reporter: Howard M. Lewis Ship
>            Priority: Minor
>
> From the JavaDoc:
>   /**
>      * Find an asset but does not attempt to localize it. If the path has no 
> prefix, it is assumed to
>      * be on the classpath.
>      * 
>      * @since 5.2.0
>      * @throws RuntimeException
>      *             if the asset can not be found
>      */
>       Asset getUnlocalizedAsset(String path);
> I discoverred that I can pass an invalid path name and it works.  
>     private Asset getLocalizedAssetFromResource(Resource unlocalized, Locale 
> locale)
>     {
>         Resource localized = locale == null ? unlocalized : 
> unlocalized.forLocale(locale);
>         if (localized == null)
>             throw new 
> RuntimeException(ServicesMessages.assetDoesNotExist(unlocalized));
>         return getAssetForResource(localized);
>     }
> That should be:
>   if (localized == null || ! localized.exists())

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to