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

KeyError: 'dx_News Item__for__NewsItem' in @@custom_migration #305

Closed
fulv opened this issue Dec 8, 2015 · 4 comments
Closed

KeyError: 'dx_News Item__for__NewsItem' in @@custom_migration #305

fulv opened this issue Dec 8, 2015 · 4 comments

Comments

@fulv
Copy link
Member

fulv commented Dec 8, 2015

I'm using 1.1b6 on Plone 4.3.7.

https://github.com/plone/plone.app.contenttypes/blob/1.1b6/plone/app/contenttypes/migration/custom_migration.py#L241

It seems that safe_dx is not that safe, after all. Indeed:

(Pdb) dx_typename
'News Item'
2015-12-07 18:06:07 ERROR Zope.SiteErrorLog 1449540367.650.827755354863 http://localhost:8080/Plone/@@custom_migration
Traceback (innermost last):
  Module ZPublisher.Publish, line 138, in publish
  Module ZPublisher.mapply, line 77, in mapply
  Module ZPublisher.Publish, line 48, in call_object
  Module plone.app.contenttypes.migration.custom_migration, line 46, in __call__
  Module plone.app.contenttypes.migration.custom_migration, line 241, in migrate
KeyError: 'dx_News Item__for__NewsItem'
fulv added a commit that referenced this issue Dec 8, 2015
@fulv
Copy link
Member Author

fulv commented Dec 8, 2015

This is how I got it to work:
1.1b6...fulv-305

I'm sure it's horrible, but hopefully it will save you some time.

@pbauer
Copy link
Member

pbauer commented Dec 8, 2015

I'm waiting for @rockfruit to fix his pull-request #284. That fixes this issue.

@pbauer
Copy link
Member

pbauer commented Dec 14, 2015

I gave up on waiting and backported it myself. Fixed.

@pbauer pbauer closed this as completed Dec 14, 2015
@gianniftp
Copy link

Plone 5.0.10 , python 2.7.18. trying to migrate to plone 5.2...
when i try to use the @@custom_migration form i have the same problem with wrong space replace . i found this issue closed. i followed the trace trough #284 and #307 and i went in dead end as the cherry picking was in 1.1.x but not from 1.2.x on.

gianniftp pushed a commit to gianniftp/plone.app.contenttypes that referenced this issue Mar 10, 2021
…m request;

added changelog in news to refer to the issue plone#305.
gianniftp pushed a commit to gianniftp/plone.app.contenttypes that referenced this issue Mar 23, 2021
…m request;

added changelog in news to refer to the issue plone#305.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants