Daniel,

While they seem to tackle the same problem, I think our extensions have very
different philosophies and can thus probably live separate lives.  Here's
some of the design decisions I made:

1) Uploaded files should live in the filesystem where the web server can
serve them efficiently to the client.
2) Although logically it would be nice to have uploaded files live in a
directory structure related to the page, page urls/slugs can change more
easily than files can be moved.  Thus it's probably better that the storage
location of files be orthogonal to the page layout.
3) Pages should be able to easily access the assets/files belonging to their
parent(s).  So, in-page tags that reference an asset also search through the
parent pages until it is found.
4) acts_as_attachment has worked well for me in the past, so I'll use that
for handling the uploads and file management.
5) The UI should mimic John's original comps as closely as possible.
6) John liked the way Basecamp allowed you to do multiple uploads (without a
popup), so let's do that similarly.

I hope this reveals to you why I did certain things and not other things.
I'll continue to look through your code to see if there's anything I can
learn that might enhance page_attachments.

Sean

On 1/22/07, Daniel Sheppard <[EMAIL PROTECTED]> wrote:

> My extension is a little rougher around the edges than yours at the
> moment, but I'm planning on spending Tuesday night fixing that up (and
> removing the dependence on my other admin_parts plugin) -

Ok. Scratch that. Just re-jigged it a lot. Pulled in a couple of things
from your design and moves to consistently using the word 'Attachment'.
Hence this is now the PartAttachments extension.

http://soxbox.no-ip.org/radiant/svn/extensions/part_assets/

So you'll need to rename that dir to part_attachments on the way down.
Still depends on the admin_parts extension (same repository), but I've
removed the obnoxious bits of the admin_parts extension, so it should
just quietly stay out of the way.


Dan.
_______________________________________________
Radiant mailing list
Post:   [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

_______________________________________________
Radiant mailing list
Post:   [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

Reply via email to