coralseait

Custom CDN usage for sure; keep that in.

Hello,

Thanks very much for the feedback. For this project, the way the AD is implemented and the complexity of their AD it is best to use title (which holds their roles within the AD).

As a work around we’ve implemented some mapping based on ADI capturing AD Title in User Meta and then mapping via ACF options. It works very well, without filtering or requiring mod to your code AND without requiring a massive change to their legacy AD structure.

mrpink84,

Perhaps you could style the ul and li (since they have classes) like bootstrap’s list-inline?

essentially if you do similar to this from bootsrap CSS:

.list-inline {
  padding-left: 0;
  margin-left: -5px;
  list-style: none;
}
.list-inline > li {
  display: inline-block;
  padding-right: 5px;
  padding-left: 5px;
}

But using rpwe-ul and rpwe-li

I typically include Boostrap and Font Awesome now in all my WordPress projects and list-inline is wonderful!

Thanks very much Satrya, very appreciated!

No worries, happy to help. Hopefully you can mod it to suit your needs!

Here you go, here’s the dev versions of our code:

// ******************** Related Posts Logic ********************

	// Help Function for Checking Null or Empty

	// Function for basic field validation (present and neither empty nor only white space
	function IsNullOrEmptyString($question){
	    return (!isset($question) || trim($question)==='');
	};

	// constants
	define('CNST_TRC_CATEGORY_ALERT_ID', 2);
	define('CNST_TRC_CATEGORY_NEWS_ID', 3);

	// defaults
	$trc_number_of_alert_posts = 2;
	$trc_number_of_news_posts = 3;
	$trc_number_of_related_posts = 5;

// This will pull post ids for alerts and news
function trc_get_related_alerts_news_post_ids( $trc_number_of_alerts_posts, $trc_number_of_news_posts) {

	// first get related alerts

	// setup our query args
	$trc_related_posts_args = array (
		'post_type' 		=> 'post',
		'cat'			=> CNST_TRC_CATEGORY_ALERT_ID,
		'posts_per_page' 	=> $trc_number_of_alerts_posts,
		'fields' 		=> 'ids',
	);

	// new wp query to return the post ids matching
	$trc_related_posts_ids_query = new WP_Query( $trc_related_posts_args);

	// save alert post ids
	foreach( $trc_related_posts_ids_query->posts as $trc_alert_post_id) {
		$trc_query_alerts_post_ids[] = $trc_alert_post_id;
	};

	// reset wp post data
	wp_reset_postdata();

	// now get related news

	// setup our query args
	// here we limit the query so it excludes alert ids we already found
	$trc_related_posts_args = array (
		'post_type' 		=> 'post',
		'post__not_in'		=> $trc_query_alerts_post_ids,
		'cat'			=> CNST_TRC_CATEGORY_NEWS_ID,
		'posts_per_page' 	=> $trc_number_of_news_posts,
		'fields' 		=> 'ids',
	);

	// new wp query to return the post ids matching
	$trc_related_posts_ids_query = new WP_Query( $trc_related_posts_args);

	// save news post ids
	foreach( $trc_related_posts_ids_query->posts as $trc_news_post_id) {
		$trc_query_news_post_ids[] = $trc_news_post_id;
	};

	// reset wp post data
	wp_reset_postdata();

	// merge our arrays of ids
	$trc_query_post_ids = array_merge( $trc_query_alerts_post_ids, $trc_query_news_post_ids);

	return $trc_query_post_ids;
}

//* Related Posts Filters
add_filter( 'rpwe_default_query_arguments', 'trc_related_posts' );
function trc_related_posts( $args ) {

	// arrays
	$trc_related_alerts_news_ids = array();
	$trc_current_post_categories = array();
	$trc_full_related_ids = array();

	// current post data
	global $post;
	$trc_current_post_id = $post->ID;

	//echo '<br>The Current Post ID: ' . $trc_current_post_id;

	// get the current post categories
	$trc_current_post_categories = wp_get_post_categories( $trc_current_post_id);

	//echo '<br>The Current Post Categories: ';
	//echo var_dump( $trc_current_post_categories);

	// First get alerts and news
	$trc_related_alerts_news_ids = trc_get_related_alerts_news_post_ids( $trc_number_of_alerts_posts, $trc_number_of_news_posts);

	// Now query WordPress:
	// Use our Current Post Categories
	// Exclude our Alerts and News IDs
	// setup our query args
	$trc_related_posts_args = array (
		'post_type' 		=> 'post',
		'post__not_in'		=> $trc_related_alerts_news_ids,
		'category__in'		=> $trc_current_post_categories,
		'posts_per_page' 	=> $trc_number_of_related_posts ,
		'fields' 		=> 'ids',
	);
	$trc_related_posts_query = new WP_Query( $trc_related_posts_args);

	// save related post ids
	foreach( $trc_related_posts_query->posts as $trc_related_post_id) {
		$trc_related_post_ids[] = $trc_related_post_id;
	};

	// reset wp post data
	wp_reset_postdata();

	//echo '<br>Related Post IDs: ';
	//echo var_dump( $trc_related_post_ids);

	// now merge related posts and alerts / news
	// check for null or empty first
	if ( IsNullOrEmptyString( $trc_related_post_ids)) {
		$trc_full_related_ids = $trc_related_alerts_news_ids;
	} else {
		$trc_full_related_ids = array_merge( $trc_related_post_ids, $trc_related_alerts_news_ids);
	};

	//echo '<br>Full Related IDs: ';
	//echo var_dump( $trc_full_related_ids);

	// pass our related post ids to the plugin args
	$args['post__in'] = $trc_full_related_ids;

	return $args;
}

No worries, if you need a sample of the code let me know.

formattingissue,

I’ve done something like you need via the ability to filter the query args going to RPWE. In our case we needed to always pull 2 post from one cat, 3 from another and append them to all other related posts. It was straight forward to accomplish this by building an array of post ids matching and sending them to RPWE via post__in arg.

We use iThemes security for xml-rpc attack protection with good results.

Perfect !!!

//* Override Genesis Easy Columns so Authors and up have button
remove_action('init', 'gc_add_button');
function trc_gc_add_button() {
	   if ( current_user_can('edit_posts'))
	   {
		  	add_filter('mce_external_plugins', 'gc_add_plugin');
			add_filter('mce_buttons', 'gc_register_button');
	   }
	}
add_action('init', 'trc_gc_add_button');

Having a similar problem with Genesis 2.0 / Landscape child theme on WP 3.6 with latest W3TC. On other installs I’m not having the issue on G 2.0, but this is my only landscape child site.

We are using GDE with W3 Total Cache (integrated with Cloudflare as well) with zero issues. It works very well, Cloudflare is handling minification and rocket loader is on and there are no problems.

Yoast will probably fix it in the next release. I’m not sure it is really causing problems with google as it seems to be processing the child xml maps ok but I won’t know until next full crawl.

Possible, I believe I’ve found the offending issue with my sitemap.

The category timestamp is different than the others

<sitemap>
<loc>https://www.coralseait.com/attachment-sitemap.xml</loc>
<lastmod>2013-06-22T00:58:11+00:00</lastmod>
</sitemap>
<sitemap>
<loc>https://www.coralseait.com/category-sitemap.xml</loc>
<lastmod>2013-06-23 13:04:02</lastmod>
</sitemap>
<sitemap>


Notice the different format, and this is indeed the line google complains about. I tried updating a category to force a new timestamp which did happen, but still in the wrong format.

Thank you ricky7622 – That solved it for me too! Cheers!