Uploaded image for project: 'mod-orders'
  1. mod-orders
  2. MODORDERS-262

PO with a POL whose productID has a paren, copied/edited from Instance cannot be saved

    XMLWordPrintable

Details

    • ACQ Sprint 74
    • 1
    • Thunderjet

    Description

      Overview: If you create a POL with data from an Inventory Instance but then change that data such that the UUID is wiped out, but the data has a Product ID containing a parenthesis, then you cannot save/open the PO

      Steps to Reproduce:

      1. Log into FOLIO-snapshot as diku_admin
      2. Go to the order app
      3. Create a PO
      4. Create a POL and populate the bib data from an Inventory Instance ("accidental atheist" title)
      5. Remove a contributor, or edit the title slightly, which should make the Instance UUID disappear
      6. Save the POL
      7. Open the PO

      Expected Results: The PO should Open

      Actual Results: There's a message "PO cannot be saved".

      Additional Information: See attached video. From piotr_kalashuk in the comments: Looks like parentheses in the productId should be somehow be escaped when building CQL query to inventory storage.

      TestRail: Results

        Attachments

          1. accidental_atheist_instance.png
            60 kB
            Piotr Kalashuk
          2. InventoryIdentifierWithParentheses.png
            135 kB
            Piotr Kalashuk
          3. Inventory with paren.PNG
            177 kB
            Ann-Marie Breaux
          4. OpenOrderWithParenthesis.png
            236 kB
            Varun Javalkar
          5. PO cannot be saved.mp4
            4.39 MB
            Ann-Marie Breaux
          6. POL with paren.PNG
            170 kB
            Ann-Marie Breaux
          7. ProductIdWithParenthesisSearch.png
            120 kB
            Varun Javalkar
          8. Try to open this one.PNG
            147 kB
            Ann-Marie Breaux

          Issue Links

            Activity

              People

                varunjavalkar Varun Javalkar
                abreaux Ann-Marie Breaux
                Craig McNally Craig McNally
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases