magento-coding-standard icon indicating copy to clipboard operation
magento-coding-standard copied to clipboard

Is there any reason why function `parse_url` is Discouraged?

Open VladyslavSikailo opened this issue 2 years ago • 1 comments

When I use parse_url() function - phpcs shows me the following error:

 39 | WARNING | The use of function parse_url() is discouraged

Is there any reason why I should not use this function? I don't see any.

Do I have any alternative functions to use? It was possible to use \Zend\Uri\Http but it's removed starting from Magento 2.4.1.

Can I somehow remove it from the Discouraged function list using a custom ruleset? Seems like I can't. Update: I have disabled it using the following sentence, but not sure whether it right way

    <rule ref="Magento2.Functions.DiscouragedFunction">
        <properties>
            <property name="forbiddenFunctions" type="array">
                <element key="!parse_url" value="null"/>
            </property>
        </properties>
    </rule>

See: https://github.com/magento/magento-coding-standard/blob/develop/Magento2/Sniffs/Functions/DiscouragedFunctionSniff.php#L103

VladyslavSikailo avatar Jun 03 '22 15:06 VladyslavSikailo

Hi @Dren7755. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this


m2-assistant[bot] avatar Jun 03 '22 15:06 m2-assistant[bot]