Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 355627 - Misleading handling of empty column values in ColumnFilter
Summary: Misleading handling of empty column values in ColumnFilter
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Scout (show other bugs)
Version: 3.7.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7.1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-24 05:38 EDT by Bruno Koeferli CLA
Modified: 2012-05-23 10:58 EDT (History)
2 users (show)

See Also:
zimmermann: indigo+
zimmermann: juno+

Attachments
Contribution to solve same issue in smartcolumn filter (5.45 KB, patch)
2011-08-29 02:54 EDT, Ivan Motsch CLA 		zimmermann: iplog+
Details | Diff
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 Bruno Koeferli CLA 2011-08-24 05:38:00 EDT 
Build Identifier: 3.7.0

In ColumnFilter exists a default entry "Empty" which stands for empty values. But a visual empty column value (f.e. an empty string "") is not considered by this filter entry and leads to an extra entry in the ColumnFilter list.

Reproducible: Always

Steps to Reproduce:
1. Build a table with at least one visible column
2. Ensure the table is filled with data (null values and visual empty values)
3. Open/activate ColumnFilter dialog
Comment 1 Ivan Motsch CLA 2011-08-24 08:41:42 EDT 
Fixed by handling empty strings and s.trim().length()==0 the same way as null.
Comment 2 Ivan Motsch CLA 2011-08-29 02:54:32 EDT 
Created attachment 202297 [details]
Contribution to solve same issue in smartcolumn filter
Comment 3 Ivan Motsch CLA 2011-08-29 02:55:11 EDT 
Contribution from Claudio Guglielmo to solve same issue in smartcolumn filter
Comment 4 Matthias Zimmermann CLA 2011-10-11 02:39:04 EDT 
shipped with 3.7.1
Comment 5 Matthias Zimmermann CLA 2012-05-23 10:58:29 EDT 
moved iplog flag to patch
Comment 6 Matthias Zimmermann CLA 2012-05-23 10:58:52 EDT 
Comment on attachment 202297 [details]
Contribution to solve same issue in smartcolumn filter

iplog flag for patch from claudio