Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Release Branch 12.04, Release Branch 13.07, Release Branch 14.12, Trunk
    • Fix Version/s: 14.12.01, 12.04.06, 13.07.03, 16.11.01
    • Component/s: order
    • Labels:
      None
    • Flags:
      Patch

      Description

      There are a couple of errors in this file. They are not related each to other.

      1. 12 & 13 use EntityQuery class that is not available in these releases. This cause error in Quick Add screen creating a purchase order.

      2. In all releases the code in that file try to collect prices for variants. Doing so it use NumberFormat.format() to transform base price into string that is part of an embed JS snippet. In case purchase order this price is rather empty. Using null with formatting method cause error.

      You may see this error in the trunk if you starts PO and and try Browser Categories->Widgets->Small Widgets. This price should appear if you select Select Units of Measure for WG-1111 but currently this does not work.

      I'm not sure these prices make sense for purchase orders but my goal was just fix ugly error message. Solution display N/A if price is not available.

      1. prod_summary_trunk_14.diff
        1 kB
        Oleg Andreyev
      2. prod_summary_13.diff
        2 kB
        Oleg Andreyev
      3. prod_summary_12.diff
        2 kB
        Oleg Andreyev

        Activity

        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Oleg,

        For R12 and R13 it was my bad. I just reverted r1639921 and r1639922

        I will review the older error later

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Oleg, For R12 and R13 it was my bad. I just reverted r1639921 and r1639922 I will review the older error later
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Oleg,

        Your patch is in
        trunk r1705004
        R14.12 r1705006
        R13.07 r1705007
        R12.04 r1705008

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Oleg, Your patch is in trunk r1705004 R14.12 r1705006 R13.07 r1705007 R12.04 r1705008

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            redcat Oleg Andreyev
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development