Jacques Le Roux

Hi Ieugen, Not yet, I'm currently focusing on the new codeQL implementation. I tried it few years ago for Java but it was not able to handle OFBiz, only JavaScript....

BTW, this is about whole OOTB OFBiz code. There is no other than this one in your PR.

For now I had only a cursory look at your work. But, as I said, I intend to have a deeper look in 2024.

Indeed @danwatford , I agree and it's easy to change for this PR and followings

Hi @dtrunk90, I remember having seen that before, but completly forgot about it. Could you please update the PR, or create anew. At 1st glance, the conflicts seem a bit...

Hi @dtrunk90, @mbrohl , I must say last time I just had a glance, notably on the Gradle Page. After reading a bit more there, with a glance as the...

As a note, here is the Gradle page summary for the catalog feature: > In summary, using a catalog is always a good engineering practice as it centralizes common definitions,...

Hi @mleila37 , Same seems like a good PR to push :)

Hi @PierreSmits , After applying and a restart, I still get : ERROR MESSAGE java.lang.RuntimeException: Error rendering included menu named [MainActionMenu] at location [component://commonext/widget/ofbizsetup/Menus.xml]: java.lang.IllegalArgumentException: Could not find menu with...

Hi Pierre, The patch does no apply in Elipse nor with git apply: > C:\projectsASF\Git\ofbiz-framework>git apply 785.patch > 785.patch:70: trailing whitespace. > 785.patch:71: trailing whitespace. > description="The main portal page...