You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Dan Haywood (JIRA)" <ji...@apache.org> on 2016/11/02 00:15:58 UTC
[jira] [Comment Edited] (ISIS-1388) Order of fields (as shown by
column ordering) should be more predictable.
[ https://issues.apache.org/jira/browse/ISIS-1388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15627150#comment-15627150 ]
Dan Haywood edited comment on ISIS-1388 at 11/2/16 12:15 AM:
-------------------------------------------------------------
overlapping to some extent by ISIS-1529
http://isis.apache.org/guides/rgsvc.html#_rgsvc_spi_TableColumnOrderService
perhaps also add a new columnOrder=... for properties?
was (Author: danhaywood):
superceded by ISIS-1529
http://isis.apache.org/guides/rgsvc.html#_rgsvc_spi_TableColumnOrderService
> Order of fields (as shown by column ordering) should be more predictable.
> ---------------------------------------------------------------------------
>
> Key: ISIS-1388
> URL: https://issues.apache.org/jira/browse/ISIS-1388
> Project: Isis
> Issue Type: Bug
> Components: Core
> Affects Versions: 1.12.1
> Reporter: Dan Haywood
> Assignee: Dan Haywood
> Fix For: 1.14.0
>
>
> At the moment it can be hard to predict, because the BS3Grid is traversed by type, eg:
> {code}
> private void traverseCols(final Grid.Visitor visitor, final BS3Row bs3Row) {
> final BS3Grid.Visitor bs3Visitor = asBs3Visitor(visitor);
> final List<BS3RowContent> cols = bs3Row.getCols();
> for (BS3RowContent rowContent : Lists.newArrayList(cols)) {
> rowContent.setOwner(bs3Row);
> if(rowContent instanceof BS3Col) {
> final BS3Col bs3Col = (BS3Col) rowContent;
> bs3Visitor.preVisit(bs3Col);
> bs3Visitor.visit(bs3Col);
> traverseDomainObject(bs3Col, visitor);
> traverseTabGroups(bs3Col, visitor);
> traverseActions(bs3Col, visitor);
> traverseFieldSets(bs3Col, visitor);
> traverseCollections(bs3Col, visitor);
> traverseRows(bs3Col, visitor);
> bs3Visitor.postVisit(bs3Col);
> } else if (rowContent instanceof BS3ClearFix) {
> final BS3ClearFix bs3ClearFix = (BS3ClearFix) rowContent;
> bs3Visitor.visit(bs3ClearFix);
> } else {
> throw new IllegalStateException(
> "Unrecognized implementation of BS3RowContent, " + rowContent);
> }
> }
> }
> {code}
> Instead, if it were traversed by element order, then the ordering of properties would be more predictable.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)