[#XRP-103] 'Configure columns' API should easily allow for columns to be skipped.

Issue Details [XML]

Key:	XRP-103
Type:	New Feature
Status:	Resolved
Resolution:	Applied
Priority:	Minor
Assignee:	Marc Portier
Reporter:	Marc Portier
Votes:	0
Watchers:	0

Operations

If you were logged in you would be able to see more operations.

xReporter

'Configure columns' API should easily allow for columns to be skipped.

Created: 14/Feb/06 05:47 AM Updated: 15/Feb/06 09:56 AM

Component/s:

None

Affects Version/s:

None

Fix Version/s:

1.3

Description

See discussion on the list: http://lists.cocoondev.org/pipermail/xreporter/2006-February/002567.html

Here is a description of the proposed changes to accomodate for a more flexible and clear end-user control over the configuration of columns (i.e. selection of order and presence of columns in the result)

[default form for end user]
- replace the 'blank' choice from the drop-down list of column names with a more self-describing label describing what the option does (no more columns)
- add an additional choice to the list that caters for the extra option: 'just skip this column' . Since the rest of the values in the list are column names this option should be encoded with a value that cannot double as a column-name (suggested value: ? (U+2209) which is the mathematical symbol for 'not an element off')

[xreporter default style]
- change the columns.xsl so it generates the form in the above described way

[xreporter backend]
- check for this new column-value during definition-building --> don't allow for this column name
- check for this option choosen to skip to the next column in the /columns

All

Comments

Change History

Sort Order:

Comment by Marc Portier [15/Feb/06 06:43 AM]

[ Permlink ]

Committed the described changes to svn revision 652: http://svn.cocoondev.org/viewsvn?rev=652&root=xreporter&view=rev

Comment by Marc Portier [15/Feb/06 09:56 AM]

[ Permlink ]

Please note the impact on required translations in possible localized versions of xreporter.
This fix introduced some new keys to be translated.

The detail of what got added is in the diff:
http://svn.cocoondev.org/viewsvn/trunk/xreporter-cocoon/src/resources/common/messages_en.xml?rev=652&root=xreporter&view=diff&r1=652&r2=651&p1=trunk/xreporter-cocoon/src/resources/common/messages_en.xml&p2=/trunk/xreporter-cocoon/src/resources/common/messages_en.xml