Behavior:

After using the already existing directory below as my storage path and submitting the configuration, I get the message:

The directory /Library/WebServer/Documents/drupal/sites/default/files/gallery/gallery2/video has been created

and so it has been, in addition to the directory that already exists, just with a different owner:

drwxrwxrwx 5 admin admin 170 Nov 15 11:31 video
drwxrwxr-x 2 www admin 68 Nov 15 12:56 video

That's confusing, to say the least. (If I want to set things up so that several applications can use the same directory, MM is making that difficult.)

Expected behavior:

MM checks to see that directory exists, and does not create it, if it does. If there are permissions errors because the directories aren't set up correctly, write that to a report.

Comments

shunting’s picture

shunting commented

Gah. Stupid. Of course, there's a trailing space in the path I pasted in from the shell.

So, can we convert this to a request that trailing spaces be stripped from the paths, or that input validation asks if that's the intent.

[wanders away, mumbling]

arthurf’s picture

arthurf commented

Is this with mm_dir ? I'd be happy to make this modification

arthurf’s picture

arthurf commented
Status: Active » Needs review

I just committed a rtrim operation on validation which should solve the issue.