Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore the "Use external editor" permission #3190

Open
ale-rt opened this issue Oct 5, 2020 · 6 comments
Open

Restore the "Use external editor" permission #3190

ale-rt opened this issue Oct 5, 2020 · 6 comments

Comments

@ale-rt
Copy link
Member

ale-rt commented Oct 5, 2020

Given the temporary lack of webdav, the permission Use external editor was removed on Python 3:
cda90e8

Should we restore it now that webdav is back or should we move the code in some other place?

@ale-rt
Copy link
Member Author

ale-rt commented Oct 5, 2020

There is also some other code that depends on the external editor:
https://github.com/plone/Products.CMFPlone/blob/master/Products/CMFPlone/tests/testExternalEditorEnabled.py

@mauritsvanrees
Copy link
Member

If I remember correctly, the external editor needs ZServer, so it won't work on Plone 6.
But on 5.2 the permission could be restored yes.

@ale-rt
Copy link
Member Author

ale-rt commented Oct 7, 2020

I think it just depends on webdav, which is now provide by Zope itself.
I am actually working on restoring the external editor in Plone 5.2.2/Python 3 and I am at a good point.
I am using this branch: zopefoundation/Products.ExternalEditor#14

@ale-rt
Copy link
Member Author

ale-rt commented Oct 7, 2020

Just for a confirmation external editor works on 6.0.

Creating a new Plone site with this buildout:

[buildout]
extends = https://raw.githubusercontent.com/plone/buildout.coredev/6.0/buildout.cfg
parts =
    instance
auto-checkout += 
    Products.ExternalEditor
   
[instance]
eggs += 
    Products.ExternalEditor

[sources]
Products.ExternalEditor= git ${remotes:zope}/Products.ExternalEditor.git pushurl=${remotes:zope_push}/Products.ExternalEditor.git branch=python3

adding a file, calling as admin:

and running zopeedit allows me to modify the file on Plone.

Of course a regular user will not have the proper permissions, which were removed by cda90e8.
I am unsure if they should be restored in Plone or added in collective.externaleditor or another package.

@mauritsvanrees
Copy link
Member

Your buildout runs, and the instance starts, so that is good. I see external editor icons in the ZMI and can click on it and it starts opening a .zem file. But I cannot really test it, because the zopeedit program fails for me.

@ale-rt
Copy link
Member Author

ale-rt commented Oct 8, 2020

I forgot to mention that for zopeedit I am using this branch https://github.com/collective/collective.zopeedit/tree/wip with Python2.7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants