Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.0
    • Fix Version/s: 5.0.1
    • Component/s: MyResearch
    • Labels:
      None

      Description

      I just noticed some undesirable behaviors in the list editing functionality.

      Problem #1.

      1.) Add an item to a list.
      2.) Go to the list.
      3.) Click the edit button next to the item you added.
      4.) Click the delete button at the bottom of the edit screen.
      5.) Click "no" in the pop-up. You now have a garbled display.

      This is a pretty obvious problem that hopefully has a straightforward solution.

      Problem #2:

      1.) From the same edit screen you were viewing in step 4 above, click the "delete" button again, but this time, select "yes."
      2.) Acknowledge the pop-up by clicking "close."
      3.) You are now on a screen that says "The record you selected is not part of any of your lists." and has a save button at the bottom.
      4.) Click "save."
      5.) You now get a "list successfully updated" acknowledgement, but nothing has been added to your list.

      This is more subtle -- this is happening because the "add to another list" drop-down is not being set to a specific list. I'm not sure that this is exactly a bug, but it's certainly counterintuitive behavior that could confuse users. At very least, perhaps if there is no data to edit on the screen, and the user hasn't selected a valid list, hitting "save" should result in an error message instead of a success message.

      When this is all fixed, we should probably add a Mink test suite to cover this functionality, as I don't believe it is currently being tested (or if it is being tested, it's not being THOROUGHLY tested).

        Activity

        Hide
        Chris Hallberg added a comment -
        I solved the first bug but didn't get the same result for the second: http://prntscr.com/k9058a

        Once that's sorted, tests do sound like a good plan.
        Show
        Chris Hallberg added a comment - I solved the first bug but didn't get the same result for the second: http://prntscr.com/k9058a Once that's sorted, tests do sound like a good plan.
        Hide
        Demian Katz added a comment -
        These bugs have been fixed, so I am closing the ticket. Adding test suite coverage is still worth doing, though!
        Show
        Demian Katz added a comment - These bugs have been fixed, so I am closing the ticket. Adding test suite coverage is still worth doing, though!

          People

          • Assignee:
            Chris Hallberg
            Reporter:
            Demian Katz
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: