[Robin W]

@clivesmith just tried an anonymous post using

//add message if reply help in moderation
add_filter ('bbp_new_reply_redirect_to' , 'rew_pending_check', 10 , 3) ;

function rew_pending_check  ($reply_url, $redirect_to, $reply_id) {
	$status = get_post_status ($reply_id) ;
	if ($status == 'pending' ) {
		$reply_url = '/moderation/' ;
	}
return $reply_url ;
}

and it does redirect to the moderation page – can you recheck yours please ?

[Clivesmith]

I ran your code on my development site as a guest and it does not work it returns the same as on the live site as a guest.
As I am the only member, I only moderate guests.

[Robin W]

Put this in your child theme’s function file – or use

Code Snippets

add_action ('bbp_template_before_replies_loop' ,'rew_show_forum' ) ;

function rew_show_forum () {
	$forum_id = bbp_get_topic_forum_id() ;
	$title = bbp_get_forum_title($forum_id) ;
	echo '<div class="rew-in-forum">In forum : '.$title.'</div>' ;
	
}

[Robin W]

so are you saying my code does nothing for anonymous users ? It seems to work for logged in users.

am doing further testing later, but your feedback would be useful

[Clivesmith]

Just realized that even without your code it comes back as /topic/adelaide-airport-adl/?moderation_pending=7608 never noticed that before

[Robin W]

or this for full name

add_shortcode ('kiki-greet' , 'kiki_greet' ) ;

function kiki_greet() {
$user = wp_get_current_user() ;
echo $user->first_name.' '.$user->last_name ;
}

[Robin W]

ah, sorry forgot that I had done that for you !

add_shortcode ('kiki-greet' , 'kiki_greet' ) ;

function kiki_greet() {
$user = wp_get_current_user() ;
echo $user->display_name ;
}

and [kiki-greet]

[Kikis]

add_shortcode (‘kiki-stats’ , ‘kiki_show_stats’ ) ;

function kiki_show_stats() {
$user_count = count_users();
$users = $user_count[‘total_users’] ;
$topic_count = wp_count_posts(‘topic’);
$topics = $topic_count->publish;
$reply_count = wp_count_posts(‘reply’);
$replies = $reply_count->publish ;
$text = ‘Our ‘.$users.’ users have created ‘.$topics. ‘ topics with ‘.$replies.’ replies.’ ;
echo $text ;
}

[Clivesmith]

Hi Robin,
Thank you, I would like to echo Chuckle’s comment, you are really appreciated.

The code above returns me to the topic page with address as /topic/ministro-pistarini-ezeiza-international-airport-eze/?moderation_pending=7364

I added a { to the function line of the code above. I assume I did not need the original code as well.

[Robin W]

authors are working on stuff – they chat on slack most days – they were discussing topic counts yesterday. But they have day jobs, and writing and testing fixes when your plugin is affecting 300,000 users takes a lot of time and careful thought.

I do this totally voluntarily, so strain is entirely self inflicted 🙂 But thanks that you are thinking of me, cheers me up !

It would be good to get some feedback on the code, but I’m chugging on looking at it as a side project whilst I wait clients coming back on paid projects.

[Robin W]

just had a thought

create a WordPress page with a permalink of ‘/moderation’ and in that put some text say

‘your post is being held pending moderation’

tehn put this in your child theme’s function file – or use

Code Snippets

//add message if reply held in moderation
add_filter ('bbp_new_reply_redirect_to' , 'rew_pending_check', 10 , 3) ;

function rew_pending_check  ($reply_url, $redirect_to, $reply_id)
	$status = get_post_status ($reply_id) ;
	if ($status == 'pending' ) {
		$reply_url = '/moderation/' ;
	}
return $reply_url ;
}

not perfect by any means, but better than the nothing they now get

If you could test for me, I’ll see if I can improve how it works

[Robin W]

@clivesmith – had a think about guest posters – we can’t show the post, as by definition they are not logged in, so unless you show all moderation replies to all users (which defeats the purpose of moderation 🙂 ) they won’t see the post.

bbpress doesn’t have the functionality that chuckie shows for topics above.

I’ll try and see if I can work some code as I dig further into this, but no promises 🙂

[Robin W]

I’ve had a play this evening with this issue.

This code is very rough and ready, but if it works as I think it does, then it shows pending REPLIES to the user who posted them and moderators and keymasters. NOT topics !!

so when a user posts a reply and it goes into moderation, they see there reply in the topic with a warning.

If you want to try it, put this in your child theme’s function file – or use

Code Snippets

add_filter ('bbp_has_replies' , 'rew_has_replies' ) ;
	
function rew_has_replies( $args = array() ) {

	/** Defaults **************************************************************/

	// Other defaults
	$default_reply_search   = bbp_sanitize_search_request( 'rs' );
	$default_post_parent    = ( bbp_is_single_topic() ) ? bbp_get_topic_id() : 'any';
	$default_post_type      = ( bbp_is_single_topic() && bbp_show_lead_topic() ) ? bbp_get_reply_post_type() : array( bbp_get_topic_post_type(), bbp_get_reply_post_type() );
	$default_thread_replies = (bool) ( bbp_is_single_topic() && bbp_thread_replies() );

	// Default query args
	$default = array(
		'post_type'              => $default_post_type,         // Only replies
		'post_parent'            => $default_post_parent,       // Of this topic
		'posts_per_page'         => bbp_get_replies_per_page(), // This many
		'paged'                  => bbp_get_paged(),            // On this page
		'orderby'                => 'date',                     // Sorted by date
		'order'                  => 'ASC',                      // Oldest to newest
		'hierarchical'           => $default_thread_replies,    // Hierarchical replies
		'ignore_sticky_posts'    => true,                       // Stickies not supported
		'update_post_term_cache' => false,                      // No terms to cache

		// Conditionally prime the cache for all related posts
		'update_post_family_cache' => true
	);

	// Only add 's' arg if searching for replies
	// See https://bbpress.trac.wordpress.org/ticket/2607
	if ( ! empty( $default_reply_search ) ) {
		$default['s'] = $default_reply_search;
	}

	// What are the default allowed statuses (based on user caps)
	if ( bbp_get_view_all( 'edit_others_replies' ) ) {

		// Default view=all statuses
		$post_statuses = array_keys( bbp_get_topic_statuses() );

		// Add support for private status
		if ( current_user_can( 'read_private_replies' ) ) {
			$post_statuses[] = bbp_get_private_status_id();
		}

		// Join post statuses together
		$default['post_status'] = $post_statuses;

	// Lean on the 'perm' query var value of 'readable' to provide statuses
	} else {
		//get public and pending (not sure if we need this or just to remove the perm status that was here ?
		$post_statuses = array_keys( rew_get_topic_statuses() );
	}

	/** Setup *****************************************************************/

	// Parse arguments against default values
	$r = bbp_parse_args( $args, $default, 'has_replies' );

	// Set posts_per_page value if replies are threaded
	$replies_per_page = (int) $r['posts_per_page'];
	if ( true === $r['hierarchical'] ) {
		$r['posts_per_page'] = -1;
	}

	// Get bbPress
	$bbp = bbpress();
	
	//now filter the query before execution
	
	// Add filter if participant
	$user_id = get_current_user_id() ;
	$role = bbp_get_user_role( $user_id );
	if ($role == 'bbp_participant'  || $role == 'bbp_moderator' || bbp_is_user_keymaster($user_id)) { 
		add_filter( 'posts_where', 'rew_where' );
	}

	// Call the query
	$bbp->reply_query = new WP_Query( $r );
	
	
		// Remove filter
	if ($role == 'bbp_participant'  || $role == 'bbp_moderator' || bbp_is_user_keymaster($user_id)) { 
		remove_filter( 'posts_where', 'rew_where' );
	}
	
	// Maybe prime the post author caches
	if ( ! empty( $r['update_post_family_cache'] ) ) {
		bbp_update_post_family_caches( $bbp->reply_query->posts );
	}

	// Add pagination values to query object
	$bbp->reply_query->posts_per_page = (int) $replies_per_page;
	$bbp->reply_query->paged          = (int) $r['paged'];

	// Never home, regardless of what parse_query says
	$bbp->reply_query->is_home        = false;

	// Reset is_single if single topic
	if ( bbp_is_single_topic() ) {
		$bbp->reply_query->is_single = true;
	}

	// Only add reply to if query returned results
	if ( ! empty( $bbp->reply_query->found_posts ) ) {

		// Get reply to for each reply
		foreach ( $bbp->reply_query->posts as &$post ) {

			// Check for reply post type
			if ( bbp_get_reply_post_type() === $post->post_type ) {
				$reply_to = bbp_get_reply_to( $post->ID );

				// Make sure it's a reply to a reply
				if ( empty( $reply_to ) || ( bbp_get_reply_topic_id( $post->ID ) === $reply_to ) ) {
					$reply_to = 0;
				}

				// Add reply_to to the post object so we can walk it later
				$post->reply_to = $reply_to;
			}
		}
	}

	// Only add pagination if query returned results
	if ( ! empty( $bbp->reply_query->found_posts ) && ! empty( $bbp->reply_query->posts_per_page ) ) {

		// Figure out total pages
		if ( true === $r['hierarchical'] ) {
			$walker      = new BBP_Walker_Reply();
			$total_pages = ceil( $walker->get_number_of_root_elements( $bbp->reply_query->posts ) / $bbp->reply_query->posts_per_page );
		} else {

			// Total for pagination boundaries
			$total_pages = ( $bbp->reply_query->posts_per_page === $bbp->reply_query->found_posts )
				? 1
				: ceil( $bbp->reply_query->found_posts / $bbp->reply_query->posts_per_page );

			// Pagination settings with filter
			$bbp_replies_pagination = apply_filters( 'bbp_replies_pagination', array(
				'base'    => bbp_get_replies_pagination_base( bbp_get_topic_id() ),
				'total'   => $total_pages,
				'current' => $bbp->reply_query->paged
			) );

			// Add pagination to query object
			$bbp->reply_query->pagination_links = bbp_paginate_links( $bbp_replies_pagination );
		}
	}

	// Filter & return
	return apply_filters( 'rew_has_replies', $bbp->reply_query->have_posts(), $bbp->reply_query );
}

