Skip to content

Fix signature

Fix signature #621

Triggered via push December 30, 2024 21:23
Status Failure
Total duration 6m 55s
Artifacts 3

ci-on-push.yml

on: push
pre-commit  /  pre-commit
3m 18s
pre-commit / pre-commit
Matrix: phpstan / php-stan
gh-travis  /  gh-travis
gh-travis / gh-travis
Matrix: windows-ci / win-test
Waiting for pending jobs
Fit to window
Zoom out
Zoom in

Annotations

12 errors and 9 warnings
phpstan / php-stan (8.2): htdocs/comm/action/index.php#L2008
Parameter #1 $arraycolor of function colorArrayToHex expects array{int, int, int}, non-empty-list<string> given.
phpstan / php-stan (8.2): htdocs/core/class/commoninvoice.class.php#L537
Variable $num might not be defined.
phpstan / php-stan (8.2): htdocs/core/class/commoninvoice.class.php#L539
Method CommonInvoice::getListOfOpenDirectDebitOrCreditTransfer() should return array<array{amount: float|int, date: int, num: string, ref: string, ref_ext?: string, fk_bank_line?: int, type: string}> but returns list<array{id: mixed, invoiceid: int, date: ''|int, amount: mixed}>.
phpstan / php-stan (8.2): htdocs/core/class/doleditor.class.php#L213
Property DolEditor::$content (string) on left side of ?? is not nullable.
phpstan / php-stan (8.2): htdocs/fourn/facture/card.php#L1642
Comparison operation "<" between '' and 0 is always true.
phpstan / php-stan (8.2): htdocs/fourn/facture/card.php#L1642
Result of || is always true.
phpstan / php-stan (8.2): htdocs/fourn/facture/card.php#L4072
If condition is always true.
phpstan / php-stan (8.2): htdocs/salaries/card.php#L472
Variable $triggermodname in empty() always exists and is not falsy.
phpstan / php-stan (8.2): htdocs/webportal/class/webportalinvoice.class.php#L341
Method WebPortalInvoice::LibStatut() overrides method CommonInvoice::LibStatut() but misses parameter #6 $nbofopendirectdebitorcredittransfer.
phpstan / php-stan (8.2)
Process completed with exit code 1.
phan / Run phan
Process completed with exit code 1.
phan / Run phan
Process completed with exit code 1.
pre-commit / pre-commit
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
pre-commit / pre-commit
Cache save failed.
phpstan / php-stan (8.2)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
phan / Run phan
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
phan / Run phan: htdocs/core/class/commoninvoice.class.php#L537
commoninvoice.class.php: PhanPossiblyNullTypeMismatchProperty: Assigning $num of type ?int to property but \CommonInvoice-&gt;nbofopendirectdebitorcredittransfer is int (null is incompatible)
phan / Run phan: htdocs/core/class/commoninvoice.class.php#L537
commoninvoice.class.php: PhanPossiblyUndeclaredVariable: Variable $num is possibly undeclared
phan / Run phan: htdocs/fourn/facture/card.php#L4070
card.php: PhanDeprecatedClassConstant: Reference to deprecated class constant \FactureFournisseur::TYPE_PROFORMA defined at htdocs/core/class/commoninvoice.class.php:238 (Deprecated because: Remove this. A "proforma invoice" is an order with a look of invoice, not an invoice !)
phan / Run phan: htdocs/webportal/class/webportalinvoice.class.php#L341
webportalinvoice.class.php: PhanParamSignatureMismatch: Declaration of function LibStatut(int $paye, int $status, int $mode = 0, int $alreadypaid = unknown, int $type = unknown) : string should be compatible with function LibStatut(int $paye, int $status, int $mode = 0, int $alreadypaid = unknown, int $type = unknown, int $nbofopendirectdebitorcredittransfer = 0) : string defined in htdocs/core/class/commoninvoice.class.php:920 (Saw fewer optional parameters in the override)
phan / Run phan: htdocs/webportal/class/webportalinvoice.class.php#L341
webportalinvoice.class.php: PhanParamSignatureRealMismatchTooFewParameters: Declaration of function LibStatut($paye, $status, $mode = 0, $alreadypaid = unknown, $type = unknown) should be compatible with function LibStatut($paye, $status, $mode = 0, $alreadypaid = unknown, $type = unknown, $nbofopendirectdebitorcredittransfer = 0) (the method override accepts 5 parameter(s), but the overridden method can accept 6) defined in htdocs/core/class/commoninvoice.class.php:920

Artifacts

Produced during runtime
Name Size
phan-srcrt Expired
858 Bytes
phpstan-srcrt Expired
930 Bytes
precommit-logs
587 Bytes