Skip to content

Commit

Permalink
Fix psalm issue
Browse files Browse the repository at this point in the history
  • Loading branch information
gfaudriga committed Apr 9, 2024
1 parent 395b5cd commit 32cdb76
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/Controller/MessagesController.php
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ public function getBody(int $id): JSONResponse {

if ($extractionLibrary === 'kitinerary') {
$itineraries = $this->itineraryService->extract($account, $mailbox, $message->getUid());
if ($itineraries) {
if (!empty($itineraries)) {

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-master

RedundantCondition

lib/Controller/MessagesController.php:253:8: RedundantCondition: Operand of type true is always truthy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-master

RedundantCondition

lib/Controller/MessagesController.php:253:9: RedundantCondition: Type Nextcloud\KItinerary\Itinerary for $itineraries is never falsy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-stable26

RedundantCondition

lib/Controller/MessagesController.php:253:8: RedundantCondition: Operand of type true is always truthy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-stable26

RedundantCondition

lib/Controller/MessagesController.php:253:9: RedundantCondition: Type Nextcloud\KItinerary\Itinerary for $itineraries is never falsy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-stable28

RedundantCondition

lib/Controller/MessagesController.php:253:8: RedundantCondition: Operand of type true is always truthy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-stable28

RedundantCondition

lib/Controller/MessagesController.php:253:9: RedundantCondition: Type Nextcloud\KItinerary\Itinerary for $itineraries is never falsy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-stable27

RedundantCondition

lib/Controller/MessagesController.php:253:8: RedundantCondition: Operand of type true is always truthy (see https://psalm.dev/122)

Check failure on line 253 in lib/Controller/MessagesController.php

View workflow job for this annotation

GitHub Actions / static-psalm-analysis dev-stable27

RedundantCondition

lib/Controller/MessagesController.php:253:9: RedundantCondition: Type Nextcloud\KItinerary\Itinerary for $itineraries is never falsy (see https://psalm.dev/122)
$json['schema'] = $itineraries;
}
} else {
Expand Down

0 comments on commit 32cdb76

Please sign in to comment.