search

WBCE / WBCE_CMS

Core package of WBCE CMS. This package includes the core and the default addons. Visit https://wbce.org (DE) or https://wbce-cms.org (EN) to learn more or to join the WBCE CMS community.
https://wbce-cms.org
GNU General Public License v2.0
32 stars 22 forks source link

Add a history function to pages . #113

Closed NorHei closed 6 years ago

NorHei commented 8 years ago

What the title says , any ideas welcome .

digitalbricks commented 8 years ago

Are you meaning a sort of versions or revisions? At least for WYSIWYG-sections there was (as far i know) a working implementation of versions in "extendedWYSIWYG" by Ralf Hertsch: https://github.com/phpManufaktur/extendedWYSIWYG

webbird commented 8 years ago

I have to say that extendedWYSIWYG seems to have some caveats, especially if used with LEPTON. (Umlauts, quotes.) Apart of that, it should be a good basis.

NorHei commented 8 years ago

Basically i would prefer full drafts and history for the complete page. And yess that would make problems whith modules like topics , procalendar or foldergallery as they do not create "real pages "

digitalbricks commented 8 years ago

As in WB(CE) every module brings its own save routine, it's probably hard to find a system wide solution for this. As far as I understand, the underlying CM system isn't actually involved in saving content from a page section – that's the job of the save.php in each module. And as long the system isn't noticed on save event, it is likely unable to track the changes.

In MODX Revolution for example there are several system events that are fired on several actions. So there is OnDocFormSave event you can hook on and do some actions. WB is lacking such event listeners and so I think i will get paintfull to get this job done. But actually i am not so deep involved in the WB(CE) code, so maybe there is a "hidden secret"?

NorHei commented 8 years ago

extendedWYSIWYG is a module issue not a core functionality. Feel Free to simply build one . If you need some extra functionality in the Core for this , have a word whith me :-)

NorHei commented 8 years ago

Sorry overlapping comments.

Yes you got it right, thats something that is really needed. Admintools already have this in 1.2.x , possibly this is only for 1.3.x but its something thats coming on the long term