Merge pull request #219 from level-level/fix/template-context-priorit…

…y-v1

Fix/template context priority v1

.github/pull_request_template.md

@@ -0,0 +1,28 @@
+
+<!--- Provide a general summary of your changes in the Title above -->
+
+## Issue link
+<!--- Link to the origin of the issue in Asana or HelpScout. -->
+
+## Description
+<!--- Describe your changes in detail -->
+
+## How Has This Been Tested?
+<!--- Please describe in detail how you tested your changes. -->
+<!--- Include details of your testing environment, tests ran to see how -->
+<!--- your change affects other areas of the code, etc. -->
+
+## Screenshots (if appropriate):
+
+## Types of changes
+<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
+
+## Checklist:
+<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
+<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
+- [ ] My code follows the code style of this project.
+- [ ] My change requires a change to the documentation.
+- [ ] I have updated the documentation accordingly.

.github/workflows/test.yml

@@ -5,7 +5,7 @@ jobs:
   build:
     strategy:
       matrix:
-        php: ['7.2', '7.3', '7.4']
+        php: ['7.4']
       fail-fast: false
     name: PHP ${{ matrix.php }}
     runs-on: ubuntu-18.04

readme.txt

@@ -22,6 +22,10 @@ Yes, at Level Level we use it for all of our new projects. It's already running
 
 == Changelog ==
 
+= 1.1.1 - March 11, 2021 =
+
+* Change the priority of the clarkson_core_template_context filter from 10 to 5 to make sure Clarkson Core fills the context first.
+
 = 1.1.0 =
 
 * Adds a new `clarkson_core_block_context_{$name}` hook to modify variables available in the twig render function of a block.

src/Template_Context.php

@@ -11,11 +11,11 @@ class Template_Context {
 	 * Register all hooks to add context to the template call.
 	 */
 	public function register_hooks():void {
-		add_filter( 'clarkson_core_template_context', array( $this, 'add_author' ), 10, 2 );
-		add_filter( 'clarkson_core_template_context', array( $this, 'add_term' ), 10, 2 );
-		add_filter( 'clarkson_core_template_context', array( $this, 'add_search_count' ), 10, 2 );
-		add_filter( 'clarkson_core_template_context', array( $this, 'add_posts' ), 10, 2 );
-		add_filter( 'clarkson_core_template_context', array( $this, 'add_post_type' ), 10, 2 );
+		add_filter( 'clarkson_core_template_context', array( $this, 'add_author' ), 5, 2 );
+		add_filter( 'clarkson_core_template_context', array( $this, 'add_term' ), 5, 2 );
+		add_filter( 'clarkson_core_template_context', array( $this, 'add_search_count' ), 5, 2 );
+		add_filter( 'clarkson_core_template_context', array( $this, 'add_posts' ), 5, 2 );
+		add_filter( 'clarkson_core_template_context', array( $this, 'add_post_type' ), 5, 2 );
 	}
 
 	/**