Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 354117 - ColumnFilterForm ignores AbstractTableField.isSortEnabled()
Summary: ColumnFilterForm ignores AbstractTableField.isSortEnabled()
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Scout (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 trivial (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-08 05:46 EDT by Beat Schwarzentrub CLA
Modified: 2021-08-19 11:06 EDT (History)
1 user (show)

See Also:
claudio.guglielmo: juno+

Attachments
ColumnFilterForm.java.patch (2.29 KB, patch)
2011-08-08 05:47 EDT, Beat Schwarzentrub CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Beat Schwarzentrub CLA 2011-08-08 05:46:55 EDT 
Build Identifier: 3.7.0.2011-Jun 2011

PROBLEM DESCRIPTION:
In some cases you don't want the user to allow custom sorting in tables. This can be achieved by setting getConfiguredSortEnabled() = false.

However, the "ColumnFilterForm" keeps showing links to sort the table by the current column. (The data is not actually sorted, but a small "sorting arrow" is shown in the column header.)

EVALUATION:
The ColumnFilterForm should disable the links for sorting, when the table has isSortEnabled() = false.

PROPOSED SOLUTION:
See attached patch.

Reproducible: Always
Comment 1 Beat Schwarzentrub CLA 2011-08-08 05:47:16 EDT 
Created attachment 201061 [details]
ColumnFilterForm.java.patch
Comment 2 Claudio Guglielmo CLA 2012-05-01 07:26:41 EDT 
Hi Beat

Thanks for the patch. Unfortunately the code has changed meanwhile because of bug 363828 so I could not apply it. Nevertheless I fixed the bug as suggested.
Comment 3 Matthias Zimmermann CLA 2012-07-09 06:50:36 EDT 
ticket closed.
deliverd as part of eclipse scout 3.8.0 (juno release train)