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

stored_file->get_author returns the wrong information

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1, 2.2
    • Fix Version/s: 2.0.6, 2.1.3
    • Component/s: Files API, Unit tests
    • Labels:
      None

      Description

      From the code (/moodle/lib/filestorage/stored_file.php):

      454: public function get_author() {
      455: return $this->file_record->license;
      456: }

      NOTE: I selected Unit Tests as one of the components because CLEARLY some are missing/broken to let something as obviously incorrect as this slip through.....

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              skodak Petr Skoda added a comment -

              Thanks a lot for the report.

              Show
              skodak Petr Skoda added a comment - Thanks a lot for the report.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated, thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              I've created MDL-29910 about to add coverage for all this stuff. Passing.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - I've created MDL-29910 about to add coverage for all this stuff. Passing.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Done, your delicious hacks have been sent upstream, many thanks!

              Closing as fixed, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Done, your delicious hacks have been sent upstream, many thanks! Closing as fixed, ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    28/Nov/11