post_process_request does not correctly pass content type

[Ryan Witt]

AccountLdapPlugin blocks other plugins from changing the content type of their response. This was a problem for me and kept RevtreePlugin from rendering its SVG.

Here's a patch that fixes it:

accountldap/accountldap.py

@@ -53 +53 @@
 
     def post_process_request(self, req, template, data, content_type):
         if not req.remote_user or req.session.has_key('email'):
-            return template, data, None
+            return template, data, content_type
         uid = req.remote_user.lower()
         name, email = self._getUserAttributes(uid)
         req.session['name'] = name
         req.session['email'] = email
-        return template, data, None
+        return template, data, content_type
     #
     #----------------------------------------- INavigationContributor interface
     #

This issue was a blocker for our new install.

[anonymous]

Thanks for posting this. This also caused problems with the datefield plugin on my install

[Ryan J Ollos]

I'll see about getting this committed to the repository, to avoid more reports like #9611 (closed as duplicate).

[Ryan J Ollos]

This patch was applied to the 0.12 branch in [5428]. The 0.11 branch has a README that says to use the 0.12 branch.