radiology-research / ispy2_mri

GUI to enter info on incoming MRI's for ISPY2 into database
GNU General Public License v3.0
0 stars 0 forks source link

Weird behavior on clear for `BSiteComboBox` #16

Open RossBoylan opened 9 months ago

RossBoylan commented 9 months ago

Steps

  1. Read in data from text file.
  2. Manually select site and visit since neither matched. This involved scrolling down site list.
  3. Save. Got popup confirming.
  4. After form was all clear except for Site, which showed Georgetown, the first.

This is under release 0.1.0.

The Problem

The site should be blank.

Reviewing the code shows that clear() sets current index to -1, which should be blank.

Theories

  1. Setting the current index to -1 does not have the effect it is documented to have.
  2. clear() is not getting called for this widget. However, if that were the case it would show the old value.
  3. The problem is peculiar to the site widget.
  4. The problem applies to all comboboxes.
  5. I hit a key without realizing it and made an unwitting selection.