Skip to content

Commit ec91292

Browse files
committed
Update arg comment in ruleset
Whitelist and ignore `append` usage in JS
1 parent d946b20 commit ec91292

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

admin/js/nginx-helper-admin.js

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,15 @@
4949
ajaxurl,
5050
args,
5151
function( data ) {
52-
news_section.find( '.inside' ).empty();
53-
news_section.find( '.inside' ).append( data );
52+
/**
53+
* Received markup is safe and escaped appropriately.
54+
*
55+
* File: admin/class-nginx-helper-admin.php
56+
* Method: nginx_helper_get_feeds();
57+
*/
58+
59+
// phpcs:ignore -- WordPressVIPMinimum.JS.HTMLExecutingFunctions.append
60+
news_section.find( '.inside' ).empty().append( data );
5461
}
5562
);
5663

phpcs.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
<!-- Check up to 8 files simultanously. -->
1414
<arg name="parallel" value="8"/>
1515

16-
<!-- Only check the PHP files. JS files are checked separately with JSCS and JSHint. -->
16+
<!-- Check PHP and JS files. -->
1717
<arg name="extensions" value="php,js"/>
1818

1919
<!-- Check all files in this directory and the directories below it. -->

0 commit comments

Comments
 (0)