Conversation
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
phpcs scanning turned up:
🚫 162 errors
hashes-api-scanning skipped
Posting will continue in further review(s)
| @@ -0,0 +1,36 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-3rfsjx.php, but found phpcs-scan-3rfsJX.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,37 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-nyoqbf.php, but found phpcs-scan-nYOqbf.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,20 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-wpyd0w.php, but found phpcs-scan-WpyD0w.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,56 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-jklocp.php, but found phpcs-scan-JKlOcP.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,466 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-ct7mm7.php, but found phpcs-scan-cT7MM7.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,100 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-pvtrqq.php, but found phpcs-scan-pVTRQq.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,52 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-zmvkjk.php, but found phpcs-scan-zmvkjK.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,44 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-tnzka4.php, but found phpcs-scan-TnZka4.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,100 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-ssnboo.php, but found phpcs-scan-ssNboo.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,34 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-vfsyyi.php, but found phpcs-scan-vfSYYI.php (WordPress.Files.FileName.NotHyphenatedLowercase).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| @@ -0,0 +1,24 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-pn9iv3.php, but found phpcs-scan-Pn9iV3.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,12 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-z7ggep.php, but found phpcs-scan-z7gGep.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,57 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-fpukvk.php, but found phpcs-scan-fPuKVK.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,30 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-onqtz6.php, but found phpcs-scan-onqTZ6.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,21 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-pxozot.php, but found phpcs-scan-pXOzot.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,45 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-trjx8p.php, but found phpcs-scan-tRJX8P.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,43 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-kiuqhd.php, but found phpcs-scan-kIuqhd.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,42 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-vpurla.php, but found phpcs-scan-VPurLA.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,16 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-dme4yy.php, but found phpcs-scan-dme4yY.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,82 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-eq6whm.php, but found phpcs-scan-eq6WHm.php (WordPress.Files.FileName.NotHyphenatedLowercase).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| @@ -0,0 +1,81 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-jtc4el.php, but found phpcs-scan-Jtc4eL.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,39 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-qpdlaa.php, but found phpcs-scan-qpdLaa.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,33 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-mjvrtz.php, but found phpcs-scan-MjVrtz.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,37 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-jifr7y.php, but found phpcs-scan-jIFr7Y.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,37 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-wir96o.php, but found phpcs-scan-wiR96o.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,33 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-xxphup.php, but found phpcs-scan-xXphuP.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,43 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-nxfvdg.php, but found phpcs-scan-nxFVdg.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,77 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-ijgmkh.php, but found phpcs-scan-IjgmkH.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,62 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-n4bpn8.php, but found phpcs-scan-n4BPN8.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,33 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-efuzga.php, but found phpcs-scan-eFUZGA.php (WordPress.Files.FileName.NotHyphenatedLowercase).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| @@ -0,0 +1,19 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-7uhvx2.php, but found phpcs-scan-7UHVX2.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,18 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-aie1bv.php, but found phpcs-scan-aIe1Bv.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,45 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-ofexdy.php, but found phpcs-scan-OfExDY.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,35 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-oytbbs.php, but found phpcs-scan-oytBbs.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| <h1 class="page-title screen-reader-text"><?php single_post_title(); ?></h1> | ||
| </header> | ||
|
|
||
| <?php |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 4 tabs, found 3 (Generic.WhiteSpace.ScopeIndent.Incorrect).
| if ( have_posts() ) : | ||
|
|
||
| if ( is_home() && ! is_front_page() ) : | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 4 tabs, found 2 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| @@ -0,0 +1,55 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-w5jv6v.php, but found phpcs-scan-w5Jv6V.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| ?> | ||
|
|
||
| <nav class="pagination-container container" aria-label="<?php esc_html_e( 'numeric pagination', 'dctx' ); ?>"> | ||
| <?php echo paginate_links( $args ); // WPCS: XSS OK. ?> |
There was a problem hiding this comment.
| <?php | ||
| // If we're doing a URL, just make this LOOK like a button but be a link. | ||
| if ( 'link' === $button_setting && $button_url ) : | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 3 tabs, found 2 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| </span> | ||
| </a> | ||
| </li><!-- .social-icon --> | ||
| <?php |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 4 tabs, found 3 (Generic.WhiteSpace.ScopeIndent.Incorrect).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| ?> | ||
| <span class="screen-reader-text"> | ||
| <?php | ||
| echo /* translators: the social network name */ sprintf( esc_html( 'Link to %s', 'dctx' ), ucwords( esc_html( $network ) ) ); // WPCS: XSS OK. |
There was a problem hiding this comment.
|
|
||
| // Only display the list item if a URL is set. | ||
| if ( ! empty( $network_url ) ) : | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 4 tabs, found 3 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| return false; | ||
| } | ||
|
|
||
| echo dctx_get_the_content( do_shortcode( $copyright_text ) ); // phpcs: xss ok. |
There was a problem hiding this comment.
| <?php | ||
| // Use absolute path in the Customizer so that icons show up in there. | ||
| if ( is_customize_preview() ) : | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 3 tabs, found 2 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| <?php | ||
| // Display description if available. | ||
| if ( $args['desc'] ) : | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 3 tabs, found 2 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| class="icon icon-<?php echo esc_attr( $args['icon'] ); ?>" | ||
| <?php | ||
| echo dctx_get_the_content( $aria_hidden ); // WPCS XSS OK. | ||
| echo dctx_get_the_content( $aria_labelledby ); // WPCS XSS OK. |
There was a problem hiding this comment.
| ?> | ||
| class="icon icon-<?php echo esc_attr( $args['icon'] ); ?>" | ||
| <?php | ||
| echo dctx_get_the_content( $aria_hidden ); // WPCS XSS OK. |
There was a problem hiding this comment.
| <?php | ||
| echo dctx_get_the_content( $height ); // WPCS XSS OK. | ||
| echo dctx_get_the_content( $width ); // WPCS XSS OK. | ||
| echo dctx_get_the_content( $fill ); // WPCS XSS OK. |
There was a problem hiding this comment.
| <svg | ||
| <?php | ||
| echo dctx_get_the_content( $height ); // WPCS XSS OK. | ||
| echo dctx_get_the_content( $width ); // WPCS XSS OK. |
There was a problem hiding this comment.
|
|
||
| <svg | ||
| <?php | ||
| echo dctx_get_the_content( $height ); // WPCS XSS OK. |
There was a problem hiding this comment.
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| * @author WDS | ||
| */ | ||
| function dctx_display_svg( $args = array() ) { | ||
| echo dctx_get_svg( $args ); // WPCS XSS Ok. |
There was a problem hiding this comment.
| $tags_list = get_the_tag_list( '', esc_html__( ', ', 'dctx' ) ); | ||
| if ( $tags_list ) { | ||
| /* translators: the post tags */ | ||
| printf( '<span class="tags-links">' . esc_html__( 'Tagged %1$s', 'dctx' ) . '</span>', $tags_list ); // WPCS: XSS OK. |
There was a problem hiding this comment.
| $categories_list = get_the_category_list( esc_html__( ', ', 'dctx' ) ); | ||
| if ( $categories_list && dctx_categorized_blog() ) { | ||
| /* translators: the post category */ | ||
| printf( '<span class="cat-links">' . esc_html__( 'Posted in %1$s', 'dctx' ) . '</span>', $categories_list ); // WPCS: XSS OK. |
There was a problem hiding this comment.
| '<span class="author vcard"><a class="url fn n" href="' . esc_url( get_author_posts_url( get_the_author_meta( 'ID' ) ) ) . '">' . esc_html( get_the_author() ) . '</a></span>' | ||
| ); | ||
|
|
||
| echo '<span class="posted-on">' . $posted_on . '</span><span class="byline"> ' . $byline . '</span>'; // WPCS: XSS OK. |
There was a problem hiding this comment.
| @@ -0,0 +1,540 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-i0jioq.php, but found phpcs-scan-i0JIoq.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| /** | ||
| * If WP is in script debug, or we pass ?script_debug in a URL - set debug to true. | ||
| */ | ||
| $debug = ( defined( 'SCRIPT_DEBUG' ) && true === SCRIPT_DEBUG ) || ( isset( $_GET['script_debug'] ) ) ? true : false; // WPCS: CSRF OK. |
There was a problem hiding this comment.
| /** | ||
| * If WP is in script debug, or we pass ?script_debug in a URL - set debug to true. | ||
| */ | ||
| $debug = ( defined( 'SCRIPT_DEBUG' ) && true === SCRIPT_DEBUG ) || ( isset( $_GET['script_debug'] ) ) ? true : false; // WPCS: CSRF OK. |
There was a problem hiding this comment.
| @@ -0,0 +1,154 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-2btfjv.php, but found phpcs-scan-2BTfjV.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| <?php | ||
| break; | ||
| default: | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 5 tabs, found 4 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| <p><strong><?php echo esc_html( $font_var ); ?>:</strong> <span style="font-family: <?php echo esc_attr( $family ); ?>"><?php echo esc_html( $family ); ?></span></p> | ||
| <?php endforeach; ?> | ||
| </div> | ||
| <?php |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 5 tabs, found 4 (Generic.WhiteSpace.ScopeIndent.Incorrect).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
|
|
||
| foreach ( $fonts as $name => $family ) : | ||
| $font_var = '$font-' . str_replace( ' ', '-', strtolower( $name ) ); | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 6 tabs, found 5 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| <?php | ||
| break; | ||
| case 'fonts': | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 5 tabs, found 4 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
|
|
||
| <?php endforeach; ?> | ||
| </div> | ||
| <?php |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 5 tabs, found 4 (Generic.WhiteSpace.ScopeIndent.Incorrect).
|
|
||
| foreach ( $colors as $name => $hex ) : | ||
| $color_var = '$color-' . str_replace( ' ', '-', strtolower( $name ) ); | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 6 tabs, found 5 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| switch ( $args['global_type'] ) : | ||
|
|
||
| case 'colors': | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 5 tabs, found 4 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| @@ -0,0 +1,257 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-giyceq.php, but found phpcs-scan-GiYcEq.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| $args = array_merge( | ||
| $args, | ||
| array( | ||
| 'tax_query' => array( |
There was a problem hiding this comment.
| @@ -0,0 +1,89 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-rgnotw.php, but found phpcs-scan-rgNotW.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,48 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-vk5ufs.php, but found phpcs-scan-Vk5uFs.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| $card_type = 'website'; | ||
|
|
||
| // Translators: get the search term. | ||
| $card_long_description = $card_description = sprintf( esc_html__( 'Search results for %s.', 'dctx' ), $search_term ); |
There was a problem hiding this comment.
🚫 Error: Assignments must be the first block of code on a line (Squiz.PHP.DisallowMultipleAssignments.Found).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| var api = wp.customize; | ||
|
|
||
| // Site title. | ||
| api( 'blogname', function( value ) { |
There was a problem hiding this comment.
🚫 Error: Only one argument is allowed per line in a multi-line function call (PEAR.Functions.FunctionCallSignature.MultipleArguments).
| var api = wp.customize; | ||
|
|
||
| // Site title. | ||
| api( 'blogname', function( value ) { |
There was a problem hiding this comment.
🚫 Error: Opening parenthesis of a multi-line function call must be the last content on the line (PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket).
| * File livepreview.js. | ||
| * | ||
| * Deal with real time changes asynchronously. | ||
| */ |
There was a problem hiding this comment.
🚫 Error: Missing @Package tag in file comment (Squiz.Commenting.FileComment.MissingPackageTag).
| * @since NEXT | ||
| */ | ||
| function dctx_display_link( $args = array() ) { | ||
| echo dctx_get_link( $args ); // WPCS: XSS Ok. |
There was a problem hiding this comment.
| margin-left: 8px; | ||
| } | ||
| </style> | ||
| <?php |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 2 tabs, found 1 (Generic.WhiteSpace.ScopeIndent.Incorrect).
| * @author WDS | ||
| */ | ||
| function dctx_flexible_content_layout_title_acf_admin_head() { | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 2 tabs, found 1 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| 'start_date' => $start_date, | ||
| 'end_date' => $end_date, | ||
| ) | ||
| ) |
There was a problem hiding this comment.
🚫 Error: Multi-line function call not indented correctly; expected 4 spaces but found 8 (PEAR.Functions.FunctionCallSignature.Indent).
|
|
||
| // If the block has expired, add "(expired)" to the title. | ||
| if ( dctx_has_block_expired( | ||
| array( |
There was a problem hiding this comment.
🚫 Error: Multi-line function call not indented correctly; expected 8 spaces but found 12 (PEAR.Functions.FunctionCallSignature.Indent).
|
|
||
| // If we have a background image, echo our background image markup inside the block container. | ||
| if ( $background_image_markup ) { | ||
| echo $background_image_markup; // WPCS XSS OK. |
There was a problem hiding this comment.
|
|
||
| // If we have a background video, echo our background video markup inside the block container. | ||
| if ( $background_video_markup ) { | ||
| echo $background_video_markup; // WPCS XSS OK. |
There was a problem hiding this comment.
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| // Parse args. | ||
| $args = wp_parse_args( $args, $defaults ); | ||
|
|
||
| $background_video_markup = $background_image_markup = ''; |
There was a problem hiding this comment.
🚫 Error: Assignments must be the first block of code on a line (Squiz.PHP.DisallowMultipleAssignments.Found).
| @@ -0,0 +1,449 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-kzv7l2.php, but found phpcs-scan-kzV7l2.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,70 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-2g259c.php, but found phpcs-scan-2g259C.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| <p class="slide-description"><?php esc_html_e( 'Slide Content', 'dctx' ); ?></p> | ||
| </div> | ||
| </section> | ||
| <?php |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 2 tabs, found 1 (Generic.WhiteSpace.ScopeIndent.Incorrect).
| 'color_picker' => 'blue', | ||
| ), | ||
| ), | ||
| 'display_options' => array( |
There was a problem hiding this comment.
| 'url' => '#', | ||
| ), | ||
| 'background_options' => array( | ||
| 'background_type' => 'color', |
There was a problem hiding this comment.
| 'data' => array( | ||
| 'title' => esc_html__( 'Call To Action Title', 'dctx' ), | ||
| 'text' => esc_html__( 'Call To Action Text', 'dctx' ), | ||
| 'button_link' => array( |
There was a problem hiding this comment.
| 'attributes' => array( | ||
| 'data' => array( | ||
| 'title' => esc_html__( 'Call To Action Title', 'dctx' ), | ||
| 'text' => esc_html__( 'Call To Action Text', 'dctx' ), |
There was a problem hiding this comment.
| 'example' => array( | ||
| 'attributes' => array( | ||
| 'data' => array( | ||
| 'title' => esc_html__( 'Call To Action Title', 'dctx' ), |
There was a problem hiding this comment.
| @@ -0,0 +1,443 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-wkeije.php, but found phpcs-scan-wKeIje.php (WordPress.Files.FileName.NotHyphenatedLowercase).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| <?php | ||
| $description = get_bloginfo( 'description', 'display' ); | ||
| if ( $description || is_customize_preview() ) : | ||
| ?> |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 5 tabs, found 4 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
|
|
||
| <?php | ||
| $description = get_bloginfo( 'description', 'display' ); | ||
| if ( $description || is_customize_preview() ) : |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 4 tabs, found 5 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| @@ -0,0 +1,72 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-fciv2p.php, but found phpcs-scan-FCiv2P.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,241 @@ | |||
| <?php | |||
| /** | |||
| * dctx functions and definitions. | |||
There was a problem hiding this comment.
🚫 Error: Doc comment short description must start with a capital letter (Generic.Commenting.DocComment.ShortNotCapital).
| @@ -0,0 +1,241 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-nlid9r.php, but found phpcs-scan-nLid9r.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,30 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-ikf9f4.php, but found phpcs-scan-iKF9F4.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,43 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-wgqlxh.php, but found phpcs-scan-WGqlxH.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,9 @@ | |||
| /* http://prismjs.com/download.html?themes=prism&languages=markup+css+css-extras+clike+javascript+scss */ | |||
There was a problem hiding this comment.
| @@ -0,0 +1,4 @@ | |||
| /*! jQuery v2.1.1 | (c) 2005, 2014 jQuery Foundation, Inc. | jquery.org/license */ | |||
There was a problem hiding this comment.
| @@ -0,0 +1,20 @@ | |||
| /** | |||
There was a problem hiding this comment.
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| }, this)); | ||
| }; | ||
|
|
||
| global.Sidebar = Sidebar; |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected at least 1 tabs, found 0 (Generic.WhiteSpace.ScopeIndent.Incorrect).
| }, this)); | ||
| }; | ||
|
|
||
| global.Sidebar = Sidebar; |
There was a problem hiding this comment.
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); | ||
| }, this)); | ||
| }; |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 1 tabs, found 0 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); | ||
| }, this)); | ||
| }; |
There was a problem hiding this comment.
|
|
||
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); | ||
| }, this)); |
There was a problem hiding this comment.
🚫 Error: Closing parenthesis of a multi-line function call must be on a line by itself (PEAR.Functions.FunctionCallSignature.CloseBracketLine).
|
|
||
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); | ||
| }, this)); |
There was a problem hiding this comment.
🚫 Error: Only one argument is allowed per line in a multi-line function call (PEAR.Functions.FunctionCallSignature.MultipleArguments).
|
|
||
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); | ||
| }, this)); |
There was a problem hiding this comment.
🚫 Error: Line indented incorrectly; expected 2 tabs, found 1 (Generic.WhiteSpace.ScopeIndent.IncorrectExact).
|
|
||
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); | ||
| }, this)); |
There was a problem hiding this comment.
🚫 Error: Tabs must be used to indent lines; spaces are not allowed (Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed).
| this.index[slug] = !this.index[slug]; | ||
|
|
||
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); |
There was a problem hiding this comment.
🚫 Error: Expected 1 spaces before closing bracket; 0 found (PEAR.Functions.FunctionCallSignature.SpaceBeforeCloseBracket).
| this.index[slug] = !this.index[slug]; | ||
|
|
||
| // Update DOM | ||
| $item.toggleClass(this.conf.collapsedClass); |
There was a problem hiding this comment.
🚫 Error: Expected 1 spaces after opening bracket; 0 found (PEAR.Functions.FunctionCallSignature.SpaceAfterOpenBracket).
wpugphil
left a comment
wpugphil
left a comment
There was a problem hiding this comment.
Previous scan continued.
| @@ -0,0 +1,108 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-fsp6to.php, but found phpcs-scan-fSP6tO.php (WordPress.Files.FileName.NotHyphenatedLowercase).
| @@ -0,0 +1,79 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Class file names should be based on the class name with "class-" prepended. Expected class-text-editor-custom-control.php, but found phpcs-scan-dgNYpY.php (WordPress.Files.FileName.InvalidClassFileName).
| @@ -0,0 +1,79 @@ | |||
| <?php | |||
There was a problem hiding this comment.
🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-dgnypy.php, but found phpcs-scan-dgNYpY.php (WordPress.Files.FileName.NotHyphenatedLowercase).
|
|
||
| /** | ||
| * Safely log to the console | ||
| * @param mixed str var to log |
There was a problem hiding this comment.
🚫 Error: There must be exactly one blank line before the tags in a doc comment (Generic.Commenting.DocComment.SpacingBeforeTags).
| // Update the customize option. | ||
| wp.customize( this.id, function ( obj ) { | ||
| obj.set( $me.val() ); | ||
| }); |
There was a problem hiding this comment.
🚫 Error: Closing parenthesis of a multi-line function call must be on a line by itself (PEAR.Functions.FunctionCallSignature.CloseBracketLine).
| var $me = $( this ); | ||
|
|
||
| // Update the customize option. | ||
| wp.customize( this.id, function ( obj ) { |
There was a problem hiding this comment.
🚫 Error: Only one argument is allowed per line in a multi-line function call (PEAR.Functions.FunctionCallSignature.MultipleArguments).
| var $me = $( this ); | ||
|
|
||
| // Update the customize option. | ||
| wp.customize( this.id, function ( obj ) { |
There was a problem hiding this comment.
🚫 Error: Opening parenthesis of a multi-line function call must be the last content on the line (PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket).
| }); | ||
| }); | ||
| } | ||
| }, 1000 ); |
There was a problem hiding this comment.
🚫 Error: Closing parenthesis of a multi-line function call must be on a line by itself (PEAR.Functions.FunctionCallSignature.CloseBracketLine).
| }); | ||
| }); | ||
| } | ||
| }, 1000 ); |
There was a problem hiding this comment.
🚫 Error: Only one argument is allowed per line in a multi-line function call (PEAR.Functions.FunctionCallSignature.MultipleArguments).
| }); | ||
| }); | ||
| } | ||
| }, 1000 ); |
There was a problem hiding this comment.
🚫 Error: Tabs must be used to indent lines; spaces are not allowed (Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed).
|
Total number of active review comments per Pull-Request has been reached and some comments might not appear as a result. Please resolve some issues to see more (commit-ID: baad9e9). |
2 participants
DCTx V2