issue/133 Added onDownloadFolder

[oliverfoster]

resolve #133

Added

• getFiles function to encapsulate the files part of the render function
• getSelectedItems function to encapsulate the selectedItems part of the render function
• canDownloadFolder and onDownloadFolder functions and their configs

Changed

• handleActionBarDownloadClick function to check if the selection is a folder using new getFiles and getSelectedItems functions
• render function uses the new getFiles and getSelectedItems functions
• Switched to using utils.isFolder rather than a size === 0 comparison for folders

[oliverfoster]

@jarekwg Is this feasible? Have you managed to take a look?

[jarekwg]

Hi @oliverfoster, sorry for the wait. I'll get one of my colleagues to take a look soon.

[oliverfoster]

No worries, happy to amend if they've suggestions.

[jlo-1]

Looks pretty good just one comment to address before merge

[venky0195]

Hi, @jarekwg @oliverfoster @olivermartinfoster onDownloadFolder not working.
I'm unable to download a folder. However, download files are working.
<FileBrowser
onDownloadFile={arrayOfSelectedItems => { console.log('downloadFile', arrayOfSelectedItems) }}
onDownloadFolder={(arrayOfSelectedFolders) => { console.log('downloadFolder', arrayOfSelectedFolders) }} />
I'm getting console for onDownloadFile, but no download button when I click on folder.
Thanks in advance

[venky0195]

Also, there is only below code in browser.js in node modules.
actionRenderer: ActionRenderer,
onCreateFolder, onRenameFile, onRenameFolder,
onDeleteFile, onDeleteFolder, onDownloadFile

There is no onDownloadFolder anywhere in the browser.js

[oliverfoster]

These changes haven't been released yet. You'd have to use the master branch. Are you sure you have the code from this pr?