Moodle

New Window setting "Allow the window to be scrolled" resets

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.8.2
  • Fix Version/s: 1.8.7, 1.9.3, 2.0
  • Component/s: SCORM
  • Labels:
    None
  • Affected Branches:
    MOODLE_18_STABLE
  • Fixed Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

Description

On the Updating SCORM/AICC page, select "Display package in New Window" and uncheck all checkboxes except "Allow the window to be resized." Save changes. Later, Update this SCORM/AICC and you will see that the "Allow the window to be scrolled" checkbox is checked. You must once again uncheck this box, or your window will have scrollbars!

There are also three missing language strings on the Updating SCORM/AICC page:

[[activation]]
[[options]]
[[updatefreq]]

Activity

Hide
Beatriz Beltran added a comment -

PLEASE DO SOMETHING ABOUT THIS. MY TEACHERS ARE GOING CRAZY HAVING TO EDIT ALL THEIR OLD RESOURCES! (more than a million)

This behavior has the same problem in all Resources, meaning text pages, web pages,etc.

When upgrading to 1.8.2 all old resources that were programmed only to:
Allow the window to be resized
Allow the window to be scrolled

Are now set back to all the window attributes checked. However they still work fine.

BUT, when editing a resource, all the attributes show checked out. If the attributes are not unchecked the window goes back to show Menu bar, tools bar, etc.

Is any place where the administrator can set those attributes a default to use throughout the system?
Would that be nice or what?

Show
Beatriz Beltran added a comment - PLEASE DO SOMETHING ABOUT THIS. MY TEACHERS ARE GOING CRAZY HAVING TO EDIT ALL THEIR OLD RESOURCES! (more than a million) This behavior has the same problem in all Resources, meaning text pages, web pages,etc. When upgrading to 1.8.2 all old resources that were programmed only to: Allow the window to be resized Allow the window to be scrolled Are now set back to all the window attributes checked. However they still work fine. BUT, when editing a resource, all the attributes show checked out. If the attributes are not unchecked the window goes back to show Menu bar, tools bar, etc. Is any place where the administrator can set those attributes a default to use throughout the system? Would that be nice or what?
Hide
Beatriz Beltran added a comment -

Going to Site Administration, Modules, Activities, Resources, Settings it is possible to change what windows attributes should be available by default.

This seems to solve our problem.

The problem is the lack of documentation!

Show
Beatriz Beltran added a comment - Going to Site Administration, Modules, Activities, Resources, Settings it is possible to change what windows attributes should be available by default. This seems to solve our problem. The problem is the lack of documentation!
Hide
Dan Marsden added a comment -

John/Beatriz - can we close this, or are there still missing strings?

thanks,

Dan

Show
Dan Marsden added a comment - John/Beatriz - can we close this, or are there still missing strings? thanks, Dan
Hide
John Isner added a comment -

Dan,
The missing strings seem to be fixed, BUT the window issues are still unresolved. (I just tested on demo.moodle.org).

Show
John Isner added a comment - Dan, The missing strings seem to be fixed, BUT the window issues are still unresolved. (I just tested on demo.moodle.org).
Hide
Dan Marsden added a comment -

thanks. - will have a look at it!

Dan

Show
Dan Marsden added a comment - thanks. - will have a look at it! Dan
Hide
Dan Marsden added a comment -

ok taking notes.....

it seems like at some point all those options were to be stored in the mdl_scorm > options field - instead of having a field for each value in the scorm table.. - don't know enough about formslib to see how that might work...

also the variables seem to be set as $SCORM_POPUP_OPTIONS in locallib.php (but not 'grabbed' from anywhere) so even if we were saving the values somewhere in the db, it isn't checking them.

there also seems to be some code at the top of locallib that doesn't seem to serve any purpose (stdoptions isn't used anywhere)

$stdoptions = '';
foreach ($SCORM_POPUP_OPTIONS as $popupopt => $value) {
$stdoptions .= $popupopt.'='.$value;
if ($popupopt != 'status') { $stdoptions .= ','; }
}

Show
Dan Marsden added a comment - ok taking notes..... it seems like at some point all those options were to be stored in the mdl_scorm > options field - instead of having a field for each value in the scorm table.. - don't know enough about formslib to see how that might work... also the variables seem to be set as $SCORM_POPUP_OPTIONS in locallib.php (but not 'grabbed' from anywhere) so even if we were saving the values somewhere in the db, it isn't checking them. there also seems to be some code at the top of locallib that doesn't seem to serve any purpose (stdoptions isn't used anywhere) $stdoptions = ''; foreach ($SCORM_POPUP_OPTIONS as $popupopt => $value) { $stdoptions .= $popupopt.'='.$value; if ($popupopt != 'status') { $stdoptions .= ','; } }
Hide
Dan Marsden added a comment -

attaching patch that should fix this.

Show
Dan Marsden added a comment - attaching patch that should fix this.
Hide
Dan Marsden added a comment -

fix now in 1.8stable, 1.9stable and HEAD

Show
Dan Marsden added a comment - fix now in 1.8stable, 1.9stable and HEAD
Hide
Piers Harding added a comment -

code reviewed and discussed.

Show
Piers Harding added a comment - code reviewed and discussed.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: