From 0ee5b971bba06a20afbaafaea08a24f6e8224c7e Mon Sep 17 00:00:00 2001
From: Dennis Ploetner <re@lloc.de>
Date: Tue, 4 Jun 2024 23:22:25 +0200
Subject: [PATCH] Plugin Check review

---
 includes/MslsAdmin.php     | 18 ++++++++++--------
 includes/MslsAdminIcon.php |  1 +
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/includes/MslsAdmin.php b/includes/MslsAdmin.php
index df0700e9..264696bb 100644
--- a/includes/MslsAdmin.php
+++ b/includes/MslsAdmin.php
@@ -308,7 +308,10 @@ public function advanced_section(): int {
 	public function rewrites_section(): int {
 		$map = array();
 		foreach ( get_post_types( array( 'public' => true ), 'objects' ) as $key => $object ) {
-			$map[ "rewrite_{$key}" ] = sprintf( __( '%s Slug', 'multisite-language-switcher' ), $object->label );
+			/* translators: %s: post type label */
+			$format = __( '%s Slug', 'multisite-language-switcher' );
+
+			$map[ "rewrite_{$key}" ] = sprintf( $format, $object->label );
 		}
 
 		return $this->add_settings_fields( $map, 'rewrites_section' );
@@ -382,14 +385,13 @@ public function reference_user() {
 		if ( count( $users ) > self::MAX_REFERENCE_USERS ) {
 			$users = array_slice( $users, 0, self::MAX_REFERENCE_USERS, true );
 
-			$message = sprintf(
-				__(
-					'Multisite Language Switcher: Collection for reference user has been truncated because it exceeded the maximum of %s users. Please, use the hook "msls_reference_users" to filter the result before!',
-					'multisite-language-switcher'
-				),
-				self::MAX_REFERENCE_USERS
+			/* translators: %s: maximum number of users */
+			$format = __(
+				'Multisite Language Switcher: Collection for reference user has been truncated because it exceeded the maximum of %s users. Please, use the hook "msls_reference_users" to filter the result before!',
+				'multisite-language-switcher'
 			);
-			trigger_error( $message );
+
+			trigger_error( sprintf( $format, self::MAX_REFERENCE_USERS ) );
 		}
 
 		echo ( new Select( 'reference_user', $users, $this->options->reference_user ) )->render();
diff --git a/includes/MslsAdminIcon.php b/includes/MslsAdminIcon.php
index ca819252..736b38b7 100644
--- a/includes/MslsAdminIcon.php
+++ b/includes/MslsAdminIcon.php
@@ -205,6 +205,7 @@ public function get_img(): string {
 	 */
 	public function get_a(): string {
 		if ( empty( $this->href ) ) {
+			/* translators: %s: blog name */
 			$format = __( 'Create a new translation in the %s-blog', 'multisite-language-switcher' );
 			$href   = $this->get_edit_new();
 		} else {