This project forked from coenjacobs/mozart
Prefix PHP namespaces and classnames to allow multiple versions of libraries to exist without conflict.
Home Page: https://brianhenryie.github.io/strauss/
License: MIT License
I am receiving an infinite loop with the recursiveGetAllDependencies() function when using the PHAR as reported by Xdebug. Attached is my composer.json.
It appears to be around the league/flysystem package. Multiple lines like so:
45.3508 30967016 30. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [1 => 'league/flysystem-local', 2 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
45.4071 25482912 31. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [2 => 'league/flysystem', 3 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
45.4556 27192200 32. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [1 => 'league/flysystem-local', 2 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
45.4996 28892624 33. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [2 => 'league/flysystem', 3 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
45.5499 30601912 34. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [1 => 'league/flysystem-local', 2 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
45.6173 25444520 35. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [2 => 'league/flysystem', 3 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
45.6785 27153808 36. BrianHenryIE\Strauss\Console\Commands\Compose->recursiveGetAllDependencies($requiredPackageNames = [1 => 'league/flysystem-local', 2 => 'league/mime-type-detection']) phar:///Users/jonerickson/PhpstormProjects/wordpress-community-hive/strauss.phar/src/Console/Commands/Compose.php:173
Any help on how to diagnose would be great.
{
"require": {
"php": "^8.1",
"roots/acorn": "^3.2",
"guzzlehttp/guzzle": "^7.0",
"web-token/jwt-framework": "^3.2"
},
"require-dev": {
"spatie/laravel-ignition": "^1.6",
"laravel/pint": "^v1.10.5",
"fzaninotto/faker": "^1.5"
},
"autoload": {
"psr-4": {
"CommunityHive\\App\\": "src/app",
"CommunityHive\\Database\\Factories\\": "src/database/factories/",
"CommunityHive\\Database\\Seeders\\": "src/database/seeders/"
}
},
"autoload-dev": {
"psr-4": {
"Tests\\": "src/tests/"
}
},
"post-autoload-dump": [
"Roots\\Acorn\\ComposerScripts::postAutoloadDump"
]
}
https://github.com/JsonMapper/JsonMapper
src/JsonMapper.php:224
throw new \JsonException(json_last_error_msg(), \json_last_error());
to:
throw new \My_Prefix_JsonExceptionJsonException\json_last_error());
https://github.com/symfony/polyfill-php73/blob/1.x/Resources/stubs/JsonException.php
Hi Brian,
I spent sometime tonight figuring out how to make this work, including setting up a composer package at https://packagist.org/packages/qriouslad/codestar-framework for the library I needed to prefix.
I've successfully executed the strauss script via composer to prefix that package, however, I noticed not all classes are properly prefixed.
For example, in setup.class.php, while if ( class_exists( 'CSF_Options' )) is successfully prefixed into if ( class_exists( 'Prefix_CSF_Options' )), the if (class_exists( 'CSF_Metabox' )) part remains the same, not getting prefixed, as well as other classes in that file.
Since this library is not using namespaces yet, I needed to use this in the package composer.json:
"autoload": { "classmap": ["src/"] },
...where the library files are inside the /src/ folder. Could this be a method that is not yet fully supported by strauss?
"/Users/brianhenry/Sites/woocommerce-gateway-bitcoin/vendor/blockchain/blockchain/composer.json" does not match the expected JSON schema:
- name : Does not match the regex pattern ^[a-z0-9]([_.-]?[a-z0-9]+)*/[a-z0-9](([_.]?|-{0,2})[a-z0-9]+)*$Script vendor/bin/strauss handling the post-update-cmd event returned with error code 1
{
"name": "Blockchain/Blockchain",
"description": "Blockchain API client library",
"keywords": ["bitcoin", "blockchain"],
"homepage": "https://github.com/blockchain/api-v1-client-php",
"license": "MIT",
"autoload": {
"psr-4": {
"Blockchain\\": "src/"
}
},
"require": {
"php": ">=5.3.0",
"ext-curl": "*"
}
}
Manually editing the package name to "blockchain/blockchain" fixed it.
Given a package that is in a directory on the local filesystem
"repositories": [
{
"type": "path",
"url": "../bh-pdf-helpers"
},
...
],
"require": {
"brianhenryie/pdf-helpers": "dev-master",
...
Error:
The "/Users/brianhenry/Sites/my-project/vendor/Sites/bh-pdf-helpers/src/" directory does not exist.
The problem is in ComposerPackage constructor.
This was introduced when supporting packages that install themselves to a folder different to their package name.
Edit: Fix written but needs testing and thinking about.
When installing this via composer the binary is called mozart and not strauss as mentioned in the documentation.
I get this:
vendor/bin/mozart
Instead of this:
vendor/bin/strauss
Have I missed something or is this a mistake?
files always get copied, even when they match an exclusion in exclude_from_copy.
when multiple directories are add for the same namespace as specified in Composer documentaion:
{
"autoload": {
"psr-4": { "Monolog\\": ["src/", "lib/"] }
}
}
it results in the error:
PHP Fatal error: Uncaught TypeError: trim(): Argument #1 ($string) must be of type string, array given in [REDACTED]/vendor/brianhenryie/strauss/src/Composer/Extra/StraussConfig.php:235
it seems like the problem is strauss always expect the configuration to be a string and try to trim() it.
First of all; thank you for this package. It works wonderfull.
Although I'm running into an issue when using delete_vendor_packages in the config. The reason I want to use this is to cleanup my vendor directory so only specific dependencies stay there and only those will be autoloaded using composer. That is because I use the composer autoloader for some packages that are not using namespaces and I also use the composer autoloader to load my custom plugin classes.
That means I include both of the autoloaders in my plugin like so:
/**
* Autoload non prefixed- and plugin classes using Composer.
* @see https://getcomposer.org/doc/01-basic-usage.md#autoloading
*/
require_once __DIR__ . '/vendor/autoload.php';
/**
* Autoload prefixed classes using Strauss.
* @see https://github.com/BrianHenryIE/strauss
*/
require_once __DIR__ . '/vendor/prefixed/autoload.php';This is my config in the composer.json:
"scripts": {
"post-install-cmd": [
"sh prefixer.sh"
],
"post-update-cmd": [
"sh prefixer.sh"
]
},
"extra": {
"strauss": {
"target_directory": "vendor/prefixed",
"namespace_prefix": "Tussendoor\\Bol\\Vendor\\",
"constant_prefix": "TSD_BOL_VENDOR_",
"include_modified_date": false,
"include_author": false,
"delete_vendor_packages": true,
"exclude_from_copy": {
"packages": [
"yahnis-elsts/plugin-update-checker",
"woocommerce/action-scheduler"
]
}
}
}As you can see I am running a custom shell script after the regular composer commands like install and update. The shell script is looking like this:
#!/bin/bash
printf "\n"
printf "Composer action is completed! Wait for the new actions. \n"
printf "\n"
printf "Starting Strauss to prefix all dependencies \n"
vendor/bin/strauss
printf "\n"
printf "Strauss action is completed! \n"
printf "Dumping the composer autoload to prevent errors. \n"
composer dump-autoload
printf "\n"
printf "Autoload has been dumped. \n"
printf "Cleaning the empty folders in the vendor directory. \n"
find vendor -type d -empty -delete
printf "\n"
printf "All done! \n"So after running vendor/bin/strauss my vendor directory is looking like this. (leaving some stuff out for simplicity):
- vendor
- bin
- composer
- strauss
- composer
- (no directories only files like autoload.php)
- excluded-package
- moved-package-directories-that-are-now-empty
- prefixed
- (all packages except excluded)
- composer
- src
- Composer
- (some folders and some files)
All good until now. But the next step is to dump the composer autoload by composer dump-autoload and this is where I run into the following errors:
PHP Warning: include(/vendor/bin/../composer/composer/bin/composer): Failed to open stream: No such file or directory in /vendor/bin/composer on line 120
Warning: include(/vendor/bin/../composer/composer/bin/composer): Failed to open stream: No such file or directory in /vendor/bin/composer on line 120
PHP Warning: include(): Failed opening '/vendor/bin/../composer/composer/bin/composer' for inclusion (include_path='.:/opt/homebrew/Cellar/[email protected]/8.1.19/share/[email protected]/pear') in /vendor/bin/composer on line 120
Warning: include(): Failed opening '/vendor/bin/../composer/composer/bin/composer' for inclusion (include_path='.:/opt/homebrew/Cellar/[email protected]/8.1.19/share/[email protected]/pear') in /vendor/bin/composer on line 120
This is originating from vendor/bin/composer on line 120. That is the following:
include __DIR__ . '/..'.'/composer/composer/bin/composer';
This file isnt present anymore after using delete_vendor_packages and it also is not in the new vendor/prefixed/composer directory. I've tried excluding the composer/composer package by adding it to the exclude_from_copy config but that resulted in even more errors.
I believe this error could be resolved when the composer directory would not be copied to the target_directory. This directory should also not be cleaned so the file that is being searched for will still be present.
I've got the following questions:
target_directory at all?I'm attempting to bundle the aws-sdk-php package in my plugin (specifically the S3 component) and am running into an odd issue.
Basically these files:
vendor/aws/aws-sdk-php/src/Endpoint/UseDualstackEndpoint/Configuration.php
vendor/aws/aws-sdk-php/src/Endpoint/UseFipsEndpoint/Configuration.php
vendor/aws/aws-sdk-php/src/S3/UseArnRegion/Configuration.php
all have a line that looks similar to this:
$this->useFipsEndpoint = Aws\boolean_value($useFipsEndpoint);and after prefixing, they become something like this:
$this->useFipsEndpoint = MyPlugin\Vendor\Aws\boolean_value($useFipsEndpoint);Which results in this error:
PHP Fatal error: Uncaught Error: Call to undefined function MyPlugin\Vendor\Aws\Endpoint\UseFipsEndpoint\MyPlugin\Vendor\Aws\boolean_value() in /app/content/plugins/my-plugin/vendor-prefixed/aws/aws-sdk-php/src/Endpoint/UseFipsEndpoint/Configuration.php:20
If I manually add a backslash at the begining of these classes, like this:
$this->useFipsEndpoint = \MyPlugin\Vendor\Aws\boolean_value($useFipsEndpoint);The issue goes away and everything works as expected.
Any ideas?
So switching from Mozart to this, has been fantastic as this is all working out of the box without the need for patches, or figuring out which versions of a composer package I need. However, when I made a new class with an admin notice add_action, just to test and ensure this is all properly working, I get the following error. Am I missing something, do I need to somehow wrap the wp core as well? I know the way I'm calling to the action is fine, but not sure why this error would trigger when calling to a default WP function.
PHP Fatal error: Uncaught Error: Call to undefined function add_action() in /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/rap-site-reset.php:34
Stack trace:
#0 /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/vendor/composer/autoload_real.php(78): require()
#1 /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/vendor/composer/autoload_real.php(61): composerRequire9a0dc73fcbf3506c717760c1fe787923()
#2 /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/vendor/autoload.php(7): ComposerAutoloaderInit9a0dc73fcbf3506c717760c1fe787923::getLoader()
#3 /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/vendor/brianhenryie/strauss/bin/strauss(5): require('/home/user/Site...')
#4 /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/vendor/brianhenryie/strauss/bin/strauss(28): {closure}()
#5 /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/vendor/bin/strauss(112): include('/home/user/Site...')
#6 {main}
thrown in /home/user/Sites/functionplayground/web/app/plugins/rap-site-reset/rap-site-reset.php on line 34
Here's my composer.json
{
"name": "kupoback/rap-site-reset",
"description": "",
"type": "wordpress-muplugin",
"require": {
"composer/installers": "^2.0",
"ext-json": "*",
"php": "^7.4|^8.0.0",
"phpseclib/phpseclib": "dev-master",
"wp-cli/db-command": "^2.0",
"wp-cli/extension-command": "^2.0",
"wp-cli/maintenance-mode-command": "2.x-dev",
"illuminate/support": "^8.0"
},
"require-dev": {
"squizlabs/php_codesniffer": "4.0.x-dev",
"roave/security-advisories": "dev-master",
"brianhenryie/strauss": "dev-master"
},
"config": {
"optimize-autoloader": true,
"preferred-install": "dist",
"allow-plugins": {
"composer/installers": true,
"cweagans/composer-patches": true
}
},
"license": "MIT",
"scripts": {
"strauss": [
"vendor/bin/strauss"
],
"post-root-package-install": [
"@strauss"
],
"post-root-package-update": [
"@strauss"
],
"test": [
"phpcs"
]
},
"authors": [
{
"name": "kupoback",
}
],
"autoload": {
"files": [ "rap-site-reset.php" ],
"psr-4": {
"RapReset\\": "src/"
}
},
"minimum-stability": "dev",
"extra": {
"strauss": {
"classmap_prefix": "RapReset",
"constant_prefix": "RR_",
"delete_vendor_files": false,
"namespace_prefix": "RapReset\\Vendor",
"target_directory": "vendor_prefix"
}
}
}
Here's my rap-site-reset.php file:
use RapReset\RapReset;
require_once __DIR__ . '/vendor_prefix/autoload.php';
add_action('admin_notices', [RapReset::class, 'adminNotice']);
And my Class
namespace RapReset;
class RapReset
{
public static function adminNotice()
{
echo "Hello World";
}
}
When utilizing libraries that contain files like fonts (tecnickcom/tcpdf) or other non-php assets (composer/ca-bundle is a library built around a cacert.pem file), these assets are not copied over by Strauss. We currently resolve this with a bash script that runs after Strauss, but it would be nice to be able to tell Strauss which folders to copy over, or even to let it copy all the files/folders for a specific library?
using just this:
"require": {
"iio/libmergepdf": "^4.0"
},
results in object being prefixed in all files. This appears to be caused by the following docblock (in: tecnickcom/tcpdf/include/tcpdf_static.php#L326-L332)
/**
* Creates a copy of a class object
* @param $object (object) class object to be cloned
* @return cloned object
* @since 4.5.029 (2009-03-19)
* @public static
*/If I change that first line to Creates a copy of a classy object, the issue is resolved
Hi,
I'm trying to use your package in a plugin where I make use of some Symfony components.
When running strauss, I'm having the following error:
/full/path/to/project/vendor/symfony/process//Tests/" directory does not exist
The "exclude-from-classmap" key doesn't seem to be handled in strauss. Could you have a look?
I do my development primarily on Windows and have had various path issues with Mozart and Strauss before but it keeps regressing.
The current release does not work on Windows either. I've hacked together a quick fix here, but it feels like a patch at best.
The recurring theme is simple: code assumes forward slashes when in fact the path (on Windows) could have both forward and backward slashes, mainly due to inconsistent usage of DIRECTORY_SEPARATOR.
I propose to stop using DIRECTORY_SEPARATOR altogether and exclusively use forward slashes. PHP itself handles this fine on Windows, and as long as there are no things shell commands where you really need the correct separator, it's simply much easier to work with.
At the same time I think we could benefit from a simple Path helper class, with a few simple functions that replace a lot of recurring operations throughout Strauss (which at the moment also have slightly differing syntaxes, making it harder to debug path related issues):
Path::normalize( $path ) to replace all backward slashes with forward slash. For performance I would suggest using a simple str_replace rather than preg_replace. (similar to wp_normalize_path in WordPress).Path::ensureTrailingSlash( $path ) to append a trailing slash if not present. (similar to trailingslashit in WordPress, which is actually referenced in some of the comments!)Path::join( [ $path_1, $path_2 ] ) joining one or more paths with a /, avoiding ugly . '/' . string concatenations.Path::getRelativePath( $path, $basePath ) get the relative path of $path to $basePath.Function names, class structure etc. don't really matter, but I think this would make all path/file related code a lot more readable and more resistant to OS related issues. I looked for existing libraries handling this but couldn't find something that fits the bill, and also realized that it's pretty simple stuff to begin with.
We are facing an issue using your library with the symfony/polyfill-intl-normalizer, as the /Resources/stubs/Normalizer.php extended class shows like this:
class Normalizer_Test_Normalizer extends Normalizer_Test\Symfony\Polyfill\Intl\Normalizer_Test_Normalizer\Normalizer
{when the actual Normalizer.php class name in the library root dir is:
namespace Normalizer_Test\Symfony\Polyfill\Intl\Normalizer_Test_Normalizer;
class Normalizer_Test_Normalizer
{The error:
Uncaught Error: Class 'Normalizer_Test\Symfony\Polyfill\Intl\Normalizer_Test_Normalizer\Normalizer' not found in /lib/symfony/polyfill-intl-normalizer/Resources/stubs/Normalizer.php:9
You can easily test this using your phar.php and this JSON in a test directory:
{
"require": {
"symfony/polyfill-intl-normalizer": "^1.23"
},
"extra": {
"strauss": {
"target_directory": "lib",
"namespace_prefix": "Normalizer_Test\\",
"classmap_prefix": "Normalizer_Test_",
"packages": [],
"override_autoload": {
},
"exclude_from_copy": {
"packages": [
],
"namespaces": [
],
"patterns": [
]
},
"exclude_from_prefix": {
"packages": [
],
"namespaces": [
],
"patterns": [
"/^psr.*$/"
]
},
"namespace_replacement_patterns" : {
},
"delete_vendor_files": false
}
},
"scripts": {
"strauss": [
"@php strauss.phar"
],
"post-install-cmd": [
"@strauss"
],
"post-update-cmd": [
"@strauss"
]
}
}
bootstrap.php
"autoload": {
"classmap": ["src"],
"files": ["bootstrap.php"]
},
in a symlinked package
"repositories":{
"brianhenryie/bh-wp-plugin-meta-kit":{
"type": "path",
"url": "../bh-wp-plugin-meta-kit/"
}
was not copied to the destination directory.
And the generated autoload-files.php referenced the original path:
require_once __DIR__ . '/Sites/bh-wp-plugin-meta-kit/bootstrap.php';
MPDF is a 27000 line file. Replacing classes in it is failing.
Error: Backtrack limit was exhausted!
{
"name": "brianhenryie/strauss-backtrack-limit-exhausted",
"minimum-stability": "dev",
"require": {
"afragen/wp-dependency-installer": "^3.1",
"mpdf/mpdf": "*"
},
"require-dev": {
"brianhenryie/strauss": "dev-master"
},
"extra": {
"strauss": {
"namespace_prefix": "BrianHenryIE\\Strauss_Backtrack_Limit_Exhausted\\",
"target_directory": "/strauss/",
"classmap_prefix": "BH_Strauss_Backtrack_Limit_Exhausted_"
}
}
}
Hey,
I'm not sure if this a bug or just not a supported functionality. I'm using strauss and it's copying over the libraries fine and generating an autoload for the classes except for the /guzzlehttp/psr7/src/functions_include.php I'm aware that this file is being deprecated but I'm just not sure if this should work or not.
Thanks,
I'm running the library and it's only pulling and namespacing the dealerdirect/phpcodesniffer-composer-installer dependency which is actually a dev dependency..
Its totally ignoring a dependency not hosted on packagist and also one hosted there so it seems to just not be pulling my changes?
I also manually tried adding the packages to the "packages" array in the composer extra but it didn't have any effect.
My composer file:
{
"repositories": [
{
"type": "git",
"url": "https://github.com/iconicwp/WordPress-Settings-Framework.git"
}
],
"scripts": {
"prefix": "vendor/bin/strauss"
},
"require-dev": {
"dealerdirect/phpcodesniffer-composer-installer": "0.7.1",
"wp-coding-standards/wpcs": "2.3",
"suin/phpcs-psr4-sniff": "^3.0",
"brianhenryie/strauss": "^0.13.0"
},
"autoload":{
"psr-4": {
"Namespace\\Subnamespace\\" : "includes/"
},
"classmap":[
"vendor/iconicwp/wordpress-settings-framework/wp-settings-framework.php"
]
},
"require": {
"iconicwp/wordpress-settings-framework": "dev-master",
"freemius/wordpress-sdk": "^2.5"
},
"extra": {
"strauss": {
"target_directory": "vendor-prefixed",
"packages": [
"iconicwp/wordpress-settings-framework",
"freemius/wordpress-sdk"
],
"override_autoload": {
},
"exclude_from_copy": {
"packages": [
],
"namespaces": [
],
"file_patterns": [
]
},
"exclude_from_prefix": {
"packages": [
],
"namespaces": [
],
"file_patterns": [
"/^psr.*$/"
]
},
"namespace_replacement_patterns" : {
},
"delete_vendor_files": false
}
},
"config": {
"allow-plugins": {
"dealerdirect/phpcodesniffer-composer-installer": true
}
}
}
When I run strauss this the vendor-prefixed library:
"autoload": {
"psr-4": {
"Automattic\\WooCommerce\\": ["src/WooCommerce"]
}
}
Which is valid according to Composer schema: https://getcomposer.org/doc/04-schema.md#psr-4
Workaround:
"extra": {
"strauss": {
"override_autoload": {
"automattic/woocommerce": {
"psr-4": {
"Automattic\\WooCommerce\\": "src/WooCommerce"
}
}
},
Hi again and thanks for fixing #23 @BrianHenryIE!
Now with Strauss 0.10.1 I could find more static method namespace issues.
Namespaces aren't replaced
For exact behavior I created a gist with all kind of scenarios.
Thank you!
The strauss package is working perfectly for prefixing classes and constants. So far it helps us avoiding dependency hell within our custom WordPress plugin and we prevented a lot of compatibility issues with other plugins.
But I'm running into an issue. The illuminate\collections is being used and this package loads global functions in the helpers.php. Which is checking if a function exist like in this example:
if (! function_exists('collect')) {
/**
* Create a collection from the given value.
*
* @param mixed $value
* @return \Custom\Prefix\Illuminate\Support\Collection
*/
function collect($value = null)
{
return new Collection($value);
}
} But when a different plugin is using the same package and this plugin is loaded first; we end up in the global function of that plugin. Which is not using my custom prefixed package and which is not using the same version as we do in our plugin.
This is resulting in this error:
#0 /home/user/public_html/wp-content/plugins/reviewx/vendor/jouleslabs/warehouse/src/Supports/common.php(58): explode()
#1 /home/user/public_html/wp-content/plugins/reviewx/vendor/jouleslabs/warehouse/src/Supports/common.php(50): array_get()
#2 /home/user/public_html/wp-content/plugins/our-custom-plugin/vendor/illuminate/collections/Arr.php(459): data_get()
#3 /home/user/public_html/wp-content/plugins/our-custom-plugin/vendor/illuminate/collections/Collection.php(688): Custom\Prefix\Illuminate\Support\Arr::pluck()
This is my composer.json:
{
"name": "custom/plugin",
"license": "GPL-2.0-or-later",
"description": "WordPress plugin developed by Tussendoor.",
"type": "wordpress-plugin",
"require": {
"php": "^8.0",
"nesbot/carbon": "^2.66.0",
"illuminate/database": "^8.0.0",
"twbs/bootstrap": "^5.2.3",
"fortawesome/font-awesome": "^6.4.0",
"adbario/php-dot-notation": "^3.3.0",
"woocommerce/action-scheduler": "^3.4.0",
"yahnis-elsts/plugin-update-checker": "^4.13"
},
"autoload": {
"psr-4": {
"Custom\\Namespace\\": "app/"
},
"files": [
"vendor/woocommerce/action-scheduler/action-scheduler.php"
],
"classmap": [
"vendor"
]
},
"config": {
"optimize-autoloader": true,
"sort-packages": true,
"platform": {
"php": "8.0.3"
}
},
"scripts": {
"pre-update-cmd": [
"sh strauss-observer.sh"
],
"post-install-cmd": [
"sh strauss-prefixer.sh"
],
"post-update-cmd": [
"sh strauss-prefixer.sh"
]
},
"extra": {
"strauss": {
"target_directory": "vendor",
"namespace_prefix": "Custom\\Prefix\\",
"constant_prefix": "TSD_CUSTOM_PREFIX_",
"include_modified_date": false,
"include_author": false,
"delete_vendor_packages": true,
"exclude_from_prefix": {
"packages": [
"yahnis-elsts/plugin-update-checker",
"woocommerce/action-scheduler"
]
},
"override_autoload": {
"nesbot/carbon": {
"autoload": {
"psr-4": {
"Carbon\\": "src/Carbon/"
}
},
"classmap": [
"lazy"
]
}
}
}
}
}I'm using the strauss.phar at the moment. Is there a way to support prefixing global functions?
#[\ReturnTypeWillChange] is a special atribute provided by PHP to silence deprecation errors for return values. Strauss changes that very specific attribute name to add a prefix, and that leads to huge amounts of deprecation notices from Laravel while using phpunit.
Is there any way to stop this issue?
Quite a few operations will change or delete files. When packages are installed by symlink these operations will break other projects also using the packages.
It would be great to have a warning before any dangerous operations are performed, and a warning in the docs.
As metapackages do not follow the standard directory structure, Strauss has trouble dealing with them, as it cannot find their composer.json. An example of this can be seen by requiring league/omnipay. A workaround for this is to require the packages from the metapackage separately, but it would be more elegant if Strauss could handle this gracefully.
This also applies to other non-standard package types. For instance, if requiring composer/installers such that we can declare our package as a wordpress-plugin, rather than a library, strauss is unable to locate the package 'composer-plugin-api' and fails to compose dependencies.
I am unsure the best way to deal with these special cases - Perhaps at least in the case of composer-plugin-api we simply exclude this package by default
Running into these errors while running strauss:
PHP Deprecated: Return type of MyCLabs\Enum\Enum::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /Users/kodiegrantham/.composer/vendor/myclabs/php-enum/src/Enum.php on line 246
Deprecated: Return type of MyCLabs\Enum\Enum::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /Users/kodiegrantham/.composer/vendor/myclabs/php-enum/src/Enum.php on line 246
I have it installed as a global package due to a conflict with the version of flysystem that strauss uses and the one my package uses.
It hangs there for a while and then ultimately fails with this error:
PHP Fatal error: Uncaught TypeError: BrianHenryIE\Strauss\Licenser::addChangeDeclarationToPhpString(): Return value must be of type string, null returned in /Users/kodiegrantham/.composer/vendor/brianhenryie/strauss/src/Licenser.php:270
I get an error when running Strauss when delete_vendor_files is enabled:
File not found at path: vendor/vendor/trustedlogin/trustedlogin-client/src/SupportRole.php
I noted in the code that vendor/ is prepended. Maybe that's not necessary? https://github.com/BrianHenryIE/strauss/blob/master/src/Cleanup.php#L43
Hi again,
I got another case in which the namespace replacements are currently not working yet: Inside array square brackets. Right now I avoid this issue by declaring a variable, but of course the better solution would be having this fixed.
The namespace replacement works for non-static methods, but not for static methods.
StraussTestPackage.php:
<?php
namespace ST;
class StraussTestPackage {
public function __construct() {
$arr = array();
$arr[ ( new \ST\StraussTestPackage2() )->test() ] = true;
$arr[ \ST\StraussTestPackage2::test2() ] = true;
}
}
Result of StraussTestPackage.php
<?php
/**
* @license proprietary
*
* Modified using Strauss.
* @see https://github.com/BrianHenryIE/strauss
*/
namespace Prefix\ST;
class StraussTestPackage {
public function __construct() {
$arr = array();
$arr[ ( new \Prefix\ST\StraussTestPackage2() )->test() ] = true;
$arr[ \ST\StraussTestPackage2::test2() ] = true;
}
}
StraussTestPackage2.php:
<?php
namespace ST;
class StraussTestPackage2 {
public function __construct() {
}
public function test() {
return 'Test';
}
public static function test2() {
return 'Test';
}
}
guzzlehttp/guzzle/src/BodySummarizer.php:24
/**
* Returns a summarized message body.
*/
public function summarize(MessageInterface $message): ?string
{
return $this->truncateAt === null
? \GuzzleHttp\Psr7\Message::bodySummary($message)
: \GuzzleHttp\Psr7\Message::bodySummary($message, $this->truncateAt);
}
PHP Fatal error: Uncaught Error: Class "GuzzleHttp\Psr7\Message" not found in /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/guzzle/src/BodySummarizer.php:31
Stack trace:
#0 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/guzzle/src/Exception/RequestException.php(113): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\BodySummarizer->summarize(Object(BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Psr7\Response))
#1 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/guzzle/src/Middleware.php(75): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Exception\RequestException::create(Object(BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Psr7\Request), Object(BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Psr7\Response), NULL, Array, NULL)
#2 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(210): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Middleware::BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\{closure}(Object(BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Psr7\Response))
#3 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(159): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise::callHandler(1, Object(BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Psr7\Response), NULL)
#4 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/TaskQueue.php(54): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise::BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\{closure}()
#5 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(254): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\TaskQueue->run(true)
#6 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(230): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise->invokeWaitFn()
#7 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(275): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise->waitIfPending()
#8 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(232): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise->invokeWaitList()
#9 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/promises/src/Promise.php(68): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise->waitIfPending()
#10 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/guzzle/src/Client.php(129): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Promise\Promise->wait()
#11 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/klaviyo/sdk/lib/API/ProfilesApi.php(157): BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Client->send(Object(BrianHenryIE\WP_Autologin_URLs\GuzzleHttp\Psr7\Request), Array)
#12 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/klaviyo/sdk/lib/API/ProfilesApi.php(135): BrianHenryIE\WP_Autologin_URLs\Klaviyo\API\ProfilesApi->exchangeWithHttpInfo(Array)
#13 /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/klaviyo/sdk/lib/Subclient.php(57): BrianHenryIE\WP_Autologin_URLs\Klaviyo\API\ProfilesApi->exchange(Array)
#14 /path/to/wp-content/plugins/bh-wp-autologin-urls/API/Integrations/class-klaviyo.php(36): BrianHenryIE\WP_Autologin_URLs\Klaviyo\Subclient->__call('exchange', Array)
#15 /path/to/wp-content/plugins/bh-wp-autologin-urls/WP_Includes/class-login.php(236): BrianHenryIE\WP_Autologin_URLs\API\Integrations\Klaviyo->get_email('3GJUdebm1Z28U02...')
#16 /path/to/wp-content/plugins/bh-wp-autologin-urls/WP_Includes/class-login.php(79): BrianHenryIE\WP_Autologin_URLs\WP_Includes\Login->klaviyo()
#17 /path/to/wp-includes/class-wp-hook.php(307): BrianHenryIE\WP_Autologin_URLs\WP_Includes\Login->process_querystring('')
#18 /path/to/wp-includes/class-wp-hook.php(331): WP_Hook->apply_filters(NULL, Array)
#19 /path/to/wp-includes/plugin.php(476): WP_Hook->do_action(Array)
#20 /path/to/wp-settings.php(461): do_action('plugins_loaded')
#21 /path/to/wp-config.php(95): require_once('/chroot/home/ab...')
#22 /path/to/wp-load.php(50): require_once('/chroot/home/ab...')
#23 /path/to/wp-blog-header.php(13): require_once('/chroot/home/ab...')
#24 /path/to/index.php(17): require('/chroot/home/ab...')
#25 {main}
thrown in /path/to/wp-content/plugins/bh-wp-autologin-urls/strauss/guzzlehttp/guzzle/src/BodySummarizer.php on line 31
I'm getting ready to use this on the new Validation library, and realized we have a need that I think Strauss would be a great fit for.
In the library we're using the WordPress __() translation function. The trick with that function is that the domain can't be a variable, it has to be a plain string for generating the POT file. That would mean the domain would need to be changed from some placeholder to whatever it ought to be during the build process.
The crew at Modern Tribe recently threw a script together that accomplishes this task. But it feels like something that would be a great fit for Strauss to handle.
Thanks!
Attempting to solve depedency conflicts with https://github.com/wp-graphql/wp-graphql
composer.json
"require": {
"wp-graphql/wp-graphql": "^1.12"
},
"extra": {
"strauss": {
"target_directory": "Dependencies",
"namespace_prefix": "MyProject\\Dependencies\\",
"classmap_prefix": "Prefix_",
"constant_prefix": "Prefix_",
"packages": [
"wp-graphql/wp-graphql"
],
"delete_vendor_packages": true,
"delete_vendor_files": true
}
},
results in a whole lot of weirdness with the WPGraphQL package ultimately not working.
The main WPGraphQL class is not namespaced, the top of the file DOES have the following commented out line added:
// Global. - namespace MyProject\Dependencies\WPGraphQL;but the actual class declaration remains untouched:
final class WPGraphQL {whereas the expectation was for the class to either be namespaced via the namespace keyword or via the classmap_prefix prefix specified in composer.json.
Either way, autoloading does not work and results in errors like:
[26-Apr-2023 23:08:28 UTC]
PHP Fatal error: Uncaught Error:
Class "MyProject\Dependencies\WPGraphQL" not found
in /path/to/project/Dependencies/wp-graphql/wp-graphql/src/Registry/Utils/PostObject.php:181which appears to be expecting the WPGraphQL to be namespaced like MyProject\Dependencies\WPGraphQL
PLease note that the autoload file generated by straus is loaded during the very first few steps of my bootstrapping process:
require_once __DIR__ . '/Dependencies/autoload.php';Anyways, not sure whether my problems are caused by a bug, or due to misconfiguration, or if there is something else i failed to do.
Thank you in advance for all of your time, effort, and talent you have commited to this project. I'll gladly help write unit tests if any fix is needed to get this working.
Hi @BrianHenryIE. I'm having issues with Strauss adding the namespace prefix twice on my files. There doesn't appear to be any way to fix it. I've removed the custom prefix and let it use the default, and that default doesn't get added twice.
See: https://www.loom.com/share/dd52ab3e791840bdbf25bd45fcdeecf3
Here's a couple samples from the autoloader.
'Dragon\Dependencies\Dragon\Dependencies\Ramsey\Uuid\FeatureSet' => $strauss . '/ramsey/uuid/src/FeatureSet.php',
'Dragon\Dependencies\Dragon\Dependencies\Doctrine\Common\EventArgs' => $strauss . '/doctrine/event-manager/lib/Doctrine/Common/EventArgs.php',Here's the relevant chunk from my composer.json file.
"extra": {
"strauss": {
"target_directory": "strauss",
"namespace_prefix": "Dragon\\Dependencies\\",
"classmap_prefix": "DRAGON_",
"constant_prefix": "DRAGON_CONST_",
"packages": [
"guzzlehttp/guzzle",
"dragon-public/framework"
],
"override_autoload": {
},
"exclude_from_copy": {
"packages": [
],
"namespaces": [
],
"file_patterns": [
]
},
"exclude_from_prefix": {
"packages": [
],
"namespaces": [
],
"file_patterns": [
"/^psr.*$/"
]
},
"namespace_replacement_patterns" : {
},
"delete_vendor_files": false
}
}Update:
It seems to only happen with nested namespace prefixes. Here's a sample after changing it to a single namespace. I'll use that, but it might be worth documenting/fixing.
'DragonDependencies\Ramsey\Uuid\FeatureSet' => $strauss . '/ramsey/uuid/src/FeatureSet.php',
'DragonDependencies\Doctrine\Common\EventArgs' => $strauss . '/doctrine/event-manager/lib/Doctrine/Common/EventArgs.php',Inherited from Mozart, if the package uses a non-standard vendor directory this won't work because 'vendor' is hardcoded in several places. See coenjacobs#132
Currently working on a fix, will submit a PR.
Default config.
"exclude_from_prefix": { "packages": [ ], "namespaces": [ ], "file_patterns": [ "/^psr.*$/" ] },
Setting a package to exclude make the file patterns be forgotten.
Hi, Brian.
I migrated my composer file from
"repositories": [
{
"url": "https://github.com/brianhenryie/mozart",
"type": "git"
},
],
"require-dev":{
"coenjacobs/mozart": "dev-rewrite",
"squizlabs/php_codesniffer": "*",
"wp-coding-standards/wpcs": "*"
},
to
With config:
"strauss": {
"dep_namespace": "MZoo\\MBO_Sandbox\\Dependencies\\",
"dep_directory": "/src/Mozart/",
"packages": [
"htmlburger/carbon-fields",
"ericmann/wp-session-manager",
"ericmann/sessionz"
],
"delete_vendor_files": false,
"override_autoload": {
"htmlburger/carbon-fields": {
"psr-4": {
"Carbon_Fields\\": "core/"
},
"files": [
"config.php",
"templates",
"assets",
"build"
]
}
}
}
}
I'm not quite sure how you are achieving the namespace prefixes, but it seems ones within Carbon Fields are breaking.
It breaks:
Uncaught Error: Class 'Carbon_Fields\Provider\Container_Condition_Provider' not found in path/to/src/Mozart/htmlburger/carbon-fields/core/Carbon_Fields.php:392
Then when I update that namespace to:
\MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Provider\Container_Condition_Provider
It breaks at:
Uncaught Error: Class 'Carbon_Fields\Carbon_Fields' not found in path/to/src/Mozart/htmlburger/carbon-fields/core/Loader/Loader.php:6
I have of course removed vendor and re-installed with re-running vendor/bin/strauss (Cool name, BTW.)
Am I missing a configuration or a step?
Diffs I see in the rendered files are, significantly:
# src/Mozart/htmlburger/carbon-fields/core/Carbon_Fields.php
And
$ioc->register( new \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Provider\Container_Condition_Provider() );
$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );
# src/Mozart/htmlburger/carbon-fields/core/Container.php
* Container proxy factory class.
* Used for shorter namespace access when creating a container.
*
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\Comment_Meta_Container make_comment_meta( string $id, string $name = null )
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\Nav_Menu_Item_Container make_nav_menu_item( string $id, string $name = null )
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\Network_Container make_network( string $id, string $name = null )
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\Post_Meta_Container make_post_meta( string $id, string $name = null )
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\Term_Meta_Container make_term_meta( string $id, string $name = null )
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\Theme_Options_Container make_theme_options( string $id, string $name = null )
- * @method static \MZoo\MBO_Sandbox\Dependencies\Carbon_Fields\Container\User_Meta_Container make_user_meta( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\Comment_Meta_Container make_comment_meta( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\Nav_Menu_Item_Container make_nav_menu_item( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\Network_Container make_network( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\Post_Meta_Container make_post_meta( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\Term_Meta_Container make_term_meta( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\Theme_Options_Container make_theme_options( string $id, string $name = null )
+ * @method static \Carbon_Fields\Container\User_Meta_Container make_user_meta( string $id, string $name = null )
Thank you and if this is a bug, please feel free to let me know how I can help.
Currently any package that declares & uses namespaced functions via the use function ... method, those function's namespaces do not get properly prefixed.
Example package you can try is code-atlantic/chophper on the develop branch:
https://github.com/code-atlantic/chophper/tree/develop.
Currently the main branch has them hard coded for each usage which does get properly prefixed and didn't throw errors.
Basically if you have the following in a package, it won't translate and results in fatals.
namespace Chophper; // This works fine.
use function Chophper\some_func; // This doesn't get prefixed properly.
some_func(); // Resulting in errors.
\Chophper\some_func(); // This works fine.phive install strauss would be nice :)
Needs a published GPG key and a signature per release.
Hi, I'm coming from coenjacobs#124 and wanted to give Strauss a go to fix my issues with mPDF & Mozart.
But Strauss is not replacing the old namespace with the new one when a static method is called. The are some cases in my codebase where also static methods get the updated namespace, but most of the times they do not.
I've created a simplified gist where I also attached the resulting file in which the non-static method gets the updated namespace and the static method does not. Is there something wrong with my composer.json or is Strauss not working correctly here? Thanks in advance!
Hi,
when I add "include_modified_date": false to the extra/strauss object it doesn't take any effect, the date is still added to the header.
I also want to get rid of the author in the header, because I don't want git to include all files again after running strauss when actually the only change is the author in the header who ran strauss.
So a second option like "include_author": false would be much appreciated.
If both options are set to false the text would then be "Modified using Strauss.", which would not change if multiple authors run strauss on multiple dates.
Thank you!
After adding strauss, it seems that I need to use both the composer autoload and the strauss autoload.
include_once('strauss/autoload.php');
include_once('vendor/autoload.php');
The staruss process doesn't generate autoload data for my own classes / namespace.
"autoload": {
"psr-4": {
"MyPlugin\\": "src/php"
}
},
Is there a setting I need add so that I only need to use one autoloader? Or am I missing something ?
Thanks
As you wished @BrianHenryIE, here's another one:
In some cases, the namespace prefix is added two or three times in strings. I couldn't find a simple code example where the prefix is added three times, but here's one where it's added two times.
Hint: If the content of the file StraussTestPackage.php is removed or the namespace in StraussTestPackage2.php is changed from ST\Namespace to ST everything works as expected.
Thanks again and keep up the good work!
StraussTestPackage.php:
<?php
namespace ST;
class StraussTestPackage {
public function __construct() {
}
}
StraussTestPackage2.php:
<?php
namespace ST\Namespace;
class StraussTestPackage2 {
public function __construct() {
$one = '\ST\Namespace';
$two = '\ST\Namespace\StraussTestPackage2';
}
}
Result of StraussTestPackage2.php:
<?php
/**
* @license proprietary
*
* Modified by __root__ on 02-July-2021 using Strauss.
* @see https://github.com/BrianHenryIE/strauss
*/
namespace StraussTest\ST\Namespace;
class StraussTestPackage2 {
public function __construct() {
$one = '\StraussTest\StraussTest\ST\Namespace';
$two = '\StraussTest\StraussTest\ST\Namespace\StraussTestPackage2';
}
}
This seems to be a regression, which I believe was fixed in Mozart by either coenjacobs#91 or coenjacobs#103
on Windows, the directory separator is not cleaned for the files to be copied in the enumerator here:
strauss/src/FileEnumerator.php
Line 119 in 836d196
adding this below that line fixes the issue:
strauss/src/FileEnumerator.php
Line 148 in 836d196
to replicate, simply use:
"require": {
"iio/libmergepdf": "^4.0"
},
Hello!
Could you clean up these +0 branches?
https://github.com/BrianHenryIE/strauss/branches
I'm planning to send at least 1 PR :)
When using strauss via the phar file I get the following error:
PHP Warning: include(phar://strauss.phar/vendor/autoload.php): Failed to open stream: phar error: invalid url or non-existent phar "phar://strauss.phar/vendor/autoload.php" in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss on line 23
Warning: include(phar://strauss.phar/vendor/autoload.php): Failed to open stream: phar error: invalid url or non-existent phar "phar://strauss.phar/vendor/autoload.php" in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss on line 23
PHP Warning: include(): Failed opening 'phar://strauss.phar/vendor/autoload.php' for inclusion (include_path='.:/usr/local/Cellar/php/8.1.2/share/php/pear') in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss on line 23
Warning: include(): Failed opening 'phar://strauss.phar/vendor/autoload.php' for inclusion (include_path='.:/usr/local/Cellar/php/8.1.2/share/php/pear') in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss on line 23
PHP Fatal error: Uncaught Error: Class "BrianHenryIE\Strauss\Console\Application" not found in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss:26
Stack trace:
#0 phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss(28): {closure}('0.8.1')
#1 /Users/kodiegrantham/www/strauss.phar(12): require('phar:///Users/k...')
#2 {main}
thrown in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss on line 26
Fatal error: Uncaught Error: Class "BrianHenryIE\Strauss\Console\Application" not found in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss:26
Stack trace:
#0 phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss(28): {closure}('0.8.1')
#1 /Users/kodiegrantham/www/strauss.phar(12): require('phar:///Users/k...')
#2 {main}
thrown in phar:///Users/kodiegrantham/www/strauss.phar/bin/strauss on line 26
Thanks for the tool, like it so far!
I wonder if it's ok to set the output directory to 'vendor' to not have 'vendor_prefix' or simliar.
Currently I run rm -rf vendor;mv vendor_prefix vendor after the strauss command to only have one vendor folder which is also in my .gitignore.
Also I like to keep brianhenryie/strauss dependency in the 'require-dev' but that would bot work if running composer update --no-dev for the production build. What's the expected approach here?
Thanks!
When running the strauss.phar command after composer install all my dependencies get prefixed correctly except nesbot/carbon.
In short:
All classes in the vendor/nesbot/carbon/src directory get prefixed correctly. But the classes in the root vendor/nesbot/carbon directory doesnt and the classes inside vendor/nesbot/carbon/lazy does not get prefixed either.
My composer.json:
{
"name": "custom/plugin",
"license": "GPL-2.0-or-later",
"description": "WordPress plugin developed by Tussendoor.",
"type": "wordpress-plugin",
"require": {
"php": "^8.0",
"nesbot/carbon": "^2.66.0",
"illuminate/database": "^8.0.0",
"twbs/bootstrap": "^5.2.3",
"fortawesome/font-awesome": "^6.4.0",
"adbario/php-dot-notation": "^3.3.0",
"woocommerce/action-scheduler": "^3.4.0",
"yahnis-elsts/plugin-update-checker": "^4.13"
},
"autoload": {
"psr-4": {
"Custom\\Namespace\\": "app/"
},
"files": [
"vendor/woocommerce/action-scheduler/action-scheduler.php"
],
"classmap": [
"vendor"
]
},
"config": {
"optimize-autoloader": true,
"sort-packages": true,
"platform": {
"php": "8.0.3"
}
},
"scripts": {
"post-install-cmd": [
"sh prefixer.sh"
],
"post-update-cmd": [
"sh prefixer.sh"
]
},
"extra": {
"strauss": {
"target_directory": "vendor",
"namespace_prefix": "Custom\\Namespace\\Vendor\\",
"constant_prefix": "TSD_CUSTOM_VENDOR_",
"include_modified_date": false,
"include_author": false,
"delete_vendor_packages": true,
"exclude_from_prefix": {
"packages": [
"yahnis-elsts/plugin-update-checker",
"woocommerce/action-scheduler"
]
}
}
}
}As you can see I am running a custom shell script after the regular composer commands like install and update. The shell script is looking like this:
#!/bin/bash
printf "\n"
printf "Composer action is completed! Wait for the new actions. \n"
printf "\n"
printf "Starting Strauss to prefix all dependencies \n"
php strauss.phar
printf "\n"
printf "Strauss action is completed! \n"
printf "Dumping the composer autoload to prevent errors. \n"
composer dump-autoload
printf "\n"
printf "Autoload has been dumped. \n"
printf "Cleaning the empty folders in the vendor directory. \n"
find vendor -type d -empty -delete
printf "\n"
printf "All done! \n"So everything works fine after running the script. The plugin gets loaded correctly and thus I thought everything worked fine :) Except I encountered the following error on a specifc page where I use the Carbon class. This fatal error is showing:
Fatal error:
Uncaught Error: Class "Carbon\AbstractTranslator" not found in /vendor/nesbot/carbon/lazy/Carbon/TranslatorStrongType.php:17
Upon investigation I found that not all classes from vendor/nesbot/carbon are getting prefixed. All classes in the vendor/nesbot/carbon/src directory get prefixed correctly. But the classes in the root vendor/nesbot/carbon directory doesnt and the classes inside vendor/nesbot/carbon/lazy does not get prefixed either. The error is originated from the lazy directory.
Its not a very big directory so I tried prefixing them manually and running composer dump-autoload afterwards. This fixes the issue.
I am wondering if this is a bug or due to misconfiguration on my end.
Thanks again for this project!
Hello, there!
As part of the university research we are currently doing regarding the security of Github Actions, we noticed that one or many of the workflows that are part of this repository are referencing vulnerable versions of the third-party actions. As part of a disclosure process, we decided to open issues to notify GitHub Community.
Please note that there are could be some false positives in our methodology, thus not all of the open issues could be valid. If that is the case, please let us know, so that we can improve on our approach. You can contact me directly using an email: ikoishy [at] ncsu.edu
Thanks in advance
The vulnerability fix that is missing by actions' versions could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider updating the reference to the action.
If you end up updating the reference, please let us know. We need the stats for the paper :-)
As of v0.14.1, PHP files with an inline namespace ( i.e. <?php namespace MyNamespace; instead of <?php \nnamespace MyNamespace;) are no longer parsed and instead the class is renamed to a global prefix.
E.g.: Source file.
(Left: 0.14.0 | Right: v0.14.1 )
The namespace being updated is TrustedLogin, a word which is also used in strings. After running Strauss, the strings get modified:
esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )esc_html__( 'Learn about GravityView\TrustedLogin', 'trustedlogin' )
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
Django
The Web framework for perfectionists with deadlines.
Laravel
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
Microsoft
Open source projects and samples from Microsoft.
Google
Google ❤️ Open Source for everyone.
Alibaba
Alibaba Open Source for everyone
D3
Data-Driven Documents codes.
Tencent
China tencent open source team.