click here for details... Sakai Executive Director Position Search now open
Issue Details (XML | Word | Printable)

Key: SAK-16668
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Zhen Qian
Reporter: Aaron R Widner
Votes: 0
Watchers: 3
Operations

If you were logged in you would be able to see more operations.
Sakai

assignments / private notes and model answers does not display proper error message for text over 255 characters

Created: 02-Jul-2009 07:09   Updated: 15-Oct-2009 17:59
Component/s: Assignments
Affects Version/s: 2.6.x
Fix Version/s: 2.6.1, 2.7.0

Time Tracking:
Not Specified

Environment:
Local Production and Stage
qa1-za (svn tags/sakai-2.6.0-rc06) using MySQL/InnoDB - Built: 07/01/09 14:05 - Sakai Revision: 64433
Issue Links:
Depend
 
Duplicate
 

 GENERAL   TESTING   
2.6.x Status: Closed
2.5.x Status: None
2.4.x Status: None
Test Plan:
1. Create assignment / Edit an existing assignment
2. Click 'Add' next to Private Note / Model Answer in the Additional Information section
3. Enter information in the text area field
4. Click 'Keep private to myself' for the Share dropdown
5. Click 'Save' to save the Private Note
6. Click the 'Post' button
7. A warning appears saying, 'Alert: Assignment due date set to be in the past. Are you sure?' Click 'Post' again
8. The page reloads, but nothing else happens. If you click 'Post' again, you'll get the same warning from before and all subsequent attempts to click 'Post' result in the assignment not being posted with the changes.


 Description  « Hide
When users enter text into the private notes/model answers boxes in assignments it does not display a proper error message when there is 255 characters or more. When attempting to post with 255 characters or more users get looped with every other screen posting the error: 'Alert: Assignment due date set to be in the past. Are you sure?' Click 'Post' again.'

 All   Comments   Work Log   Change History   Subversion Commits   git Commits      Sort Order: Ascending order - Click to sort in descending order
Zhen Qian added a comment - 17-Jul-2009 12:11
r64791.

Updated the text field length to be 16777215 (2^24 - 1) as for medium text.

- Zhen

Zhen Qian added a comment - 17-Jul-2009 12:53
I have also updated the 2.6 conversion script inside the reference module to reflect the column type changes.

Zhen Qian added a comment - 17-Jul-2009 12:53
The conversion script change is in r64794.

Sam Ottenhoff added a comment - 30-Jul-2009 15:21
Confirmed fixed on latest trunk code.

To test:

1) Confirmed that 256 character answer would trigger this bug.
2) Tested on nightly2 and local build of trunk.
3) Created Assignment
4) "Model Answer" => Add
5) Pasted 256 character model answer
6) Selected a share option
7) Save (Model Answer)
8) Post (Assignment)

Received expect result of no error messages

Zhen Qian added a comment - 01-Sep-2009 07:45
merged r64791 of SAK-16668 into 2-6-x branch

merged r64794 of SAK-16668 into 2-6-x branch

Anthony Whyte added a comment - 14-Oct-2009 05:44
2.6.1 branch merge, r67752. 2.6 migration and release notes pages updated.

Anthony Whyte added a comment - 14-Oct-2009 05:45
Matt or David, please merge Matt's notes additions to 2.6.x

Anthony Whyte added a comment - 15-Oct-2009 17:59
Notes added to 2.6.x conversion scripts as of r67766.