Portlet Bridge
  1. Portlet Bridge
  2. PBR-267

Support Liferay 6.0.6 with Glassfish 3.1

    Details

    • Similar Issues:
      Show 10 results 

      Description

      JBoss portletbridge can work with Liferay portal with some modification.

      1. liferay_support_edited.patch
        170 kB
        Sverker Abrahamsson
      2. liferay_support.patch
        242 kB
        Balazs Zsoldos

        Activity

        Hide
        Balazs Zsoldos
        added a comment -

        With the patch most of the JSF and Richfaces components work in Liferay portal (did not find any that does not). With the patch https://issues.jboss.org/browse/RF-11055 FileUpload works as well (in the portlet app we must define FileUploadExternalContextFactory as the factory to make it first in the queue)

        LIMITATION: only ajax page navigation works not the full page refresh.

        Tested with the SUN JSF implementation. The jsf.js had to be changed in order to make it work in a portal environment and also with full page refresh inside a portlet.
        With the change script and link elements will get id attribute and the client side js will only update those that do not exist yet on the page.

        Some general source code formatting was made on the classes I worked (tabs to spaces on the beginning of lines) that is why the patch got a bit bigger.

        If you have any question why I did a change and where please do not hesitate to ask.

        Show
        Balazs Zsoldos
        added a comment - With the patch most of the JSF and Richfaces components work in Liferay portal (did not find any that does not). With the patch https://issues.jboss.org/browse/RF-11055 FileUpload works as well (in the portlet app we must define FileUploadExternalContextFactory as the factory to make it first in the queue) LIMITATION: only ajax page navigation works not the full page refresh. Tested with the SUN JSF implementation. The jsf.js had to be changed in order to make it work in a portal environment and also with full page refresh inside a portlet. With the change script and link elements will get id attribute and the client side js will only update those that do not exist yet on the page. Some general source code formatting was made on the classes I worked (tabs to spaces on the beginning of lines) that is why the patch got a bit bigger. If you have any question why I did a change and where please do not hesitate to ask.
        Hide
        joodie B
        added a comment -

        How can I use this patch to get my richfaces-portlets working in liferay? Is this patch in the jboss-portletbridge svn? Can you provide a jar-file with this fix?

        Show
        joodie B
        added a comment - How can I use this patch to get my richfaces-portlets working in liferay? Is this patch in the jboss-portletbridge svn? Can you provide a jar-file with this fix?
        Hide
        Balazs Zsoldos
        added a comment -

        Hi,

        I will provide a sample portlet war and upload it into the liferay community somewhen in the next week. That will contain every jar that someone needs. I guess the patch is not in the svn of portletbridge yet as I uploaded it yesterday .

        Regards,
        Balazs

        Show
        Balazs Zsoldos
        added a comment - Hi, I will provide a sample portlet war and upload it into the liferay community somewhen in the next week. That will contain every jar that someone needs. I guess the patch is not in the svn of portletbridge yet as I uploaded it yesterday . Regards, Balazs
        Hide
        Attila Bak
        added a comment -

        Hi Balazs,

        This would interest me too!

        Thanks
        attila

        Show
        Attila Bak
        added a comment - Hi Balazs, This would interest me too! Thanks attila
        Hide
        Balazs Zsoldos
        added a comment -

        The sample portlet is ready. You will be able to find more information at http://www.liferay.com/community/forums/-/message_boards/message/9386795

        Show
        Balazs Zsoldos
        added a comment - The sample portlet is ready. You will be able to find more information at http://www.liferay.com/community/forums/-/message_boards/message/9386795
        Hide
        Attila Bak
        added a comment -

        Tested on Tomcat works as well!
        Great jobs Balazs, hopefully this needed fix makes it into the bridge!

        Show
        Attila Bak
        added a comment - Tested on Tomcat works as well! Great jobs Balazs, hopefully this needed fix makes it into the bridge!
        Hide
        Sverker Abrahamsson
        added a comment -

        Hi
        I noticed that the supplied patch contains big sections where the only change is indention. Neither would it apply cleanly on the portletbridge-3.0.X branch.

        I've therefore manually applied the acctual code changes without changing formatting as much as possible (I see now in the patch that there was some small adjustments on the formatting) which I hope makes it easier to review it so that hopefully it can be accepted.

        No changes has been made to the actual code.

        Show
        Sverker Abrahamsson
        added a comment - Hi I noticed that the supplied patch contains big sections where the only change is indention. Neither would it apply cleanly on the portletbridge-3.0.X branch. I've therefore manually applied the acctual code changes without changing formatting as much as possible (I see now in the patch that there was some small adjustments on the formatting) which I hope makes it easier to review it so that hopefully it can be accepted. No changes has been made to the actual code.
        Hide
        Ken Finnigan
        added a comment -

        The proposed changes don't really match with the current code base. Please re-open if necessary

        Show
        Ken Finnigan
        added a comment - The proposed changes don't really match with the current code base. Please re-open if necessary

          People

          • Assignee:
            Unassigned
            Reporter:
            Balazs Zsoldos
          • Votes:
            5 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: