Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-37896

qtype essay uses the wrong PARAM type

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.4, 2.4.1, 2.5
    • Fix Version/s: 2.3.5, 2.4.2
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide
      1. Edit your profile and set When editing text to Use standard web forms.
      2. Preview an essay question using Response format: HTML editor.
      3. Enter some HTML (see below) selecting one of the following format:
        • Moodle
        • Plain text
        • Markdown
      4. Click Save.
      5. Verify that your response is processed appropriately.
      6. Click Submit all and finish.
      7. Verify that your response is processed appropriately.

      Sample text to input:

      <p><b>To: John Doe</b> <johndoe@acme.com>
      <br/>From: Jane Doe <doe@xyz.com>
      <p>
      

      Show
      Edit your profile and set When editing text to Use standard web forms. Preview an essay question using Response format: HTML editor. Enter some HTML (see below) selecting one of the following format: Moodle Plain text Markdown Click Save. Verify that your response is processed appropriately. Click Submit all and finish. Verify that your response is processed appropriately. Sample text to input: <p><b>To: John Doe</b> <johndoe@acme.com> <br/>From: Jane Doe <doe@xyz.com> <p>
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Discovered while testing MDL-37847.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              timhunt Tim Hunt added a comment -

              Petr, does https://github.com/timhunt/moodle/compare/master...MDL-37896 look OK to you? particularly the change to the PHPdoc comment for PARAM_CLEANHTML.

              Show
              timhunt Tim Hunt added a comment - Petr, does https://github.com/timhunt/moodle/compare/master...MDL-37896 look OK to you? particularly the change to the PHPdoc comment for PARAM_CLEANHTML.
              Hide
              skodak Petr Skoda added a comment -

              looks ok

              Show
              skodak Petr Skoda added a comment - looks ok
              Hide
              timhunt Tim Hunt added a comment -

              Thanks Petr, submitting for integration.

              Show
              timhunt Tim Hunt added a comment - Thanks Petr, submitting for integration.
              Hide
              damyon Damyon Wiese added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              Cheers!

              Show
              damyon Damyon Wiese added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. Cheers!
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated (23, 24 & master), thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (23, 24 & master), thanks!
              Hide
              andyjdavis Andrew Davis added a comment -

              This seems to be working as described. Passing.

              Show
              andyjdavis Andrew Davis added a comment - This seems to be working as described. Passing.
              Hide
              damyon Damyon Wiese added a comment -

              Congratulations this fix has been added to Moodle!

              You may want to dedicate this issue to someone special on this Valentines day.

              Thanks!

              Show
              damyon Damyon Wiese added a comment - Congratulations this fix has been added to Moodle! You may want to dedicate this issue to someone special on this Valentines day. Thanks!

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    11/Mar/13