Fix all types

Author: szepeviktor

phpstan.neon

@@ -7,8 +7,8 @@ parameters:
         - tiny-nav-menu-cache.php
         - tiny-translation-cache.php
     ignoreErrors:
-#        - '#^Function do_action invoked with [345] parameters, 1-2 required\.$#'
         - '#^Function apply_filters invoked with [345] parameters, 2 required\.$#'
+        - '#^Function do_action invoked with [345] parameters, 1-2 required\.$#'
         -
             path: tiny-nav-menu-cache.php
             message: '#^Access to an undefined property object::\$menu_id\.$#'

tiny-cache.php

@@ -32,6 +32,7 @@ function tiny_cache_skip_cache() {
  *
  * @param string $more_link_text
  * @param bool   $strip_teaser
+ * @return void
  */
 function the_content_cached( $more_link_text = null, $strip_teaser = false ) {
 
@@ -50,6 +51,7 @@ function the_content_cached( $more_link_text = null, $strip_teaser = false ) {
 
     // Cache hit.
     if ( $found ) {
+        /** @var string $cached */
         // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
         print $cached;
 
@@ -81,6 +83,7 @@ function the_content_cached( $more_link_text = null, $strip_teaser = false ) {
  *
  * @param string $more_link_text
  * @param bool   $strip_teaser
+ * @return string
  */
 function get_the_content_cached( $more_link_text = null, $strip_teaser = false ) {
 
@@ -97,6 +100,7 @@ function get_the_content_cached( $more_link_text = null, $strip_teaser = false )
 
     // Cache hit.
     if ( $found ) {
+        /** @var string $cached */
         return $cached;
     }
 
@@ -148,6 +152,7 @@ function tiny_cache_save_the_content( $content ) {
  * @param string $slug
  * @param string $name
  * @param string $version_hash
+ * @return void
  */
 function get_template_part_cached( $slug, $name = null, $version_hash = '' ) {
 
@@ -174,6 +179,7 @@ function get_template_part_cached( $slug, $name = null, $version_hash = '' ) {
 
     // Cache hit.
     if ( $found ) {
+        /** @var string $cached */
         // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
         print $cached;
 
@@ -199,6 +205,7 @@ function get_template_part_cached( $slug, $name = null, $version_hash = '' ) {
  * Delete cached content by ID.
  *
  * @param int $post_id
+ * @return void
  */
 function tiny_cache_delete_the_content( $post_id ) {
 
@@ -211,6 +218,7 @@ function tiny_cache_delete_the_content( $post_id ) {
  * @param string   $new_status
  * @param string   $old_status
  * @param \WP_Post $post
+ * @return void
  */
 function tiny_cache_post_transition( $new_status, $old_status, $post ) {
 
@@ -224,6 +232,8 @@ function tiny_cache_post_transition( $new_status, $old_status, $post ) {
 
 /**
  * Hook cache delete actions.
+ *
+ * @return void
  */
 function tiny_cache_actions() {
 

tiny-translation-cache.php

@@ -26,11 +26,20 @@ public function __construct() {
         add_action( 'muplugins_loaded', array( $this, 'init' ) );
     }
 
+    /**
+     * @return void
+     */
     public function init() {
 
         add_filter( 'override_load_textdomain', array( $this, 'load_textdomain' ), 30, 3 );
     }
 
+    /**
+     * @param bool $override
+     * @param string $domain
+     * @param string $mofile
+     * @return bool
+     */
     public function load_textdomain( $override, $domain, $mofile ) {
 
         // Copied from core
@@ -39,8 +48,9 @@ public function load_textdomain( $override, $domain, $mofile ) {
 
         $mo    = new \MO();
         $key   = $this->get_key( $domain, $mofile );
-        $found = null;
+        $found = false;
         // @TODO Compress stored data unserialize() and gzinflate( $ )
+        /** @var array{entries?: string, headers?: array<mixed>} $cache */
         $cache = wp_cache_get( $key, self::GROUP, false, $found );
 
         if ( $found && isset( $cache['entries'], $cache['headers'] ) ) {
@@ -73,13 +83,21 @@ public function load_textdomain( $override, $domain, $mofile ) {
         return true;
     }
 
+    /**
+     * @param string $domain
+     * @param string $mofile
+     * @return string
+     */
     private function get_key( $domain, $mofile ) {
 
         // @FIXME Why do we need text domain? Isn't the full path exact enough?
         // Hash of text domain and .mo file path
         return md5( $domain . $mofile );
     }
 
+    /**
+     * @return void
+     */
     private function exit_with_instructions() {
 
         $doc_root = isset( $_SERVER['DOCUMENT_ROOT'] ) ? $_SERVER['DOCUMENT_ROOT'] : ABSPATH; // WPCS: input var, sanitization ok.