Issue 124349

but when  i just sorted by 2 fields, it works! just cannot grouped report

Created attachment 82768 [details]
see reports in my base

i make 2 reports for your inquiry

confirmed with AOO 3.4.1 and 4.1-dev

generated report uses Command: 
> SELECT "Addresse"."ID", "Addresse"."card", "Addresse"."name", "Addresse"."date", "Addresse"."polis", "Addresse"."adress", "Addresse"."class", "Addresse"."school" FROM "Addresse" "Addresse" ORDER BY "Addresse"."date" DESC

while it should be:
> SELECT "Addresse"."ID", "Addresse"."card", "Addresse"."name", "Addresse"."date", "Addresse"."polis", "Addresse"."adress", "Addresse"."class", "Addresse"."school" FROM "Addresse" "Addresse" ORDER BY "Addresse"."school" ASC, "Addresse"."date" DESC

the command is available through
Edit report -> Form-Navigator (Form Design Toolbar):  Forms > ReportSource >  Command > Properties

workaround: edit Command or 
create query sorted by date and build report from query only sorted by school

probably related bug or cause:

if you go back from wizard step 5 (layout) to 4 (sorting) the first search criteria is missing but only if you have a group assigned in step 3 (if you have 2 groups assigned the first 2 criteria are missing)

this leads to
> main\wizards\com\sun\star\wizards\report\ReportWizard.java

I see nothing suspicious in enterStep(): line 157.

leaveStep(): line 214 however has a call [IReportDocument].setSorting(String[][])
which has two implementations but which one is called?

> main\wizards\com\sun\star\wizards\report\ReportTextImplementation.java (1 hit)
>	Line 609:     public void setSorting(String[][] aSortFieldNames)

> main\wizards\com\sun\star\wizards\reportbuilder\ReportBuilderImplementation.java (1 hit)
>	Line 507:     public void setSorting(String[][] _aSortFieldNames)

(In reply to j.nitschke from comment #4)
> which has two implementations but which one is called?
startReportWizard Line 459: loads class "com.sun.star.wizards.reportbuilder.ReportBuilderImplementation"

> main\wizards\com\sun\star\wizards\reportbuilder\ReportBuilderImplementation.java (1 hit)
>	Line 507:     public void setSorting(String[][] _aSortFieldNames)

since it works without groups
> main\wizards\com\sun\star\wizards\reportbuilder\ReportBuilderImplementation.java (2 hits)
>	Line 512:             getRecordParser().prependSortFieldNames(groupNames);
is a hot candidate
implementation:
> main\wizards\com\sun\star\wizards\db\CommandMetaData.java (1 hit)
>	Line 163:     public void prependSortFieldNames(String[] _fieldnames)
I don't see how groupnames get removed from SortFieldNames there

need someone with basic java skills to take a look

looked at the wrong place
not setSorting is the problem
getSortFieldNames shows unexpected behavior:
it returns array of pairs field names and sort direction
but excludes duplicates (OK) and fields marked READ_ONLY (What Why?)
main/wizards/com/sun/star/wizards/ui/SortingComponent.java 
> if (!((Boolean) CurUnoDialog.getControlProperty("lstSort" + (i + 1), PropertyNames.READ_ONLY)))

what happend before is:
- entering sort step of the wizard
- sort fields are filled with the group fields and marked READ_ONLY
main/wizards/com/sun/star/wizards/report/ReportWizard.java : line 160

leaving sort step only the non READ_ONLY fields are fetched

remove exclusion of read only fields in getSortFieldNames() fixes the wizard

let's see who calls getSortFieldNames():
* ReportWizard.leaveStep this where we want it all SortFieldNames,
  result is used in setSorting which brings us to comment 4,
  setSorting has 2 implementation of which one has a hack, it prepends the
  group names to the list of SortFieldNames
  (see main/wizards/com/sun/star/wizards/reportbuilder/ReportBuilderImplementation.java line 507)
  seems this implementation is only used when the extension "Oracle Report Builder" is installed (huh?)
  
* QueryWizard.leaveStep 
  QueryWizard makes no use of READ_ONLY property in sort step

planed solution:
* remove exclusion in SortingComponent.getSortFieldNames()
* remove hack in ReportBuilderImplementation

Created attachment 83065 [details]
remove READ_ONLY check and workaround for missing fields

patch tested on linux with trunk
works without Oracle Report Builder
also seems to work with Oracle Report Builder but this extension is very unstable on my machine (ORB does not support 4.x according to website)

this will also fix Issue 122604 and Issue 122605

don't know why this exception was made in first place, but couldn't find any other affected code beside report wizard

I volunteer to review the provided patch on Windows.
But I will not have the time resources this week -  I think I will have a look next week.