tapestry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Howard M. Lewis Ship (Updated) (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (TAP5-1425) AssetSource.getUnlocalizedAsset() should throw an exception if asset is found but does not actually exist
Date Mon, 24 Oct 2011 15:39:35 GMT

     [ https://issues.apache.org/jira/browse/TAP5-1425?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Howard M. Lewis Ship updated TAP5-1425:
---------------------------------------

    Summary: AssetSource.getUnlocalizedAsset() should throw an exception if asset is found
but does not actually exist  (was: AssetSource.getUnlocalizedAsset() should throw an exception
if asset not found but does not actually exist)
    
> 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

        

Mime
View raw message