Merge lp:~davidc3/resource-centre/jp-exclude-from-rss into lp:resource-centre

[David Callé]

Exclude lang:jp tagged posts from the main RSS feed

Filtering out posts containing the "lang:jp" tag, so the future Japanese blog can be derived on tag basis from blog.ubuntu.com without impacting the english RSS feed. A matching change will be made on blog.ubuntu.com to filter out these posts from being displayed.

[David Callé (davidc3)]

I was able to test and got tag exclusion working using `tag__not_in`.

[Anthony Dillon (ya-bo-ng)]

The blog loads the data via the built-in WordPress Rest API. The content is not loaded from the RSS although I guess it should be hidden from that too for people using the RSS.

I have applied your branch (and updated the tag id) to [our testing enviroment](http://insights.canonicalwebteam.com).

I created an article with the tag `lang:jp`.

This article is still [visible in the API](http://insights.canonicalwebteam.com/wp-json/wp/v2/posts?per_page=3&page=1&_embed=True).

A similar approach may be to add a condition to [blog.ubuntu.com code base](https://github.com/canonical-websites/blog.ubuntu.com/). Keeping all the logic in one place, [example PR](https://github.com/canonical-websites/blog.ubuntu.com/pull/392)

With that this branch is good.

review: Approve

[David Callé (davidc3)]

Thanks for testing, doing the same filtering as the custom snapcraft tag was the intended plan of action.

=== modified file 'functions/rss.php'
--- functions/rss.php	2017-03-02 16:07:24 +0000
+++ functions/rss.php	2018-11-20 10:49:17 +0000
@@ -6,6 +6,7 @@
 function myFeedExcluder($query) {
  if ($query->is_feed) {
    $query->set('cat','-1554,-1551,-1552,-1553,-344,-347,-346,-345');
+   $query->set('tag__not_in','3184');
  }
 return $query;
 }