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: