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

undefined $fullpath in throw new base_logger_exception('error_opening_file', $fullpath);

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Backup
    • Labels:

      Description

      in file_logger.class.php in backup code...

       throw new base_logger_exception('error_opening_file', $fullpath);

      ...should be...

       throw new base_logger_exception('error_opening_file', $this->fullpath);

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            to integrators: please cherry pick to stable branches

            Show
            skodak Petr Skoda added a comment - to integrators: please cherry pick to stable branches
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Petr - this has been integrated now.

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Petr - this has been integrated now.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Confirmed no testing required

            Show
            samhemelryk Sam Hemelryk added a comment - Confirmed no testing required
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many.

            Nah, joking, many thanks! Closing this a fixed, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many. Nah, joking, many thanks! Closing this a fixed, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Mar/12