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

get_ajaxselect_options is no longer used in plone.app.z3cform #194

Merged
merged 9 commits into from Apr 11, 2019

Conversation

jensens
Copy link
Sponsor Member

@jensens jensens commented Mar 27, 2019

Deprecate it.

Refactor get_relateditems_options to be more specific.

More cleanup would be needed.

in plone.app.z3c.form, deprecate

refactor get_relateditems_options to be more specific
@mister-roboto
Copy link

@jensens thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

@jensens jensens marked this pull request as ready for review March 28, 2019 11:18
@pbauer pbauer changed the title get_ajaxselect_options is no longer used in plone.app.z3c.form get_ajaxselect_options is no longer used in plone.app.z3cform Apr 1, 2019
@gforcada
Copy link
Sponsor Contributor

gforcada commented Apr 1, 2019

@jenkins-plone-org please run jobs

@jensens
Copy link
Sponsor Member Author

jensens commented Apr 2, 2019

@gforcada this need to be run together with other PRs and theres still a problem left as I found at my last run.

@pbauer
Copy link
Sponsor Member

pbauer commented Apr 2, 2019

I saw the same issue as jenkins did during some job: Rendering the relateditems widgets with a previously set value breaks.

  File "/home/gitlab-runner/builds/44df7dbf/0/kbo/kbo-buildout/src-mrd/plone.app.z3cform/plone/app/z3cform/widget.py", line 116, in render
    _base_args = self._base_args()
  File "/home/gitlab-runner/builds/44df7dbf/0/kbo/kbo-buildout/src-mrd/plone.app.z3cform/plone/app/z3cform/widget.py", line 599, in _base_args
    field_name,
  File "/home/gitlab-runner/builds/44df7dbf/0/kbo/kbo-buildout/src-mrd/plone.app.widgets/plone/app/widgets/utils.py", line 144, in get_relateditems_options
    'v': value.split(separator)
  File "/home/gitlab-runner/builds/44df7dbf/0/kbo/kbo-buildout/src-mrd/plone.app.vocabularies/plone/app/vocabularies/catalog.py", line 635, in __call__
    parsed = parseQueryString(context, query['criteria'])
TypeError: list indices must be integers, not str

@jensens
Copy link
Sponsor Member Author

jensens commented Apr 3, 2019

Yes, I am aware. It's not ready yet.

@pbauer
Copy link
Sponsor Member

pbauer commented Apr 8, 2019

Looks good to me.

@jensens
Copy link
Sponsor Member Author

jensens commented Apr 9, 2019

5.1 failure see plone/jenkins.plone.org#258 - this is no blocker for merging!

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

Successfully merging this pull request may close these issues.

None yet

4 participants