From 0a6cb4da47ecfc18d8570031e62b522628dd8301 Mon Sep 17 00:00:00 2001
From: Erik van der Bas <erik@basedonline.nl>
Date: Tue, 21 May 2024 09:35:50 +0200
Subject: [PATCH] feat: rework of PHP files. (#152)

* feat: rework of PHP files.
* chore: update namespace
* fix: don't provide post again
* Remove $dirname since we only use the default location now.
* chore: remove unneeded get_post(s) calls
---
 404.php             | 15 ++++++------
 archive.php         | 50 ++++++++++++++++++-------------------
 author.php          | 26 ++++++++++++--------
 functions.php       | 17 +++++++------
 index.php           | 29 ++++++++++++----------
 page.php            | 22 ++++++-----------
 search.php          | 20 +++++++--------
 single.php          | 23 +++++++++--------
 src/StarterSite.php | 60 ++++++++++++++++++++++++++-------------------
 9 files changed, 136 insertions(+), 126 deletions(-)

diff --git a/404.php b/404.php
index e56dea257..869be1374 100644
--- a/404.php
+++ b/404.php
@@ -1,13 +1,12 @@
 <?php
+
 /**
- * The template for displaying 404 pages (Not Found)
- *
- * Methods for TimberHelper can be found in the /functions sub-directory
- *
- * @package  WordPress
- * @subpackage  Timber
- * @since    Timber 0.1
+ * The template for the 404 page
  */
 
+namespace App;
+
+use Timber\Timber;
+
 $context = Timber::context();
-Timber::render( '404.twig', $context );
+Timber::render('404.twig', $context);
diff --git a/archive.php b/archive.php
index 0bba43c27..1305089e4 100644
--- a/archive.php
+++ b/archive.php
@@ -1,40 +1,40 @@
 <?php
+
 /**
  * The template for displaying Archive pages.
  *
  * Used to display archive-type pages if nothing more specific matches a query.
  * For example, puts together date-based pages if no date.php file exists.
  *
- * Learn more: http://codex.wordpress.org/Template_Hierarchy
- *
- * Methods for TimberHelper can be found in the /lib sub-directory
+ * Learn more: https://developer.wordpress.org/themes/basics/template-hierarchy/
  *
- * @package  WordPress
- * @subpackage  Timber
- * @since   Timber 0.2
  */
 
-$templates = array( 'archive.twig', 'index.twig' );
+namespace App;
+
+use Timber\Timber;
 
-$context = Timber::context();
+$templates = array('archive.twig', 'index.twig');
 
-$context['title'] = 'Archive';
-if ( is_day() ) {
-	$context['title'] = 'Archive: ' . get_the_date( 'D M Y' );
-} elseif ( is_month() ) {
-	$context['title'] = 'Archive: ' . get_the_date( 'M Y' );
-} elseif ( is_year() ) {
-	$context['title'] = 'Archive: ' . get_the_date( 'Y' );
-} elseif ( is_tag() ) {
-	$context['title'] = single_tag_title( '', false );
-} elseif ( is_category() ) {
-	$context['title'] = single_cat_title( '', false );
-	array_unshift( $templates, 'archive-' . get_query_var( 'cat' ) . '.twig' );
-} elseif ( is_post_type_archive() ) {
-	$context['title'] = post_type_archive_title( '', false );
-	array_unshift( $templates, 'archive-' . get_post_type() . '.twig' );
+$title = 'Archive';
+if (is_day()) {
+	$title = 'Archive: ' . get_the_date('D M Y');
+} elseif (is_month()) {
+	$title = 'Archive: ' . get_the_date('M Y');
+} elseif (is_year()) {
+	$title = 'Archive: ' . get_the_date('Y');
+} elseif (is_tag()) {
+	$title = single_tag_title('', false);
+} elseif (is_category()) {
+	$title = single_cat_title('', false);
+	array_unshift($templates, 'archive-' . get_query_var('cat') . '.twig');
+} elseif (is_post_type_archive()) {
+	$title = post_type_archive_title('', false);
+	array_unshift($templates, 'archive-' . get_post_type() . '.twig');
 }
 
-$context['posts'] = Timber::get_posts();
+$context = Timber::context([
+	'title' => $title,
+]);
 
-Timber::render( $templates, $context );
+Timber::render($templates, $context);
diff --git a/author.php b/author.php
index 22935423a..2d63a05a5 100644
--- a/author.php
+++ b/author.php
@@ -1,21 +1,27 @@
 <?php
+
 /**
  * The template for displaying Author Archive pages
  *
  * Methods for TimberHelper can be found in the /lib sub-directory
  *
- * @package  WordPress
- * @subpackage  Timber
- * @since    Timber 0.1
  */
 
+namespace App;
+
+use Timber\Timber;
+
 global $wp_query;
 
-$context          = Timber::context();
-$context['posts'] = Timber::get_posts();
-if ( isset( $wp_query->query_vars['author'] ) ) {
-	$author            = Timber::get_user( $wp_query->query_vars['author'] );
-	$context['author'] = $author;
-	$context['title']  = 'Author Archives: ' . $author->name();
+$title = false;
+if (isset($wp_query->query_vars['author'])) {
+	$author = Timber::get_user($wp_query->query_vars['author']);
+	$title  = 'Author Archives: ' . $author->name();
 }
-Timber::render( array( 'author.twig', 'archive.twig' ), $context );
+
+$context = Timber::context([
+	'title'  => $title,
+	'author' => $author,
+]);
+
+Timber::render(array('author.twig', 'archive.twig'), $context);
diff --git a/functions.php b/functions.php
index 43cddcbec..1cae0e0a9 100644
--- a/functions.php
+++ b/functions.php
@@ -1,17 +1,20 @@
 <?php
+
 /**
- * Timber starter-theme
- * https://github.com/timber/starter-theme
+ * Functions and definitions
+ *
+ * @link https://developer.wordpress.org/themes/basics/theme-functions/
+ * @link https://github.com/timber/starter-theme
  */
 
+namespace App;
+
+use Timber\Timber;
+
 // Load Composer dependencies.
 require_once __DIR__ . '/vendor/autoload.php';
-
 require_once __DIR__ . '/src/StarterSite.php';
 
-Timber\Timber::init();
-
-// Sets the directories (inside your theme) to find .twig files.
-Timber::$dirname = [ 'templates', 'views' ];
+Timber::init();
 
 new StarterSite();
diff --git a/index.php b/index.php
index 30dd82f4d..ed3d2c254 100644
--- a/index.php
+++ b/index.php
@@ -1,23 +1,26 @@
 <?php
+
 /**
  * The main template file
+ *
  * This is the most generic template file in a WordPress theme
  * and one of the two required files for a theme (the other being style.css).
  * It is used to display a page when nothing more specific matches a query.
- * E.g., it puts together the home page when no home.php file exists
+ * E.g., it puts together the home page when no home.php file exists.
  *
- * Methods for TimberHelper can be found in the /lib sub-directory
- *
- * @package  WordPress
- * @subpackage  Timber
- * @since   Timber 0.1
+ * @link https://developer.wordpress.org/themes/basics/template-hierarchy/
  */
 
-$context          = Timber::context();
-$context['posts'] = Timber::get_posts();
-$context['foo']   = 'bar';
-$templates        = array( 'index.twig' );
-if ( is_home() ) {
-	array_unshift( $templates, 'front-page.twig', 'home.twig' );
+use Timber\Timber;
+
+$templates = array('index.twig');
+
+if (is_home()) {
+	array_unshift($templates, 'front-page.twig', 'home.twig');
 }
-Timber::render( $templates, $context );
+
+$context = Timber::context([
+	'foo'   => 'bar',
+]);
+
+Timber::render($templates, $context);
diff --git a/page.php b/page.php
index 256fb9d11..e0612797a 100644
--- a/page.php
+++ b/page.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * The template for displaying all pages.
  *
@@ -7,22 +8,13 @@
  * and that other 'pages' on your WordPress site will use a
  * different template.
  *
- * To generate specific templates for your pages you can use:
- * /mytheme/templates/page-mypage.twig
- * (which will still route through this PHP file)
- * OR
- * /mytheme/page-mypage.php
- * (in which case you'll want to duplicate this file and save to the above path)
- *
- * Methods for TimberHelper can be found in the /lib sub-directory
- *
- * @package  WordPress
- * @subpackage  Timber
- * @since    Timber 0.1
  */
 
+namespace App;
+
+use Timber\Timber;
+
 $context = Timber::context();
+$post = $context['post'];
 
-$timber_post     = Timber::get_post();
-$context['post'] = $timber_post;
-Timber::render( array( 'page-' . $timber_post->post_name . '.twig', 'page.twig' ), $context );
+Timber::render(array('page-' . $post->post_name . '.twig', 'page.twig'), $context);
diff --git a/search.php b/search.php
index 069b3aae7..61f28f7e1 100644
--- a/search.php
+++ b/search.php
@@ -1,18 +1,16 @@
 <?php
+
 /**
  * Search results page
- *
- * Methods for TimberHelper can be found in the /lib sub-directory
- *
- * @package  WordPress
- * @subpackage  Timber
- * @since   Timber 0.1
+ * @link https://developer.wordpress.org/themes/basics/template-hierarchy/
  */
 
-$templates = array( 'search.twig', 'archive.twig', 'index.twig' );
+use Timber\Timber;
+
+$templates = array('search.twig', 'archive.twig', 'index.twig');
 
-$context          = Timber::context();
-$context['title'] = 'Search results for ' . get_search_query();
-$context['posts'] = Timber::get_posts();
+$context = Timber::context([
+   'title' => 'Search results for ' . get_search_query(),
+]);
 
-Timber::render( $templates, $context );
+Timber::render($templates, $context);
diff --git a/single.php b/single.php
index e47be9045..531fb1c3a 100644
--- a/single.php
+++ b/single.php
@@ -1,20 +1,19 @@
 <?php
+
 /**
  * The Template for displaying all single posts
- *
- * Methods for TimberHelper can be found in the /lib sub-directory
- *
- * @package  WordPress
- * @subpackage  Timber
- * @since    Timber 0.1
+ * @link https://developer.wordpress.org/themes/basics/template-hierarchy/
  */
 
-$context         = Timber::context();
-$timber_post     = Timber::get_post();
-$context['post'] = $timber_post;
+namespace App;
+
+use Timber\Timber;
+
+$context = Timber::context();
+$post = $context['post'];
 
-if ( post_password_required( $timber_post->ID ) ) {
-	Timber::render( 'single-password.twig', $context );
+if (post_password_required($post->ID)) {
+	Timber::render('single-password.twig', $context);
 } else {
-	Timber::render( array( 'single-' . $timber_post->ID . '.twig', 'single-' . $timber_post->post_type . '.twig', 'single-' . $timber_post->slug . '.twig', 'single.twig' ), $context );
+	Timber::render(array('single-' . $post->ID . '.twig', 'single-' . $post->post_type . '.twig', 'single-' . $post->slug . '.twig', 'single.twig'), $context);
 }
diff --git a/src/StarterSite.php b/src/StarterSite.php
index 9e4c9141d..3d83669a0 100644
--- a/src/StarterSite.php
+++ b/src/StarterSite.php
@@ -1,19 +1,24 @@
 <?php
 
+namespace App;
+
 use Timber\Site;
+use Timber\Timber;
 
 /**
  * Class StarterSite
  */
-class StarterSite extends Site {
-	public function __construct() {
-		add_action( 'after_setup_theme', array( $this, 'theme_supports' ) );
-		add_action( 'init', array( $this, 'register_post_types' ) );
-		add_action( 'init', array( $this, 'register_taxonomies' ) );
-
-		add_filter( 'timber/context', array( $this, 'add_to_context' ) );
-		add_filter( 'timber/twig', array( $this, 'add_to_twig' ) );
-		add_filter( 'timber/twig/environment/options', [ $this, 'update_twig_environment_options' ] );
+class StarterSite extends Site
+{
+	public function __construct()
+	{
+		add_action('after_setup_theme', array($this, 'theme_supports'));
+		add_action('init', array($this, 'register_post_types'));
+		add_action('init', array($this, 'register_taxonomies'));
+
+		add_filter('timber/context', array($this, 'add_to_context'));
+		add_filter('timber/twig', array($this, 'add_to_twig'));
+		add_filter('timber/twig/environment/options', [$this, 'update_twig_environment_options']);
 
 		parent::__construct();
 	}
@@ -21,15 +26,15 @@ public function __construct() {
 	/**
 	 * This is where you can register custom post types.
 	 */
-	public function register_post_types() {
-
+	public function register_post_types()
+	{
 	}
 
 	/**
 	 * This is where you can register custom taxonomies.
 	 */
-	public function register_taxonomies() {
-
+	public function register_taxonomies()
+	{
 	}
 
 	/**
@@ -37,7 +42,8 @@ public function register_taxonomies() {
 	 *
 	 * @param string $context context['this'] Being the Twig's {{ this }}.
 	 */
-	public function add_to_context( $context ) {
+	public function add_to_context($context)
+	{
 		$context['foo']   = 'bar';
 		$context['stuff'] = 'I am a value set in your functions.php file';
 		$context['notes'] = 'These values are available everytime you call Timber::context();';
@@ -47,9 +53,10 @@ public function add_to_context( $context ) {
 		return $context;
 	}
 
-	public function theme_supports() {
+	public function theme_supports()
+	{
 		// Add default posts and comments RSS feed links to head.
-		add_theme_support( 'automatic-feed-links' );
+		add_theme_support('automatic-feed-links');
 
 		/*
 		 * Let WordPress manage the document title.
@@ -57,14 +64,14 @@ public function theme_supports() {
 		 * hard-coded <title> tag in the document head, and expect WordPress to
 		 * provide it for us.
 		 */
-		add_theme_support( 'title-tag' );
+		add_theme_support('title-tag');
 
 		/*
 		 * Enable support for Post Thumbnails on posts and pages.
 		 *
 		 * @link https://developer.wordpress.org/themes/functionality/featured-images-post-thumbnails/
 		 */
-		add_theme_support( 'post-thumbnails' );
+		add_theme_support('post-thumbnails');
 
 		/*
 		 * Switch default core markup for search form, comment form, and comments
@@ -98,7 +105,7 @@ public function theme_supports() {
 			)
 		);
 
-		add_theme_support( 'menus' );
+		add_theme_support('menus');
 	}
 
 	/**
@@ -106,7 +113,8 @@ public function theme_supports() {
 	 *
 	 * @param string $text being 'foo', then returned 'foo bar!'.
 	 */
-	public function myfoo( $text ) {
+	public function myfoo($text)
+	{
 		$text .= ' bar!';
 		return $text;
 	}
@@ -116,14 +124,15 @@ public function myfoo( $text ) {
 	 *
 	 * @param Twig\Environment $twig get extension.
 	 */
-	public function add_to_twig( $twig ) {
+	public function add_to_twig($twig)
+	{
 		/**
 		 * Required when you want to use Twig’s template_from_string.
 		 * @link https://twig.symfony.com/doc/3.x/functions/template_from_string.html
 		 */
 		// $twig->addExtension( new Twig\Extension\StringLoaderExtension() );
 
-		$twig->addFilter( new Twig\TwigFilter( 'myfoo', [ $this, 'myfoo' ] ) );
+		$twig->addFilter(new \Twig\TwigFilter('myfoo', [$this, 'myfoo']));
 
 		return $twig;
 	}
@@ -137,9 +146,10 @@ public function add_to_twig( $twig ) {
 	 *
 	 * @return array
 	 */
-	function update_twig_environment_options( $options ) {
-	    // $options['autoescape'] = true;
+	function update_twig_environment_options($options)
+	{
+		// $options['autoescape'] = true;
 
-	    return $options;
+		return $options;
 	}
 }