Tapestry 5
  1. Tapestry 5
  2. TAP5-1425

AssetSource.getUnlocalizedAsset() should throw an exception if asset is found but does not actually exist

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.3, 5.2
    • Fix Version/s: 5.3
    • Component/s: tapestry-core
    • Labels:
      None

      Description

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

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Massimo Lusetti
            Reporter:
            Howard M. Lewis Ship
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development