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

Row sorter to Summary Report #3631

Closed
asfimport opened this issue Jul 21, 2015 · 5 comments
Closed

Row sorter to Summary Report #3631

asfimport opened this issue Jul 21, 2015 · 5 comments

Comments

@asfimport
Copy link
Collaborator

mchassagneux (Bug 58161):
I made this modification to add row sorter by column in the Summary Report component.

Could you look at it ?

Created attachment RowSorter.patch: Row sorter to Summary Report

RowSorter.patch
238,241d237
< 
<         // Add Row Sorter to JTable
<         myJTable.setAutoCreateRowSorter(true);
< 

OS: All

Duplicated by:

@asfimport
Copy link
Collaborator Author

mchassagneux (migrated from Bugzilla):
The patch was made on "src\components\org\apache\jmeter\visualizers\SummaryReport.java" file

@asfimport
Copy link
Collaborator Author

UbikLoadPack support (migrated from Bugzilla):
Thanks, sounds interesting but same note as for #3631 regarding the patch.

@asfimport
Copy link
Collaborator Author

mchassagneux (migrated from Bugzilla):
New version of the patch

Created attachment RowSorter.patch: Row sorter to Summary Report

RowSorter.patch
diff  C:/perso_max/workspace/JMeter_2.12/src/components/org/apache/jmeter/visualizers/SummaryReport.java C:/perso_max/workspace/JMeter/src/components/org/apache/jmeter/visualizers/SummaryReport.java
238,241d237
< 
<         // Add Row Sorter to JTable
<         myJTable.setAutoCreateRowSorter(true);
< 

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Hi,
Thanks for patch but it's not working for me.
It throws either ClassCastException (String to double conversion) or ArraysOutOfBoundsException.

Regards

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
This bug has been marked as a duplicate of #2792

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