njvack / libstats

Automatically exported from code.google.com/p/libstats
GNU General Public License v2.0
1 stars 0 forks source link

False SQL query in ByInitials.php report #9

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run the current commit of the ByInitials report
2. Report return will only list "In-Person" rather than all question format
types

What is the expected output? What do you see instead?

All of the question format types for each initial should display in the report

What version of the product are you using? On what operating system?

1.0.6

Please provide any additional information below.
The sql query need to be changed...

Line 36:   GROUP BY initials, questions.time_spent_id
Should be: GROUP BY initials, questions.question_format_id

Original issue reported on code.google.com by jedij...@gmail.com on 12 Nov 2009 at 8:54

GoogleCodeExporter commented 9 years ago
I've attached a patch file with this change, since I don't have commit access 
to the 
trunk.

Original comment by dtk.john...@gmail.com on 10 Dec 2009 at 8:10

Attachments:

GoogleCodeExporter commented 9 years ago
Confirmed issue. Wish I'd have found this before I spent two hours debugging 
the SQL
query. The fix given does work, also.

Original comment by greenwoo...@gmail.com on 15 Apr 2010 at 2:08