dg/composer-backslasher

Composer plugin that speeds up your application by adding backslashes to all PHP internal functions and constants.

Composer Backslasher

Composer plugin that speeds up your application by adding backslashes to all PHP internal functions and constants in /vendor. Does not modify files of your application.

Installation

composer require dg/composer-backslasher

Then simply use composer update.

How it works?

It simply turns this code:

namespace A;

if (preg_match('/(foo)(bar)(baz)/', 'foobarbaz', $matches, PREG_OFFSET_CAPTURE)) {
	// ...
}

into this code:

namespace A;

if (\preg_match('/(foo)(bar)(baz)/', 'foobarbaz', $matches, \PREG_OFFSET_CAPTURE)) {
	// ...
}

to avoid double lookup for global functions and constants.

Configuration

If you want to ignore certain functions or constants, specify them in the configuration. Simply add a extra > backslasher-ignore section to composer.json file:

{
	"extra": {
		"backslasher-ignore": [
			"GuzzleHttp\\Promise\\each"
		]
	}
}

Support Project

Do you like Composer Backslasher? Are you looking forward to the new features?

composer-backslasher's Issues

doesn't change files

• bug report? yes
• feature request? no

Description

after installing this package and running composer update, i can see the message "Composer Backslasher: Added 39 backslashes." but in reality no file has changed (i have nothing to commit)

Steps To Reproduce

1. run composer require dg/composer-backslasher
2. git status:

	modified:   composer.json
	modified:   composer.lock
	modified:   symfony.lock

3. run composer update
4. see "Composer Backslasher: Added 39 backslashes."
5. git status:

	modified:   composer.json
	modified:   composer.lock
	modified:   symfony.lock

6. 🤔

wrong parsing

• bug report? yes
• feature request? no

Description

backslash is added to the function name and it breaks the application
public static function NAN() became public static function \NAN()

Steps To Reproduce

define function with name public static function NAN() in vendor
(f.e. "phpoffice/phpspreadsheet": "^1.1")
include this package and run composer update
code will change to public static function \NAN()

Local function overriding the one from PHP API

• bug report? yes
• feature request? yes

Description

This tool is incompatible with package https://github.com/guzzle/promises which override PHP function each by its own version (and maybe some other).

Steps To Reproduce

Use library with following content:

<?php
namespace A;

function settle($promises)
{
    $results = [];
    return each(
        $promises,
        function () {},
        function () {}
    )->then(function () use (&$results) {
        ksort($results);
        return $results;
    });
}

function each(
    $iterable,
    callable $onFulfilled = null,
    callable $onRejected = null
) {
    return 'baz';
}

Expected result after processed by this tool

<?php
namespace A;

function settle($promises)
{
    $results = [];
    return each(
        $promises,
        function () {},
        function () {}
    )->then(function () use (&$results) {
        \ksort($results);
        return $results;
    });
}

function each(
    $iterable,
    callable $onFulfilled = null,
    callable $onRejected = null
) {
    return 'baz';
}

What is produced instead

<?php
namespace A;

function settle($promises)
{
    $results = [];
    return \each( // notice this back slash, targeting global method instead local one
        $promises,
        function () {},
        function () {}
    )->then(function () use (&$results) {
        \ksort($results);
        return $results;
    });
}

function each(
    $iterable,
    callable $onFulfilled = null,
    callable $onRejected = null
) {
    return 'baz';
}

Proposed solution

• Add possibility to blacklist some directories
• Check existence of overriding functions

add to require-dev instead of require

• bug report? no
• feature request? no

Description

this package is useful in dev environment but useless in any other (testing, production, ...)
i would even say it is harmful in production as it has to be autoloaded

Suggestion

install via composer require dg/composer-backslasher --dev