Submitted by : stefan rank at: 2004-11-30T06:36:53+00:00 (13 years ago)
Name :
Category : Severity : Status :
Optional subject :  
Optional comment :

there seems to be a bug in the change to the mailhost check introduced with

Thu Nov 25 05:13:24 Westeuropõische Normalzeit 2004 simon@joyful.com
  • we no longer require the mailhost id to be MailHost, but isMailoutEnabled which controlled the skin link did

everything is fine if you have a MailHost object in your zwiki folder, but if there is none, you get an

AttributeError? in

Module Products.ZWiki.Mail, line 578, in mailhost

and that for every page (and then its returned as an incomplete html page only containing a table, no html/head/body tags, which gets rendered as an extremely wide and blank page in opera... thats why the severity is critical :)

the offending line reads:

mhosts = folder.objectValues(spec='Mail Host')

(thats the code that now gets called by the above change, I suppose there is some "try, except" missing or something. i dont know why objectValues gives an AttributeError? if there is no matching object???)

this is with zwiki freshly pulled just now, and zope 2.7 on windows.

to reproduce just call a zwiki-page that has no MailHost object in its folder.

(new) --Simon Michael, Tue, 30 Nov 2004 10:26:48 -0800 reply

Argh, thanks!

property change --simon, Tue, 30 Nov 2004 14:52:29 -0800 reply

Severity: critical => serious

... --simon, Wed, 29 Dec 2004 12:44:01 -0800 reply

> i dont know why objectValues gives an AttributeError?

I don't know either. I'm looking at it periodically, it's hard to reproduce.

fix applied for 0.37 --simon, Sun, 02 Jan 2005 14:28:58 -0800 reply

Status: open => closed

property change --Tue, 04 Jan 2005 14:11:07 -0800 reply