add extractArchive bool parameter:

[revolunet]

This is allows to sync non-zip files too :

• make it possible de specify non-zip files in sync method
• backup the destination file in the specified location

(iOS only)

ex:

var sync = ContentSync.sync({
    src: 'http://path/to/video.mp4',
    // unique name
    id: 'episode/42/video.mp4',
    // cachable
    type: 'local',
    // Not a zip
    extractArchive: false
});

Some remarks/questions :

• why not using hash objects to pass parameters from plugin JS api to the native APIs, instead of long list of parameters ? (is there any cordova plugin boilerplate using this?)
• the copyRootApp and copyCordovaAssets should imho be a separate method for this plugin, for example : ContentSync.copyRootApp(destination)

[revolunet]

/cc @denisbabineau

[mwbrooks]

Before merging this, we will want to have an Android and WP implementation. We are trying to be strict with ensuring that this is a cross-platform plugin with parity between Android, iOS, and WP.

Personally, I'm reluctant to continue adding features to this plugin. If we consider moving forward with this, then can we assume that all files that do have a application/zip mime-type are standard files? That way we can avoid the additional option parameter.

[purplecabbage]

Why not just sync files, and if they are zip files unzip them ... not sure why you would need an extra option.

[revolunet]

thanks, agree :)

[imhotep]

@revolunet what about ContentSync.download?

[revolunet]

yes the problem is that this one doesnt check for file existence before downloading :)

[revolunet]

removed the extractArchive parameter in favor of file type detection, based on server content-type and file extension