Changeset 7006 for trunk/src/includes/common/formatting.php

Timestamp:

11/24/2019 01:38:37 PM (5 years ago)

Author:

netweb

Message:

Build Tools: Add PHPCS with custom bbPress ruleset

• This changeset adds an initial bbPress PHPCS ruleset
• Future iteration of the rules in the ruleset can follow later
• To run the PHPCS check run `grunt phpcs

Fixes #3294.

File:

• trunk/src/includes/common/formatting.php (modified) (8 diffs)

trunk/src/includes/common/formatting.php

@@ -97 +97 @@
  */
 function bbp_kses_data( $data = '' ) {
-    return wp_kses( $data , bbp_kses_allowed_tags() );
+    return wp_kses( $data, bbp_kses_allowed_tags() );
 }
 
@@ -112 +112 @@
 function bbp_code_trick( $content = '' ) {
     $content = str_replace( array( "\r\n", "\r" ), "\n", $content );
-    $content = preg_replace_callback( "|(`)(.*?)`|",      'bbp_encode_callback', $content );
+    $content = preg_replace_callback('|(`)(.*?)`|',      'bbp_encode_callback', $content );
     $content = preg_replace_callback( "!(^|\n)`(.*?)`!s", 'bbp_encode_callback', $content );
 
@@ -131 +131 @@
     // Setup variables
     $openers = array( '<p>', '<br />' );
-    $content = preg_replace_callback( "!(<pre><code>|<code>)(.*?)(</code></pre>|</code>)!s", 'bbp_decode_callback', $content );
+    $content = preg_replace_callback( '!(<pre><code>|<code>)(.*?)(</code></pre>|</code>)!s', 'bbp_decode_callback', $content );
 
     // Do the do
@@ -197 +197 @@
 
     // Trim inline code, not pre blocks (to prevent removing indentation)
-    if ( "`" === $matches[1] ) {
+    if ( '`' === $matches[1] ) {
         $content = trim( $matches[2] );
     } else {
@@ -215 +215 @@
 
     // Wrap blocks in pre tags
-    if ( "`" !== $matches[1] ) {
+    if ('`' !== $matches[1] ) {
         $content = "\n<pre>" . $content . "</pre>\n";
     }
@@ -235 +235 @@
     // Setup variables
     $trans_table = array_flip( get_html_translation_table( HTML_ENTITIES ) );
-    $amps        = array( '&#38;','&#038;', '&amp;' );
-    $single      = array( '&#39;','&#039;'          );
+    $amps        = array( '&#38;', '&#038;', '&amp;' );
+    $single      = array( '&#39;', '&#039;'          );
     $content     = $matches[2];
     $content     = strtr( $content, $trans_table );
@@ -314 +314 @@
     // Bail on links that match the current domain
     if ( preg_match( '%href=["\'](' . preg_quote( set_url_scheme( $home_url, 'http'  ) ) . ')%i', $text ) ||
-         preg_match( '%href=["\'](' . preg_quote( set_url_scheme( $home_url, 'https' ) ) . ')%i', $text )
+         preg_match( '%href=["\'](' . preg_quote( set_url_scheme( $home_url, 'https' ) ) . ')%i', $text )
     ) {
         return "<a {$text}>";
@@ -393 +393 @@
 
     // Cleanup of accidental links within links
-    return preg_replace( '#(<a([ \r\n\t]+[^>]+?>|>))<a [^>]+?>([^>]+?)</a>([^<]*)</a>#i', "$1$3$4</a>", $r );
+    return preg_replace( '#(<a([ \r\n\t]+[^>]+?>|>))<a [^>]+?>([^>]+?)</a>([^<]*)</a>#i', '$1$3$4</a>', $r );
 }