-
-
Notifications
You must be signed in to change notification settings - Fork 53
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
api.content.create fails to create a Plone4 AT Collection #413
Comments
please be careful when fixing this or we could find an issue similar with the one I described in #407. |
Hi @hvelarde I will add a test to plone.api showing the issue. Actually a simple create fails, we could also fix it in archetypes.querywidget but that does not seems correct to me. I will add a test that creates every ATCT default types. Thank you for advice, Gauthier |
@gbastien we already created a failing test on the referenced pull request. |
@gbastien A lot of time has passed, and don't remember exactly why I chose The purpose of the fix was for the All of my findings are described on #99 (comment) |
thank you for feedback, I think {'':''} will have the same behavior and it solves the problem... I proposed this change to @idgserpro and @hvelarde Gauthier |
fixed in #414 |
Hi everybody, @david-batranu
it fails in archetypes/querywidget/widget.py because the form given to process_form is {None: None} and values are not iterable.
This is due to a recent change, so either we change {None: None} to {'': ''} or we have to change archetypes.querywidget. Would {'': ''} sove your problem @david-batranu because it solves our ;-)
Thank you!
Gauthier
The text was updated successfully, but these errors were encountered: