Re: [cargotracker-dev] (Review/approve PR)

["reza_rahman@xxxxxxxxx" <reza_rahman@xxxxxxxxx>]

What specifically was the original issue with MyFaces and what is the current issue with Mojarra? Can a code change be made to the application to support both Mojarra and MyFaces?

 

---

From: cargotracker-dev <cargotracker-dev-bounces@xxxxxxxxxxx> on behalf of Gilbert Kwan <gkwan@xxxxxxxxxx>
Sent: Wednesday, September 7, 2022 3:39 PM
To: cargotracker developer discussions <cargotracker-dev@xxxxxxxxxxx>
Subject: Re: [cargotracker-dev] (Review/approve PR)

 

The current code from the master works with Payara but not Liberty. The current code from the liberty-experimental branch works with Liberty but not Payara. The change is to narrow down the gap between Payara and Liberty. Eventually, we want that the master branch can carry the Liberty profile, and both Payara and Liberty work well with the same code.

 

Thanks

Gilbert

 

 

From: cargotracker-dev <cargotracker-dev-bounces@xxxxxxxxxxx> on behalf of reza_rahman@xxxxxxxxx <reza_rahman@xxxxxxxxx>
Date: Wednesday, September 7, 2022 at 3:30 PM
To: cargotracker developer discussions <cargotracker-dev@xxxxxxxxxxx>
Subject: [EXTERNAL] Re: [cargotracker-dev] (Review/approve PR)

Does the current code work with Payara and Liberty? If so, why is the change needed at all? From: cargotracker-dev <cargotracker-dev-bounces@ eclipse. org> on behalf of Gilbert Kwan <gkwan@ ca. ibm. com> ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍

ZjQcmQRYFpfptBannerStart

This Message Is From an External Sender

This message came from outside your organization.

ZjQcmQRYFpfptBannerEnd

Does the current code work with Payara and Liberty? If so, why is the change needed at all?

 

---

From: cargotracker-dev <cargotracker-dev-bounces@xxxxxxxxxxx> on behalf of Gilbert Kwan <gkwan@xxxxxxxxxx>
Sent: Wednesday, September 7, 2022 3:26 PM
To: cargotracker developer discussions <cargotracker-dev@xxxxxxxxxxx>
Subject: Re: [cargotracker-dev] (Review/approve PR)

 

Hi Reza,

 

In my understanding from Chanun, there is a JSF implementation difference between Payara and Liberty. Payara is using Mojarra, and Liberty is using MyFaces. Chanun updated the following 3 files on the liberty-experimental branch that are favourite for MyFaces, so that Liberty does not show any warnings.

1. src/main/webapp/WEB-INF/templates/common/admin.xhtml
2. src/main/webapp/WEB-INF/templates/common/public.xhtml
3. src/main/webapp/WEB-INF/web.xml

I cannot judge that the difference is MyFaces bug or Mojarra bug. I don’t see Payara or Liberty will change their implementation. To avoid the difference, Liberty can use Mojarra with the jsfContainer feature. That why I submitted the PR #231 that restores the above 3 files to original (same as master branch) and make Liberty to use Mojarra.

 

Hope it can clarify what the changes are.

 

Thanks

Gilbert

 

From: cargotracker-dev <cargotracker-dev-bounces@xxxxxxxxxxx> on behalf of reza_rahman@xxxxxxxxx <reza_rahman@xxxxxxxxx>
Date: Wednesday, September 7, 2022 at 8:59 AM
To: cargotracker developer discussions <cargotracker-dev@xxxxxxxxxxx>
Subject: [EXTERNAL] Re: [cargotracker-dev] (Review/approve PR)

I looked at it, but honestly I am having trouble understanding the changes. One of the goals as an official Jakarta EE project is to highlight standards based implementation neutrality. Why is it necessary to switch to Mojarra? Are we running

ZjQcmQRYFpfptBannerStart

This Message Is From an External Sender

This message came from outside your organization.

ZjQcmQRYFpfptBannerEnd

I looked at it, but honestly I am having trouble understanding the changes. One of the goals as an official Jakarta EE project is to highlight standards based implementation neutrality. Why is it necessary to switch to Mojarra? Are we running into a MyFaces bug or do we have code that isn’t quite standard?

 

---

From: cargotracker-dev <cargotracker-dev-bounces@xxxxxxxxxxx> on behalf of Gilbert Kwan <gkwan@xxxxxxxxxx>
Sent: Wednesday, September 7, 2022 8:49 AM
To: cargotracker-dev@xxxxxxxxxxx <cargotracker-dev@xxxxxxxxxxx>
Subject: Re: [cargotracker-dev] (Review/approve PR)

 

Hi Reza,

 

Do you have a chance to review and merge my PR https://github.com/eclipse-ee4j/cargotracker/pull/231?

 

Thanks

Gilbert

 

From: Gilbert Kwan <gkwan@xxxxxxxxxx>
Date: Tuesday, August 30, 2022 at 2:12 PM
To: cargotracker-dev@xxxxxxxxxxx <cargotracker-dev@xxxxxxxxxxx>
Subject: FW: [EXTERNAL] [cargotracker-dev] (no subject)

Hi Reza,

 

FYI, I signed the ECA.

 

Thanks

Gilbert

 

From: cargotracker-dev <cargotracker-dev-bounces@xxxxxxxxxxx> on behalf of Gilbert Kwan <gkwan@xxxxxxxxxx>
Date: Friday, August 26, 2022 at 1:06 PM
To: cargotracker-dev@xxxxxxxxxxx <cargotracker-dev@xxxxxxxxxxx>
Subject: [EXTERNAL] [cargotracker-dev] (no subject)

Hi Reza, I continue Chanun work. I just submitted a new PR to liberty-experimental with some updates, hoping that it can be merged soon. Enable Open Liberty to use jsfContainer feature and Mojarra library ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍ ‍

ZjQcmQRYFpfptBannerStart

This Message Is From an External Sender

This message came from outside your organization.

ZjQcmQRYFpfptBannerEnd

Hi Reza,

 

I continue Chanun work. I just submitted a new PR to liberty-experimental with some updates, hoping that it can be merged soon.

1. Enable Open Liberty to use jsfContainer feature and Mojarra library
2. Restore 2 xhtml and web.xml files to original

 

Thanks

Gilbert Kwan