ExternalContextImpl.getResponseContentType() does not conform to Section 6.1.3.1 of the JSR 329 Spec

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Won't Fix
• Affects Version/s: bridge-impl-2.0.0, bridge-impl-3.0.0, bridge-impl-4.0.0
• Fix Version/s: bridge-impl-2.1.0, bridge-ext-1.0.1, bridge-ext-2.0.1, bridge-ext-3.0.1, bridge-impl-3.1.0, bridge-impl-4.1.0, bridge-ext-5.0.1
• Component/s: Liferay Faces Bridge Ext, Liferay Faces Bridge Impl / Demos / Tests
• Labels: None

Regarding ExternalContext.getResponseContentType(), Section 6.1.3.1 of the JSR 329 Spec (titled "Methods that deviate from Faces 1.2 Javadoc") states:

Return the MIME Content-Type for this response. If called during the RENDER_PHASE or RESOURCE_PHASE, returns the value from the corresponding render response.getContentType()[6.62, 6.129]. If called during the ACTION_PHASE or EVENT_PHASE it throws an IllegalStateException[6.63, 6.130].

The Liferay Faces Bridge implementation does not fully conform to this requirement. While it does call MimeResponse.getContentType() in order to get the content-type, if that value is null then it calls PortletRequest.getResponseContentType() as a fallback, which it should not do.