Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2, 2.1
    • Fix Version/s: 2.0.3
    • Component/s: Repositories
    • Environment:
      Moodle 2.0.2+ (20110427), Moodle 2.1dev (20110427)
    • Testing Instructions:
      Hide

      To test it:

      Enable dropbox repository plugin at Site administration->plugins->repositories->manage repositories
      Go to user private files area, open file picker, choose dropbox, a login button should appear, click it, you should see dropbox login page instead of dropbox warning page about url changed.

      Show
      To test it: Enable dropbox repository plugin at Site administration->plugins->repositories->manage repositories Go to user private files area, open file picker, choose dropbox, a login button should appear, click it, you should see dropbox login page instead of dropbox warning page about url changed.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
      git@github.com:dongsheng/moodle.git
    • Pull Master Branch:
      s10_MDL-27330_dropbox_authorization_url_master
    • Rank:
      17430

      Description

      In Moodle 2.0.2 and Moodle 2.1 you will get an error with the Dropbox authorization URL.
      Moodle links to https://api.dropbox.com/0/oauth/
      The url should be https://www.dropbox.com/0/oauth/
      I don't know if Dropbox changed this url during the last time.

      It's a blocker.

        Issue Links

          Activity

          Hide
          Ralf Krause added a comment -
          Show
          Ralf Krause added a comment - Also look at http://moodle.org/mod/forum/discuss.php?d=173457
          Hide
          Dongsheng Cai added a comment -

          Thanks for reporting.

          Yes, dropbox changed the url, we should fix it next development cycle.

          Show
          Dongsheng Cai added a comment - Thanks for reporting. Yes, dropbox changed the url, we should fix it next development cycle.
          Hide
          Dongsheng Cai added a comment -

          Hi, Ralf, just to confirm, seems dropbox change url back, https://www.dropbox.com/0/oauth/ gives 404 error now.

          Show
          Dongsheng Cai added a comment - Hi, Ralf, just to confirm, seems dropbox change url back, https://www.dropbox.com/0/oauth/ gives 404 error now.
          Hide
          Dongsheng Cai added a comment -

          Please ignore my last comment, http://www.dropbox.com/0/oauth is correct

          Show
          Dongsheng Cai added a comment - Please ignore my last comment, http://www.dropbox.com/0/oauth is correct
          Hide
          Dongsheng Cai added a comment -

          Fixed in my github

          Show
          Dongsheng Cai added a comment - Fixed in my github
          Hide
          Sam Hemelryk added a comment -

          Thanks Dongsheng, due to the release of 2.0.3 planned within the next day or two this was integrated immediately.

          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Thanks Dongsheng, due to the release of 2.0.3 planned within the next day or two this was integrated immediately. Cheers Sam
          Hide
          Dongsheng Cai added a comment -

          Thanks!

          Regards,
          Dongsheng

          Show
          Dongsheng Cai added a comment - Thanks! Regards, Dongsheng
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Uhm, I can connect and browse my files perfectly but when I pick one image I end with:

          Show
          Eloy Lafuente (stronk7) added a comment - Uhm, I can connect and browse my files perfectly but when I pick one image I end with:
          Hide
          Eloy Lafuente (stronk7) added a comment - - edited

          Uhm, I can connect and browse my files perfectly but when I pick one image I end with:

          ERROR: Invalid JSON string
          
          ( ! ) Warning: file_put_contents(eloy_avatar2006_gorila4skype.gif) [function.file-put-contents]: failed to open stream: Permission denied in /repository/dropbox/locallib.php on line 79
          Call Stack
          #TimeMemoryFunctionLocation
          10.0001643696{main}(  )../repository_ajax.php:0
          20.05424360536repository_dropbox->get_file(  )../repository_ajax.php:213
          30.05424360752dropbox->get_file(  )../lib.php:254
          40.64074369200file_put_contents
          (  )../locallib.php:79
          
          
          
          ( ! ) Warning: filesize() [function.filesize]: stat failed for eloy_avatar2006_gorila4skype.gif in /repository/repository_ajax.php on line 220
          Call Stack
          #TimeMemoryFunctionLocation
          10.0001643696{main}(  )../repository_ajax.php:0
          20.64114368248filesize
          (  )../repository_ajax.php:220
          
          {"error":"Can not read file, either file does not exist or there are permission problems","stacktrace":"* line 1048 of \/lib\/filestorage\/file_storage.php: file_exception thrown\n* line 810 of \/lib\/filestorage\/file_storage.php: call to file_storage->add_file_to_pool()\n* line 1003 of \/repository\/lib.php: call to file_storage->create_file_from_pathname()\n* line 243 of \/repository\/repository_ajax.php: call to repository::move_to_filepool()\n","debuginfo":null,"reproductionlink":"http:\/\/127.0.0.1\/~stronk7\/integration\/"}
          

          I never had tried this before so I don't know I'm missing anything in the Dropbox side in order to get this working.

          Show
          Eloy Lafuente (stronk7) added a comment - - edited Uhm, I can connect and browse my files perfectly but when I pick one image I end with: ERROR: Invalid JSON string ( ! ) Warning: file_put_contents(eloy_avatar2006_gorila4skype.gif) [function.file-put-contents]: failed to open stream: Permission denied in /repository/dropbox/locallib.php on line 79 Call Stack #TimeMemoryFunctionLocation 10.0001643696{main}( )../repository_ajax.php:0 20.05424360536repository_dropbox->get_file( )../repository_ajax.php:213 30.05424360752dropbox->get_file( )../lib.php:254 40.64074369200file_put_contents ( )../locallib.php:79 ( ! ) Warning: filesize() [function.filesize]: stat failed for eloy_avatar2006_gorila4skype.gif in /repository/repository_ajax.php on line 220 Call Stack #TimeMemoryFunctionLocation 10.0001643696{main}( )../repository_ajax.php:0 20.64114368248filesize ( )../repository_ajax.php:220 { "error" : "Can not read file, either file does not exist or there are permission problems" , "stacktrace" : "* line 1048 of \/lib\/filestorage\/file_storage.php: file_exception thrown\n* line 810 of \/lib\/filestorage\/file_storage.php: call to file_storage->add_file_to_pool()\n* line 1003 of \/repository\/lib.php: call to file_storage->create_file_from_pathname()\n* line 243 of \/repository\/repository_ajax.php: call to repository::move_to_filepool()\n" , "debuginfo" : null , "reproductionlink" : "http:\/\/127.0.0.1\/~stronk7\/integration\/" } I never had tried this before so I don't know I'm missing anything in the Dropbox side in order to get this working.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          debugging a bit, it seems that the line:

          file_put_contents($saveas, $content);

          is the one casing the 1st warning above, with $saveas being, simply, the name of the file eloy_avatar2006_gorila4skype.gif, without any path at all, so it tries to write the file... into some "incorrect" place.

          I'm sorry but I think this cannot be passed.

          Show
          Eloy Lafuente (stronk7) added a comment - debugging a bit, it seems that the line: file_put_contents($saveas, $content); is the one casing the 1st warning above, with $saveas being, simply, the name of the file eloy_avatar2006_gorila4skype.gif, without any path at all, so it tries to write the file... into some "incorrect" place. I'm sorry but I think this cannot be passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          (changing tester to me, as far as i decided to fail this)

          Show
          Eloy Lafuente (stronk7) added a comment - (changing tester to me, as far as i decided to fail this)
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Just tried it once more with one file without spaces nor underscores (lolo.git) and the same problem persists, tried both locally and @ qa.moodle.net

          Show
          Eloy Lafuente (stronk7) added a comment - Just tried it once more with one file without spaces nor underscores (lolo.git) and the same problem persists, tried both locally and @ qa.moodle.net
          Hide
          Eloy Lafuente (stronk7) added a comment -

          OKi, finally, I'm passing this as far as the connection happens ok and it's possible to browse the files perfectly.

          I've created MDL-27382 (to be fixed ASAP) as followup for the problems found.

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - OKi, finally, I'm passing this as far as the connection happens ok and it's possible to browse the files perfectly. I've created MDL-27382 (to be fixed ASAP) as followup for the problems found. Ciao
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Closing as fixed. Many thanks!

          Note: Don't forget the followup, plz!

          Show
          Eloy Lafuente (stronk7) added a comment - Closing as fixed. Many thanks! Note: Don't forget the followup, plz!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: