It is clear from some of the other open issues that the current DropBox PHP class that is in the module (the DropboxUploader by Jaka Jančar - https://github.com/jakajancar/DropboxUploader/) is no longer meeting this module's needs. To that end, three of us decided to port the module to utilize the Dropbox API PHP class by "BenTheDesigner" (https://github.com/BenTheDesigner/Dropbox).

I'll include the patch in the first comment below, so far, it looks like it is working, but obviously further testing is needed. There are some tasks that we still need to figure out though, including:

  1. Add some help text to the DropBox "add destination" settings page.
  2. Figure out if we're handing the 32-byte encryption key stuff correctly.
  3. Figure out if we're doing the autoloading of the DropBox API PHP classes the "Drupal" way.
  4. Create README.txt file
  5. Run the module through Coder.

If this patch does get accepted, please attribute it to not only myself (ultimike: http://drupal.org/user/51132) but also:

The three of us attacked this issue as a team (basically pair-programming the entire day, but with three of us. What's that called? Trio-programming? Tri-programming?)

Thanks,
-mike

Comments

ultimike’s picture

...and the patch. Enjoy!

Be sure to download the lastest version of the new PHP API from https://github.com/BenTheDesigner/Dropbox, then rename the top-level directory to "dropbox" and place it in your "libraries" folder. The end result should be something like this:

sites/all/libraries/dropbox/Dropbox/API.php

(obviously, this has to be documented in the module as well)

Thanks,
-mike

vegastech’s picture

I'm new to using that patch command in Linux.
Accordingly, I'm not sure which file needs to be patched.
If you can be more specific on how to install this patch, it would be greatly helpful.
Thanks

wundo’s picture

Status: Active » Fixed

Committed!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

okeedoak’s picture

The installer file says the API.php file should be at 'sites/all/libraries/Dropbox-master/Dropbox/API.php' Your comment says: 'sites/all/libraries/dropbox/Dropbox/API.php'

Which is it?

okeedoak’s picture

Status: Closed (fixed) » Active
ultimike’s picture

The comment above is correct (sites/all/libraries/dropbox/Dropbox/API.php), the backup_migrate_dropbox.install file needs to be updated.

Thanks,
-mike

brycefisherfleig’s picture

Status: Active » Closed (fixed)

I realize this comment is a bit dated, but I've submitted a patch to solve the problem you've stumbled onto:
https://drupal.org/node/2060517#comment-7737103

brycefisherfleig’s picture

Issue summary: View changes

Added two additional "to do" items.