Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide
      1. Log in as the administrator
      2. Go to [Settings->Site administration->Reports->Logs]
      3. Select some logs (All participants, All days, All activities, All actions) and select 'Download in text format'. Click 'Get these logs'.
      4. Open the file in a text editor or spreadsheet application.

      [Test]
      Make sure that the data is in the appropriate format and that no errors have been generated.

      Show
      Log in as the administrator Go to [Settings->Site administration->Reports->Logs] Select some logs (All participants, All days, All activities, All actions) and select 'Download in text format'. Click 'Get these logs'. Open the file in a text editor or spreadsheet application. [Test] Make sure that the data is in the appropriate format and that no errors have been generated.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-34429-master

      Description

      Log reports can be downloaded from admin>reports>logs

      Use the new class in lib/csvlib.class.php to export data into a csv, tsv or other delimited file.
      Examples can be found in:

      admin/user/usre_bulk_download.php
      grade/export/txt/grade_export_txt.php
      lib/tablelib.php
      mod/data/lib.php

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              ankit_frenz Ankit Agarwal added a comment -

              Hi Adrian,
              Is the change to $filename intentional ?

              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - Hi Adrian, Is the change to $filename intentional ? Thanks
              Hide
              ankit_frenz Ankit Agarwal added a comment - - edited

              Adrian explained that csvexporter class was adding the same date information to the filename. Hence the change was needed.

              All looks good to me.

              +1 for integration.
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - - edited Adrian explained that csvexporter class was adding the same date information to the filename. Hence the change was needed. All looks good to me. +1 for integration. Thanks
              Hide
              abgreeve Adrian Greeve added a comment -

              Thanks for the peer review Ankit and the discussion about titles in the csv file.

              Show
              abgreeve Adrian Greeve added a comment - Thanks for the peer review Ankit and the discussion about titles in the csv file.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday.

              This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody.

              This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P

              Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday. This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody. This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao
              Hide
              stronk7 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
              stronk7 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
              nebgor Aparup Banerjee added a comment -

              Thanks, thats been integrated into master only.

              Show
              nebgor Aparup Banerjee added a comment - Thanks, thats been integrated into master only.
              Hide
              phalacee Jason Fowler added a comment -

              this works fine - would be nice to be able to export as a CSV vs TSV though

              Show
              phalacee Jason Fowler added a comment - this works fine - would be nice to be able to export as a CSV vs TSV though
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              I'm so proud...of you, many thanks!

              http://youtu.be/n64CdfDRnZY

              Closing as fixed, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - I'm so proud...of you, many thanks! http://youtu.be/n64CdfDRnZY Closing as fixed, ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    3/Dec/12