Moodle QA
  1. Moodle QA
  2. MDLQA-1190 Moodle 2.2 QA
  3. MDLQA-1196

CLONE - In a single simple discussion forum only teachers can start discussions

    Details

    • Type: Functional Test Functional Test
    • Status: Passed
    • Priority: Minor Minor
    • Affects Version/s: Moodle 2.2
    • Fix Version/s: Moodle 2.2
    • Component/s: Forum, Student, Teacher
    • Labels:
      None

      Description

      1. Login as a teacher and start a discussion in a single simple discussion forum.
      2. Login as a student and reply to the discussion.
      3. Check that, as a student, it is not possible to start a discussion.

        Issue Links

          Activity

          Hide
          Charles Fulton added a comment -

          This works as expected through the interface, however it is possible to circumvent the limitations in the interface by manually entering an URL.

          Example:

          1. Login as a teacher and start a discussion in a single simple discussion forum.
          2. Enter the URL of the forum in order to create a new discussion (e.g. MOODLE/mod/forum/post.php?forum=n).
          3. You'll get the interface and be able to add a new discussion topic. This also worked for both a teacher AND a student.

          Afterwards if you view the forum and have debugging turned on you'll get an error thrown because get_record() will get multiple results:

          Error: mdb->get_record() found more than one record!
          line 1302 of /lib/dml/moodle_database.php: call to debugging()
          line 1262 of /lib/dml/moodle_database.php: call to moodle_database->get_record_sql()
          line 1242 of /lib/dml/moodle_database.php: call to moodle_database->get_record_select()
          line 144 of /mod/forum/view.php: call to moodle_database->get_record()

          This does mean you'll never see more than one discussion topic regardless.

          Show
          Charles Fulton added a comment - This works as expected through the interface, however it is possible to circumvent the limitations in the interface by manually entering an URL. Example: 1. Login as a teacher and start a discussion in a single simple discussion forum. 2. Enter the URL of the forum in order to create a new discussion (e.g. MOODLE/mod/forum/post.php?forum=n). 3. You'll get the interface and be able to add a new discussion topic. This also worked for both a teacher AND a student. Afterwards if you view the forum and have debugging turned on you'll get an error thrown because get_record() will get multiple results: Error: mdb->get_record() found more than one record! line 1302 of /lib/dml/moodle_database.php: call to debugging() line 1262 of /lib/dml/moodle_database.php: call to moodle_database->get_record_sql() line 1242 of /lib/dml/moodle_database.php: call to moodle_database->get_record_select() line 144 of /mod/forum/view.php: call to moodle_database->get_record() This does mean you'll never see more than one discussion topic regardless.
          Hide
          Helen Foster added a comment -

          Charles, thanks for your feedback. Is the situation you describe reported as a Moodle (MDL) issue? (I did a quick search but couldn't find anything.) If not, please could you create an issue for it.

          Show
          Helen Foster added a comment - Charles, thanks for your feedback. Is the situation you describe reported as a Moodle (MDL) issue? (I did a quick search but couldn't find anything.) If not, please could you create an issue for it.
          Hide
          Charles Fulton added a comment -

          Yes, MDL-30273.

          Show
          Charles Fulton added a comment - Yes, MDL-30273 .
          Hide
          Martin Dougiamas added a comment -

          I've passed this because it works as expected through the interface. The functionality tests OK.

          Totally agree about the bug in the backend though - there should be more checks.

          Show
          Martin Dougiamas added a comment - I've passed this because it works as expected through the interface. The functionality tests OK. Totally agree about the bug in the backend though - there should be more checks.
          Hide
          Eloy Lafuente (stronk7) added a comment - - edited

          Please, if there are backends issues not blocking this, don't create links saying it's being blocked, grrrr. TIA!

          Show
          Eloy Lafuente (stronk7) added a comment - - edited Please, if there are backends issues not blocking this, don't create links saying it's being blocked, grrrr. TIA!
          Hide
          Michael de Raadt added a comment -

          Updating the links for this. We will continue to work on the linked MDL issue, but it is no longer a blocker.

          Show
          Michael de Raadt added a comment - Updating the links for this. We will continue to work on the linked MDL issue, but it is no longer a blocker.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: