sonata-project / SonataDoctrinePhpcrAdminBundle

[Abandoned] Symfony Sonata / Integrate Doctrine PHPCR into the SonataAdminBundle
https://sonata-project.org
MIT License
34 stars 92 forks source link

Audit / Versionning #12

Closed rande closed 5 years ago

rande commented 12 years ago

Hello,

I just add a support for Audit/Versionning, I have started to add interfaces in the AdminBundle and the implementation in DoctrineORMAdminBundle. I am not sure if the interface can match the PHPCR requirements, can you please have a look the the following comments and provides any comments you might have.

Thanks.

dbu commented 12 years ago

we plan to add versioning to phpcr-odm soon, so this comes really at the right time. the planned interface will look something like this: https://github.com/doctrine/phpcr-odm/wiki/Versioning

i glanced over your code and it looks like we can do this. things i wonder:

rande commented 12 years ago
dbu commented 12 years ago
rande commented 12 years ago

I guess, there is a difference between audit and revision ... for now the AdminBundle try to solve the Audit one : tracking every changes on an object.

@beberlei : maybe the AuditEntity can be altered to match the Audit (Quality Control) and the Revision (User Content) requirements.

dbu commented 12 years ago

versioning support is now available in phpcr-odm https://github.com/doctrine/phpcr-odm/pull/96 (when using jackalope-jackrabbit, that is ;-)

dbu commented 12 years ago

is #27 a duplicate of this issue?

INHack20 commented 9 years ago

+1 +1 +1

will support versioning of documents? symfony an alternative to version documents?

dbu commented 9 years ago

@INHack20 doctrine phpcr-odm supports versioning. but nobody yet built support for the functionality into sonata admin. maybe this can simply be an admin extension, but probably direct support in sonata admin would allow a better user experience.

versioning is explained at http://doctrine-phpcr-odm.readthedocs.org/en/latest/reference/versioning.html

core23 commented 5 years ago

This issue is very old. Can you please check this against the latest stable / master version?

Feel free to reopen, if the error still exists.

A PR would be welcome if you reopen this issue.