[#SAK-4684] Post 'em / error when uploading csv file with more than 17 rows

Sakai

Post 'em / error when uploading csv file with more than 17 rows

Created: 13-Feb-2006 14:42 Updated: 06-Nov-2008 10:46

Component/s:

Post'Em

Affects Version/s:

2.1.0

Fix Version/s:

2.2.1

Time Tracking:

Not Specified

File Attachments:	1. Test3-1.csv (0.7 kB) 2. Test3-1.csv (0.7 kB) 3. Test3.csv (0.7 kB)
Environment:	All OS / All Browsers

2.6.x Status:	None
2.5.x Status:	None
2.4.x Status:	None

Description

« Hide

When uploading a CSV file via Post 'Em, if the file contains over 17 rows of data and does not include a '0' in every assignment, it displays an error -"Number of columns (2) in row 17 does not match number in row one!" . Adding a 0 in every assignment grade field OR reducing the number of rows to less than 17 fixes the problem and it uploads fine. I've attached copies of the three test files I used when verifying the bug.

All

Comments

Work Log

Change History

Subversion Commits

git Commits

Sort Order:

[ Permlink | « Hide ]

Kristol Hancock added a comment - 21-Feb-2006 09:35

I've changed the attachments for this bug to make the problem a bit more clear. Test3.csv can be uploaded without any error messages. Test3-1.csv, which has one additional row, cannot be uploaded. The error message that is received is 'Number of columns (2) in row 17 does not match number in row one!' as Kevin stated above.

[ Permlink | « Hide ]

Jarrod Lannan added a comment - 19-Apr-2006 10:15

Last row needs commas in the attached file.

[ Permlink | « Hide ]

Megan May added a comment - 25-Apr-2006 09:31

Jarrod, today at our team meeting we decided to address this problem via the application

[ Permlink | « Hide ]

Michelle Wagner added a comment - 18-Jul-2006 10:47

The problem was that the csv files that were used did not have the same number of columns in row 17 as the header row. The same error occurs if you remove any columns from any other row. To fix this, if any of the rows (including the header row) have more columns than the other rows, we will add blank columns to the other rows to ensure that all rows have the same number of columns.

[ Permlink | « Hide ]

Michelle Wagner added a comment - 24-Jul-2006 13:16

r12748
Made small change - now appends blank space instead of null

[ Permlink | « Hide ]

Megan May added a comment - 28-Jul-2006 06:04

verified on local instance (r13002)