search

mysticfall / pivot4j

Pivot4J provides a common API for OLAP servers which can be used to build an analytical service frontend with pivot style GUI.
Other
128 stars 101 forks source link

Swap axes does not work properly with sorting. #148

Closed alegiuliani closed 10 years ago

alegiuliani commented 10 years ago

I found a minor error, that happens when I swap axes then try to use the sort function.

Steps to reproduce:

  1. Run this MDX: SELECT NON EMPTY {[Measures].[Unit Sales]} ON COLUMNS, NON EMPTY {[Store].Children} ON ROWS FROM [Sales Ragged]
  2. Swap the axes
  3. Try to sort the measure.
alegiuliani commented 10 years ago

Xavier,

Tested & Approved!

Thanks!

alegiuliani commented 10 years ago

Just to comment:

Try to run this mdx: SELECT NON EMPTY {[Measures].[Store Sales], [Measures].[Store Cost], [Measures].[Unit Sales]} ON COLUMNS, NON EMPTY {[Store].[Israel], [Store].[USA], [Store].[Vatican]} ON ROWS FROM [Sales Ragged]

Now, sort one measure. Look to the others measures sort icons. They change too. Is that correct? Thank you.

With the same mdx: SELECT NON EMPTY {[Measures].[Store Sales], [Measures].[Store Cost], [Measures].[Unit Sales]} ON COLUMNS, NON EMPTY {[Store].[Israel], [Store].[USA], [Store].[Vatican]} ON ROWS FROM [Sales Ragged]

I found something strange. Try to drill Israel >> Haifa

The icon of Haifa doesn't chance to -

Yes, it might be another thread of bug, but I don't know if it's a real bug.

mysticfall commented 10 years ago

Probably a different issue, I guess. I'll reopen the issue anyway so I won't forget to review it this weekend.

mysticfall commented 10 years ago

@alegiuliani I confirmed the problem and posted another issue as #150.

By the way, I couldn't reproduced the problem with session expiration you mentioned on IRC. So, it'd be helpful if you can post an issue about it with some instruction to reproduce it on my end.

Thanks!

P.S. And please post a separate issue for each problem. It's much easier to track them that way :)

alegiuliani commented 10 years ago

Hi Xavier,

Actually I didn't mention any bug of session expiration(or at least I didn't remember), maybe it was someone else. I'll test the last commit.

Thanks!

mysticfall commented 10 years ago

@alegiuliani Urg... sorry! Seems I have a short memory :)

By the way, the code are committed just now so you might want to check if the build really contains the relevant changes before you test.

alegiuliani commented 10 years ago

Hi Xavier, Sorry for the late(holliday guilt)! The main error was tested & approved!(I'll open another related to the sort icon to separate the subjects)

Thanks