robfelty WordPress plugin forum » Topic: introduction - I am working on linking mailman with buddypress forums

robfelty WordPress plugin forum » Topic: introduction - I am working on linking mailman with buddypress forums http://forum.robfelty.com/ robfelty WordPress plugin forum » Topic: introduction - I am working on linking mailman with buddypress forums en Wed, 08 Feb 2012 09:35:35 +0000 andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-3023 Tue, 30 Mar 2010 13:26:49 +0000 andrew_s1 3023@http://forum.robfelty.com/ Thanks Rob. robfelty on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-3021 Mon, 29 Mar 2010 19:52:59 +0000 robfelty 3021@http://forum.robfelty.com/ I put those changes into trunk. andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-3019 Mon, 29 Mar 2010 00:40:48 +0000 andrew_s1 3019@http://forum.robfelty.com/ Revised patch - sorry for pasting direct code, but pastebin's not online at the moment. Features are as above: smtp fix, and adding a way for a plugin filter to block a post completely. Additionally, this fixes the handling of :end, which isn't handled well for HTML posts currently. <pre><code>Index: postie-functions.php =================================================================== --- postie-functions.php (revision 220496) +++ postie-functions.php (working copy) @@ -173,12 +173,21 @@ 'post_status' => $post_status ); $details = apply_filters('postie_post', $details); - DisplayEmailPost($details); - PostToDB($details,$isReply, $post_to_db, - $custom_image_field); - if ($confirmation_email) - MailToRecipients($mimeDecodedEmail, false, - array($postAuthorDetails['email']), false, false); + if ( empty( $details ) ) { + // It is possible that the filter has removed the post, in which case, it should not be posted. + // And if we created a placeholder post (because this was not a reply to an existing post), + // then it should be removed + if ( !$is_reply ) { + wp_delete_post($post_id); + } + } else { + DisplayEmailPost( $details ); + PostToDB( $details,$isReply, $post_to_db, $custom_image_field ); + if ($confirmation_email) { + MailToRecipients( $mimeDecodedEmail, false, + array($postAuthorDetails['email']), false, false ); + } + } } /** FUNCTIONS **/ @@ -902,10 +911,11 @@ function checkSMTP($mimeDecodedEmail, $smtpservers) { -if ( empty( $smtpservers ) ) return true; - + if ( empty( $smtpservers ) ) return true; foreach ( (array) $mimeDecodedEmail->headers['received'] as $received ) { - if ( in_array( strtolower($received), $smtpservers ) ) return true; + foreach ($smtpservers as $smtp) { + if ( stristr( $received, $smtp ) !== false ) return true; + } } return false; } @@ -954,20 +964,12 @@ * @param string * @param filter */ -function EndFilter( &$content,$filter) { -$arrcontent = explode("\n", $content); -$i = 0; -for ($i = 0; $i<=count($arrcontent); $i++) { - $line = $arrcontent[$i]; - $nextline = $arrcontent[$i+1]; - if (preg_match("/^$filter/",trim($line))) { - //print("Found in $line"); - break; - } - $strcontent .= $line ."\n"; +function EndFilter( &$content, $end ) { + $pos = strpos( $content, $end ); + if ( $pos === false ) + return $content; + return $content = substr($content, 0, $pos); } - $content = $strcontent; -} //filter content for new lines function FilterNewLines ( $content, $convertNewLines=false ) {</code></pre> andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2972 Mon, 22 Mar 2010 13:32:39 +0000 andrew_s1 2972@http://forum.robfelty.com/ Sorry, I let a small problem through with the smtp filter. <a href='http://wordpress.pastebin.ca/1849602'>This patch fixes it</a>, as well as adding in a new feature: if a postie filter returns false, then the email is not turned into a wordpress post: this provides a way for the filter to suppress an email. andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2948 Wed, 17 Mar 2010 13:55:56 +0000 andrew_s1 2948@http://forum.robfelty.com/ Ah, ok, I'll just back away from that one slowly then! robfelty on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2947 Wed, 17 Mar 2010 12:04:44 +0000 robfelty 2947@http://forum.robfelty.com/ Andrew, Thanks for that. However, I think that someone had requested this awhile ago, to use this: <a href="http://htmlpurifier.org/" rel="nofollow">http://htmlpurifier.org/</a> andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2944 Tue, 16 Mar 2010 23:55:54 +0000 andrew_s1 2944@http://forum.robfelty.com/ Thanks for that, Rob, that's great. I saw that "html purify" was top of the TODO list in postie-functions: this patch should do it: <a href="http://wordpress.pastebin.ca/1843301" rel="nofollow">http://wordpress.pastebin.ca/1843301</a> Regards, Andrew robfelty on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2941 Tue, 16 Mar 2010 11:53:42 +0000 robfelty 2941@http://forum.robfelty.com/ Andrew, I have committed your changes into HEAD now. I also fixed the cron updating, which you had noted was not working. Thanks so much for your help. I apologize for the ugly code. Quite a few people had their hands on postie before me, and it also was dormant for a couple years. I have slowly been working on bringing it back up to wordpress standards. The reason I started hacking postie in the first place was to use it with mailman lists for a friend. I had it working fairly well, but he never ended up really using it. Rob andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2938 Mon, 15 Mar 2010 08:05:23 +0000 andrew_s1 2938@http://forum.robfelty.com/ <a href='http://wordpress.pastebin.ca/1841258'>revised patch here</a>, with nested serialized arrays, as discusssed. I'm still testing: it seems stable so far... andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2935 Sun, 14 Mar 2010 14:10:25 +0000 andrew_s1 2935@http://forum.robfelty.com/ OK, I'll have a go at this, and get back to you tomorrow, all being well. Re <code>GetConfig/GetDBConfig</code> - as the only place the old config is used, is in the activation routine, I moved the relevant code into <code>function activate_postie()</code> in <code>postie.php</code>. But I'll reverse those particular changes, so that you & all postie users can be confident that their old settings will be carried over securely. Regards, Andrew robfelty on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2934 Sun, 14 Mar 2010 11:45:43 +0000 robfelty 2934@http://forum.robfelty.com/ I had a chance to look over the patch a little bit. It looks good for the most part. The main thing that I would change is to undo any changes you made to the GetConfig (and GetDBConfig) functions. Under the new settings scheme, I don't use that function any more, except to convert the old-style settings into the new style. So it needs to be left untouched in order to smoothly transition from old style to new style settings. If you can get nested serialized arrays to work, that would be great. Rob andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2933 Sun, 14 Mar 2010 11:24:38 +0000 andrew_s1 2933@http://forum.robfelty.com/ Hi Rob, <code>register_setting</code> will do all the serialisation for you. The good news is that I really didn't need to touch much of your options code to get all that working, with serialised options - I just adjusted the place where <code>register_setting</code> was called, and tweaked a couple of other things. Most of my patch is concerned with tidying up the handling of default options, and stripping out the old database-handling of the old-format options. I think we can get serialised arrays to work. If we can, would you prefer to stick with those? They all get turned into arrays at processing-time anyway. They'd just get turned into <code>\n</code>-delimited strings for processing in the <code>textarea</code> on the options screen. Regards, Andrew robfelty on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2932 Sun, 14 Mar 2010 10:57:32 +0000 robfelty 2932@http://forum.robfelty.com/ Andrew, I was switching these to strings because I could not get nested serialized arrays to work. I haven't gotten a chance to look at your patch yet. I was moving towards storing all postie settings in one serialized array, so that there would only be one database lookup necessary. Is that the way register_settings works too? andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2931 Sun, 14 Mar 2010 01:17:22 +0000 andrew_s1 2931@http://forum.robfelty.com/ <a href='http://wordpress.pastebin.ca/1838810'>patch updated</a> here. About the options that were previously stored as arrays (e.g. smtp or sig_patterns_list), is it better to store those as arrays, or as strings with array elements joined with a \n separator? From the code, it looks like you're switching to the latter - have I got that right? Regards, Andrew andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2926 Fri, 12 Mar 2010 15:14:01 +0000 andrew_s1 2926@http://forum.robfelty.com/ register_setting allows you to use the WP backend to do these useful tasks: database updates, some validation and sanitisation, and handling the _nonce_ fields for security. EDIT2: Whereas I add <code>settings_fields('postie-settings')</code> into the main options form, to generate the _nonce_fields for security. I did have some difficulty the first time I tried to get register_setting to work, and had to try a few different combinations. The key is that (as you can see in my patch) you need to run the register_setting call from a function that is hooked into admin_init. So, in the main body of postie.php I add: <pre><code>if (is_admin()) { add_action('admin_init','postie_admin_settings'); }</code></pre> and in postie-functions.php, I add: <pre><code>function postie_admin_settings() { register_setting('postie-settings','postie-settings','postie_validate_settings'); }</code></pre> where the last argument is a validation function. Then all of the handling of $_POST variables can be removed from the options form, and options can be submitted/retrieved just using get_option('postie-settings') and update_option('postie_settings,$options); NB that in all three cases, register_setting, get_option and update_option, it's singular not plural (e.g. 'option' not 'options') - that catches me out every time! But in the case of <code>settings_fields</code>, it's plural "settings" (and "fields"), not singular "setting"! Regards, Andrew robfelty on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2925 Fri, 12 Mar 2010 13:01:20 +0000 robfelty 2925@http://forum.robfelty.com/ Andrew, Thanks for the changes. I will look at them soon. I have been working on updating the settings mechanisms. I couldn't get <code>register_settings()</code> to work, but I am using <code>settings_fields</code>. What is the difference really? Rob andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2922 Thu, 11 Mar 2010 04:32:35 +0000 andrew_s1 2922@http://forum.robfelty.com/ I've taken a pass at rationalising the options handling, and getting it all working with register_settings and the like, leaving all the database updating to wordpress. <a href='http://wordpress.pastebin.ca/1833538'>Here's my diff file</a> It touches quite a lot of code - sorry about that. But it does knock the options handling into shape. It strips out all the old database-options handling, apart from the bit that's run at plugin activation, that imports settings from the old database-options. All that's now in the activation call within postie.php. I hope this is useful. Feedback very welcome. I don't have legacy postie configs to test it against, but will happily fix any bugs arising from my changes. andrew_s1 on "introduction - I am working on linking mailman with buddypress forums" http://forum.robfelty.com/topic/introduction-i-am-working-on-linking-mailman-with-buddypress-forums#post-2919 Wed, 10 Mar 2010 06:45:53 +0000 andrew_s1 2919@http://forum.robfelty.com/ Thank you Rob - good work on taking postie forward. I'm working on a plugin to link mailman to buddypress. <a href='http://andrewsinlondon.wordpress.com/'>I've written some thoughts on the process</a>. I think Postie will do the email-> wordpress job I need, with minor mods. It's a very impressive plugin. I'm wrestling with the HEAD revision of postie in the SVN now - it wasn't possible to change settings on a clean install, so I've done some patching to fix that. [edit - just tried another clean install, and it worked fine, as long as I commented out postie-functions.php line 5 - Suhosin does not like that ini_set] Anyway, just wanted to check in, let you know what I'm working on.