Hi, I'd like to bulk upload files into a specific folder. At this moment, all files are upload ed into /sites/default/files folder, which makes the folder really untidy when you have to deal with lots of files. To better organize files, I think that could be useful to introduce the option of select a folder where the files must be copied.
To address this issue, I developed a patch. Please test my patch and tell me what do you think about (it is my first contribution).

Best regards.
Loris.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lorisbel created an issue. See original summary.

lorisbel’s picture

lorisbel’s picture

joseph.olstad’s picture

Thanks for the patch, this is functionality that is helpful.

parser says corrupt patch at line 84.

Can you try cloning the 7.x-2.x branch of media and then make the patch from that?

Thanks for your time.

joseph.olstad’s picture

to clone from 7.x-2.x branch do this:

git clone --branch 7.x-2.x https://git.drupal.org/project/media.git
cd media
lorisbel’s picture

lorisbel’s picture

joseph.olstad’s picture

Nice work thanks for fixing the patch.

joseph.olstad’s picture

Version: 7.x-2.11 » 7.x-2.x-dev
Status: Patch (to be ported) » Fixed

Did some minor cleanup, I cleaned up indenting (two spaces per indent, not 6, not 4), whitespace, and fixed a couple code standards compliance issues with the concatenation space between the dot.
else should also be

}
else {

Thanks.

Status: Fixed » Closed (fixed)

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