Mismatch between extended class name and the actual class name about strauss HOT 3 OPEN

@BrianHenryIE as a more general question, would you say polyfills like these should be excluded from Strauss treatments?

Considering that for example this Normalizer polyfill defines missing global functions like normalizer_is_normalized and normalizer_normalize only when they're not defined in the first place, and that those functions aren't renamed themselves (or contained in a namespace)

from strauss.

I didn't mean to close this. Still more work to do!

from strauss.

OK, @alexmigf , the actual bug is fixed.

But as @Spreeuw points out, this might not be worth prefixing.

bootstrap.php operates in the global namespace. Strauss adds prefixes to existing namespaces, it does not add a namespace to files with none in them (although that's an interesting idea and could work as an alternative to prefixing classnames).

So when our plugin autoloader runs, Symphony's boostrap is going to check for a global normalizer_is_normalized() function, and add it if absent. If another plugin has loaded an older (or newer) version of polyfill-intl-normalizer, normalizer_is_normalized() will already be defined and that code used.

So the problem Strauss aims to solve is not correctly addressed here. And I don't think there is any benefit to prefixing the namespace.

I've seen suggestions for unit testing global functions by redefining them inside the project root namespace. i.e. Normalizer_Test\normalizer_is_normalized(). I think if normalizer_is_normalized() is called within the Normalizer_Test namespace, it will call normalizer_is_normalized() if available and Normalizer_Test\normalizer_is_normalized() otherwise.

I'll leave this open for discussion and until I test that idea out.

This is the config for excluding it, and this may end up being the default configuration:

{
    "require": {
        "symfony/polyfill-intl-normalizer": "^1.23"
    },
    "extra": {
        "strauss": {
            "exclude_from_prefix": {
                "packages": [
                        "symfony/polyfill-intl-normalizer"
                ]
            }
        }
    }
}

from strauss.