FIX Codesniffer style violations with comments

This commit is contained in:
Robbie Averill 2018-06-11 09:55:18 +12:00
parent a8f14f96ed
commit d842225df6
3 changed files with 20 additions and 20 deletions

View File

@ -114,8 +114,8 @@ class HTMLEditorSanitiser
if ($attrType === '!') { if ($attrType === '!') {
$element->attributesRequired[] = $attrName; $element->attributesRequired[] = $attrName;
$attr->required = true; $attr->required = true;
} // Denied from global } elseif ($attrType === '-') {
elseif ($attrType === '-') { // Denied from global
unset($element->attributes[$attrName]); unset($element->attributes[$attrName]);
continue; continue;
} }
@ -126,12 +126,12 @@ class HTMLEditorSanitiser
if ($prefix === '=') { if ($prefix === '=') {
$element->attributesDefault[$attrName] = $value; $element->attributesDefault[$attrName] = $value;
$attr->defaultValue = $value; $attr->defaultValue = $value;
} // Forced value } elseif ($prefix === ':') {
elseif ($prefix === ':') { // Forced value
$element->attributesForced[$attrName] = $value; $element->attributesForced[$attrName] = $value;
$attr->forcedValue = $value; $attr->forcedValue = $value;
} // Required values } elseif ($prefix === '<') {
elseif ($prefix === '<') { // Required values
$attr->validValues = explode('?', $value); $attr->validValues = explode('?', $value);
} }
} }
@ -290,8 +290,8 @@ class HTMLEditorSanitiser
// If it's a script or style, we don't keep contents // If it's a script or style, we don't keep contents
if ($el->tagName === 'script' || $el->tagName === 'style') { if ($el->tagName === 'script' || $el->tagName === 'style') {
$el->parentNode->removeChild($el); $el->parentNode->removeChild($el);
} // Otherwise we replace this node with all it's children } else {
else { // Otherwise we replace this node with all it's children
// First, create a new fragment with all of $el's children moved into it // First, create a new fragment with all of $el's children moved into it
$frag = $doc->createDocumentFragment(); $frag = $doc->createDocumentFragment();
while ($el->firstChild) { while ($el->firstChild) {
@ -301,8 +301,8 @@ class HTMLEditorSanitiser
// Then replace $el with the frags contents (which used to be it's children) // Then replace $el with the frags contents (which used to be it's children)
$el->parentNode->replaceChild($frag, $el); $el->parentNode->replaceChild($frag, $el);
} }
} // Otherwise tidy the element } else {
else { // Otherwise tidy the element
// First, if we're supposed to pad & this element is empty, fix that // First, if we're supposed to pad & this element is empty, fix that
if ($elementRule->paddEmpty && !$el->firstChild) { if ($elementRule->paddEmpty && !$el->firstChild) {
$el->nodeValue = '&nbsp;'; $el->nodeValue = '&nbsp;';

View File

@ -622,8 +622,8 @@ class SQLSelect extends SQLConditionalExpression
if (!empty($this->having)) { if (!empty($this->having)) {
$records = $this->execute(); $records = $this->execute();
return $records->numRecords(); return $records->numRecords();
} // Choose a default column } elseif ($column == null) {
elseif ($column == null) { // Choose a default column
if ($this->groupby) { if ($this->groupby) {
$column = 'DISTINCT ' . implode(", ", $this->groupby); $column = 'DISTINCT ' . implode(", ", $this->groupby);
} else { } else {

View File

@ -573,8 +573,8 @@ class ShortcodeParser
} elseif ($location == self::AFTER) { } elseif ($location == self::AFTER) {
// Move after block parent // Move after block parent
$this->insertAfter($node, $parent); $this->insertAfter($node, $parent);
} // Split parent at node } elseif ($location == self::SPLIT) {
elseif ($location == self::SPLIT) { // Split parent at node
$at = $node; $at = $node;
$splitee = $node->parentNode; $splitee = $node->parentNode;
@ -593,8 +593,8 @@ class ShortcodeParser
} }
$this->insertAfter($node, $parent); $this->insertAfter($node, $parent);
} // Do nothing } elseif ($location == self::INLINE) {
elseif ($location == self::INLINE) { // Do nothing
if (in_array(strtolower($node->tagName), self::$block_level_elements)) { if (in_array(strtolower($node->tagName), self::$block_level_elements)) {
user_error( user_error(
'Requested to insert block tag ' . $node->tagName . ' inline - probably this will break HTML compliance', 'Requested to insert block tag ' . $node->tagName . ' inline - probably this will break HTML compliance',
@ -646,11 +646,11 @@ class ShortcodeParser
// If no shortcodes defined, don't try and parse any // If no shortcodes defined, don't try and parse any
if (!$this->shortcodes) { if (!$this->shortcodes) {
$continue = false; $continue = false;
} // If no content, don't try and parse it } elseif (!trim($content)) {
elseif (!trim($content)) { // If no content, don't try and parse it
$continue = false; $continue = false;
} // If no shortcode tag, don't try and parse it } elseif (strpos($content, '[') === false) {
elseif (strpos($content, '[') === false) { // If no shortcode tag, don't try and parse it
$continue = false; $continue = false;
} }