[ZCM] [ZC] 2157/ 2 Accept "Improved detail in SystemError message on Class not found"

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

Issue #2157 Update (Accept) "Improved detail in SystemError message on Class not found"
 Status Accepted, Zope/bug+solution medium
To followup, visit:
  http://www.zope.org/Collectors/Zope/2157

==============================================================
= Accept - Entry #2 by tseaver on Jul 23, 2006 7:55 pm

 Status: Pending => Accepted

 Supporters added: tseaver

*Sigh* -- there are no tests at all for OFS.Uninstalled.  I will
try to write a minimal set of tests for them, so that I can show
that this bug is fixed when I apply the patch.
________________________________________
= Request - Entry #1 by russf on Jul 23, 2006 2:03 pm


Uploaded:  "Uninstalled.py.diff"
 - http://www.zope.org/Collectors/Zope/2157/Uninstalled.py.diff/view
>Preamble...
>From ZODB-Dev list:
>On 23 Jul 2006, at 17:46, Tres Seaver wrote:
>BTW, some might call your change a "feature":  I would back
>characterizing it as a bugfix (and describing the defect).  As such, we
>can then deploy it into release line code, rather than only on the trunk.
>Tres.

Background...

When a module is renamed, as sometimes occurs between upgrades, object instances cannot be reconciles to their filesystem classes.

In such cases curretnly a SystemError is raised with an inadequate message, that does not help find the source of the problem.

Fix...

Applying the attached patch in OFS will provide an early clue as to the source of the problem, and reduce time to fix.

Tres's comment...
On 23 Jul 2006, at 17:46, Tres Seaver wrote:
"It looks fine to me."


==============================================================