javaserverfaces-spec-public
  1. javaserverfaces-spec-public
  2. JAVASERVERFACES_SPEC_PUBLIC-914

add ResourceWrapper#getContentType() or define resource behaviour for null values

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1, 2.2
    • Component/s: Components/Renderers
    • Labels:
      None
    • Status Whiteboard:
      Hide

      size_medium importance_medium

      Show
      size_medium importance_medium
    • Tags:

      Description

      Component libraries which use the ResouceWrapper but do not explicitely override Resource#getContentType() will cause an undefined content Type (aka null).

      So we either also provide delegation for this method in ResourceWrapper() or we define that the resource handler implementation must use the external context to determine the default contentType to use as fallback.

        Issue Links

          Activity

          Hide
          rogerk added a comment -

          I would prefer to go the route of providing delegation in ResourceWrapper - see: http://java.net/jira/browse/JAVASERVERFACES-1879
          However, this may be classified as a spec change and therefore we may not be able to get it into 2.1 (but let's see). To speed things up, can you provide a patch for the implementation workaround option?

          Show
          rogerk added a comment - I would prefer to go the route of providing delegation in ResourceWrapper - see: http://java.net/jira/browse/JAVASERVERFACES-1879 However, this may be classified as a spec change and therefore we may not be able to get it into 2.1 (but let's see). To speed things up, can you provide a patch for the implementation workaround option?
          Hide
          rogerk added a comment -

          Assign

          Show
          rogerk added a comment - Assign
          Hide
          rogerk added a comment -

          Need to defer to 2.2 since this is a spec change.

          Show
          rogerk added a comment - Need to defer to 2.2 since this is a spec change.
          Hide
          Hanspeter Duennenberger added a comment -
          Show
          Hanspeter Duennenberger added a comment - The patch is attached to http://java.net/jira/browse/JAVASERVERFACES-1879
          Hide
          Ed Burns added a comment -

          This should be done for 2.2

          Show
          Ed Burns added a comment - This should be done for 2.2
          Hide
          rogerk added a comment -

          triage

          Show
          rogerk added a comment - triage
          Hide
          Hanspeter Duennenberger added a comment -

          This issue was resolved by JAVASERVERFACES_SPEC_PUBLIC-917

          Show
          Hanspeter Duennenberger added a comment - This issue was resolved by JAVASERVERFACES_SPEC_PUBLIC-917
          Hide
          Manfred Riem added a comment -

          Closing resolved issue out

          Show
          Manfred Riem added a comment - Closing resolved issue out

            People

            • Assignee:
              Hanspeter Duennenberger
              Reporter:
              Mark Struberg
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: