VueJs::fromStringMultiple() - Code Metrics - Inspection of "Merge pull request #223 from briedis/multiple-doma..." - oscarotero/Gettext - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a20241...2c1d12 )

by Oscar

created 2019-09-15 10:40 UTC

VueJs::fromStringMultiple() B

↳ Parent: VueJs

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	71
c	0
b	0
f	0
cc	3
nc	4
nop	3
rs	8.6327

How to fix Long Method

<?php
/** @noinspection PhpComposerExtensionStubsInspection */

namespace Gettext\Extractors;

use DOMAttr;
use DOMDocument;
use DOMElement;
use DOMNode;
use Exception;
use Gettext\Translations;
use Gettext\Utils\JsFunctionsScanner;

/**
 * Class to get gettext strings from VueJS template files.
 */
class VueJs extends Extractor implements ExtractorInterface, ExtractorMultiInterface
{
    public static $options = [
        'constants' => [],

        'functions' => [
            'gettext' => 'gettext',
            '__' => 'gettext',
            'ngettext' => 'ngettext',
            'n__' => 'ngettext',
            'pgettext' => 'pgettext',
            'p__' => 'pgettext',
            'dgettext' => 'dgettext',
            'd__' => 'dgettext',
            'dngettext' => 'dngettext',
            'dn__' => 'dngettext',
            'dpgettext' => 'dpgettext',
            'dp__' => 'dpgettext',
            'npgettext' => 'npgettext',
            'np__' => 'npgettext',
            'dnpgettext' => 'dnpgettext',
            'dnp__' => 'dnpgettext',
            'noop' => 'noop',
            'noop__' => 'noop',
        ],
    ];

    /**
     * @inheritDoc
     * @throws Exception
     */
    public static function fromFileMultiple($file, array $translations, array $options = [])

    {
        foreach (self::getFiles($file) as $file) {
            $options['file'] = $file;
            static::fromStringMultiple(self::readFile($file), $translations, $options);
        }
    }

    /**
     * @inheritdoc
     * @throws Exception
     */
    public static function fromString($string, Translations $translations, array $options = [])
    {
        self::fromStringMultiple($string, [$translations], $options);
    }

    /**
     * @inheritDoc
     * @throws Exception
     */
    public static function fromStringMultiple($string, array $translations, array $options = [])
    {
        $options += self::$options;
        $options += [
            // HTML attribute prefixes we parse as JS which could contain translations (are JS expressions)
            'attributePrefixes' => [
                ':',
                'v-bind:',
                'v-on:',
                'v-text',
            ],
            // HTML Tags to parse
            'tagNames' => [
                'translate',
            ],
            // HTML tags to parse when attribute exists
            'tagAttributes' => [
                'v-translate',
            ],
            // Comments
            'commentAttributes' => [
                'translate-comment',
            ],
            'contextAttributes' => [
                'translate-context',
            ],
            // Attribute with plural content
            'pluralAttributes' => [
                'translate-plural',
            ],
        ];

        // Ok, this is the weirdest hack, but let me explain:
        // On Linux (Mac is fine), when converting HTML to DOM, new lines get trimmed after the first tag.
        // So if there are new lines between <template> and next element, they are lost
        // So we insert a "." which is a text node, and it will prevent that newlines are stripped between elements.
        // Same thing happens between template and script tag.
        $string = str_replace('<template>', '<template>.', $string);
        $string = str_replace('</template>', '</template>.', $string);

        // Normalize newlines
        $string = str_replace(["\r\n", "\n\r", "\r"], "\n", $string);

        // VueJS files are valid HTML files, we will operate with the DOM here
        $dom = self::convertHtmlToDom($string);

        $script = self::extractScriptTag($string);

        // Parse the script part as a regular JS code
        if ($script) {
            $scriptLineNumber = $dom->getElementsByTagName('script')->item(0)->getLineNo();
            self::getScriptTranslationsFromString(
                $script,
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
                $translations,
                $options,
                $scriptLineNumber - 1
            );
        }

        // Template part is parsed separately, all variables will be extracted
        // and handled as a regular JS code
        $template = $dom->getElementsByTagName('template')->item(0);
        if ($template) {
            self::getTemplateTranslations(
                $template,
                $translations,
                $options,
                $template->getLineNo() - 1
            );
        }
    }

    /**
     * Extracts script tag contents using regex instead of DOM operations.
     * If we parse using DOM, some contents may change, for example, tags within strings will be stripped
     *
     * @param $string
     * @return bool|string
     */
    private static function extractScriptTag($string)
    {
        if (preg_match('#<\s*?script\b[^>]*>(.*?)</script\b[^>]*>#s', $string, $matches)) {
            return $matches[1];
        }

        return '';
    }

    /**
     * @param string $html
     * @return DOMDocument
     */
    private static function convertHtmlToDom($html)
    {
        $dom = new DOMDocument;

        libxml_use_internal_errors(true);
        $dom->loadHTML($html);

        libxml_clear_errors();

        return $dom;
    }

    /**
     * Extract translations from script part
     *
     * @param string $scriptContents Only script tag contents, not the whole template
     * @param Translations|Translations[] $translations One or multiple domain Translation objects
     * @param array $options
     * @param int $lineOffset Number of lines the script is offset in the vue template file
     * @throws Exception
     */
    private static function getScriptTranslationsFromString(
        $scriptContents,
        $translations,
        array $options = [],
        $lineOffset = 0
    ) {
        $functions = new JsFunctionsScanner($scriptContents);
        $options['lineOffset'] = $lineOffset;
        $functions->saveGettextFunctions($translations, $options);
    }

    /**
     * Parse template to extract all translations (element content and dynamic element attributes)
     *
     * @param DOMNode $dom
     * @param Translations|Translations[] $translations One or multiple domain Translation objects
     * @param array $options
     * @param int $lineOffset Line number where the template part starts in the vue file
     * @throws Exception
     */
    private static function getTemplateTranslations(
        DOMNode $dom,
        $translations,
        array $options,
        $lineOffset = 0
    ) {
        // Build a JS string from all template attribute expressions
        $fakeAttributeJs = self::getTemplateAttributeFakeJs($options, $dom);

        // 1 line offset is necessary because parent template element was ignored when converting to DOM
        self::getScriptTranslationsFromString($fakeAttributeJs, $translations, $options, $lineOffset);

        // Build a JS string from template element content expressions
        $fakeTemplateJs = self::getTemplateFakeJs($dom);
        self::getScriptTranslationsFromString($fakeTemplateJs, $translations, $options, $lineOffset);

        self::getTagTranslations($options, $dom, $translations);
    }

    /**
     * @param array $options
     * @param DOMNode $dom
     * @param Translations|Translations[] $translations
     */
    private static function getTagTranslations(array $options, DOMNode $dom, $translations)
    {
        // Since tag scanning does not support domains, we always use the first translation given
        $translations = is_array($translations) ? reset($translations) : $translations;

        $children = $dom->childNodes;
        for ($i = 0; $i < $children->length; $i++) {
            $node = $children->item($i);

            if (!($node instanceof DOMElement)) {
                continue;
            }

            $translatable = false;

            if (in_array($node->tagName, $options['tagNames'], true)) {
                $translatable = true;
            }

            $attrList = $node->attributes;
            $context = null;
            $plural = "";
            $comment = null;

            for ($j = 0; $j < $attrList->length; $j++) {

                /** @var DOMAttr $domAttr */
                $domAttr = $attrList->item($j);
                // Check if this is a dynamic vue attribute
                if (in_array($domAttr->name, $options['tagAttributes'])) {
                    $translatable = true;
                }
                if (in_array($domAttr->name, $options['contextAttributes'])) {
                    $context = $domAttr->value;
                }
                if (in_array($domAttr->name, $options['pluralAttributes'])) {
                    $plural = $domAttr->value;
                }
                if (in_array($domAttr->name, $options['commentAttributes'])) {
                    $comment = $domAttr->value;
                }
            }

            if ($translatable) {
                $translation = $translations->insert($context, trim($node->textContent), $plural);
                $translation->addReference($options['file'], $node->getLineNo());
                if ($comment) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                    $translation->addExtractedComment($comment);
                }
            }

            if ($node->hasChildNodes()) {
                self::getTagTranslations($options, $node, $translations);
            }
        }
    }

    /**
     * Extract JS expressions from element attribute bindings (excluding text within elements)
     * For example: <span :title="__('extract this')"> skip element content </span>
     *
     * @param array $options
     * @param DOMNode $dom
     * @return string JS code
     */
    private static function getTemplateAttributeFakeJs(array $options, DOMNode $dom)
    {
        $expressionsByLine = self::getVueAttributeExpressions($options['attributePrefixes'], $dom);

        if (empty($expressionsByLine)) {
            return '';
        }

        $maxLines = max(array_keys($expressionsByLine));
        $fakeJs = '';

        for ($line = 1; $line <= $maxLines; $line++) {
            if (isset($expressionsByLine[$line])) {
                $fakeJs .= implode("; ", $expressionsByLine[$line]);
            }
            $fakeJs .= "\n";
        }

        return $fakeJs;
    }

    /**
     * Loop DOM element recursively and parse out all dynamic vue attributes which are basically JS expressions
     *
     * @param array $attributePrefixes List of attribute prefixes we parse as JS (may contain translations)
     * @param DOMNode $dom
     * @param array $expressionByLine [lineNumber => [jsExpression, ..], ..]
     * @return array [lineNumber => [jsExpression, ..], ..]
     */
    private static function getVueAttributeExpressions(
        array $attributePrefixes,
        DOMNode $dom,
        array &$expressionByLine = []
    ) {
        $children = $dom->childNodes;

        for ($i = 0; $i < $children->length; $i++) {
            $node = $children->item($i);

            if (!($node instanceof DOMElement)) {
                continue;
            }
            $attrList = $node->attributes;

            for ($j = 0; $j < $attrList->length; $j++) {

                /** @var DOMAttr $domAttr */
                $domAttr = $attrList->item($j);

                // Check if this is a dynamic vue attribute
                if (self::isAttributeMatching($domAttr->name, $attributePrefixes)) {
                    $line = $domAttr->getLineNo();
                    $expressionByLine += [$line => []];
                    $expressionByLine[$line][] = $domAttr->value;
                }
            }

            if ($node->hasChildNodes()) {
                $expressionByLine = self::getVueAttributeExpressions($attributePrefixes, $node, $expressionByLine);
            }
        }

        return $expressionByLine;
    }

    /**
     * Check if this attribute name should be parsed for translations
     *
     * @param string $attributeName
     * @param string[] $attributePrefixes
     * @return bool
     */
    private static function isAttributeMatching($attributeName, $attributePrefixes)
    {
        foreach ($attributePrefixes as $prefix) {
            if (strpos($attributeName, $prefix) === 0) {
                return true;
            }
        }
        return false;
    }

    /**
     * Extract JS expressions from within template elements (excluding attributes)
     * For example: <span :title="skip attributes"> {{__("extract element content")}} </span>
     *
     * @param DOMNode $dom
     * @return string JS code
     */
    private static function getTemplateFakeJs(DOMNode $dom)
    {
        $fakeJs = '';
        $lines = explode("\n", $dom->textContent);

        // Build a fake JS file from template by extracting JS expressions within each template line
        foreach ($lines as $line) {
            $expressionMatched = self::parseOneTemplateLine($line);

            $fakeJs .= implode("; ", $expressionMatched) . "\n";
        }

        return $fakeJs;
    }

    /**
     * Match JS expressions in a template line
     *
     * @param string $line
     * @return string[]
     */
    private static function parseOneTemplateLine($line)
    {
        $line = trim($line);

        if (!$line) {
            return [];
        }

        $regex = '#\{\{(.*?)\}\}#';

        preg_match_all($regex, $line, $matches);

        $matched = array_map(function ($v) {
            return trim($v, '\'"{}');
        }, $matches[1]);

        return $matched;
    }
}


