Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ArgumentsPanel; bad row delete when editing a cell #767

Closed
asfimport opened this issue Sep 20, 2001 · 3 comments
Closed

ArgumentsPanel; bad row delete when editing a cell #767

asfimport opened this issue Sep 20, 2001 · 3 comments

Comments

@asfimport
Copy link
Collaborator

khammond25 (Bug 3747):
In ArgumentsPanel, if you are in edit mode for a cell and click the delete
button, the value you are editing will not be deleted but the other column's
value will.

Here's an example using WebTesting's Parameter Table:

  1. Create two rows: 1st row contains "param1" and "value1".
    2nd row contains "param2" and "value2".
  2. Get into edit mode for the NAME column in row Expand / Collapse all buttons #1 by double-clicking on that
    cell. You should see a vertical bar in the text field.
  3. Click the DELETE button.

The expected result is for row #1 to be deleted, and row #2 moved up and into
row #1's place. So, the table should now only have one row, with
values "param2" and "value2". Instead, our single row table contains the
values "param1" and "value2". The contents of the cell that we were editing
remains intact.

Severity: normal
OS: All

@asfimport
Copy link
Collaborator Author

khammond25 (migrated from Bugzilla):
I submitted a patch that includes a fix for this bug and #753. See bug
#3628 for the patch.

@asfimport
Copy link
Collaborator Author

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.

@asfimport
Copy link
Collaborator Author

Jordi Salvat i Alabart (migrated from Bugzilla):
Bulk-closing all bugs RESOLVED before JMeter 1.8 release date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant