Elias Torres wrote:

Allen Gilliland wrote:

Elias Torres wrote:
Allen Gilliland wrote:
[snip]

To do this I am thinking that we can basically leave the FileManager the
way it is now, except that where methods currently accept a path we will
just modify the implementation to make sure that path is only ever a
single level deep.

correct.

On the UI I think I'll change the "create subdirectory" part to only be
shown when the person is at the root of their uploads, otherwise it
won't be there.  That should provide the necessary functionality right?

yup

So, I don't mind doing it, as long as everyone else agrees.  Can I get a
show of hands?

I am +0

I am +1

Note: If I try the photo well approach in that feature I'm experimenting
with I'll still need a getAllFiles() or maybe I'll do some navigation
based on buckets/single-folder only. Therefore, your changes are simply
a simplification to what we already have and nothing more. If we have a
big need in the future, we can open up n-deep functionality.

hmmm, I didn't think you would need that method with these changes. If you call getFiles(weblog, "bucket") then that will give you all files in a bucket. The only time when you would be getting more files than the ones in a single bucket is if you wanted the files at the root, *plus* all files all in buckets. i.e. you wanted all files for the weblog.

I thought you were trying to avoid that for performance reasons.

-- Allen



Thanks.

-Elias

Reply via email to