Uploaded image for project: '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
    • Status: Passed
    • Priority: 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

          Attachments

            Issue Links

              Activity

              Hide
              cfulton 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
              cfulton 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
              tsala 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
              tsala 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
              cfulton Charles Fulton added a comment -

              Yes, MDL-30273.

              Show
              cfulton Charles Fulton added a comment - Yes, MDL-30273 .
              Hide
              dougiamas 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
              dougiamas 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
              stronk7 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
              stronk7 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
              salvetore 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
              salvetore 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: