[ZCM] [ZC] 1457/ 3 Comment "ZCTextIndex: QueryError and ParseError not available for import"

[Collector: Zope Bugs, Features, and Patches ...]

Issue #1457 Update (Comment) "ZCTextIndex: QueryError and ParseError not available for import"
 Status Pending, Zope/bug medium
To followup, visit:
  http://zope.org/Collectors/Zope/1457

==============================================================
= Comment - Entry #3 by Caseman on Aug 5, 2004 10:22 am

That looks good to me.
________________________________________
= Comment - Entry #2 by chrisw on Aug 5, 2004 8:11 am

The following works for me, is it the right thing to do?

# Make QueryErrors and ParseError public
ModuleSecurityInfo('Products').declarePublic('ZCTextIndex')
ModuleSecurityInfo('Products.ZCTextIndex').declarePublic('ParseTree')
ModuleSecurityInfo('Products.ZCTextIndex.ParseTree').declarePublic('QueryError')
ModuleSecurityInfo('Products.ZCTextIndex.ParseTree').declarePublic('ParseError')
________________________________________
= Request - Entry #1 by chrisw on Aug 5, 2004 7:37 am

Products.ZCTextIndex.ParseTree.QueryError
Products.ZCTextIndex.ParseTree.ParseError

...neither are importable in "safe" code, so you can't gracefully and safe(w.r.t. conflict errors)ly handle people putting things ZCTextIndex doesn't understand in search boxes.

There should be security assertions that allow these to be imported.
==============================================================