Details
-
Type: Improvement
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 3.1.0.BETA1
-
Fix Version/s: EE-3.3.0.GA_P02, 4.0
-
Component/s: ACE-Components
-
Labels:None
-
Environment:Portlets
Description
The FileEntryPhaseListener#setPortletRequestWrapper(Object) method contains some workarounds for some things that should be fixed in the bridge. One such hack is getting the BridgeContext instance by calling PortletRequest.getAttribute("javax.portlet.faces.bridgeContext") so that the setPortletRequest(PortletRequest) method can be called.
I think the latest version of Liferay Faces Bridge may have fixed all the problems that these workarounds are there to solve. In addition, the following issue documents that the "javax.portlet.faces.bridgeContext" request attribute has been deprecated:
http://issues.liferay.com/browse/FACES-1253
I think the latest version of Liferay Faces Bridge may have fixed all the problems that these workarounds are there to solve. In addition, the following issue documents that the "javax.portlet.faces.bridgeContext" request attribute has been deprecated:
http://issues.liferay.com/browse/FACES-1253
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
I'll take a look. Since we probably have to work with both PortletFaces and Liferay Faces Bridge for some set of future releases, it may not be possible to remove the workarounds, however.
Show
Deryk Sinotte
added a comment - I'll take a look. Since we probably have to work with both PortletFaces and Liferay Faces Bridge for some set of future releases, it may not be possible to remove the workarounds, however.
Resolving as fixed as the dependency on the 'javax.portlet.faces.bridgeContext' request attribute has been removed. I've opened the following cases to address issues found along the way:
ICE-9954- Resource coalescing doesn't work with Liferay 6.2ICE-9953- Namespace our parametersICE-9952- Styling of showcase examples problematic in Liferay 6.2PUSH-311- Push not working with portlets