Moodle
  1. Moodle
  2. MDL-25582

two notices diplayed adding a new instance of a non core module

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: None
    • Component/s: Other
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Rank:
      13234

      Description

      Download http://download.moodle.org/plugins/mod/NEWMODULE.zip.
      Change NEWMODULE to WIZARD and newmodule to wizard.
      Install and go to add a new instance of wizard.
      you will get two notices
      undefined variable introeditor in course/modedit.php at line 430 and 432.

      Fix (found by David):
      change line 401 of moodle20/course/modedit.php
      from
      if (plugin_supports('mod', $fromform->modulename, FEATURE_MOD_INTRO, false)) {
      to
      if (plugin_supports('mod', $fromform->modulename, FEATURE_MOD_INTRO, true)) {

      Thanks David.

        Activity

        Hide
        David Mudrak added a comment -

        The reason for theis error is following: at the line 401, the condition

        if (plugin_supports('mod', $fromform->modulename, FEATURE_MOD_INTRO, false)) {
        

        is used to check whether $introeditor shall be intialized. Note the default 'false' here.

        Later at the line 429, the condition

        if (plugin_supports('mod', $fromform->modulename, FEATURE_MOD_INTRO, true)) {
        

        is used to check whether $introeditor shall be processed.

        Because the fresh new module does not explicitly declare the features, the default values are used and that produces this mess. Elsewhere in Moodle code, we expect the default true for this feature (so the editor is rendered in the form, for example). So I am going to expect here, too.

        Show
        David Mudrak added a comment - The reason for theis error is following: at the line 401, the condition if (plugin_supports('mod', $fromform->modulename, FEATURE_MOD_INTRO, false )) { is used to check whether $introeditor shall be intialized. Note the default 'false' here. Later at the line 429, the condition if (plugin_supports('mod', $fromform->modulename, FEATURE_MOD_INTRO, true )) { is used to check whether $introeditor shall be processed. Because the fresh new module does not explicitly declare the features, the default values are used and that produces this mess. Elsewhere in Moodle code, we expect the default true for this feature (so the editor is rendered in the form, for example). So I am going to expect here, too.
        Hide
        Petr Škoda added a comment -

        +1, thanks!

        Show
        Petr Škoda added a comment - +1, thanks!
        Hide
        David Mudrak added a comment -

        Fixed in CVS HEAD. Thanks Daniele for spotting this and thanks Petr for the review.

        Show
        David Mudrak added a comment - Fixed in CVS HEAD. Thanks Daniele for spotting this and thanks Petr for the review.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: