Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue With Lumberjack WordPress #639

Closed
4 tasks
Androlax2 opened this issue Dec 10, 2021 · 3 comments
Closed
4 tasks

Issue With Lumberjack WordPress #639

Androlax2 opened this issue Dec 10, 2021 · 3 comments

Comments

@Androlax2
Copy link

Androlax2 commented Dec 10, 2021

Before submitting an issue please check that you’ve completed the following steps:

  • Made sure you’re on the latest version
  • Used the search feature to ensure that the bug hasn’t been reported before

Describe the bug
A clear and concise description of what the bug is.

There is a conflict between Lumberjack-core and imagify-plugin

To Reproduce
Steps to reproduce the behavior:

Use roots/bedrock and this composer.json :

{
  "name": "roots/bedrock",
  "type": "project",
  "license": "MIT",
  "description": "WordPress boilerplate with modern development tools, easier configuration, and an improved folder structure",
  "homepage": "https://roots.io/bedrock/",
  "authors": [
    {
      "name": "Scott Walkinshaw",
      "email": "scott.walkinshaw@gmail.com",
      "homepage": "https://github.com/swalkinshaw"
    },
    {
      "name": "Ben Word",
      "email": "ben@benword.com",
      "homepage": "https://github.com/retlehs"
    }
  ],
  "keywords": [
    "bedrock",
    "composer",
    "roots",
    "wordpress",
    "wp",
    "wp-config"
  ],
  "support": {
    "issues": "https://github.com/roots/bedrock/issues",
    "forum": "https://discourse.roots.io/category/bedrock"
  },
  "repositories": [
    {
      "type": "composer",
      "url": "https://wpackagist.org",
      "only": [
        "wpackagist-plugin/*",
        "wpackagist-theme/*"
      ]
    },
    {
      "type": "composer",
      "url": "https://auth-acf-composer-proxy.pivvenit.net/wordpress-muplugin/"
    },
    {
      "type": "git",
      "url": "https://github.com/tabrisrp/wp-background-processing"
    }
  ],
  "require": {
    "php": ">=7.4",
    "composer/installers": "^1.11",
    "vlucas/phpdotenv": "^5.3",
    "oscarotero/env": "^2.1",
    "roots/bedrock-autoloader": "^1.0",
    "roots/wp-config": "^1.0",
    "roots/wp-password-bcrypt": "^1.0",
    "roots/wordpress": "^5.7.2",
    "wp-media/imagify-plugin": "^1.10",
    "rareloop/lumberjack-core": "^5.0"
  },
  "require-dev": {
    "squizlabs/php_codesniffer": "^3.6",
    "roave/security-advisories": "dev-master",
  },
  "config": {
    "optimize-autoloader": true,
    "preferred-install": "dist"
  },
  "minimum-stability": "dev",
  "prefer-stable": true,
  "extra": {
    "installer-paths": {
      "htdocs/app/mu-plugins/{$name}/": [
        "type:wordpress-muplugin",
      ],
      "htdocs/app/plugins/{$name}/": [
        "type:wordpress-plugin"
      ],
      "htdocs/app/themes/{$name}/": [
        "type:wordpress-theme"
      ]
    },
    "wordpress-install-dir": "htdocs/wp"
  },
  "scripts": {
    "post-root-package-install": [
      "php -r \"copy('.env.example', '.env');\""
    ],
    "test": [
      "phpcs"
    ]
  }
}

And now you have this error : Fatal error: Uncaught TypeError: Argument 2 passed to Laminas\ZendFrameworkBridge\Autoloader::createPrependAutoloader() must be an instance of Composer\Autoload\ClassLoader

If you remove wp-media/imagify-plugin and composer update, no error found. So I assume it come from this repo.

Backlog Grooming (for WP Media dev team use only)

  • Reproduce the problem
  • Identify the root cause
  • Scope a solution
  • Estimate the effort
@piotrbak piotrbak changed the title Issue With Lumberjack Wordpress Issue With Lumberjack WordPress Dec 13, 2021
@Androlax2
Copy link
Author

@Androlax2
Copy link
Author

Is this package needed for this one to works properly ?

Asked for a pull request to add a configuration to remove the suffix if the user wants to, but I feel like a suffix should not be added to the internal composer.

Androlax2 added a commit to Androlax2/imagify-plugin that referenced this issue Dec 21, 2021
@Androlax2 Androlax2 mentioned this issue Mar 31, 2022
@Androlax2
Copy link
Author

Androlax2 commented Apr 8, 2022

This issue is resolved thanks to Composer v2 where we can allow-plugins now :

"allow-plugins": {
      "dangoodman/composer-for-wordpress": false
},

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants