Uploaded image for project: 'Portlet Bridge'
  1. Portlet Bridge
  2. PBR-267

Support Liferay 6.0.6 with Glassfish 3.1

    Details

      Description

      JBoss portletbridge can work with Liferay portal with some modification.

        Gliffy Diagrams

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

          Activity

          Hide
          zs_b 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
          zs_b 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 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 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
          zs_b 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
          zs_b 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
          baki Attila Bak added a comment -

          Hi Balazs,

          This would interest me too!

          Thanks
          attila

          Show
          baki Attila Bak added a comment - Hi Balazs, This would interest me too! Thanks attila
          Hide
          zs_b 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
          zs_b 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
          baki Attila Bak added a comment -

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

          Show
          baki 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 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 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
          kenfinni Ken Finnigan added a comment -

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

          Show
          kenfinni 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:
              zs_b Balazs Zsoldos
            • Votes:
              5 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development