OFBiz
  1. OFBiz
  2. OFBIZ-1690

Set widget default url encode value to true

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: Release 4.0, Trunk
    • Fix Version/s: None
    • Component/s: framework
    • Labels:
      None

      Description

      The bug is explained here: http://www.nabble.com/Question-about-ofbiz-url-encoding-to15638660.html#a15638660
      Can someone with framework access commit it please.

      Thanks,
      Bilgin

      1. encode.patch
        5 kB
        Bilgin Ibryam
      2. encode.patch
        2 kB
        Bilgin Ibryam

        Activity

        Hide
        Bilgin Ibryam added a comment -

        Adrian you are right,
        but setting encode attribute to false for external links should solve this issue?

        Show
        Bilgin Ibryam added a comment - Adrian you are right, but setting encode attribute to false for external links should solve this issue?
        Hide
        Adrian Crum added a comment -

        Bilgin,

        My concern is with external links - will the jsessionid parameter get appended to them also.

        Show
        Adrian Crum added a comment - Bilgin, My concern is with external links - will the jsessionid parameter get appended to them also.
        Hide
        Bilgin Ibryam added a comment -

        Adrian,

        I updated the patch according to your remarks.
        Can you tell me what kind of existing code this patch could break?

        Thanks for your review and comments!

        Show
        Bilgin Ibryam added a comment - Adrian, I updated the patch according to your remarks. Can you tell me what kind of existing code this patch could break? Thanks for your review and comments!
        Hide
        Adrian Crum added a comment -

        Bilgin,

        I'd like to hear more comments on the subject. Your patch only changes the default in the widget's xsd, it doesn't address the scenario where the xsd is not available. To handle that case, the model widgets would have to default the setting to true also. That could break a lot of existing code.

        Show
        Adrian Crum added a comment - Bilgin, I'd like to hear more comments on the subject. Your patch only changes the default in the widget's xsd, it doesn't address the scenario where the xsd is not available. To handle that case, the model widgets would have to default the setting to true also. That could break a lot of existing code.

          People

          • Assignee:
            Unassigned
            Reporter:
            Bilgin Ibryam
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development