Issue Details (XML | Word | Printable)

Key: MDL-13280
Type: Bug Bug
Status: Closed Closed
Resolution: Duplicate
Priority: Major Major
Assignee: Dan Marsden
Reporter: safdar javed
Votes: 0
Watchers: 1
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Javascript error of form validation on scorm add/edit page.

Created: 01/Feb/08 10:16 PM   Updated: 11/Nov/08 10:31 AM
Return to search
Component/s: SCORM
Affects Version/s: 1.7
Fix Version/s: 1.8.7, 1.9.3

Environment: This bug has no issue with OS or browser.
Issue Links:
Duplicate
 

Database: MySQL
Participants: Dan Marsden and safdar javed
Security Level: None
Resolved date: 28/Jul/08
Affected Branches: MOODLE_17_STABLE
Fixed Branches: MOODLE_18_STABLE, MOODLE_19_STABLE


 Description  « Hide
When we add a scorm there is a javascript error on that page. this error is on Packege File element of page. Because element name is reference[value] while when validation function called to validate it it calls it with the name form.elements['reference']. Due to this javascript error comes when we submit the form.

DETAIL:
function validate_mod_scorm_mod_form(frm)

in this function there is call to function validate_mod_scorm_mod_form_reference(frm.elements['reference'])

in this call frm.elements['reference'] is passes as argument while the element name is reference[value]

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Dan Marsden made changes - 24/Jun/08 07:07 AM
Field Original Value New Value
Assignee Roberto Pinna [ upo ] Nobody [ nobody ]
Dan Marsden made changes - 24/Jun/08 07:33 AM
Assignee Nobody [ nobody ] Dan Marsden [ danmarsden ]
Dan Marsden made changes - 28/Jul/08 05:26 AM
Fix Version/s 1.9.3 [ 10290 ]
Affects Version/s 1.7.3 [ 10212 ]
Affects Version/s 1.7.4 [ 10243 ]
Fix Version/s 1.8.7 [ 10291 ]
Affects Version/s 1.7.1 [ 10151 ]
Fix Version/s 2.0 [ 10122 ]
Affects Version/s 1.7.2 [ 10174 ]
Dan Marsden made changes - 28/Jul/08 05:27 AM
Link This issue duplicates MDL-9077 [ MDL-9077 ]
Dan Marsden added a comment - 28/Jul/08 05:28 AM
this is now fixed in 18stable, 19stable, and HEAD, please wait until wednesday for this to appear in the Weekly build available on the download page.

thanks!

Dan


Dan Marsden made changes - 28/Jul/08 05:28 AM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Duplicate [ 3 ]
Dan Marsden made changes - 11/Nov/08 10:31 AM
Fix Version/s 2.0 [ 10122 ]
Status Resolved [ 5 ] Closed [ 6 ]