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

Book module should implement reset framework

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.2, 2.4
    • Fix Version/s: 2.3.3
    • Component/s: Book
    • Labels:
    • Testing Instructions:
      Hide
      1. Goto a course which has atleast one book instance (or create one)
      2. Goto course admin>reset
      3. make sure under the heading "Reset not implemented" Book is not listed (click show advanced)
      4. Reset the course.
      5. Make sure no warning msgs related to book is generated.
      Show
      Goto a course which has atleast one book instance (or create one) Goto course admin>reset make sure under the heading "Reset not implemented" Book is not listed (click show advanced) Reset the course. Make sure no warning msgs related to book is generated.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-34944-master

      Description

      At present whenever you reset a course that contains a book instances, it popups a msg saying Reset is not implemented for the book module. Maybe we should just declare the reset function in book/lib.php . This would be consistent behaviour with other resources like URL and Folder.

      /**
       * This function is used by the reset_course_userdata function in moodlelib.
       * @param $data the data submitted from the reset course.
       * @return array status array
       */
      function book_reset_userdata($data) {
          return array();
      }

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

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

                  Dates

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