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

File XML Properties file process error

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 18.12.05, Upcoming Branch, 22.01.01
    • 22.01.01
    • framework/common
    • None

    Description

      The following error message is displayed when starting OfBiz:

      2023-01-05 10:15:46,913 |sse-nio-8443-exec-19 |UtilXml                       |E| XmlFileLoader: File XML Properties file process error. Line: 39. Error message: cvc-complex-type.2.2: Element 'value' must have no element [children], and the value must be valid.
      2023-01-05 10:15:46,913 |sse-nio-8443-exec-19 |UtilXml                       |E| XmlFileLoader: File XML Properties file process error. Line: 40. Error message: cvc-complex-type.2.2: Element 'value' must have no element [children], and the value must be valid. 

      This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common.

      Attachments

        Activity

          cshan Chenghu Shan added a comment -

          I intend to provide a solution for this issue.

          cshan Chenghu Shan added a comment - I intend to provide a solution for this issue.

          Commit 2a535b5ba50bfd205bd158b3972acdbe7288dab9 in ofbiz-framework's branch refs/heads/release18.12 from cshan-ecomify
          [ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=2a535b5ba5 ]

          Fixed: File XML Properties file process error (OFBIZ-12738) (#580)

          This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common.
          Fixed escape syntax for <>.

          jira-bot ASF subversion and git services added a comment - Commit 2a535b5ba50bfd205bd158b3972acdbe7288dab9 in ofbiz-framework's branch refs/heads/release18.12 from cshan-ecomify [ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=2a535b5ba5 ] Fixed: File XML Properties file process error ( OFBIZ-12738 ) (#580) This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common. Fixed escape syntax for <>.

          Commit 9be3b2f3d8134e5f2c0c06c9a23afde654e96989 in ofbiz-framework's branch refs/heads/release22.01 from Jacques Le Roux
          [ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=9be3b2f3d8 ]

          Fixed: File XML Properties file process error (OFBIZ-12738) (#580)

          This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common.
          Fixed escape syntax for <>.

          Thanks: Chenghu Shan

          jira-bot ASF subversion and git services added a comment - Commit 9be3b2f3d8134e5f2c0c06c9a23afde654e96989 in ofbiz-framework's branch refs/heads/release22.01 from Jacques Le Roux [ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=9be3b2f3d8 ] Fixed: File XML Properties file process error ( OFBIZ-12738 ) (#580) This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common. Fixed escape syntax for <>. Thanks: Chenghu Shan

          Commit 870adbe31f410251cfc02ef0937e05a99e28e354 in ofbiz-framework's branch refs/heads/trunk from Jacques Le Roux
          [ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=870adbe31f ]

          Fixed: File XML Properties file process error (OFBIZ-12738) (#580)

          This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common.
          Fixed escape syntax for <>.

          Thanks: Chenghu Shan

          jira-bot ASF subversion and git services added a comment - Commit 870adbe31f410251cfc02ef0937e05a99e28e354 in ofbiz-framework's branch refs/heads/trunk from Jacques Le Roux [ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=870adbe31f ] Fixed: File XML Properties file process error ( OFBIZ-12738 ) (#580) This is caused by unescaped <> brackets in QRCodeUiLabels.xml in framework/common. Fixed escape syntax for <>. Thanks: Chenghu Shan

          Thanks Chenghu Shan,

          Actually when we fix a bug we do it in the trunk and then backport in supported branche/s. You provided a PR based on the 18.12 branch. But this branch is no longer supported.
          So rather than fixing in the branch you spot the bug it's better, easier for committers, for you to fix the bug in the trunk. We then backport the fix where it's needed.

          Here are more information: https://cwiki.apache.org/confluence/display/OFBIZ/OFBiz+Contributors+Best+Practices

          jleroux Jacques Le Roux added a comment - Thanks Chenghu Shan, Actually when we fix a bug we do it in the trunk and then backport in supported branche/s. You provided a PR based on the 18.12 branch. But this branch is no longer supported. So rather than fixing in the branch you spot the bug it's better, easier for committers, for you to fix the bug in the trunk. We then backport the fix where it's needed. Here are more information: https://cwiki.apache.org/confluence/display/OFBIZ/OFBiz+Contributors+Best+Practices

          Ha, I see that you also provided a PR for the trunk, I actually already fixed and backported

          jleroux Jacques Le Roux added a comment - Ha, I see that you also provided a PR for the trunk, I actually already fixed and backported
          cshan Chenghu Shan added a comment -

          Hi,

          I was told to provide three versions as this bugfix was rather small.
          One for 18.12, one for 22.01 and one for trunk.

          Should I continue and create one for 22.01?

          cshan Chenghu Shan added a comment - Hi, I was told to provide three versions as this bugfix was rather small. One for 18.12, one for 22.01 and one for trunk. Should I continue and create one for 22.01?

          Ah OK, actually then I jumped too fast . For 22.01 it's already fixed as you can see above.

          jleroux Jacques Le Roux added a comment - Ah OK, actually then I jumped too fast . For 22.01 it's already fixed as you can see above.
          cshan Chenghu Shan added a comment -

          Ok, thank you.

          cshan Chenghu Shan added a comment - Ok, thank you.

          People

            cshan Chenghu Shan
            cshan Chenghu Shan
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: