[jira] [Commented] (TILES-554) Compatibility bug in BaseLocaleUrlDefinitionDAO.loadDefinitionsFromURL

["Nicolas Le Bas (JIRA)" <ji...@apache.org>]

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

Nicolas Le Bas commented on TILES-554:
--------------------------------------

Handling IOException instead of FileNotFoundException sounds reasonable to me, since the DefinitionsReader cannot throw an IOException.

However I'm not sure if Tiles should allow some the configuration files to be missing, the refreshRequired method certainly doesn't. Perhaps this issue should be tackled where the l10n is handled (i.e. in LocaleUrlDefinitionDAO, not BaseLocaleUrlDefinitionDAO). There are [other issues|https://issues.apache.org/jira/browse/TILES-547] with localization in 2.2, too.

Last but not least this works differently in tiles 3.0. Doesn't mean it works with Apache Felix (we've not tested for it).
                
> Compatibility bug in BaseLocaleUrlDefinitionDAO.loadDefinitionsFromURL
> ----------------------------------------------------------------------
>
>                 Key: TILES-554
>                 URL: https://issues.apache.org/jira/browse/TILES-554
>             Project: Tiles
>          Issue Type: Bug
>          Components: tiles-core
>    Affects Versions: 2.2.2
>            Reporter: Silvano Maffeis
>
> I found a compatibility issue when running tiles under Apache Felix.
> The Tiles method BaseLocaleUrlDefinitionDAO.loadDefinitionsFromURL(URL url)
> assumes a FileNotFoundException is thrown by the underlying URL handler, when the resource pointed to by the URL cannot be found. However, 
> a URL does not necessarily have to denote a file in the filesystem.
> Therefore I think you should not expect a FileNotFoundException here.
> Therefore the Apache Felix URL loader will through a IOException,
> and not a FileNotFoundException. Therefore the finding of localized
> resources will fail when running Tile under Apache Felix.
> Proposed fix: Can BaseLocaleUrlDefinitionDAO.loadDefinitionsFromURL be changed, so that only IOException is treated, the same way as you currently treat FileNotFoundException?
> Actually the Apache Felix folks think the current Felix URL handler implementation (throwing IOException and not FileNotFoundException!) is correct:
> http://mail-archives.apache.org/mod_mbox/felix-users/201202.mbox/%3C4F3305CF.9040707@ungoverned.org%3E

--
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