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

Dropbox does not return file as link

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.2
    • Fix Version/s: 2.3.3
    • Component/s: Repositories
    • Labels:
    • Testing Instructions:
      Hide
      1. Enable and setup Dropbox repository
      2. Add at least one image to your dropbox repository
      3. Open any page with textarea field (i.e. course editing page or add any resource with description)
      4. Insert an image from Dropbox and link to it directly
      5. Make sure image is inserted and there are no errors
      Show
      Enable and setup Dropbox repository Add at least one image to your dropbox repository Open any page with textarea field (i.e. course editing page or add any resource with description) Insert an image from Dropbox and link to it directly Make sure image is inserted and there are no errors
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      wip-MDL-35542-master

      Description

      Invalid JSON string
       
      Notice: Undefined property: stdClass::$url in /var/www/repositories/master/moodle/repository/dropbox/lib.php on line 509
      {"file":"pink_elephant.jpg","type":"link","url":""}

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski 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
            poltawski Dan Poltawski 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 -

            Thanks Marina, i've integrated this now.

            Show
            poltawski Dan Poltawski added a comment - Thanks Marina, i've integrated this now.
            Hide
            fred Frédéric Massart added a comment -

            Test passed. Thanks!

            Show
            fred Frédéric Massart added a comment - Test passed. Thanks!
            Hide
            poltawski Dan Poltawski added a comment -

            Congratulations, you've done it!

            Nf n erjneq sbe fhpprfshy vagrtengvba vagb guvf jrrxf eryrnfr, V pna abj qvfpybfr gb lbh gur rkvfgnapr bs shapgvba fge_ebg13(), gb tb va lbhe gbbyxvg nybat jvgu uggc://cuc.arg/znahny/ra/shapgvba.tmtrgff.cuc

            Cyrnfr qb abg nyybj guvf vasbezngvba gb cnff shegure.

            Show
            poltawski Dan Poltawski added a comment - Congratulations, you've done it! Nf n erjneq sbe fhpprfshy vagrtengvba vagb guvf jrrxf eryrnfr, V pna abj qvfpybfr gb lbh gur rkvfgnapr bs shapgvba fge_ebg13(), gb tb va lbhe gbbyxvg nybat jvgu uggc://cuc.arg/znahny/ra/shapgvba.tmtrgff.cuc Cyrnfr qb abg nyybj guvf vasbezngvba gb cnff shegure.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Nov/12