0.11 compatibility

[bewst]

I need trac 0.11 compatibility and unfortunately don't understand the templating system well enough to do the job myself. I can offer this patch which I hope does part of it:

notification.py

@@ -74 +74 @@
         subject = self.topic['subject']
 
         # Set set e-mail template values.
-        self.hdf.set_unescaped('discussion.re', re)
-        self.hdf.set_unescaped('discussion.prefix', prefix)
-        self.hdf.set_unescaped('discussion.title', title)
-        self.hdf.set_unescaped('discussion.id', id)
-        self.hdf.set_unescaped('discussion.author', author)
-        self.hdf.set_unescaped('discussion.time', time)
-        self.hdf.set_unescaped('discussion.moderators', moderators)
-        self.hdf.set_unescaped('discussion.subject', subject)
-        self.hdf.set_unescaped('discussion.body', body)
-        self.hdf.set_unescaped('discussion.link', link)
+        try:
+            hdf = self.hdf
+        except AttributeError:
+            set_data = self.data.__setitem__
+        else:
+            set_data = hdf.set_unescaped
+
+        for a in 're prefix title id author time moderators subject body link'.split():
+            set_data(a, locals()[a])
 
         # Render body and send notification.
         subject = self.hdf.render('discussion-notify-subject.cs')

[Radek Bartoň]

Trac 0.11 uses Genshi templates instead of ClearSilver ones. You need to rewrite all templates so it is quite a lot of job. I hope that I find a time to do it during summer but some help would be great.

[anonymous]

Well, 0.11 contains clearsilver backward compatibility hacks that work if CS is installed. I haven't had time yet to port all the templates, but all the other fixes in the attached patch are required to get this plugin to work, plus the patch at ​http://trac.edgewall.org/ticket/4547

I guess this plugin was developed on a case-insensitive filesystem as there are some case errors in imported trac module names. Also emacs complains that the utf-8 designation of all the files I touched is inaccurate, so I removed it.

[Dave Abrahams <dave@…>]

complete 0.11 compatibility (using clearsilver)

[Radek Bartoň]

I have already done a most of porting to Genshi templates and API changes. Only thing left is message list. Timeline events and searching is not tested as well. But I have bought a new laptop recently so I'm spending time with clear Gentoo installation and getting quite new hardware to work. Please be patient, I'll finish this during this or next week in worst case. I can commit changes already done right now, if you like?

[Dave Abrahams <dave@…>]

Thanks; I'm in no rush now as I have a complete working port under clearsilver. I can use my port until yours is done. [Sorry about that crack about the case-insensitive filesystem. Seems that earlier versions of Trac had capitalized some module names.]

[bewst]

updated patch that also handles notification properly

[bewst]

Replying to Blackhex:

I have already done a most of porting to Genshi templates and API changes. Only thing left is message list. Timeline events and searching is not tested as well. But I have bought a new laptop recently so I'm spending time with clear Gentoo installation and getting quite new hardware to work. Please be patient, I'll finish this during this or next week in worst case. I can commit changes already done right now, if you like?

I have updated my patch; I suggest you at least review my changes in case you missed something.

[Radek Bartoň]

Port to 0.11 is here. There was so many changes (208.4 kB diff) that I coudn't commit them together. Please test it well. I did't test e-mail notification because my ISP's spam-filter refuses me to send e-mails from Trac's notification, but I'll rewrite notification system soon (If I find a way how to test it).

[bewst]

I got email notification working in my patch; did you compare changes?

[Radek Bartoň]

I've checked what you've changed but there is much more changes in my version so it is not comparable. Isn't my version working for you? I'm able to send e-mail, but how it look like I can't get know becasue SMTP spam refusion error occurs to me so there will be some e-mail content bugs maximally.

[anonymous]

hello, I just installed the latest revision and got this error: ​http://www.dx9.de/tracdiscussionerror.png

is this may be something wrong on my side or what do you think? I set for me or my group the permission for DISCUSSION_ADMIN, but it is still the same. any suggestions?

[Radek Bartoň]

I think that problem is with your Trac 0.11 revision. I have tested with lastest 0.11dev-r5790. First try to update your Trac to lastest SVN revision and then if your error would still occur post a new ticket where you would describe your system info and post debug log. See BugReporting for further details.

Generally always use lastest trunk revision when trying DiscussionPlugin for 0.11 and make a separate ticket for each bug you'll find.

[bewst]

Replying to Blackhex:

I've checked what you've changed but there is much more changes in my version so it is not comparable. Isn't my version working for you? I'm able to send e-mail, but how it look like I can't get know becasue SMTP spam refusion error occurs to me so there will be some e-mail content bugs maximally.

Haven't dared to try it yet :-)

More seriously, I've just had other priorities, but I'll try it tomorrow.

[bewst]

The result is #1715