#12 — Enable Upload for More Than Folderish Objects

State Resolved
Version: 0.10
Area User interface
Issue type Patch
Severity Low
Submitted by Mark Jones
Submitted on Jun 13, 2010
Responsible Harald Friessnegger
Target release:
Thank you for making this upload component. Well done. I followed your integration instructions (add to policy product: profiles/default/actions.xml).

I modified it as follows:
url_expr = ${folder_url} instead of ${object_url}
and removed "and plone_context_state.is_folderish()"
from the available_expr property. This allows it to work throughout the site.

We have many 'default pages' so the object being viewed isn't often a folderish obect. This made the Upload tab disappear undesirably.
Steps to reproduce:
remove 'and plone_context_state.is_folderish()' from available_expr
change ${object_url} to ${folder_url} in url_expr
Added by Ramon Bartl on Jun 13, 2010 09:20 PM
Issue state: UnconfirmedConfirmed
Severity: MediumLow
Responsible manager: (UNASSIGNED)ramonski
Hi Mark,

thanks a lot for this hint!:)

I'll change this in the Documentation for the next Release!

Greets, Ramonski
Added by Mark Jones on Jun 13, 2010 09:26 PM
I've actually removed the available_expr entirely (allowing uploads to unpublished folders) and it seems to work just great.

Thank you for making it!

//ark
Added by Harald Friessnegger on Jun 22, 2010 11:16 AM
Issue state: ConfirmedResolved
the documentation has been updated to support uploads on folders having a default page (http://dev.plone.org/collective/changeset/119563)

we still show the action only for public folders. (this is discussed further on in #10)
Added by Ramon Bartl on Jun 22, 2010 11:26 AM
Issue state: ResolvedConfirmed
Responsible manager: ramonskifrisi
Hi frisi,

thanks for doing the changes:) I think we can remove the available expression completely as described in #12. This should work just fine.

I'll reopen this issue that we not forget to remove L 40 in collective.uploadify/trunk/README.txt
Added by Harald Friessnegger on Jun 22, 2010 12:20 PM
Issue state: ConfirmedResolved
the condition for the wf-state has been removed too:
http://dev.plone.org/collective/changeset/119565

No responses can be added.