Ticket #9561 (closed PLIP: wontfix)

Opened 6 years ago

Last modified 5 years ago

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

Replying to dukebody:

 http://plone.org/products/plone/roadmap/199

Make files previewable in Plone

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.

comment:2 Changed 6 years ago by grahamperrin

Considerations might include:

  • #7678 pdftohtml binary installer for Mac OS X
  • #7679 wv (wvhtml binary) installer for Mac OS X

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.

comment:8 Changed 4 years ago by davisagli

  • Component changed from Infrastructure to General
Note: See TracTickets for help on using tickets.