Skip to content

Commit 92797d2

Browse files
lloclloc
committed
esc_html__ instead of __
1 parent acc36aa commit 92797d2

File tree

3 files changed

+35
-35
lines changed

3 files changed

+35
-35
lines changed

‎includes/MslsAdmin.php

Lines changed: 22 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -116,12 +116,10 @@ public function __call( $method, $args ) {
116116
);
117117

118118
if ( isset( $checkboxes[ $method ] ) ) {
119-
echo ( new Group() )->add( new Checkbox( $method, $this->options->$method ) )->add(
120-
new Label(
121-
$method,
122-
$checkboxes[ $method ]
123-
)
124-
)->render();
119+
echo ( new Group() )
120+
->add( new Checkbox( $method, $this->options->$method ) )
121+
->add( new Label( $method, $checkboxes[ $method ] ) )
122+
->render();
125123
} else {
126124
$value = ! empty( $this->options->$method ) ? $this->options->$method : '';
127125
echo ( new Text( $method, $value ) )->render();
@@ -146,7 +144,7 @@ public function has_problems(): bool {
146144
} elseif ( 1 == count( $this->options->get_available_languages() ) ) {
147145
/* translators: %1$s: URL to a page at WordPress.orgs */
148146
$format = __(
149-
'TNo language files are currently installed. Learn how to install various languages in WordPress by <a href="%1$s">reading more here</a>.',
147+
'No language files are currently installed. Learn how to install various languages in WordPress by <a href="%1$s">reading more here</a>.',
150148
'multisite-language-switcher'
151149
);
152150
$message = sprintf(
@@ -262,18 +260,18 @@ public function language_section(): int {
262260
*/
263261
public function main_section(): int {
264262
$map = array(
265-
'display' => __( 'Display', 'multisite-language-switcher' ),
266-
'admin_display' => __( 'Admin Display', 'multisite-language-switcher' ),
267-
'sort_by_description' => __( 'Sort languages', 'multisite-language-switcher' ),
268-
'output_current_blog' => __( 'Current language link', 'multisite-language-switcher' ),
269-
'only_with_translation' => __( 'Translation links', 'multisite-language-switcher' ),
270-
'description' => __( 'Description', 'multisite-language-switcher' ),
271-
'before_output' => __( 'Text/HTML before the list', 'multisite-language-switcher' ),
272-
'after_output' => __( 'Text/HTML after the list', 'multisite-language-switcher' ),
273-
'before_item' => __( 'Text/HTML before each item', 'multisite-language-switcher' ),
274-
'after_item' => __( 'Text/HTML after each item', 'multisite-language-switcher' ),
275-
'content_filter' => __( 'Available translations hint', 'multisite-language-switcher' ),
276-
'content_priority' => __( 'Hint priority', 'multisite-language-switcher' ),
263+
'display' => esc_html__( 'Display', 'multisite-language-switcher' ),
264+
'admin_display' => esc_html__( 'Admin Display', 'multisite-language-switcher' ),
265+
'sort_by_description' => esc_html__( 'Sort languages', 'multisite-language-switcher' ),
266+
'output_current_blog' => esc_html__( 'Current language link', 'multisite-language-switcher' ),
267+
'only_with_translation' => esc_html__( 'Translation links', 'multisite-language-switcher' ),
268+
'description' => esc_html__( 'Description', 'multisite-language-switcher' ),
269+
'before_output' => esc_html__( 'Text/HTML before the list', 'multisite-language-switcher' ),
270+
'after_output' => esc_html__( 'Text/HTML after the list', 'multisite-language-switcher' ),
271+
'before_item' => esc_html__( 'Text/HTML before each item', 'multisite-language-switcher' ),
272+
'after_item' => esc_html__( 'Text/HTML after each item', 'multisite-language-switcher' ),
273+
'content_filter' => esc_html__( 'Available translations hint', 'multisite-language-switcher' ),
274+
'content_priority' => esc_html__( 'Hint priority', 'multisite-language-switcher' ),
277275
);
278276

279277
return $this->add_settings_fields( $map, 'main_section' );
@@ -288,11 +286,11 @@ public function main_section(): int {
288286
*/
289287
public function advanced_section(): int {
290288
$map = array(
291-
'activate_autocomplete' => __( 'Autocomplete', 'multisite-language-switcher' ),
292-
'image_url' => __( 'Custom URL for flag-images', 'multisite-language-switcher' ),
293-
'reference_user' => __( 'Reference user', 'multisite-language-switcher' ),
294-
'exclude_current_blog' => __( 'Exclude blog', 'multisite-language-switcher' ),
295-
'activate_content_import' => __( 'Content import', 'multisite-language-switcher' ),
289+
'activate_autocomplete' => esc_html__( 'Autocomplete', 'multisite-language-switcher' ),
290+
'image_url' => esc_html__( 'Custom URL for flag-images', 'multisite-language-switcher' ),
291+
'reference_user' => esc_html__( 'Reference user', 'multisite-language-switcher' ),
292+
'exclude_current_blog' => esc_html__( 'Exclude blog', 'multisite-language-switcher' ),
293+
'activate_content_import' => esc_html__( 'Content import', 'multisite-language-switcher' ),
296294
);
297295

298296
return $this->add_settings_fields( $map, 'advanced_section' );

‎includes/MslsAdminIcon.php

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -231,24 +231,24 @@ public function get_icon(): string {
231231

232232
switch ( $this->icon_type ) {
233233
case self::TYPE_FLAG:
234-
$icon = sprintf(
234+
$class = ( new IconSvg() )->get( $this->language );
235+
$icon = sprintf(
235236
'<span class="flag-icon %s">%s</span>',
236-
( new IconSvg() )->get( $this->language ),
237-
$this->language
237+
esc_attr( $class ),
238+
esc_html( $this->language )
238239
);
239240
break;
240241
case self::TYPE_LABEL:
242+
$text = ( new IconLabel() )->get( $this->language );
241243
$icon = sprintf(
242244
'<span class="language-badge %s">%s</span>',
243-
$this->language,
244-
( new IconLabel() )->get( $this->language )
245+
esc_attr( $this->language ),
246+
esc_html( $text )
245247
);
246248
break;
247249
default:
248-
$icon = sprintf(
249-
'<span class="dashicons %s"></span>',
250-
empty( $this->href ) ? 'dashicons-plus' : 'dashicons-edit'
251-
);
250+
$class = empty( $this->href ) ? 'dashicons-plus' : 'dashicons-edit';
251+
$icon = sprintf( '<span class="dashicons %s"></span>', esc_attr( $class ) );
252252
}
253253

254254
return $icon;

‎tests/phpunit/TestMslsAdmin.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -368,15 +368,17 @@ function test_language_section(): void {
368368
function test_main_section(): void {
369369
$obj = $this->get_sut();
370370

371-
Functions\when( 'add_settings_field' )->returnArg();
371+
Functions\expect( 'add_settings_field' )->times( 12 )->andReturnFirstArg();
372+
Functions\expect( 'esc_html__' )->times( 12 )->andReturnFirstArg();
372373

373374
$this->assertEquals( 12, $obj->main_section() );
374375
}
375376

376377
function test_advanced_section(): void {
377378
$obj = $this->get_sut();
378379

379-
Functions\when( 'add_settings_field' )->returnArg();
380+
Functions\expect( 'add_settings_field' )->times( 5 )->andReturnFirstArg();
381+
Functions\expect( 'esc_html__' )->times( 5 )->andReturnFirstArg();
380382

381383
$this->assertEquals( 5, $obj->advanced_section() );
382384
}

0 commit comments

Comments
 (0)