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

Exception when run plugin without options #44

Closed
bochkovprivate opened this issue Mar 30, 2017 · 6 comments
Closed

Exception when run plugin without options #44

bochkovprivate opened this issue Mar 30, 2017 · 6 comments

Comments

@bochkovprivate
Copy link
Contributor

I have a project like this:

...
test:
    php_parallel_lint:
    php_unit:
        config:
            - "test/phpunit.xml"

When I run the build it fails with messages like this:

RUNNING PLUGIN: PHP Parallel Lint (Stage: Test)
Exception: Catchable Fatal Error: Argument 3 passed to PHPCensor\Plugin\PhpParallelLint::__construct() must be of the type array, null given in /var/www/php-censor/src/PHPCensor/Plugin/PhpParallelLint.php line 46
PLUGIN: FAILED

This happens for all plugins I ran without params. That is not the desired behaviour because all params for these plugins are optional.

The same time in PHPCI this configurations works.

If I add params than it works like a charm in php-censor.

Looks like the reason is the difference between php-censor and PHPCI here:

@corpsee corpsee self-assigned this Mar 30, 2017
@corpsee corpsee added the bug label Mar 30, 2017
@corpsee
Copy link
Member

corpsee commented Mar 30, 2017

Thx @bochkovprivate! Fixed in master branch. Can you check it?

@bochkovprivate
Copy link
Contributor Author

Not working yet.
The options now occasionally pass from one plugin to anther. Here is what it looks like in the console:

RUNNING PLUGIN: MySQL (Stage: Setup)
Plugin options: ["CREATE DATABASE rfm_db_test_%BUILD%;","CREATE DATABASE rfm_db_test_%BUILD%_%ENVIRONMENT%;"]
PLUGIN: SUCCESS

RUNNING PLUGIN: Shell (Stage: Setup)
Plugin options: ["CREATE DATABASE rfm_db_test_%BUILD%;","CREATE DATABASE rfm_db_test_%BUILD%_%ENVIRONMENT%;"]
Args: ["CREATE DATABASE rfm_db_test_48;"]
Command: CREATE DATABASE rfm_db_test_48;
sh: CREATE: command not found

@corpsee
Copy link
Member

corpsee commented Mar 31, 2017

I will try to fix it...

@corpsee
Copy link
Member

corpsee commented Apr 1, 2017

@bochkovprivate I fixed this problem in branch bugfix-44. Can you check it, please?

@bochkovprivate
Copy link
Contributor Author

Yes, now it works!
Thanks for great work man.

@corpsee
Copy link
Member

corpsee commented Apr 2, 2017

No problem 😄

@corpsee corpsee closed this as completed Apr 2, 2017
@corpsee corpsee added this to the Version 1.0 milestone Dec 23, 2018
@corpsee corpsee removed this from the Version 1.0.* (patch) milestone May 7, 2019
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