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 when removing interface #1

Closed
mauritsvanrees opened this issue Aug 2, 2016 · 4 comments
Closed

KeyError when removing interface #1

mauritsvanrees opened this issue Aug 2, 2016 · 4 comments

Comments

@mauritsvanrees
Copy link
Member

See collective/collective.searchandreplace#34 for an example. Last part of traceback:

Module z3c.relationfield.event, line 76, in updateRelations
Module zc.relation.catalog, line 546, in unindex
Module zc.relation.catalog, line 556, in unindex_doc
Module zc.relation.catalog, line 621, in _remove
KeyError: InterfaceClass Products.CMFDynamicViewFTI.interfaces.ISelectableBrowserDefault

@do3cc I see this post from you where you have a workaround for fixing an already broken relation catalog: http://do3.cc/blog/2012/09/264/debugging-zcrelations---broken-btrees/

Did you get anywhere with properly fixing this problem? I do see a branch https://github.com/zopefoundation/zc.relation/tree/do3cc_foot_meet_gun but nothing really happened there yet. And on master you have made this change which may help partially:
10b9f88

(BTW, last release is 1.0 from 2008.)

@do3cc
Copy link
Member

do3cc commented Aug 2, 2016

Unfortunately no. Fixing this properly would take quite some time, enough to actually consider rewriting the functionality to something simpler. the underlaying packages allow much more than we actually need and we are using some features wrong (The docs even say not to use persistent objects as keys, and we still have them)
But I don't see me working on this any time soon :-(

@do3cc
Copy link
Member

do3cc commented Aug 2, 2016

Oh, I just noticed, my comments were with the plone packages for relations in mind, not for zc.relation

@do3cc
Copy link
Member

do3cc commented Aug 2, 2016

This is not a zc.relation issue, it is a z3c.relationship issue. This is the bug: zopefoundation/z3c.relationfield#4

@do3cc do3cc closed this as completed Aug 2, 2016
@mauritsvanrees
Copy link
Member Author

Ah, thanks for the clarification!

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

2 participants