Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-46038

Undefined variable CFG in equella and boxnet

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.7.1
    • Component/s: Repositories
    • Labels:
    • Testing Instructions:
      Hide

      Proper test (the one that was missed in MDL-40079):

      1. Setup equella and boxnet repositories
      2. Add shortcuts to files there in some filearea (i.e. private files)
      3. Wait for one day
      4. Open filearea for editing and make sure there are no warnings

      Reasonable test:

      1. Review the code
      Show
      Proper test (the one that was missed in MDL-40079 ): Setup equella and boxnet repositories Add shortcuts to files there in some filearea (i.e. private files) Wait for one day Open filearea for editing and make sure there are no warnings Reasonable test: Review the code
    • Affected Branches:
      MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      wip-MDL-46038-master

      Description

      On editing file area that contained references to Equella I received error:

      Notice: Undefined variable: CFG in /home/marina/repositories/master/moodle/repository/equella/lib.php on line 211
       
      Notice: Trying to get property of non-object in /home/marina/repositories/master/moodle/repository/equella/lib.php on line 211
       
      Notice: Trying to get property of non-object in /home/marina/repositories/master/moodle/repository/equella/lib.php on line 219
       
      Notice: Undefined variable: CFG in /home/marina/repositories/master/moodle/repository/equella/lib.php on line 219
      

      I found that this is a regression from MDL-40079 and the same error should happen in boxnet

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              marina Marina Glancy
              Reporter:
              marina Marina Glancy
              Peer reviewer:
              Mark Nelson
              Integrator:
              Damyon Wiese
              Tester:
              Simey Lameze
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/Jul/14