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

grade export of user profile fields fails unless they are all lowercase

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Log in as admin
      2. Create a course and enrol a user as a student
      3. Create several custom profile fields, with and without upper case letters in the short names
      4. Add these fields to $CFG->grade_export_customprofilefields
        • Site administration > Grades > General Settings
      5. Log in as the student
      6. fill the values for the custom profile fields
      7. Log in as admin / teacher of the course
      8. Export gradebook in different formats and make sure the custom profile fields are populated correctly
      Show
      Log in as admin Create a course and enrol a user as a student Create several custom profile fields , with and without upper case letters in the short names Add these fields to $CFG->grade_export_customprofilefields Site administration > Grades > General Settings Log in as the student fill the values for the custom profile fields Log in as admin / teacher of the course Export gradebook in different formats and make sure the custom profile fields are populated correctly
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_32_STABLE, MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Pull Master Branch:
      wip-MDL-38129-master
    • Sprint:
      3.4 Sprint 3

      Description

      Steps to reproduce

      1. add a new user profile field (e.g. textfield) with the name "Studiengang".
      2. append "Studiengang" to grade_export_customprofilefields
      3. fill this new field in some user profile
      4. export grades via ODS or XLS

      Result: The column "Studiengang" will be either empty or "0".

      Changing the name to "studiengang" (all lowercase) worksaround the bug.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              marina Marina Glancy
              Reporter:
              forouher Dariush Forouher
              Peer reviewer:
              Ankit Agarwal
              Integrator:
              Andrew Nicols
              Tester:
              Ryan Wyllie
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              3 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Sep/17