Skip to content

Commit

Permalink
Merge pull request #455 from jrfnl/feature/update-phpstan
Browse files Browse the repository at this point in the history
PHPStan: update to latest version
  • Loading branch information
JamesHeinrich authored Sep 11, 2024
2 parents 421324d + 0f8f08e commit 78dc28a
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,8 @@ jobs:
- uses: "shivammathur/setup-php@v2"
with:
php-version: "8.2"
tools: "phpstan:1.10.57"
tools: "phpstan:1.12.3"
coverage: "none"
- uses: "ramsey/composer-install@v3"
- name: "Run PHPStan"
run: "phpstan analyse -c phpstan.neon -l 4 getid3"
run: "phpstan"
2 changes: 1 addition & 1 deletion getid3/module.audio-video.quicktime.php
Original file line number Diff line number Diff line change
Expand Up @@ -1650,7 +1650,7 @@ public function QuicktimeParseAtom($atomname, $atomsize, $atom_data, $baseoffset
@list($all, $latitude, $longitude, $altitude) = $matches;
$info['quicktime']['comments']['gps_latitude'][] = floatval($latitude);
$info['quicktime']['comments']['gps_longitude'][] = floatval($longitude);
if (!empty($altitude)) {
if (!empty($altitude)) { // @phpstan-ignore-line
$info['quicktime']['comments']['gps_altitude'][] = floatval($altitude);
}
} else {
Expand Down
2 changes: 1 addition & 1 deletion getid3/module.audio-video.riff.php
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ public function Analyze() {
$info['avdataend'] = $info['filesize'];
}

$nextRIFFoffset = $Original['avdataoffset'] + 8 + $thisfile_riff['header_size']; // 8 = "RIFF" + 32-bit offset
$nextRIFFoffset = (int) $Original['avdataoffset'] + 8 + (int) $thisfile_riff['header_size']; // 8 = "RIFF" + 32-bit offset
while ($nextRIFFoffset < min($info['filesize'], $info['avdataend'])) {
try {
$this->fseek($nextRIFFoffset);
Expand Down
3 changes: 3 additions & 0 deletions phpstan.neon
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
parameters:
level: 4
paths:
- getid3
excludes_analyse:
polluteScopeWithLoopInitialAssignments: true
dynamicConstantNames:
Expand Down

0 comments on commit 78dc28a

Please sign in to comment.