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

All forms should at least have one fieldset

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: None
    • Component/s: Competencies
    • Labels:
    • Testing Instructions:
      Hide

      Create or edit a competency framework. Check General header appears.
      Add or edit a competency in a framework. Check General header appears.
      Create or edit a template framework. Check General header appears.
      In the 'List of learning plan templates' click on the number under 'Cohorts' column.
      In the 'Cohort synced to this template' screen, check General header appears.
      In the 'List of learning plan templates' click on the number under 'User plans' column.
      In the 'User plans' screen, check General header appears.
      Create or edit a user learning plan. Check General header appears.

      Show
      Create or edit a competency framework. Check General header appears. Add or edit a competency in a framework. Check General header appears. Create or edit a template framework. Check General header appears. In the 'List of learning plan templates' click on the number under 'Cohorts' column. In the 'Cohort synced to this template' screen, check General header appears. In the 'List of learning plan templates' click on the number under 'User plans' column. In the 'User plans' screen, check General header appears. Create or edit a user learning plan. Check General header appears.
    • Affected Branches:
      MOODLE_31_STABLE
    • Pull Master Branch:
      MDL-53476_master
    • Sprint:
      CBE Sprint 7

      Description

      Some forms do not declare the first fieldset, they should. E.g. (editing a plan)

      // This is missing:
      $mform->addElement('header', 'generalhdr', get_string('general'));
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                sergeg55 Serge Gauthier
                Reporter:
                fred Frédéric Massart
                Peer reviewer:
                Frédéric Massart
                Participants:
                Component watchers:
                Damyon Wiese, Jean-Philippe Gaudreau, Steve Massicotte, Serge Gauthier, Issam Taboubi, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: