Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: Upcoming Release
    • Component/s: order
    • Labels:
      None

      Description

      use https://localhost:8443/ordermgr/control/orderview?orderId=DEMO10091
      notice widget exceptions

      console output...
      2017-07-09 16:22:35,812 |jsse-nio-8443-exec-1 |ScreenFactory |I| Got 40 screens in 0.021s from: file:/ofbiz-trunk/applications/party/widget/partymgr/PartyScreens.xml
      2017-07-09 16:22:35,871 |jsse-nio-8443-exec-1 |FreeMarkerWorker |E| null
      freemarker.core.NonBooleanException: For "!" right-hand operand: Expected a boolean, but this has evaluated to a string (wrapper: f.t.SimpleScalar):
      ==> "ORDER_COMPLETED" [in template "component://order/template/order/OrderContactInfo.ftl" at line 21, column 10]


      FTL stack trace ("~" means nesting-related):

      • Failed at: #if (!"ORDER_COMPLETED" == orderHeade... [in template "component://order/template/order/OrderContactInfo.ftl" in macro "updateOrderContactMech" at line 21, column 3]
      • Reached through: @updateOrderContactMech orderHeader=(... [in template "component://order/template/order/OrderContactInfo.ftl" at line 123, column 19]

        at freemarker.core.Expression.modelToBoolean(Expression.java:163) ~[freemarker-2.3.25-incubating.jar:2.3.25]

      1. OFBIZ-9481.patch
        8 kB
        Devanshu Vyas
      2. OFBIZ-9481.patch
        2 kB
        Devanshu Vyas

        Issue Links

          Activity

          Hide
          devanshu.vyas Devanshu Vyas added a comment -

          Uploaded the patch for the bug fix.

          Show
          devanshu.vyas Devanshu Vyas added a comment - Uploaded the patch for the bug fix.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Hi Devanshu,

          I think we can use !=

          Show
          deepak.dixit Deepak Dixit added a comment - Hi Devanshu, I think we can use !=
          Hide
          devanshu.vyas Devanshu Vyas added a comment -

          Thanks Deeapk, for the suggestion.
          I found such occurrences quite a few in the application. Will fix all those and upload a patch soon.

          Show
          devanshu.vyas Devanshu Vyas added a comment - Thanks Deeapk, for the suggestion. I found such occurrences quite a few in the application. Will fix all those and upload a patch soon.
          Hide
          devanshu.vyas Devanshu Vyas added a comment -

          As discussed, here is the updated patch with the fix for all the instances of the pattern discussed.

          Show
          devanshu.vyas Devanshu Vyas added a comment - As discussed, here is the updated patch with the fix for all the instances of the pattern discussed.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          This has been fixed at ofbiz framework trunk at r#1801457
          Thanks Wai and Devanshu for your contribution.

          This is only part of develop no need to backport on older branches.

          Show
          deepak.dixit Deepak Dixit added a comment - This has been fixed at ofbiz framework trunk at r#1801457 Thanks Wai and Devanshu for your contribution. This is only part of develop no need to backport on older branches.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Thank you for your last comment Deepak

          Show
          jacques.le.roux Jacques Le Roux added a comment - Thank you for your last comment Deepak
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Thanks Jacques

          Show
          deepak.dixit Deepak Dixit added a comment - Thanks Jacques

            People

            • Assignee:
              deepak.dixit Deepak Dixit
              Reporter:
              wt Wai
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development