function rew_get_topic_statuses( $topic_id = 0 ) {

	// Filter & return
	return (array) apply_filters( 'bbp_get_topic_statuses', array(
		bbp_get_public_status_id()  => _x( 'Open',    'Open the topic',      'bbpress' ),
		), $topic_id );
}

function rew_where( $where ) {
	$user_id = get_current_user_id() ;
    global $wpdb;
	$posts = $wpdb->posts ;
    return $where . " OR ( 
             ".$posts.".post_author = ".$user_id."
        AND  ".$posts.".post_status = 'pending'
		                 
    ) ";
}

add_action ('bbp_theme_before_reply_content' , 'rew_pending' );

function rew_pending () {
	$id = bbp_get_reply_id() ;
	$status = get_post_status ($id) ;
	if ($status == 'pending' ) {
	echo '<i><b>This reply is pending review and can only be seen by you and the administrators</b></i>' ;
	}
	
}

[Robin W]

nor have I.

The function is called by core/filters.php

Line 203: add_filter( 'bbp_get_form_forum_content', 'bbp_code_trick_reverse' );
Line 206: add_filter( 'bbp_get_form_topic_content', 'bbp_code_trick_reverse' );
Line 209: add_filter( 'bbp_get_form_reply_content', 'bbp_code_trick_reverse' );

so you could try

remove_filter ('bbp_get_form_forum_content', 'bbp_code_trick_reverse' );

etc.
in your child theme and see what removing it does

[Chuckie]

Thanks.

I see line 218: $content = "\n<pre>" . $content . "</pre>\n";

I suppose that could become:

$content = "\n<pre class=\"suitable-class-name\">" . $content . "</pre>\n";

And that styling be moved into that class. But:

Line 133: $content = preg_replace_callback( "!(<pre><code>|<code>)(.*?)(</code></pre>|</code>)!s", 'bbp_decode_callback', $content );

I have no clue about what that code is doing.

[cedluck]

Hi there,

cancasstudio stated :

So the issue with topic going into pending, is that the topic or reply most likely has invalid formatted code. Example: User type is a URL and hit submit, it is gone. Well, no, it is pending. because https://domain.com/something is not allowed. And the app does tell the user of the error.

Is there any way we can allow people to type url in there topics/replies?

[Robin W]

the only place I can find <pre> is in

includes/common/formatting.php line 120 onwards

