Phpstan issues 1 (#405)

* Components fixed
* Ongoing PHPStan issue addressed
* Workflow added
* Integration of Scrutinizer deleted
* Badges added & design modified

Author: lloc

.github/workflows/deploy.yml

@@ -7,6 +7,11 @@ jobs:
   tag:
     name: New tag
     runs-on: ubuntu-latest
+    permissions:
+      id-token: write
+      contents: read
+      attestations: write
+    timeout-minutes: 70
     steps:
       - name: Checkout project
         uses: actions/checkout@v4
@@ -16,7 +21,14 @@ jobs:
         run: composer run-script build
       - name: WordPress Plugin Deploy
         uses: 10up/action-wordpress-plugin-deploy@stable
+        id: deploy
         env:
           SVN_PASSWORD: ${{ secrets.SVN_PASSWORD }}
           SVN_USERNAME: ${{ secrets.SVN_USERNAME }}
           SLUG: multisite-language-switcher
+        with:
+          generate-zip: true
+      - name: Attest build provenance
+        uses: johnbillion/action-wordpress-plugin-attestation@0.5.0
+        with:
+          zip-path: ${{ steps.deploy.outputs.zip-path }}

.scrutinizer.yml

@@ -2,14 +2,16 @@
 
 A simple, powerful, and user-friendly plugin that adds multilingual support to your [WordPress multisite](https://wordpress.org/documentation/article/create-a-network/) installation, whether using multiple subdomains or folders. Multisite Language Switcher allows you to effortlessly manage translations for posts, pages, custom post types, categories, tags, and custom taxonomies.
 
-[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/lloc/Multisite-Language-Switcher/badges/quality-score.png?s=a2e5dbac2690bbd427f2d48b84473482e7e24fdb)](https://scrutinizer-ci.com/g/lloc/Multisite-Language-Switcher/) 
-[![codecov](https://codecov.io/gh/lloc/Multisite-Language-Switcher/graph/badge.svg?token=IlD4bX4KZ4)](https://codecov.io/gh/lloc/Multisite-Language-Switcher) 
+[![WordPress Plugin Version](https://img.shields.io/wordpress/plugin/v/multisite-language-switcher.svg)](https://wordpress.org/plugins/multisite-language-switcher/)
+[![License](https://img.shields.io/badge/license-GPL_v2%2B-blue.svg)](http://opensource.org/licenses/GPL-2.0)
+[![WordPress Tested](https://img.shields.io/wordpress/v/multisite-language-switcher.svg)](https://wordpress.org/plugins/multisite-language-switcher/)
+[![codecov](https://codecov.io/gh/lloc/Multisite-Language-Switcher/graph/badge.svg?token=IlD4bX4KZ4)](https://codecov.io/gh/lloc/Multisite-Language-Switcher)
 
 ## Where to get the plugin
 
 [Download](http://downloads.wordpress.org/plugin/multisite-language-switcher.zip) the latest stable from the [WordPress Plugin Directory](https://wordpress.org/plugins/multisite-language-switcher/), and please remember to give this plugin [a five-star rating](https://wordpress.org/support/plugin/multisite-language-switcher/reviews/#new-post).
 
-_Please note that the version of Multisite Language Switcher on GitHub is a work in progress._
+_Please note that while the master branch on GitHub is intended to be stable, it is not recommended for production use. Instead, please use the [official releases](https://github.com/lloc/Multisite-Language-Switcher/releases) for deployment._
 
 If you plan to use the GitHub repository on a server, don't forget to run `composer install --no-dev`.
 

README.md

@@ -34,7 +34,7 @@
   },
   "scripts": {
     "test": "vendor/bin/phpunit",
-    "coverage": "php -d xdebug.mode=coverage vendor/bin/phpunit --coverage-html tests/coverage",
+    "coverage": "@php -d xdebug.mode=coverage vendor/bin/phpunit --coverage-html tests/coverage",
     "analyze": "vendor/bin/phpstan analyze",
     "playwright": "npx playwright test",
     "php74": "phpcs -p ./*.php includes/ --standard=vendor/phpcompatibility/php-compatibility/PHPCompatibility --runtime-set testVersion 7.4",

composer.json

@@ -9,9 +9,9 @@
 interface Importer {
 
 	/**
-	 * @param array $data
+	 * @param array<string, mixed> $data
 	 *
-	 * @return array
+	 * @return array<string, mixed>
 	 */
 	public function import( array $data );
 

includes/ContentImport/Importers/Importer.php

@@ -22,6 +22,11 @@ public static function info() {
 		);
 	}
 
+	/**
+	 * @param array<string, mixed> $data
+	 *
+	 * @return array<string, mixed>
+	 */
 	public function import( array $data ) {
 		$source_blog_id = $this->import_coordinates->source_blog_id;
 		$source_post_id = $this->import_coordinates->source_post_id;

includes/ContentImport/Importers/PostMeta/Duplicating.php

@@ -32,6 +32,11 @@ public static function info() {
 		);
 	}
 
+	/**
+	 * @param array<string, mixed> $data
+	 *
+	 * @return array<string, mixed>
+	 */
 	public function import( array $data ) {
 		$source_blog_id = $this->import_coordinates->source_blog_id;
 		$source_post_id = $this->import_coordinates->source_post_id;
@@ -142,7 +147,7 @@ function ( &$entry ) {
 	/**
 	 * @param int $source_post_thumbnail_id
 	 *
-	 * @return array
+	 * @return array<string, mixed>
 	 */
 	protected function get_attachment_meta( $source_post_thumbnail_id ) {
 		$keys = array( '_wp_attached_file', '_wp_attachment_metadata', '_wp_attachment_image_alt' );

includes/ContentImport/Importers/PostThumbnail/Linking.php

@@ -15,6 +15,11 @@ class AdminNoticeLogger extends MslsRegistryInstance implements LogWriter {
 	 */
 	protected $import_coordinates;
 
+	/**
+	 * @param array<string, mixed> $data
+	 *
+	 * @return void
+	 */
 	public function write( array $data ) {
 		/* translators: %1$d: source post ID, %2$d: source blog ID, %3$d: destination post ID, %4$d: destination blog ID */
 		$format = esc_html__( 'From post %1$d on site %2$d to post %3$d on site %4$d', 'multisite-language-switcher' );
@@ -108,9 +113,9 @@ public function write( array $data ) {
 	}
 
 	/**
-	 * @param string $section_title
-	 * @param array  $entries
-	 * @param bool   $escape_entries
+	 * @param string   $section_title
+	 * @param string[] $entries
+	 * @param bool     $escape_entries
 	 *
 	 * @return string
 	 */

includes/ContentImport/LogWriters/AdminNoticeLogger.php

@@ -2,14 +2,13 @@
 
 namespace lloc\Msls\ContentImport\LogWriters;
 
-
 interface LogWriter {
 	/**
 	 * Writes the log to the destination.
 	 *
-	 * @param array $data An array of data to log.
+	 * @param array<string, mixed> $data An array of data to log.
 	 *
 	 * @return mixed
 	 */
 	public function write( array $data );
-}
+}

includes/ContentImport/LogWriters/LogWriter.php

@@ -14,6 +14,9 @@
 
 class MetaBox extends MslsRegistryInstance {
 
+	/**
+	 * @var array<string, mixed>
+	 */
 	protected array $data = array();
 
 	/**
@@ -90,6 +93,11 @@ function ( $lang ) use ( $mydata ) {
 		echo wp_kses( $output, Component::get_allowed_html() );
 	}
 
+	/**
+	 * @param array<string, mixed> $data
+	 *
+	 * @return string
+	 */
 	protected function inline_thickbox_url( array $data = array() ): string {
 		$args = array_merge(
 			array(
@@ -111,6 +119,12 @@ public function print_modal_html(): void {
 		echo $this->inline_thickbox_html( true, $this->data );
 	}
 
+	/**
+	 * @param bool                 $echo
+	 * @param array<string, mixed> $data
+	 *
+	 * @return string
+	 */
 	protected function inline_thickbox_html( $echo = true, array $data = array() ): string {
 		if ( ! isset( $data['msls_import'] ) ) {
 			return '';