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.

        Gliffy Diagrams

          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: