Moodle
  1. Moodle
  2. MDL-43773

Name column headings in Choice text file export reversed

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.4.8, 2.5.4, 2.6.1, 2.7
        • Fix Version/s: 2.5.5, 2.6.2
        • Component/s: Choice
        • Labels:

          Description

          The firstname and lastname column headings in the text file export for the Choice activity are the wrong way around, and don't match the data. Lastname should come first, as in the other two export formats.

            Gliffy Diagrams

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              CiBoT added a comment -

              Results for MDL-43773

              Show
              CiBoT added a comment - Results for MDL-43773 Remote repository: https://github.com/lucisgit/moodle Remote branch MDL-43773 -master to be integrated into upstream master Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/672 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/672/artifact/work/smurf.html
              Hide
              Permalink
              Dan Marsden added a comment -

              looks ok to me - feel free to submit to integration when ready... I wonder how long that has been a problem!

              Show
              Dan Marsden added a comment - looks ok to me - feel free to submit to integration when ready... I wonder how long that has been a problem!
              Hide
              Permalink
              Michael de Raadt added a comment -

              Thanks for working on this, Tony. Please continue.

              It would be good if your fix could be backported to older supported branches also.

              Show
              Michael de Raadt added a comment - Thanks for working on this, Tony. Please continue. It would be good if your fix could be backported to older supported branches also.
              Hide
              Permalink
              Tony Butler added a comment -

              Thanks Michael. I've added branches for 2.4, 2.5 and 2.6.

              Show
              Tony Butler added a comment - Thanks Michael. I've added branches for 2.4, 2.5 and 2.6.
              Hide
              Permalink
              Tony Butler added a comment -

              Thanks Dan, but I can't find an option to submit for integration. Is this something I can do?

              Show
              Tony Butler added a comment - Thanks Dan, but I can't find an option to submit for integration. Is this something I can do?
              Hide
              Permalink
              CiBoT added a comment -

              Results for MDL-43773

              Show
              CiBoT added a comment - Results for MDL-43773 Remote repository: https://github.com/lucisgit/moodle Remote branch MDL-43773 -MOODLE_24_STABLE to be integrated into upstream MOODLE_24_STABLE Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/690 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/690/artifact/work/smurf.html Remote branch MDL-43773 -MOODLE_25_STABLE to be integrated into upstream MOODLE_25_STABLE Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/691 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/691/artifact/work/smurf.html Remote branch MDL-43773 -MOODLE_26_STABLE to be integrated into upstream MOODLE_26_STABLE Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/692 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/692/artifact/work/smurf.html Remote branch MDL-43773 -master to be integrated into upstream master Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/693 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/693/artifact/work/smurf.html
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              Permalink
              CiBoT added a comment -

              Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!

              Show
              CiBoT added a comment - Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              hehe, nice (since ever) one. While I'm going to integrate this, really we should move the report to provide the information adjusted to user/site preferences and configuration (instead of current harcoded last+first). Please consider creating another issue for that.

              Proceeding with this...

              Show
              Eloy Lafuente (stronk7) added a comment - hehe, nice (since ever) one. While I'm going to integrate this, really we should move the report to provide the information adjusted to user/site preferences and configuration (instead of current harcoded last+first). Please consider creating another issue for that. Proceeding with this...
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              Integrated (25, 26 & master), thanks!

              Note 24_STABLE is out of support and only security fixes land there.

              Ciao

              Show
              Eloy Lafuente (stronk7) added a comment - Integrated (25, 26 & master), thanks! Note 24_STABLE is out of support and only security fixes land there. Ciao
              Hide
              Permalink
              Andrew Davis added a comment -

              Works as described. Passing.

              Show
              Andrew Davis added a comment - Works as described. Passing.
              Hide
              Permalink
              Marina Glancy added a comment -

              Thanks for your hard work. Your code has now become a part of Moodle!

              Show
              Marina Glancy added a comment - Thanks for your hard work. Your code has now become a part of Moodle!

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: