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

PHP Code Sniffer & PHP Copy/Paste Detector Error - SQLSTATE[HY000]: General error 7 #66

Closed
BrunoJunior opened this issue May 17, 2017 · 6 comments

Comments

@BrunoJunior
Copy link

BrunoJunior commented May 17, 2017

Error messages :
image
.php-censor.yml :

build_settings:
  clone_depth: 1 # depth of 1 is a shallow clone, remove this line to clone entire repo
  ignore:
    - "vendor"
    - "tests"

setup:
  composer:
    action: "install"

test:
  php_mess_detector:
    allow_failures: true
  php_code_sniffer:
    standard: "PSR2"
  php_cpd:
    allow_failures: true
@corpsee corpsee self-assigned this May 17, 2017
@corpsee corpsee added the bug label May 17, 2017
@corpsee
Copy link
Member

corpsee commented May 17, 2017

Thanks for the report.

@corpsee
Copy link
Member

corpsee commented May 21, 2017

@BrunoJunior Now is this bug reproducing on master branch of PHP Censor? What database do you use (PostgreSQL/MySQL)?

@corpsee corpsee assigned BrunoJunior and unassigned corpsee May 29, 2017
@BrunoJunior
Copy link
Author

I use PostgreSQL.
I reproduced error on master branch :

RUNNING PLUGIN: PHP Code Sniffer (Stage: Test)
Plugin options: {"path":"fwk"}
Looking for binary: phpcs
Found in /vendor/bin: phpcs
Args: ["/vagrant/php-censor/runtime/builds/6_7b158/vendor/bin/phpcs --report=json %s %s %s %s %s "%s" %s %s %s"," --standard=PSR2"," --extensions=php"," --ignore=vendor","","","/vagrant/php-censor/runtime/builds/6_7b158/","","",""]
Command: /vendor/bin/phpcs --report=json --standard=PSR2 --extensions=php --ignore=vendor "/"
Execution status: 2
Exception: SQLSTATE[HY000]: General error: 7 number of parameters must be between 0 and 65535
PLUGIN: FAILED

@corpsee
Copy link
Member

corpsee commented May 29, 2017

@BrunoJunior Can you check again on bugfix-issue-66 branch please?

@BrunoJunior
Copy link
Author

Hi ! I'm sorry but I'll have less time to test fixes because of my work.
I explain :
I had to test different self hosted CI for our project (wich is quite big : 250K lines of PHP code) and PHP-censor was one of them.
I also test Jenkins and for our project it's faster and more stable.
My boss told me to not test PHP-Censor anymore.
I think you understand that my company don't want to pay me to pass too much time to test an unused project ^^
I'll tri to make some tests on my private time but I can't promise you anything !

@corpsee
Copy link
Member

corpsee commented May 30, 2017

Ok, @BrunoJunior. Anyway thank you for honest explanation and your reports!
I think I fixed this issue and I close it for now with merging code into master branch. Feel free to reopen this issue if you will test it again and had fail.

@corpsee corpsee closed this as completed May 30, 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