Skip to content
This repository has been archived by the owner on Sep 5, 2023. It is now read-only.

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

Closed
glassfishrobot opened this issue Nov 30, 2010 · 13 comments

Comments

@glassfishrobot
Copy link

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.

Affected Versions

[2.1]

@glassfishrobot
Copy link
Author

Reported by struberg

@glassfishrobot
Copy link
Author

Issue-Links:
blocks
JAVASERVERFACES_SPEC_PUBLIC-917

@glassfishrobot
Copy link
Author

Was assigned to dueni

@glassfishrobot
Copy link
Author

rogerk said:
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?

@glassfishrobot
Copy link
Author

rogerk said:
Assign

@glassfishrobot
Copy link
Author

rogerk said:
Need to defer to 2.2 since this is a spec change.

@glassfishrobot
Copy link
Author

dueni said:
The patch is attached to http://java.net/jira/browse/JAVASERVERFACES-1879

@glassfishrobot
Copy link
Author

@edburns said:
This should be done for 2.2

@glassfishrobot
Copy link
Author

rogerk said:
triage

@glassfishrobot
Copy link
Author

dueni said:
This issue was resolved by #917

@glassfishrobot
Copy link
Author

Marked as fixed on Thursday, May 19th 2011, 7:39:17 am

@glassfishrobot
Copy link
Author

@manfredriem said:
Closing resolved issue out

@glassfishrobot
Copy link
Author

This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-914

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant