Ticket #7055 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Load Dialog: enter a new filename and hit enter too quick -> wrong loader

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: critical Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description (last modified by Nick Draper) (diff)

If you enter a new filename (which would cause a change of loader). If you hit enter or click ok before the new loader has been assigned then it will try to use the old loader.

Improvement ideas:

  1. Disable execution until the loader search has completed
  2. Cache the execute request until the loader has been selected (probably better)
  3. Possibly test with the previous loader (before the rest) quickcheck and filecheck first, as 90% of file will use the last loader.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:2 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to the Backlog after R2.6

comment:3 Changed 7 years ago by Nick Draper

  • Description modified (diff)
  • Milestone changed from Backlog to Release 3.0

comment:4 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

Do a manual file search when run is clicked on LoadDialog

If the text field got updated but didn't emit the filesFound signal then the dialog would load the old file. When run is clicked a search is forced but it only performs a new search if something has actually changed in the box. Refs #7055

Changeset: 0e59f569070c44711746b3f07601565076dcfabd

comment:5 Changed 7 years ago by Martyn Gigg

  • Status changed from inprogress to verify
  • Resolution set to fixed

Branch: bugfix/7055_loaddialog_invalid_input

Tester: This can be difficult to reproduce. Ticket #8062 has a set of instructions that should be checked although I could not reproduce the error with those. The best way I found was to browse an load a file first then run load again and edit the box using only the keyboard to point to a different file and immediately press return when but do NOT click outside of the field. The dialog should now load the new file.

comment:6 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:7 Changed 7 years ago by Michael Reuter

I was able to reproduce the issue in #8062. After applying the fix, I see that the File widget has a star next to it, but no dialog appears. Once the file is found, the load automatically starts. I also performed the procedure requested in this ticket. No star is seen and the load starts when the file is found.

comment:8 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Merge remote branch 'origin/bugfix/7055_loaddialog_invalid_input'

Full changeset: 1a3e35a91d2259e9bda8846b5d47d5a428e4ba89

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7901

Note: See TracTickets for help on using tickets.