Uploaded image for project: 'ERM Platform'
  1. ERM Platform
  2. ERM-1691

Sort on Agreement Status should sort by label

    XMLWordPrintable

    Details

    • Template:
      Standard Bug Write-Up Format
    • Sprint:
      ERM Sprint 115, ERM Sprint 116, ERM Sprint 117
    • Development Team:
      ERM

      Description

      Overview:
      Steps to Reproduce:

      1. Create at least two agreements with different statuses
      2. Do a search which displays agreements with a variety of statuses
      3. Click "Status" column header in the results MCL
      4. Note that the status does not sort alphabetically by displayed value in the results MCL

      Expected Results:

      Should sort alphabetically ASC by status label
      Actual Results:

      Sorts ASC by status id (I believe)

      Additional Information:

      Currently the sort applied is `sort=agreementStatus%3Basc`

      I think the correct sort is `sort=agreementStatus.label%3Basc`

      URL:
      Interested parties:

       

      Dev task breakdown

      It appears that there is no neat way to map between column name and sort term in SASQ, rather that it directly grabs from the column name. We can intefere with the `onSort` function and insert `.label` where applicable, or take the slightly  easier route of renaming the column.

       

      Changing lines:

       -- 

      From "agreementStatus" to "agreementStatus.label" should do the trick.

      NOTE: We don't want to change the internationalised column header, or any of the formatters, just the keys for AgreementStatus

       

      See comments section below for new recommendation:

      https://issues.folio.org/browse/ERM-1691?focusedCommentId=105812&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-105812

        TestRail: Results

          Attachments

            Issue Links

              Activity

                People

                Assignee:
                ostephens Owen Stephens
                Reporter:
                ostephens Owen Stephens
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    TestRail: Runs

                      TestRail: Cases