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

XHTML not well formed for editor with required rule

    Details

    • Testing Instructions:
      Hide

      1. Log in as admin/teacher
      2. Enable XML strict headers under "Settings -> Site administration -> Development -> debugging"
      3. Create feedback activity and you should not see any error.

      Show
      1. Log in as admin/teacher 2. Enable XML strict headers under "Settings -> Site administration -> Development -> debugging" 3. Create feedback activity and you should not see any error.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull Master Branch:
      wip-mdl-29539

      Description

      While testing MDL-29190, I figured that there is a missing space in adding blur event, which break XHTML rule.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              nebgor Aparup Banerjee 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.

              TIA and cheers!

              Show
              nebgor Aparup Banerjee 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. TIA and cheers!
              Hide
              rajeshtaneja Rajesh Taneja added a comment -

              Branches rebased.

              Show
              rajeshtaneja Rajesh Taneja added a comment - Branches rebased.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks Raj - this has been integrated now

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks Raj - this has been integrated now
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              Hi,
              No issues were found while creating a feedback activity, however adding questions to the feedback broke the xhtml.
              Created MDL-29621 with details.
              test passed.
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - Hi, No issues were found while creating a feedback activity, however adding questions to the feedback broke the xhtml. Created MDL-29621 with details. test passed. Thanks
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              git repositories have been updated with your awesome changes, thanks! Closing.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - git repositories have been updated with your awesome changes, thanks! Closing.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    10/Oct/11