bbPress plugin browser Topic: Allow Images

bbPress plugin browser Topic: Allow Images http://bbpress.org/plugins/ bbPress plugin browser Topic: Allow Images en Sun, 27 Jul 2008 01:50:24 +0000 _ck_ on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1514 Wed, 23 Jul 2008 19:22:06 +0000 _ck_ 1514@http://bbpress.org/plugins/ No, no, don't hack the core. There is no need. Very easy to add tags in bbpress via plugin. The full plugin probably breaks code sections because it tries to do it's own bad tag encoding. You can try a more direct method instead. <pre><code><?php /* Plugin Name: Allow Images (mini-mod) */ add_filter( 'bb_allowed_tags', 'allow_img_tag' ); function allow_img_tag( $tags ) {$tags['img'] = array('src' => array(), 'title' => array(), 'alt' => array()); return $tags;} ?></code></pre> . try that instead - note it makes no attempt to verify images corenominal on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1507 Wed, 23 Jul 2008 01:24:03 +0000 corenominal 1507@http://bbpress.org/plugins/ I have the same problem as greenshady, basically the plugin breaks code placed within backticks. This is quite a big problem as allowing users to post code snippets is very important, see below :) So, does anyone have any information about whether a fix is being worked on? Note, it is possible to enable images by hacking bb-includes/formatting-functions.php, changing the following function: <pre><code>function bb_allowed_tags() { $tags = array( 'a' => array( 'href' => array(), 'title' => array(), 'rel' => array()), 'blockquote' => array('cite' => array()), 'br' => array(), 'code' => array(), 'pre' => array(), 'em' => array(), 'strong' => array(), 'ul' => array(), 'ol' => array(), 'li' => array() ); return apply_filters( 'bb_allowed_tags', $tags ); }</code></pre> To this: <pre><code>function bb_allowed_tags() { $tags = array( 'a' => array( 'href' => array(), 'title' => array(), 'rel' => array()), 'blockquote' => array('cite' => array()), 'br' => array(), 'code' => array(), 'pre' => array(), 'em' => array(), 'img' => array( 'src' => array(), 'alt' => array()), 'strong' => array(), 'ul' => array(), 'ol' => array(), 'li' => array() ); return apply_filters( 'bb_allowed_tags', $tags ); }</code></pre> However, it is probably[?] not a wise idea to do so as I am pretty sure this will not provide any sanity checks -- meaning users could possibly use the image tag to call on malicious scripts etc. Therefore, fixing this plugin would be a preferable option. pjrobertson on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1412 Thu, 03 Jul 2008 13:32:44 +0000 pjrobertson 1412@http://bbpress.org/plugins/ The easiest way to sort out image size problems is to add this code to your .css file: <pre><code>.threadpost img { width:100%; }</code></pre> it'll limit the width of the image to the width of the thread post area. greenshady on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1406 Wed, 25 Jun 2008 19:19:36 +0000 greenshady 1406@http://bbpress.org/plugins/ Plugin works great for me except for one small problem (well, major in my case). Whenever users post code in between backticks, as is the preferred method with bbPress, the code is deleted upon submission of the post form. Guymed on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1275 Tue, 20 May 2008 20:04:00 +0000 Guymed 1275@http://bbpress.org/plugins/ <blockquote> xD </blockquote> _ck_ on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1067 Thu, 24 Apr 2008 06:58:55 +0000 _ck_ 1067@http://bbpress.org/plugins/ Image size restriction has to be done either via javascript or via your own upload (something I am working on). For now you can make sure large images don't mess up your layout by applying <code>overflow:hidden;</code> to the CSS - my educated guess would be to the <code>.threadpost</code> element, or <code>#thread</code> (or <code>#thread li</code>) Farkel on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/3/#post-1064 Wed, 23 Apr 2008 23:38:33 +0000 Farkel 1064@http://bbpress.org/plugins/ Thanks for this plugin, it works great. Is there any way, however, to restrict the image size? If someone posts a large image it completely messed up my layout :( bensig on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-988 Tue, 08 Apr 2008 19:06:31 +0000 bensig 988@http://bbpress.org/plugins/ Great idea for a plug-in, but it's still not as easy to put an image in a post with bbpress as it is in other forums. As the most downloaded plug-in, shouldn't this be a feature of the forum? Isn't it the case that more administrators would prefer to have the option of attaching\linking files than those who would not want it? Speak up here to help let the dev team know that this is an important feature for a future release of bbpress: <a href="http://bbpress.org/forums/topic/priority-1-imagefile-attachments" rel="nofollow">http://bbpress.org/forums/topic/priority-1-imagefile-attachments</a> toosalty on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-892 Tue, 04 Mar 2008 14:06:39 +0000 toosalty 892@http://bbpress.org/plugins/ Hi, For those still stuck with the problem of your img tags being reduced to <img />, I am guessing your bbpress installation is integrated with wordpress. When you integrate, you include the wp-settings file, which unfortunately, to preserve compatibility with older plugins and php4, automatically magic quotes everything. Hence, adding stripslashes would solve the problem: function allow_images_encode_bad( $text ) { $text = wp_specialchars( stripslashes($text) ); baptiste on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-770 Sat, 02 Feb 2008 14:34:06 +0000 baptiste 770@http://bbpress.org/plugins/ Not sure if this was already mentioned, but images with capitalized suffixes don't get allowed. Easy to fix. In allow-images.php, look for the $matches AS $match loop and in particular $match[3]. Wrap $match[3] with strtolower() so it's strtolower($match[3]) and it'll work. What I did was before the if stmt, do: $match3low = strtolower($match[3]); and then in the code below in two spots, replaced $match[3] with $match3low Saves one conversion n3wjack on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-697 Fri, 21 Dec 2007 14:50:08 +0000 n3wjack 697@http://bbpress.org/plugins/ on 0.8.3 this plugin works but you have to supply the additional xhtml closing slash so <ul> <li><img src="foobar.png"> doesn't work <li><img src="foobar.png" /> does </ul> itissue on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-684 Sat, 15 Dec 2007 02:26:49 +0000 itissue 684@http://bbpress.org/plugins/ Nevermind i figured it out. It's on the allow-images.php page. Duh. XD Now if only we can set the width of our images. bbPress isn't letting me do that and some of my images are pretty big and are wider than the width of my layout. I need to be able to set the width. itissue on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-683 Sat, 15 Dec 2007 02:21:08 +0000 itissue 683@http://bbpress.org/plugins/ I'm having the same issue as iBodhi. And the image tag in my signature doesn't work like it used to. I'm so confused. I wonder what page schmitt edited. I'd like to try that disabling image filters thing. iBodhi on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-672 Mon, 10 Dec 2007 12:29:11 +0000 iBodhi 672@http://bbpress.org/plugins/ All the img tags in the posts that I had re-edited disappered... schmitt on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-640 Mon, 19 Nov 2007 20:00:37 +0000 schmitt 640@http://bbpress.org/plugins/ I simply disabled all image filters... and things work great. Am I inviting disaster? <pre><code>remove_filter( 'pre_post', 'encode_bad' ); add_filter( 'pre_post', 'allow_images_encode_bad', 9 ); // add_filter( 'pre_post', 'allow_images', 52 ); add_filter( 'bb_allowed_tags', 'allow_images_allowed_tags' );</code></pre> schmitt on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-639 Mon, 19 Nov 2007 19:49:29 +0000 schmitt 639@http://bbpress.org/plugins/ Plugin works great except I have the same need as a previous poster - I need to link to dynamically generated images. The have srs URL's like this: <a href="http://stockcharts.com/c-sc/sc?s=$NDX&p=W&yr=4&mn=0&dy=0&i=p55776753068&a=121012681&r=1979" rel="nofollow">http://stockcharts.com/c-sc/sc?s=$NDX&p=W&yr=4&mn=0&dy=0&i=p55776753068&a=121012681&r=1979</a> What would I need to add to the allow_images function to whitelist a domain? Security is not a major issue for me as users are approved and misbehavior is unlikely, a brute force disable is OK too. I don't think sever harm can come from an image link no? Xenuan on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-638 Mon, 19 Nov 2007 18:32:25 +0000 Xenuan 638@http://bbpress.org/plugins/ Not working. I'm using bbpress 0.8.2.1 and the latest version of this plugin. neyoung on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-538 Sat, 25 Aug 2007 07:41:54 +0000 neyoung 538@http://bbpress.org/plugins/ got it working with a fresh install of bbpress. neyoung on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-534 Sun, 19 Aug 2007 23:07:01 +0000 neyoung 534@http://bbpress.org/plugins/ I'm getting the same problem as Arlo. I've tried the double space trick with no luck. Using bbpress 0.8.2.1 and Allow images 0.7.1 I attempted the fix that Tweek proposed but it didn't solve the problem. My image url's always get truncated down to <img /> Anyone have any ideas? thierryyyyyyy on "Allow Images" http://bbpress.org/plugins/topic/allow-images/page/2/#post-422 Mon, 23 Jul 2007 21:23:44 +0000 thierryyyyyyy 422@http://bbpress.org/plugins/ And I think that the comparision of the file extension and the "authorized extension list" should be case insensitive (see : <a href="http://bbpress.org/forums/topic/allow-images-plugin-does-not-allow-caps-filename-extensions?replies=1" rel="nofollow">http://bbpress.org/forums/topic/allow-images-plugin-does-not-allow-caps-filename-extensions?replies=1</a>) I would propose to change : <code>in_array(substr($match[3], -4), [...]</code> by <code>in_array(strtolower(substr($match[3], -4)) , [...]</code> (and same change for the next line with jpeg) I'm waiting for finding the exact solution of the "backtick code" problem to release a new version for this plugin