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

User Description Field breaks in 2.8.2

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Goto Site administration -> Plugins -> Authentication -> manual
      2. Lock description field
      3. Go to user X profile
      4. Edit description field
      5. Save user X profile
      6. Login in with user X
      7. Goto Edit user profile
      8. Save user profile
      Show
      Goto Site administration -> Plugins -> Authentication -> manual Lock description field Go to user X profile Edit description field Save user X profile Login in with user X Goto Edit user profile Save user profile
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull Master Branch:
      wip-MDL-48978-master

      Description

      Creating this issue is a bit of a mystery for me because it doesn't exist in the moodle QA site, however after going through a diff of our update from 2.8.1+ and 2.8.2, the problem seems to unequivocally be in user/edit_form with a fix from MDL-37704.

      The error that I recieve when i try to update student user preferences

      I was able to reverse the problem by reverting to the 2.8.1+ version of this code in user/edit_form.php:

      if ($field === 'description')

      { + // Hard coded hack for description field. See MDL-37704 for details. + $formfield = 'description_editor'; + }

      else

      { + $formfield = $field; + }

      + if (!$mform->elementExists($formfield))

      { continue; }

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              marina Marina Glancy
              Reporter:
              jjbass jasper boyd
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Eloy Lafuente (stronk7)
              Participants:
              Component watchers:
              Barbara Ramiro, Bas Brands
              Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                2/Feb/15