Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker 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
    • Rank:
      28298

      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

        Activity

        Hide
        Martin Dougiamas added a comment -

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

        Show
        Martin Dougiamas added a comment - Petr can you please solve this? You know the circumstances best there.
        Hide
        Petr Škoda 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
        Petr Škoda 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
        Roberto Pinna added a comment -

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

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

        thanks, I will get to this problem soon

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

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

        Thanks!

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

        At the first check it seems to work rigthly.

        Many thanks!!!

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

        thanks for testing, closing now

        Show
        Petr Škoda 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: