[phpesp-dev] [ phpesp-Bugs-743518 ] Several bugs in 1.6.1

[SourceForge.net <no...@so...>]

Bugs item #743518, was opened at 2003-05-26 18:02
Message generated for change (Comment added) made by angek
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=743518&group_id=8956

Category: None
Group: None
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Kon Angelopoulos (angek)
Summary: Several bugs in 1.6.1

Initial Comment:
Great package... but:

Cross tabulate includes complete="N" records;

Cros tab grand total incorrect;

survey_results does not display records where a column

'key' is 0;

Error messages are not prominent in espresponse.inc;

----------------------------------------------------------------------

>Comment By: Kon Angelopoulos (angek)
Date: 2003-09-25 18:52

Message:
Logged In: YES 
user_id=198398

Correction made and commited to cvs.



- thanks Charlie

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2003-09-25 01:15

Message:
Logged In: NO 

The grand_total var is being initialized inside the outer loop.  

The initialization for this should be moved outside the outer 

loop.  I'd wanted to submit this as a patch but have not been 

successful in using cvs to get the most current developer 

version.   The fix is as follows:

crosstab.inc:

                // cws: 9/17/03 moved initialization for grand 

total from inside

                //      fisrt for loop below to above first for loop 

so that it didn't

                //      get reset on adding up each column's total.

                $grand_total = 0;

                for ($f = 0; $f < $cids_col_count; $f++){

                        // $grand_total = 0;

                        for ($g = 0; $g < $cids_row_count; $g++){









Charlie Smith

sm...@ld...

9/24/03

----------------------------------------------------------------------

Comment By: Kon Angelopoulos (angek)
Date: 2003-09-13 22:36

Message:
Logged In: YES 
user_id=198398

cross tabulate does not include complete="N" records.

As for the totals I cannot replicate the mathematical error

you claim to exist with the totals.

The prominence of error message should be a feature request

and not part of a bug submission.



If you feel that any of what I've said is incorrect please

provide examples for me to see as I cannot duplicate these bugs.



----------------------------------------------------------------------

Comment By: Kon Angelopoulos (angek)
Date: 2003-07-21 09:13

Message:
Logged In: YES 
user_id=198398

I'll start looking into this and see what can be done.

----------------------------------------------------------------------

Comment By: Kon Angelopoulos (angek)
Date: 2003-07-21 09:12

Message:
Logged In: YES 
user_id=198398

I'll start looking into this and see what can be done.

----------------------------------------------------------------------

Comment By: Owen Duffy (owenduffy)
Date: 2003-05-27 07:36

Message:
Logged In: YES 
user_id=787353

Bug report from sourceforge user owenduffy.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=743518&group_id=8956