You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
LookupError is the documented exception that IVocabularyRegistry.get is supposed to raise. VocabularyRegistryError is a subclass of that used by the specific default implementation. This means that when zope.vocabularyregistry is used, the generic LookupError from zope.component propagates.
Moreover, when Choice does catch VocabularyRegistryError it just raises a plain ValueError, with no indication of the failing field, value, or name. It should raise a ValidationError with the field and value. For backwards compatibility, this should also subclass ValueError and LookupError.
The text was updated successfully, but these errors were encountered:
LookupError
is the documented exception thatIVocabularyRegistry.get
is supposed to raise.VocabularyRegistryError
is a subclass of that used by the specific default implementation. This means that whenzope.vocabularyregistry
is used, the genericLookupError
from zope.component propagates.Moreover, when Choice does catch
VocabularyRegistryError
it just raises a plainValueError
, with no indication of the failing field, value, or name. It should raise aValidationError
with the field and value. For backwards compatibility, this should also subclassValueError
andLookupError
.The text was updated successfully, but these errors were encountered: