Skip to content

AbstractTag: add cached instance of the config, to improve performance #143

AbstractTag: add cached instance of the config, to improve performance

AbstractTag: add cached instance of the config, to improve performance #143

Triggered via pull request October 6, 2024 22:58
Status Success
Total duration 2m 12s
Artifacts

mt.yaml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
PHP 8.3
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
PHP 8.3
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
PHP 8.3: src/Liquid/AbstractBlock.php#L115
Escaped Mutant for Mutator "ProtectedVisibility": @@ @@ * * @param string $token */ - protected function whitespaceHandler($token) + private function whitespaceHandler($token) { /* * This assumes that TAG_START is always '{%', and a whitespace control indicator
PHP 8.3: src/Liquid/AbstractBlock.php#L121
Escaped Mutant for Mutator "MBString": @@ @@ * This assumes that TAG_START is always '{%', and a whitespace control indicator * is exactly one character long, on a third position. */ - if (mb_substr($token, 2, 1) === Liquid::get('WHITESPACE_CONTROL')) { + if (substr($token, 2, 1) === Liquid::get('WHITESPACE_CONTROL')) { $previousToken = end($this->nodelist); if (is_string($previousToken)) { // this can also be a tag or a variable
PHP 8.3: src/Liquid/AbstractBlock.php#L132
Escaped Mutant for Mutator "MBString": @@ @@ * This assumes that TAG_END is always '%}', and a whitespace control indicator * is exactly one character long, on a third position from the end. */ - self::$trimWhitespace = mb_substr($token, -3, 1) === Liquid::get('WHITESPACE_CONTROL'); + self::$trimWhitespace = substr($token, -3, 1) === Liquid::get('WHITESPACE_CONTROL'); } /** * Render the block.
PHP 8.3: src/Liquid/AbstractBlock.php#L160
Escaped Mutant for Mutator "LogicalAnd": @@ @@ { $result = ''; foreach ($list as $token) { - if (is_object($token) && method_exists($token, 'render')) { + if (is_object($token) || method_exists($token, 'render')) { $value = $token->render($context); } else { $value = $token;
PHP 8.3: src/Liquid/AbstractBlock.php#L188
Escaped Mutant for Mutator "ProtectedVisibility": @@ @@ /** * An action to execute when the end tag is reached */ - protected function endTag() + private function endTag() { // Do nothing by default }
PHP 8.3: src/Liquid/AbstractBlock.php#L206
Escaped Mutant for Mutator "Concat": @@ @@ { switch ($tag) { case 'else': - throw new ParseException($this->blockName() . " does not expect else tag"); + throw new ParseException(" does not expect else tag" . $this->blockName()); case 'end': throw new ParseException("'end' is not a valid delimiter for " . $this->blockName() . " tags. Use " . $this->blockDelimiter()); default:
PHP 8.3: src/Liquid/AbstractBlock.php#L206
Escaped Mutant for Mutator "ConcatOperandRemoval": @@ @@ { switch ($tag) { case 'else': - throw new ParseException($this->blockName() . " does not expect else tag"); + throw new ParseException(" does not expect else tag"); case 'end': throw new ParseException("'end' is not a valid delimiter for " . $this->blockName() . " tags. Use " . $this->blockDelimiter()); default:
PHP 8.3: src/Liquid/AbstractBlock.php#L208
Escaped Mutant for Mutator "Concat": @@ @@ case 'else': throw new ParseException($this->blockName() . " does not expect else tag"); case 'end': - throw new ParseException("'end' is not a valid delimiter for " . $this->blockName() . " tags. Use " . $this->blockDelimiter()); + throw new ParseException($this->blockName() . "'end' is not a valid delimiter for " . " tags. Use " . $this->blockDelimiter()); default: throw new ParseException("Unknown tag {$tag}"); }
PHP 8.3: src/Liquid/AbstractBlock.php#L208
Escaped Mutant for Mutator "ConcatOperandRemoval": @@ @@ case 'else': throw new ParseException($this->blockName() . " does not expect else tag"); case 'end': - throw new ParseException("'end' is not a valid delimiter for " . $this->blockName() . " tags. Use " . $this->blockDelimiter()); + throw new ParseException($this->blockName() . " tags. Use " . $this->blockDelimiter()); default: throw new ParseException("Unknown tag {$tag}"); }
PHP 8.3: src/Liquid/AbstractBlock.php#L208
Escaped Mutant for Mutator "ConcatOperandRemoval": @@ @@ case 'else': throw new ParseException($this->blockName() . " does not expect else tag"); case 'end': - throw new ParseException("'end' is not a valid delimiter for " . $this->blockName() . " tags. Use " . $this->blockDelimiter()); + throw new ParseException("'end' is not a valid delimiter for " . " tags. Use " . $this->blockDelimiter()); default: throw new ParseException("Unknown tag {$tag}"); }