1		<?php
2		/** @noinspection PhpComposerExtensionStubsInspection */
3
4		namespace Gettext\Extractors;
5
6		use DOMAttr;
7		use DOMDocument;
8		use DOMElement;
9		use DOMNode;
10		use Exception;
11		use Gettext\Translations;
12		use Gettext\Utils\JsFunctionsScanner;
13
14		/**
15		* Class to get gettext strings from VueJS template files.
16		*/
17		class VueJs extends Extractor implements ExtractorInterface, ExtractorMultiInterface
18		{
19		public static $options = [
20		'constants' => [],
21
22		'functions' => [
23		'gettext' => 'gettext',
24		'__' => 'gettext',
25		'ngettext' => 'ngettext',
26		'n__' => 'ngettext',
27		'pgettext' => 'pgettext',
28		'p__' => 'pgettext',
29		'dgettext' => 'dgettext',
30		'd__' => 'dgettext',
31		'dngettext' => 'dngettext',
32		'dn__' => 'dngettext',
33		'dpgettext' => 'dpgettext',
34		'dp__' => 'dpgettext',
35		'npgettext' => 'npgettext',
36		'np__' => 'npgettext',
37		'dnpgettext' => 'dnpgettext',
38		'dnp__' => 'dnpgettext',
39		'noop' => 'noop',
40		'noop__' => 'noop',
41		],
42		];
43
44		/**
45		* @inheritDoc
46		* @throws Exception
47		*/
48	View Code Duplication	public static function fromFileMultiple($file, array $translations, array $options = [])
		0 ignored issues – show Duplication introduced 2019-09-09 12:52 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
49		{
50		foreach (self::getFiles($file) as $file) {
51		$options['file'] = $file;
52		static::fromStringMultiple(self::readFile($file), $translations, $options);
53		}
54		}
55
56		/**
57		* @inheritdoc
58		* @throws Exception
59		*/
60		public static function fromString($string, Translations $translations, array $options = [])
61		{
62		self::fromStringMultiple($string, [$translations], $options);
63		}
64
65		/**
66		* @inheritDoc
67		* @throws Exception
68		*/
69		public static function fromStringMultiple($string, array $translations, array $options = [])
70		{
71		$options += self::$options;
72		$options += [
73		// HTML attribute prefixes we parse as JS which could contain translations (are JS expressions)
74		'attributePrefixes' => [
75		':',
76		'v-bind:',
77		'v-on:',
78		'v-text',
79		],
80		// HTML Tags to parse
81		'tagNames' => [
82		'translate',
83		],
84		// HTML tags to parse when attribute exists
85		'tagAttributes' => [
86		'v-translate',
87		],
88		// Comments
89		'commentAttributes' => [
90		'translate-comment',
91		],
92		'contextAttributes' => [
93		'translate-context',
94		],
95		// Attribute with plural content
96		'pluralAttributes' => [
97		'translate-plural',
98		],
99		];
100
101		// Ok, this is the weirdest hack, but let me explain:
102		// On Linux (Mac is fine), when converting HTML to DOM, new lines get trimmed after the first tag.
103		// So if there are new lines between <template> and next element, they are lost
104		// So we insert a "." which is a text node, and it will prevent that newlines are stripped between elements.
105		// Same thing happens between template and script tag.
106		$string = str_replace('<template>', '<template>.', $string);
107		$string = str_replace('</template>', '</template>.', $string);
108
109		// Normalize newlines
110		$string = str_replace(["\r\n", "\n\r", "\r"], "\n", $string);
111
112		// VueJS files are valid HTML files, we will operate with the DOM here
113		$dom = self::convertHtmlToDom($string);
114
115		$script = self::extractScriptTag($string);
116
117		// Parse the script part as a regular JS code
118		if ($script) {
119		$scriptLineNumber = $dom->getElementsByTagName('script')->item(0)->getLineNo();
120		self::getScriptTranslationsFromString(
121		$script,
		0 ignored issues – show Bug introduced 2018-08-27 07:02 UTC by Report Bug Copy Issue Report It seems like `$script` defined by `self::extractScriptTag($string)` on line `115` can also be of type `boolean`; however, `Gettext\Extractors\VueJs...ranslationsFromString()` does only seem to accept `string`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
122		$translations,
123		$options,
124		$scriptLineNumber - 1
125		);
126		}
127
128		// Template part is parsed separately, all variables will be extracted
129		// and handled as a regular JS code
130		$template = $dom->getElementsByTagName('template')->item(0);
131		if ($template) {
132		self::getTemplateTranslations(
133		$template,
134		$translations,
135		$options,
136		$template->getLineNo() - 1
137		);
138		}
139		}
140
141		/**
142		* Extracts script tag contents using regex instead of DOM operations.
143		* If we parse using DOM, some contents may change, for example, tags within strings will be stripped
144		*
145		* @param $string
146		* @return bool\|string
147		*/
148		private static function extractScriptTag($string)
149		{
150		if (preg_match('#<\s?script\b[^>]>(.?)</script\b[^>]>#s', $string, $matches)) {
151		return $matches[1];
152		}
153
154		return '';
155		}
156
157		/**
158		* @param string $html
159		* @return DOMDocument
160		*/
161		private static function convertHtmlToDom($html)
162		{
163		$dom = new DOMDocument;
164
165		libxml_use_internal_errors(true);
166		$dom->loadHTML($html);
167
168		libxml_clear_errors();
169
170		return $dom;
171		}
172
173		/**
174		* Extract translations from script part
175		*
176		* @param string $scriptContents Only script tag contents, not the whole template
177		* @param Translations\|Translations[] $translations One or multiple domain Translation objects
178		* @param array $options
179		* @param int $lineOffset Number of lines the script is offset in the vue template file
180		* @throws Exception
181		*/
182		private static function getScriptTranslationsFromString(
183		$scriptContents,
184		$translations,
185		array $options = [],
186		$lineOffset = 0
187		) {
188		$functions = new JsFunctionsScanner($scriptContents);
189		$options['lineOffset'] = $lineOffset;
190		$functions->saveGettextFunctions($translations, $options);
191		}
192
193		/**
194		* Parse template to extract all translations (element content and dynamic element attributes)
195		*
196		* @param DOMNode $dom
197		* @param Translations\|Translations[] $translations One or multiple domain Translation objects
198		* @param array $options
199		* @param int $lineOffset Line number where the template part starts in the vue file
200		* @throws Exception
201		*/
202		private static function getTemplateTranslations(
203		DOMNode $dom,
204		$translations,
205		array $options,
206		$lineOffset = 0
207		) {
208		// Build a JS string from all template attribute expressions
209		$fakeAttributeJs = self::getTemplateAttributeFakeJs($options, $dom);
210
211		// 1 line offset is necessary because parent template element was ignored when converting to DOM
212		self::getScriptTranslationsFromString($fakeAttributeJs, $translations, $options, $lineOffset);
213
214		// Build a JS string from template element content expressions
215		$fakeTemplateJs = self::getTemplateFakeJs($dom);
216		self::getScriptTranslationsFromString($fakeTemplateJs, $translations, $options, $lineOffset);
217
218		self::getTagTranslations($options, $dom, $translations);
219		}
220
221		/**
222		* @param array $options
223		* @param DOMNode $dom
224		* @param Translations\|Translations[] $translations
225		*/
226		private static function getTagTranslations(array $options, DOMNode $dom, $translations)
227		{
228		// Since tag scanning does not support domains, we always use the first translation given
229		$translations = is_array($translations) ? reset($translations) : $translations;
230
231		$children = $dom->childNodes;
232		for ($i = 0; $i < $children->length; $i++) {
233		$node = $children->item($i);
234
235		if (!($node instanceof DOMElement)) {
236		continue;
237		}
238
239		$translatable = false;
240
241		if (in_array($node->tagName, $options['tagNames'], true)) {
242		$translatable = true;
243		}
244
245		$attrList = $node->attributes;
246		$context = null;
247		$plural = "";
248		$comment = null;
249
250		for ($j = 0; $j < $attrList->length; $j++) {
		0 ignored issues – show Bug introduced 2018-04-29 13:11 UTC by Report Bug Copy Issue Report The property `length` does not seem to exist in `DOMNamedNodeMap`. An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
251		/** @var DOMAttr $domAttr */
252		$domAttr = $attrList->item($j);
253		// Check if this is a dynamic vue attribute
254		if (in_array($domAttr->name, $options['tagAttributes'])) {
255		$translatable = true;
256		}
257		if (in_array($domAttr->name, $options['contextAttributes'])) {
258		$context = $domAttr->value;
259		}
260		if (in_array($domAttr->name, $options['pluralAttributes'])) {
261		$plural = $domAttr->value;
262		}
263		if (in_array($domAttr->name, $options['commentAttributes'])) {
264		$comment = $domAttr->value;
265		}
266		}
267
268		if ($translatable) {
269		$translation = $translations->insert($context, trim($node->textContent), $plural);
270		$translation->addReference($options['file'], $node->getLineNo());
271		if ($comment) {
		0 ignored issues – show Bug Best Practice introduced 2019-04-15 14:06 UTC by Report Bug Copy Issue Report The expression `$comment` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
272		$translation->addExtractedComment($comment);
273		}
274		}
275
276		if ($node->hasChildNodes()) {
277		self::getTagTranslations($options, $node, $translations);
278		}
279		}
280		}
281
282		/**
283		* Extract JS expressions from element attribute bindings (excluding text within elements)
284		* For example: <span :title="__('extract this')"> skip element content </span>
285		*
286		* @param array $options
287		* @param DOMNode $dom
288		* @return string JS code
289		*/
290		private static function getTemplateAttributeFakeJs(array $options, DOMNode $dom)
291		{
292		$expressionsByLine = self::getVueAttributeExpressions($options['attributePrefixes'], $dom);
293
294		if (empty($expressionsByLine)) {
295		return '';
296		}
297
298		$maxLines = max(array_keys($expressionsByLine));
299		$fakeJs = '';
300
301		for ($line = 1; $line <= $maxLines; $line++) {
302		if (isset($expressionsByLine[$line])) {
303		$fakeJs .= implode("; ", $expressionsByLine[$line]);
304		}
305		$fakeJs .= "\n";
306		}
307
308		return $fakeJs;
309		}
310
311		/**
312		* Loop DOM element recursively and parse out all dynamic vue attributes which are basically JS expressions
313		*
314		* @param array $attributePrefixes List of attribute prefixes we parse as JS (may contain translations)
315		* @param DOMNode $dom
316		* @param array $expressionByLine [lineNumber => [jsExpression, ..], ..]
317		* @return array [lineNumber => [jsExpression, ..], ..]
318		*/
319		private static function getVueAttributeExpressions(
320		array $attributePrefixes,
321		DOMNode $dom,
322		array &$expressionByLine = []
323		) {
324		$children = $dom->childNodes;
325
326		for ($i = 0; $i < $children->length; $i++) {
327		$node = $children->item($i);
328
329		if (!($node instanceof DOMElement)) {
330		continue;
331		}
332		$attrList = $node->attributes;
333
334		for ($j = 0; $j < $attrList->length; $j++) {
		0 ignored issues – show Bug introduced 2019-04-15 14:06 UTC by Report Bug Copy Issue Report The property `length` does not seem to exist in `DOMNamedNodeMap`. An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
335		/** @var DOMAttr $domAttr */
336		$domAttr = $attrList->item($j);
337
338		// Check if this is a dynamic vue attribute
339		if (self::isAttributeMatching($domAttr->name, $attributePrefixes)) {
340		$line = $domAttr->getLineNo();
341		$expressionByLine += [$line => []];
342		$expressionByLine[$line][] = $domAttr->value;
343		}
344		}
345
346		if ($node->hasChildNodes()) {
347		$expressionByLine = self::getVueAttributeExpressions($attributePrefixes, $node, $expressionByLine);
348		}
349		}
350
351		return $expressionByLine;
352		}
353
354		/**
355		* Check if this attribute name should be parsed for translations
356		*
357		* @param string $attributeName
358		* @param string[] $attributePrefixes
359		* @return bool
360		*/
361		private static function isAttributeMatching($attributeName, $attributePrefixes)
362		{
363		foreach ($attributePrefixes as $prefix) {
364		if (strpos($attributeName, $prefix) === 0) {
365		return true;
366		}
367		}
368		return false;
369		}
370
371		/**
372		* Extract JS expressions from within template elements (excluding attributes)
373		* For example: <span :title="skip attributes"> {{__("extract element content")}} </span>
374		*
375		* @param DOMNode $dom
376		* @return string JS code
377		*/
378		private static function getTemplateFakeJs(DOMNode $dom)
379		{
380		$fakeJs = '';
381		$lines = explode("\n", $dom->textContent);
382
383		// Build a fake JS file from template by extracting JS expressions within each template line
384		foreach ($lines as $line) {
385		$expressionMatched = self::parseOneTemplateLine($line);
386
387		$fakeJs .= implode("; ", $expressionMatched) . "\n";
388		}
389
390		return $fakeJs;
391		}
392
393		/**
394		* Match JS expressions in a template line
395		*
396		* @param string $line
397		* @return string[]
398		*/
399		private static function parseOneTemplateLine($line)
400		{
401		$line = trim($line);
402
403		if (!$line) {
404		return [];
405		}
406
407		$regex = '#\{\{(.*?)\}\}#';
408
409		preg_match_all($regex, $line, $matches);
410
411		$matched = array_map(function ($v) {
412		return trim($v, '\'"{}');
413		}, $matches[1]);
414
415		return $matched;
416		}
417		}
418

oscarotero / Gettext

Push — master ( a20241...2c1d12 )

VueJs::fromStringMultiple() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like