Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.7
    • Component/s: None
    • Labels:
      None
    • Affected Branches:
      MOODLE_17_STABLE
    • Fixed Branches:
      MOODLE_17_STABLE

      Description

      One of the last security patch on file.php blocks the scorm module from use.
      It checks for not hidden resources but not for not hidden scorm so the result is access denied.

      One patch is to check for not hidden scorms and then permit to access to files in the moddata/scorm/<scorm id> directory and childrens.

      Bobo

        Gliffy Diagrams

          Activity

          Hide
          dougiamas Martin Dougiamas added a comment -

          Petr can you please solve this? You know the circumstances best there.

          Show
          dougiamas Martin Dougiamas added a comment - Petr can you please solve this? You know the circumstances best there.
          Hide
          skodak Petr Skoda added a comment -

          I did not like the ugly Resource hack in the first place, because the correct way is to store module data in moddata directory, course files should be always 'open' for anybody who can access course itself.

          Where is scorm storing its data?

          Show
          skodak Petr Skoda added a comment - I did not like the ugly Resource hack in the first place, because the correct way is to store module data in moddata directory, course files should be always 'open' for anybody who can access course itself. Where is scorm storing its data?
          Hide
          upo Roberto Pinna added a comment -

          The scorm data are stored in moddata/scorm/<scorm id>/

          Show
          upo Roberto Pinna added a comment - The scorm data are stored in moddata/scorm/<scorm id>/
          Hide
          skodak Petr Skoda added a comment -

          thanks, I will get to this problem soon

          Show
          skodak Petr Skoda added a comment - thanks, I will get to this problem soon
          Hide
          skodak Petr Skoda added a comment -

          If I understand it properly, the problem shoud be now fixed in cvs, please test it and report back

          Thanks!

          Show
          skodak Petr Skoda added a comment - If I understand it properly, the problem shoud be now fixed in cvs, please test it and report back Thanks!
          Hide
          upo Roberto Pinna added a comment -

          At the first check it seems to work rigthly.

          Many thanks!!!

          Show
          upo Roberto Pinna added a comment - At the first check it seems to work rigthly. Many thanks!!!
          Hide
          skodak Petr Skoda added a comment -

          thanks for testing, closing now

          Show
          skodak Petr Skoda added a comment - thanks for testing, closing now

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                7/Nov/06