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

HTML Block should require the config_text field

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.0.7, 2.1.4, 2.2.1
    • 2.1.4, 2.2.1
    • Blocks
    • MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • MOODLE_21_STABLE, MOODLE_22_STABLE
    • MDL-30805-master-1
    • Hide

      Before

      • create a new HTML block
      • edit the block settings
      • save the form without filling in any content

      With Patch

      • create a new HTML block
      • edit the block settings
      • attempt to save the form without any content
      • ensure that the form is rejected without content
      • enter content
      • ensure that the form can now be submitted
      Show
      Before create a new HTML block edit the block settings save the form without filling in any content With Patch create a new HTML block edit the block settings attempt to save the form without any content ensure that the form is rejected without content enter content ensure that the form can now be submitted

    Description

      The HTML block defines a 'config_text' element to contain the HTML, but it isn't required.
      Given the nature of the block, it should ideally be a required field

      Attachments

        Issue Links

          will help resolve

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-30806 Audit existing forms for inconsistent behaviour, missing required fields, field order

          • Minor - Minor loss of function where workaround is possible
          • Open

          Activity

            People

              dobedobedoh Andrew Lyons
              dobedobedoh Andrew Lyons
              Rossiani Wijaya Rossiani Wijaya
              Sam Hemelryk Sam Hemelryk
              Petr Skoda Petr Skoda
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Sujith Haridasan
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                9/Jan/12