Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-5312

Proposal: URL-Generation Changes (mostly for SEO reasons but not only)

    Details

      Description

      This was proposed by Paul Piper in Nabble 7 months ago. Here is quoted Paul's proposal

      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

      There is even a patch, mostly done by Jinghai Shi, that I attach here. Even if it has been already used in Syracus 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 ?

        Attachments

        1. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          109 kB
          Jacques Le Roux
        2. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          109 kB
          Jacques Le Roux
        3. SeoContextFilter.java.patch
          7 kB
          Jacques Le Roux
        4. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          640 kB
          Jacques Le Roux
        5. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          640 kB
          Jacques Le Roux
        6. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          654 kB
          Jacques Le Roux
        7. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          654 kB
          Jacques Le Roux
        8. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          229 kB
          Jacques Le Roux
        9. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          224 kB
          Parimal Gain
        10. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          229 kB
          Jacques Le Roux
        11. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          258 kB
          Jacques Le Roux
        12. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          258 kB
          Jacques Le Roux
        13. OFBiz-5312-product-ecommerce-seo.patch
          164 kB
          Shi Jinghai
        14. OFBiz-5312-product-ecommerce-seo-for-seo-branch.patch
          2 kB
          Shi Jinghai
        15. OFBIZ-5312 - ofbiz-ecommerce-seo.patch
          317 kB
          Jacques Le Roux
        16. OFBiz-5312-product-ecommerce-seo-20131031.patch
          79 kB
          Shi Jinghai
        17. OFBiz-5312-allowpaths-email-jsp-patch-20140625.patch
          21 kB
          Shi Jinghai
        18. classloaderpatch.txt
          2 kB
          Ingo Wolfmayr
        19. OFBIZ-5312-infinite-loop-20141209.patch
          8 kB
          Shi Jinghai

          Issue Links

            Activity

              People

              • Assignee:
                jacques.le.roux Jacques Le Roux
                Reporter:
                jacques.le.roux Jacques Le Roux
              • Votes:
                6 Vote for this issue
                Watchers:
                18 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: