Ticket #9561 (closed PLIP: wontfix)
Integration of ARFilePreview in Plone core (preview of office and other binary files)
Reported by: | dukebody | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | Future |
Component: | General | Version: | |
Keywords: | Cc: | grahamperrin@… |
Description
http://plone.org/products/plone/roadmap/199
Make files previewable in Plone
Proposed by
Thierry Benita
Seconded by
Jean-Nicolas Bès
Proposal type
User interface
Repository branch
ARFilePreview-2.3-plone.transforms
State
being-discussed
Definitions
Motivation
Plone doesn't show a preview of binary (office?) files. This proposal is about adding an automatic/configurable preview on ATFile content type and subclasses.
Assumptions
We assume that a content transformation engine is available (PortalTransforms or plone.transforms).
Proposal
- make it a zope3 style product (ie: an egg)
- define a new viewlet
- make it use plone.transforms by default and PloneTransforms as a fallback
- make it store the html preview and subobjects as blob or file objects instead of strings
- remove the AR (atReal) prefixes
- add some translations
- make it uninstalled by default (just like Marshall or Iterate)
Implementation
The preview and subobjects are currently stored as objects annotations.
The html transform is triggered by an IObjectEdited event.
Deliverables
- plone.preview
o documentation o localization o unit tests
Risks
The main risk is that uploading files could take a little more time because the transformation is triggered by a synchronous event.
We could work around this issue by :
- limiting the maximum size of previewable files
- triggering the preview manually (content action)
- (HARD) implement an asynchronous preview engine that queues previewing jobs and process them later (later => in another thread/request)
The overhead of registering the configure.zcml but disabling the feature will be an event subscriber triggered to do nothing but checking the activation setting on each modifications of an object.
Progress log
The product is working but needs to be polished/reworked to be as pretty as the other Plone products.
Participants
Jean-Nicolas Bès (drjnut) atReal
Thierry Benita (tbenita) atReal
Change History
comment:1 in reply to: ↑ description ; follow-up: ↓ 6 Changed 6 years ago by grahamperrin
- Cc grahamperrin@… added
comment:2 Changed 6 years ago by grahamperrin
Considerations might include:
Is the notion to use ARFilePreview with
AROfficeTransforms
or with
Products.PortalTransforms?
In any case: I recall a discussion long ago (maybe in < irc://irc.freenode.net/#plone>) suggesting that whilst ARFilePreview and/or AROfficeTransforms are fine add-ons, a Plone core approach was unlikely. That discussion possibly based on the (then) difficulty that was sometimes experienced with requirements for AROfficeTransforms (outlined at < http://pypi.python.org/pypi/Products.AROfficeTransforms> but not at < http://plone.org/products/arofficetransforms>).
In either case — core or add-on — I'm very much in favour of progressing things in this area.
comment:3 Changed 6 years ago by grahamperrin
This PLIP is mentioned at ticket:8755#comment:40.
comment:4 in reply to: ↑ description Changed 6 years ago by grahamperrin
Replying to dukebody:
Repository branch
ARFilePreview-2.3-plone.transforms
< http://plone.org/products/arfilepreview/roadmap> also mentions that branch but
< https://svn.plone.org/svn/collective/ARFilePreview/branches/ARFilePreview-2.3-plone.transforms> is 404, not found.
comment:5 Changed 6 years ago by grahamperrin
Cross reference
< http://manage.plone.org/products/arfilepreview/issues/34>,
preview ATBlob files (plone.app.blob with Plone 3.x)
comment:6 in reply to: ↑ 1 Changed 6 years ago by witsch
Replying to grahamperrin:
Roadmap items such as that one are no longer found. Any idea where they have moved to?
they have been deleted by limi. i've complained several times now, but he's reluctant to restore all of them, even though some people felt the same in the thread i brought up on plone-dev. please feel free to add your +1 — perhaps at some point he'll reconsider... :(
comment:7 Changed 5 years ago by rossp
- Status changed from new to closed
- Resolution set to wontfix
PLEASE READ THIS AND RE-OPEN VALID PLIPS!
As we launch the new PLIP process we'd like to see which PLIPs:
- are still appropriate/needed
- still have owners/proposers/champions
- still have available implementers
If this PLIP should still be considered for future releases of Plone please do re-open this ticket and assign an appropriate milestone. If it should be considered for the next release of Plone, use the 4.2 milestone. Also be sure to update the PLIP description, requester, owner, etc. and include a comment detailing recent progress and new plans. We will use all these details in the new continuous PLIP process.
Replying to dukebody:
Roadmap items such as that one are no longer found. Any idea where they have moved to?
< http://plone.org/search?SearchableText=Make+files+previewable+in+Plone> draws a blank.
Notes, references etc.
< http://plone.org/products/arfilepreview>
< http://pypi.python.org/pypi/Products.ARFilePreview>
A few months ago I made some very minor edits to a version of ARFilePreview. I don't have a full record of the edits but I'll attach maybe a couple of files.