[ https://issues.apache.org/jira/browse/OFBIZ-5312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13944254#comment-13944254 ]
Jacques Le Roux commented on OFBIZ-5312: ---------------------------------------- Anil, all committers, I thank you for giving your opinion. You are against, but at least you made the effort to express yourself and set out your arguments. I expected more feedback from other committers. Even if, like Jacopo said (in a not so polite way regarding the work done) <<(the task history is a huge mess)>>, it's not easy now to follow how things progressed in this issue (was of course easier to follow from start...). For a such important issue, I'd have expected much earlier committers's comments. Notably from you who are against after 6 months of exchanges and work! Anyway adding rants to this issues will not help. So now, that I have said what I had on the heart, it's enough, let's see things in a positive way. To answer your concern about keeping OFBiz lean. I also understand this worry. We went too far adding stuff in OFBiz. And some are now hardly maintained. Without speaking about the issue of having new releases of ~150MB (w/o specialpurpose components but ecommerce) and especially a just checked out trunk (so with svn data) of ~500MB! So I agree with you about reusing as much as possible existing features. But going the ecomclone way, the new feature will be feature rich and as lean as possible. I don't expect much more than 200KB with around 20 files max. Also I did not all that work w/o the ambition of maintaining this code, nobody will have to worry about it, for hopefully a long period... So to clarify, I will create a Jira issue, and attach a patch in the following weeks. Then we can discuss about it again... > Proposal: URL-Generation Changes (mostly for SEO reasons but not only) > ---------------------------------------------------------------------- > > Key: OFBIZ-5312 > URL: https://issues.apache.org/jira/browse/OFBIZ-5312 > Project: OFBiz > Issue Type: New Feature > Components: specialpurpose/ecommerce > Affects Versions: SVN trunk > Reporter: Jacques Le Roux > Assignee: Jacques Le Roux > Priority: Minor > Labels: changes, ecommerce, friendly, seo, url > Fix For: SVN trunk > > Attachments: OFBIZ-5312 - ofbiz-ecommerce-seo.patch, OFBIZ-5312 - > ofbiz-ecommerce-seo.patch, OFBIZ-5312 - ofbiz-ecommerce-seo.patch, OFBIZ-5312 > - ofbiz-ecommerce-seo.patch, OFBIZ-5312 - ofbiz-ecommerce-seo.patch, > OFBIZ-5312 - ofbiz-ecommerce-seo.patch, OFBIZ-5312 - > ofbiz-ecommerce-seo.patch, OFBIZ-5312 - ofbiz-ecommerce-seo.patch, OFBIZ-5312 > - ofbiz-ecommerce-seo.patch, OFBIZ-5312 - ofbiz-ecommerce-seo.patch, > OFBIZ-5312 - ofbiz-ecommerce-seo.patch, OFBIZ-5312 - > ofbiz-ecommerce-seo.patch, OFBiz-5312-product-ecommerce-seo-20131031.patch, > OFBiz-5312-product-ecommerce-seo-for-seo-branch.patch, > OFBiz-5312-product-ecommerce-seo.patch, SeoContextFilter.java.patch > > > [This was proposed by Paul Piper in Nabble 7 months > ago|http://ofbiz.135035.n4.nabble.com/Proposal-URL-Generation-Changes-td4639289.html]. > Here is quoted Paul's proposal > {quote} > Hey Everyone, > over at ilscipio (www.ilscipio.com) we developed a set of functional OFBiz > changes that we believe the entire community could benefit from. The changes > have been implemented in parts in Syracus (www.syracus.net) for a while now, > but we figured that some of which are too crucial for ofbiz' success in the > long run, so we are considering the contribution (as we did with the SOLR > component). > As you are probably aware, OFBiz has a pretty uncommon way of generating > URLs. Most of this has to do with the fact that OFBiz uses a servlet > (ControlServlet) to handle all requests. The servlet is mounted at /control, > so that it won't interfere with other servlets. Though functionally valid, > this has the sideeffect that all urls are actually created on /control, which > is neither pretty, nor good by any measures of SEO. It also means that a few > 302 redirects are necessary to forward the user from / to /control/main. It > also makes requests more complicated, since many forwards are necessary > whenever somebody wants to move away from this implementation. > Since this is hurtful to many of the implementers, I wanted to discuss > whether or not you guys would be interested in the changes we have made. The > functional changes contain: > * Removal of /control out of all the urls > * SEO-friendly URLS > * Configurable product/category and other URLs > * Frontpage mapping from /main to / > It was tested on our end and contains all necessary improvements (Transforms, > Sample Configuration, Servlets & Filters) for it to be applicable. > If interested, I would create a new JIRA ticket for this and after a few > minor internal discussions, we will gladly provide the rest of you with it. > Regards, > Paul > {quote} > There is even a patch, mostly done by Jinghai Shi, that I attach here. Even > if it has been already used in [Syracus|http://syracus.net/] since early this > year, some help would be needed to test it thoroughly in OFBiz. > Then we should discuss if it's the way to go. I believe it is. Who needs a > /control/ or /main by default in ecommerce urls? Would you not prefer > http://localhost:8080/ecommerce/ over > http://localhost:8080/ecommerce/control/main ? -- This message was sent by Atlassian JIRA (v6.2#6252)