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

Deprecate zope.formlib #38

Closed
gforcada opened this issue Oct 5, 2016 · 1 comment
Closed

Deprecate zope.formlib #38

gforcada opened this issue Oct 5, 2016 · 1 comment

Comments

@gforcada
Copy link
Sponsor Member

gforcada commented Oct 5, 2016

A quick search reveals some usage as well: https://github.com/plone/plone.app.vocabularies/search?utf8=%E2%9C%93&q=formlib&type=Code

Is there any porting guide out of it? Should that be replaced by z3c.form?

@jensens
Copy link
Sponsor Member

jensens commented Oct 6, 2016

What we have

From formlib theres only the ISourceQueryView interface used:

I seems this is an old definition for what then was expressed by zope.browser.interfaces.ITerms https://github.com/zopefoundation/zope.browser/blob/master/src/zope/browser/interfaces.py#L88

But even this is kind of deprecated, because the "All source implementations will be deprecated"

<!-- All source implementations will be deprecated -->

Proposed Solution:

  • Really deprecate all source implemenations (let them emit a DeprecationWarning), see http://docs.plone.org/develop/styleguide/deprecation.html
  • conditional import from zope.formlib, if not available provide stub Interface
  • conditional registration in zcml (only if zope.formlib is available)
  • mark code for removal in Plone 6

gforcada added a commit that referenced this issue Oct 18, 2016
Make it optional and mark all vocabularies based on it as deprecated.

This fixes #38
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue Oct 18, 2016
Branch: refs/heads/master
Date: 2016-10-17T23:58:51-04:00
Author: Gil Forcada (gforcada) <gforcada@gnome.org>
Commit: plone/plone.app.vocabularies@32a4264

Remove zope.formlib dependency

Make it optional and mark all vocabularies based on it as deprecated.

This fixes plone/plone.app.vocabularies#38

Files changed:
M CHANGES.rst
M plone/app/vocabularies/catalog.py
M plone/app/vocabularies/configure.zcml
M plone/app/vocabularies/groups.py
M plone/app/vocabularies/users.py
M setup.py
Repository: plone.app.vocabularies
Branch: refs/heads/master
Date: 2016-10-18T17:41:24-04:00
Author: Gil Forcada Codinachs (gforcada) <gil.gnome@gmail.com>
Commit: plone/plone.app.vocabularies@53c193b

Merge pull request #42 from plone/gforcada-remove-formlib

Remove zope.formlib dependency

Files changed:
M CHANGES.rst
M plone/app/vocabularies/catalog.py
M plone/app/vocabularies/configure.zcml
M plone/app/vocabularies/groups.py
M plone/app/vocabularies/users.py
M setup.py
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue Oct 18, 2016
Branch: refs/heads/master
Date: 2016-10-17T23:58:51-04:00
Author: Gil Forcada (gforcada) <gforcada@gnome.org>
Commit: plone/plone.app.vocabularies@32a4264

Remove zope.formlib dependency

Make it optional and mark all vocabularies based on it as deprecated.

This fixes plone/plone.app.vocabularies#38

Files changed:
M CHANGES.rst
M plone/app/vocabularies/catalog.py
M plone/app/vocabularies/configure.zcml
M plone/app/vocabularies/groups.py
M plone/app/vocabularies/users.py
M setup.py
Repository: plone.app.vocabularies
Branch: refs/heads/master
Date: 2016-10-18T17:41:24-04:00
Author: Gil Forcada Codinachs (gforcada) <gil.gnome@gmail.com>
Commit: plone/plone.app.vocabularies@53c193b

Merge pull request #42 from plone/gforcada-remove-formlib

Remove zope.formlib dependency

Files changed:
M CHANGES.rst
M plone/app/vocabularies/catalog.py
M plone/app/vocabularies/configure.zcml
M plone/app/vocabularies/groups.py
M plone/app/vocabularies/users.py
M setup.py
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