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 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 ]