1 ethan5150 Sep 05, 2012 20:58
3 ethan5150 Sep 06, 2012 05:24
Thanks - will this prevent me from enabling comment posting elsewhere or only when disabled?
4 tilqicom Sep 06, 2012 08:45
is this supposed to be a bug ?
sam2kb wrote:
Edit the following in /htsrv/comment_post.php line 46
if( ! $commented_Item->can_comment( NULL ) ) { $Messages->add( T_('You cannot leave comments on this post!'), 'error' ); // Exit asap header_redirect(); }
5 sam2kb Sep 06, 2012 19:40
Thanks - will this prevent me from enabling comment posting elsewhere or only when disabled?
The extra string is just an extra layer of security, it doesn't change b2evo functionality.
is this supposed to be a bug ?
A flaw for sure.
The comment should be rejected anyway since we add an error message. But who knows what plugins do, they may clear errors and the comment will go through. Anyway, there's no reason to process comments if they are not allowed.
I already added that to SVN.
6 fplanque Sep 24, 2012 03:15
Alex I cannot even post a comment when logged as admin on a plain install of trunk!
7 sam2kb Sep 24, 2012 03:52
@fplanque
It's not me, I only added header_redirect() which must be there anyway. Look at new Item::is_locked() method, I believe that's the actual cause of the error.
8 fplanque Sep 24, 2012 22:58
Ah ok sorry Alex. (and it's been fixed today)
Edit the following in /htsrv/comment_post.php line 46