Skip to content

Commit

Permalink
QUAL Ignore some phan annotations
Browse files Browse the repository at this point in the history
  • Loading branch information
mdeweerd committed Dec 6, 2023
1 parent 816fa86 commit bc36e61
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/phan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
tools: cs2pr,phan
- name: Run Phan analysis
run: |
phan --quick -k .phan/config.php -B .phan/baseline.txt --analyze-twice --minimum-target-php-version 7.1 --output-mode=checkstyle -o _phan.xml
phan --quick -k .phan/config.php -B .phan/baseline.txt --analyze-twice --minimum-target-php-version 7.0 --output-mode=checkstyle -o _phan.xml
- name: Add results to PR
if: ${{ always() }}
run: |
Expand Down
1 change: 0 additions & 1 deletion .phan/config.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@
// should be added to the `directory_list` as
// to `exclude_analysis_directory_list`.
"exclude_analysis_directory_list" => [
'.',
'htdocs/includes/',
'htdocs/core/class/lessc.class.php', // External library

Expand Down
1 change: 1 addition & 0 deletions htdocs/adherents/class/adherent.class.php
Original file line number Diff line number Diff line change
Expand Up @@ -1827,6 +1827,7 @@ public function subscriptionComplementaryActions($subscriptionid, $option, $acco
$vattouse = get_default_tva($mysoc, $mysoc, $idprodsubscription);
}
//print xx".$vattouse." - ".$mysoc." - ".$customer;exit;
// @phan-suppress-next-line PhanPluginSuspiciousParamOrder
$result = $invoice->addline($label, 0, 1, $vattouse, 0, 0, $idprodsubscription, 0, $datesubscription, '', 0, 0, '', 'TTC', $amount, 1);

Check warning on line 1831 in htdocs/adherents/class/adherent.class.php

View workflow job for this annotation

GitHub Actions / Run phan

Suspicious order for argument label - This is getting passed to parameter #24 (string $label) of \Facture::addline(string $desc, float $pu_ht, float $qty, float $txtva, float|int $txlocaltax1 = 0, float|int $txlocaltax2 = 0, int $fk_product = 0, float|int $remise_percent = 0, int|string $date_start = '', int|string $date_end = '', int $ventil = 0, int $info_bits = 0, int|string $fk_remise_except = '', string $price_base_type = 'HT', float|int $pu_ttc = 0, int $type = 0, int $rang = unknown, int $special_code = 0, string $origin = '', int $origin_id = 0, int $fk_parent_line = 0, int $fk_fournprice = null, int $pa_ht = 0, string $label = '', array|int $array_options = 0, int $situation_percent = 100, int $fk_prev_id = 0, string $fk_unit = null, float|int $pu_ht_devise = 0, string $ref_ext = '', int $noupdateafterinsertline = 0) defined at htdocs/compta/facture/class/facture.class.php:3777
if ($result <= 0) {
$this->error = $invoice->error;
Expand Down
1 change: 1 addition & 0 deletions htdocs/admin/mails_emailing.php
Original file line number Diff line number Diff line change
Expand Up @@ -450,6 +450,7 @@ function change_smtp_auth_method() {

// SuperAdministrator access only
if (!isModEnabled('multicompany') || ($user->admin && !$user->entity)) {
// @phan-suppress-next-line PhanPluginSuspiciousParamOrder
print $form->selectarray('MAIN_MAIL_SMTPS_OAUTH_SERVICE_EMAILING', $oauthservices, $conf->global->MAIN_MAIL_SMTPS_OAUTH_SERVICE_EMAILING);
} else {
$text = $oauthservices[getDolGlobalString('MAIN_MAIL_SMTPS_OAUTH_SERVICE_EMAILING')];
Expand Down

0 comments on commit bc36e61

Please sign in to comment.