Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.6, 2.2.3, 2.3
    • Fix Version/s: 2.1.7, 2.2.4
    • Component/s: Course
    • Labels:
    • Testing Instructions:
      Hide

      Do this for every course format that lets you edit section names.

      1. Create a course in Topics format
      2. Turn editing on.
      3. Edit the introduction to one section, and try to set the name to '0'.
      4. Save changes.
      5. Make sure that the section appears with name '0'.

      Show
      Do this for every course format that lets you edit section names. 1. Create a course in Topics format 2. Turn editing on. 3. Edit the introduction to one section, and try to set the name to '0'. 4. Save changes. 5. Make sure that the section appears with name '0'.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      41679

      Description

      1. Create a course in Topics format
      2. Turn editing on.
      3. Edit the introduction to one section, and try to set the name to '0'.
      4. Save changes. You are ignored.

      (If you look in the DB, you can see the new name has been saved, but because of various if !empty checks, it is not used.

        Issue Links

          Activity

          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Passing, tested under 21_STABLE and master, weeks and topics formats.

          Show
          Eloy Lafuente (stronk7) added a comment - Passing, tested under 21_STABLE and master, weeks and topics formats.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Big thanks for the effort. This is now part of Moodle upstream. Let's wait for regressions, yay! LOL

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Big thanks for the effort. This is now part of Moodle upstream. Let's wait for regressions, yay! LOL Ciao

            People

            • Assignee:
              Tim Hunt
              Reporter:
              Tim Hunt
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Eloy Lafuente (stronk7)
              Participants:
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: