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

Flickr Public repository "Link to the file directly" option fails

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3, 2.4
    • Fix Version/s: 2.3.5, 2.4.2
    • Component/s: Repositories
    • Labels:
      None

      Description

      Attempting to use the "Link to the file directly" option in the Flickr public repository generates this fatal error:

      Call to undefined method repository_flickr_public::build_photo_id()

      Which causes an invalid JSON response, which prevents the user from being able to embed the image.

      I am assuming this is supposed to be a call to repository_flickr_public::build_photo_url(). My patch changes this and the feature seems to work as intended now.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            fred Frédéric Massart added a comment -

            Thanks for your patch Ashley. I've added some testing instructions and am pushing this straight to integration as I'm acting as a peer reviewer here.

            Cheers,
            Fred

            Show
            fred Frédéric Massart added a comment - Thanks for your patch Ashley. I've added some testing instructions and am pushing this straight to integration as I'm acting as a peer reviewer here. Cheers, Fred
            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
            poltawski Dan Poltawski added a comment -

            Integrated to 24, 23 and master.

            Thanks Ashley.

            Show
            poltawski Dan Poltawski added a comment - Integrated to 24, 23 and master. Thanks Ashley.
            Hide
            phalacee Jason Fowler added a comment -

            Great work Fred, now all the pictures of monkeys playing the drums show up in my moodle correctly!

            Show
            phalacee Jason Fowler added a comment - Great work Fred, now all the pictures of monkeys playing the drums show up in my moodle correctly!
            Hide
            poltawski Dan Poltawski added a comment -

            Hurray! We did it! Thanks to all the reporters, testers, user and watchers for a bumper week of Moodling!

            Show
            poltawski Dan Poltawski added a comment - Hurray! We did it! Thanks to all the reporters, testers, user and watchers for a bumper week of Moodling!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Mar/13