Announcer will not send in html

[paul@…]

I am running trac 0.13 (upgraded today) + announcer 0.11.1-r10458

all previous versons will / have never done this.

there are plenty of hits on this issue with no clear resolution.

also announcer latest developer trunk 0.12 once compiles etc the environment would not allow an upgrade (attribute errors etc)

thus i backed of to 0.11.1-r10458

i can provide login access to the server if nessesary.

any help would be appreciated.

[anonymous]

Hey Paul, just to make it interesting, one of our TRAC projects is sending me messages in plain text AND html - one of each.

I think the Plain Text is from me being on the global CCall. And the HTML is from my announcer subscription to all activity.

Did you go into your user preferences and change your e-mail announcer preference to HTML?

[Steffen Hoffmann]

Thanks for the report.

But I'm afraid that you rather get bitten by several unrelated issues, so your description is not very helpful for pushing a solution. I'll step it down priority-wise to give room for the other, better documented issues, that may even reflect some/all of the issues that you experienced. We already know much more now i.e. about db upgrade issues: See and follow #5774 and #9742 instead.

Ultimately, if you got around this by now due to local installation/configuration issues, close this ticket yourself, please.

[Steffen Hoffmann]

Just noticed, that EmailDistributer._get_preferred_format still looks into Trac db table session_attribute for user's preferred format.

This is the old, now unsupported preference store, and plainly wrong. We need to replace it with the complementary lookup done in announcer.model.SubscriptionAttribute.

Understand, that I strongly encourage replacement of the potentially erroneous code, but won't care for fixing the old code. The OP revision is even before the new subscription model (db schema revision <3). The current db schema (revision 6, almost identical to 5, short upgrade path from 4) has been fixed for good reasons, mostly to overcome previous errors and limitations, and anything else is obsolete now.

[Steffen Hoffmann]

(In [12346]) TracAnnouncer: Read the notification format from subscriptions, refs #9282.

Now EmailDistributor cares for the preferred format, that is configured per user in the subscription db table, obsoleting related 'announcer_email_format_<realm>' entries stored in session_attribute db table before.

While this is yet another regression, because this setting has been per target (resource) realm before, in theory it could even be more fine-grained. In fact, its possible to store the preferred format per subscriber in subscription. Only it is unified per transport for now, and I see no real value in changing this, yet.