[Chuckie]

Well, the issue here is that bbPress is using a catch all for pre when it should be applying a class to the pre objects that it creates. Then we wouldn’t have this issue would we? Because they would just be classes that are used by either application with their own pre code usage.

The issue is that I don’t know how or where bbPress makes use of the pre tag so that the logic can be adjusted to use a class to indicate the styling rather than that global pre catch all.

[teazmo]

Nevermind. I found out, that I‘ve used the wrong shortcode in Notify, so formatting was removed by Notify

[Chuckie]

Turns out @robin-w that my additional css is not the solution. When the user clicks “Raw” on the code block it is supposed to toggle the raw code block to be visible. And since I added that css style it has forced it to never display.

so I need another way for bbpress to do what it wants with pre blocks that does not conflict with enlighterjs.

[Robin W]

unfortunately we can’t see that link without login details (which please do not show).

You should create an ordinary WordPress page, add a block with your text, and then follow this with a shortcode block with

[bbp-forum-index]

in it

[Alpo]

Hi everybody!

I have users from several places in our service. That means there are also several bbpress- forums in use.

I am looking for a short code which shows the forums where a user has joined. Do you have any idea? That information is available on the Buddypress profile, but it needs some clicking and our users are frustrated.

It would be nice to place the short code on sidebar so users can go directly to the forum without clicking hassle 🙂

Thanks you in advance, bbpress is dynamite 🙂

[Chuckie]

For the other issue, the only thing I can come up with is Additional CSS:

.enlighter-default .enlighter-raw {
	display:none !important;
}

But is there a more robust solution that avoids the need for this?

[Chuckie]

Hi

I have some queries and I wonder if you can help me get to the bottom of them. I would be grateful for your guidance. Here is the background.

I am using the latest bbPress plugin and I notice that the bbpress.css file is 1702 lines long.

I am using a premium theme (seos-video-premium) and even though I am using a child theme, I have noticed that my Support forum is actually using:

wp-content/themes/seos-video-premium/seos-video-premium/css/bbpress.css

I assume this is because there is no bbpress.css in my child theme css folder so it uses the themes one instead of the bbPress plugin’s one?

So I have two specific questions here.

1/ the bbpress.css file in the premium theme folder is actually 1408 lines of code. So it is 300+ lines shorter than the plugin version. To be honest, I was not expecting to find a bbpress.css file inside the theme. So what am I supposed to do? Simply replace the theme version with your plugin version?

2/ I have been trying to use a beta version of EnlighterJS Plugin (it is their beta that uses EnlighterJS v3. There was an issue with these styles:

#bbpress-forums div.bbp-topic-content pre,
#bbpress-forums div.bbp-reply-content pre {
	display: block;
	line-height: 18px;
	margin: 0 0 24px;
	padding: 5px 10px;
	white-space: pre;
	overflow: auto;
}

In their classes they have this styling:

.enlighter-default .enlighter-raw {
    display: none;
    min-width: 100%;
    line-height: inherit;
    font-size: 12px;
    font-family: inherit;
    margin: 0;
    padding: 0;
    white-space: pre-wrap;
    word-wrap: break-word;
    border: none;
    box-shadow: none;
}

The HTML is:

<pre class="enlighter-raw">.textMaterial {
  /* Uncomment to hide the material */
  /* display:none;*/
  font-size: 10pt;
  font-style: italic;
  font-weight: 700;
  background-color: yellow;
}
.textMethod {
  /* Uncomment to hide the method */
  /* display:none;*/
  font-size: 10pt;
  font-style: italic;
  font-weight: 700;
  background-color: cyan;
}</pre>

Notice that ttheir CSS style uses display: none;? The bbpress CSS file pre class has a display:block;. This causes a problem with the plugin I am trying to use.

The author does not want to use !important because he says it is bad design. So how do we fix this? How can we allow bbpress to do what it wants with pre and EnlighterJS do what it wants?

[Robin W]

yes, participants should be able to create topics and replies viz :

bbPress User Roles and Capabilities

I strongly suspect the issue is with active directory, you already have said that it rolls back settings, which makes me suspect that the entire WordPress database is being rolled back – if you create say a test post on the website, does it stay overnight, or does it disappear – this would help determine where to go next.