search

Raptus / raptus.article.core

Provides a configurable article content type (replaces the default Page content type).
0 stars 4 forks source link

Support Versioning via CMFEditions and diffs #8

Open frisi opened 10 years ago

frisi commented 10 years ago

currently we can't fully enable versioning (including comparing revisions) for raptus.article since it uses Annotationstorage. (see ticket https://dev.plone.org/ticket/11887)

some of our clients would love to have the same versioning features for raptus.article as they have for plone's stock document type.

i think this would required to either fix ticket/11887 or to migrate all raptus.article fields (or at least those that we need to be available in the revision comparision view) to attributestorage.

before i start to work on this, i'd love to hear your feedback on this. especially if you're pro or con such a change and if you see and drawbacks

skaeser commented 10 years ago

From my point of view, fixing the issue in CMFEditions would be the way to go. Should not be such a big issue, or what do you think?

frisi commented 10 years ago

basically you are right. fixing th problem instead of working around it should always be considered first.

i've been talking with @alecpm about https://dev.plone.org/ticket/11887 years ago.

iirc the main problem is, that for folderish types, annotationstorage is completely ignored since it contains information about the items contained in the folder (which cmfediton does not or can not take care of) at that time, alec - being the author of the package - said that this will be a not trivial thing to do. i did not yet had a look at the code to see if i can understand what's going on and where to hook in.

maybe we should try to get some people familiar with cmfeditions to take a look at this.

for raptus.article it might be more interesting to migrate to dexterity. dunno if versioning and diffs do work for dexterity types at all.