Closed asfimport closed 21 years ago
khammond25 (migrated from Bugzilla): Created attachment patch.LoopControlPanel.diff: Patch for LoopControlPanel.java; file contains the diff
khammond25 (migrated from Bugzilla): Created attachment patch2.LoopControlPanel.diff: NEW Patch for LoopControlPanel.java; file contains the diff
khammond25 (migrated from Bugzilla): Disregard the first patch and look at latest one (9/18/01 06:38). I changed the order that we report the error. Now the flow is: 1) display the error dialog, 2) clear the text field.
This way, when the user gets the dialog, they can see that "oh yeah, I did have a typo". Previously, I coded it so that the text field was cleared before displaying the window. This wasn't very user friendly.
I have also updated ThreadGroup and ThreadGroupGui to have the same behavior.
Look for a patch soon!
khammond25 (migrated from Bugzilla): Fixed 2001.09.27.
Jordi Salvat i Alabart (migrated from Bugzilla): Marking all bugs RESOLVED before JMeter 1.8's release date as VERIFIED. Yes, it's pretty poor QA procedure, but there's bugs here lingering since JMeter 1.6, and we need to clean up a little.
Jordi Salvat i Alabart (migrated from Bugzilla): Bulk-closing all bugs RESOLVED before JMeter 1.8 release date.
khammond25 (Bug 3682): In the latest version of org.apache.jmeter.gui.LoopControlPanel (, there are two problems:
When the "infinite" checkbox is checked, the text field is disabled, but the text label "Number of iterations" remains enabled. The defacto standard is to disable both the label and the text field.
When the "infinite" checkbox is unchecked, and you enter an non-numeric value, you get an Error Dialog. This is fine, but when you click OK to leave the dialog, the text field you were typing in is cleared out and the "infinite" checkbox is checked. Since we are validating this field after every keypress, one typing mistake and your penalized. You have to deselect the checkbox and type in your value again. It's an extra mouse click. It's fine to clear out the text field, but since the user was trying to enter a valid iteration count, let him try it again without having him deselect the checkbox again.
Severity: normal OS: All