[Mondrian] MONDRIAN-465: "ordinalColumn displayed instead ofcaptionColumn", was not solved

[Julian Hyde]

Please add your comments to the bug. 

> -----Original Message-----
> From: Alexander Korsukov [mailto:akorsukov at gmail.com] 
> Sent: Tuesday, October 13, 2009 7:03 AM
> To: jhyde at pentaho.com; Mondrian developer mailing list
> Subject: Re: [Mondrian] MONDRIAN-465: "ordinalColumn 
> displayed instead ofcaptionColumn", was not solved
> 
> Julian.
> 
> > Thanks. I have reopened the bug with your comments.
> 
> Sorry, I mistake. Necessary to change the following lines of code:
> 
> mondrian-3.1.2.13008\src\main\mondrian\rolap\SqlTupleReader.java
> line 968
> 
>                if (!ordinalSql.equals(keySql)) {
>                    alias = sqlQuery.addSelect(ordinalSql);
>                    if (needsGroupBy) {
>                        sqlQuery.addGroupBy(ordinalSql, alias);
>                    }
>                }
> 
>                if (captionSql != null) {
>                    alias = sqlQuery.addSelect(captionSql);
>                    if (needsGroupBy) {
>                        sqlQuery.addGroupBy(captionSql, alias);
>                    }
>                }
> 
> should be:
> 
>                if (captionSql != null) {
>                    alias = sqlQuery.addSelect(captionSql);
>                    if (needsGroupBy) {
>                        sqlQuery.addGroupBy(captionSql, alias);
>                    }
>                }
> 
>                if (!ordinalSql.equals(keySql)) {
>                    alias = sqlQuery.addSelect(ordinalSql);
>                    if (needsGroupBy) {
>                        sqlQuery.addGroupBy(ordinalSql, alias);
>                    }
>                }
> 
> see SqlTupleReader.Target.internalAddRow(), caption read from
> resultSet before ordinal.
> 
> I apologize again for the mistake.
> 
> -- 
> Alexander Korsukov
> _____________________________________________________
> mailto:akorsukov at gmail.com                icq:8572488
> (2E0A2052)                        xmpp:shko at jabber.ru
>