OFBIZ-11317 is NOT a new feature. It is an improvement. If you want to discuss the implications (and impact), of 11306 I suggest to start a new thread in this ml (or voice your concerns in comment postings in the appropriate ticket).
As for There's currently only few questions left for me: we have one configuration which uses org.apache.ofbiz.webapp.ftl.OfbizUrlTransform for ofbizUrl and this would not support the controlPath configuration if I don't miss something. Wouldn't it be better to change this to UrlRegexpTransform and deprecate OfbizUrlTransform? UrlRegexpTransform is located inside the product component but the macro is used in several other components as well. I think it belongs to the framework/webapp instead of applications/product. What do you think? I also suggest to start either a new thread in this ml, or open a ticket in JIRA. Best regards, Pierre Smits *Proud* *contributor* (but without privileges)* of* Apache OFBiz <https://ofbiz.apache.org/>, since 2008 *Apache Trafodion <https://trafodion.apache.org>, Vice President* *Apache Directory <https://directory.apache.org>, PMC Member* Apache Incubator <https://incubator.apache.org>, committer Apache Steve <https://steve.apache.org>, committer On Thu, Feb 13, 2020 at 1:06 PM Michael Brohl <[email protected]> wrote: > Hi Jacques, > > inline... > > Am 13.02.20 um 11:43 schrieb Jacques Le Roux: > >> > >> Yes, I confused the date (Jan vs. Feb, time goes by too quick). > >> > >> I speak of the commits towards > >> https://issues.apache.org/jira/browse/OFBIZ-11317. The issue was > >> created and on the same day it was committed. It was not yesterday > >> but the timeline between submit and commit is the same. > > > > I don't want to argue too much about that, so I hope it will be the > > end of this exchange. You are right about the Jira and commit moment, > > they are same. > > > > But I see 2 points here: > > > > * It's something you can review in ½ a hour, if not even the "famous" > > 10 minutes. You can even use a regexp to help you... > > It's a new feature and IMO the community should have a chance to review > and decide if a feature should go into the codebase. > > A quick commit on the same day simply removes this chance and builds the > impression that the committer does not care about what others think. IMO > that's not good for community work. > > But I'm getting tired of trying to get this spirit transported somehow. > Especially when it seems that nobody else really cares about this > approach. So we can end this exchange, like you hoped. > > > > >> > >> I'll have some questions towards OFBIZ-11317 also but I need time to > >> dig deeper. > > > > Sure, shoot :) > > > There's currently only few questions left for me: we have one > configuration which uses org.apache.ofbiz.webapp.ftl.OfbizUrlTransform > for ofbizUrl and this would not support the controlPath configuration if > I don't miss something. Wouldn't it be better to change this to > UrlRegexpTransform and deprecate OfbizUrlTransform? > > UrlRegexpTransform is located inside the product component but the macro > is used in several other components as well. I think it belongs to the > framework/webapp instead of applications/product. What do you think? > > Thanks, > > Michael Brohl > > ecomify GmbH - www.ecomify.de > > > > >
