Achievo/ATK - Bug 269 – Order by "Project/Phase" does not work in employee time survey form

Bug 269 - Order by "Project/Phase" does not work in employee time survey form

Summary:

Order by "Project/Phase" does not work in employee time survey form

Status:	RESOLVED FIXED

Product:	Achievo
Component:	Employee management
Version:	SVN
Platform:	Mozilla Linux

Importance:	P3 normal
Assigned To:	Ivo Jansch

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2004-02-20 04:14 by Mark Andersen
Modified:	2004-04-13 10:31 (History)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Mark Andersen 2004-02-20 04:14:22

Form : "Reports/Time Survey" , selecting "Order by" "Project/Phase" does not
work. The results are unsorted. 

Order by works for all the other fields, it is just the "project/phase" that
does not work.

Tested with achievo 1.0.2 and has same problem. 

Also reproduced using demo achievo from achievo.org.  (vsn 1.0.0)

------- Comment #1 From Ivo Jansch 2004-02-20 15:27:43 -------

Thanks for the report. I could reproduce this in my installation (CVS) too.

The problem seems to be that on line 258 in
modules/reports/class.hoursurvey.inc, the sort columns are added as if they were
columns in the hours table. The result is that the table is sorted by phase_id,
instead of 'project.name, phase.name'.

------- Comment #2 From Ivo Jansch 2004-04-07 21:07:17 -------

The problem is fixed, in such a way, that sorting on relations (for example
'coordinator' in the project screen) now works properly. (it used to sort by id,
instead of the sortfield of the relation)

The fix was sponsored by Petrosys Pty, Australia.

------- Comment #3 From Ivo Jansch 2004-04-13 09:24:17 -------

Reopened, sorting by phase (within the project/phase column) does not work
properly yet.

------- Comment #4 From Ivo Jansch 2004-04-13 10:31:42 -------

Problem solved. The latter was not a bug, but due to a leading space in the
phase name.