[#MDL-16113] Add warning to scorm modedit page when Slasharg disabled

Moodle

Add warning to scorm modedit page when Slasharg disabled

Created: 19/Aug/08 05:18 AM Updated: 27/Aug/08 06:37 AM

Component/s:

SCORM

Affects Version/s:

1.9, 1.8

Fix Version/s:

1.8.7, 1.9.3, 2.0

File Attachments:

scorm-slashargs.patch (2 kb)

Participants:	Dan Marsden, Helen Foster and Petr Škoda
Security Level:	None
QA Assignee:	Helen Foster

Description

« Hide

Scorm will not work correctly when Slasharguments are disabled.

We need to add a check and throw a warning on the modedit page to warn users that their objects will not work with slasharguments disabled.

- should add details about this to a FAQ in the documentation somewhere as well.

All

Comments

Change History

Version Control

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Dan Marsden - 19/Aug/08 08:15 AM

adding patch that displays warning on modedit page if slashargs is enabled.

[ Permlink | « Hide ]

Dan Marsden - 20/Aug/08 10:34 AM

patch now in HEAD, 19stable and 18stable

:-)

Dan

[ Permlink | « Hide ]

Helen Foster - 26/Aug/08 05:11 PM

Hi Dan,

Thanks for adding a warning when slash arguments are disabled.

I tested with the latest 1.9.2 and found it to be working fine i.e. when slash arguments are disabled, the message "WARNING: slash arguments is disabled on this site and objects may not function as expected!" appears on the edit SCORM page.

I was a bit confused though, because, despite the warning, the object (an eXe package in the Moodle Features Demo course) worked fine! What does "... not function as expected" actually mean?

Also, how about adding an additional warning about SCORM not working when slash arguments are disabled to the configslasharguments lang string and rewording the warning on the edit SCORM page to make it less scary for teachers, for example

WARNING: This SCORM package may not work properly, as the "slash arguments" site admin setting is disabled. Please contact your site administrator.

[ Permlink | « Hide ]

Petr Škoda - 26/Aug/08 06:05 PM

hi Helen,
the scorm code ignores the slashargument setting now and always uses file.php/xx/xx that is why it works for you. It would not work before, the new message is just an extra warning to let ppl know that working slasharguments are required (mostly required for MS IIS).

Petr

[ Permlink | « Hide ]

Dan Marsden - 27/Aug/08 06:37 AM

thanks Helen - how about:

WARNING: the site admin setting "slash arguments" is disabled. SCORM/AICC objects require and force the use of slash arguments. If your server does not support this, objects will not work correctly. Please contact your site administrator if this object does not work.