Skip to content

Fix a lot of psalm issues #2628

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.

Sign up for GitHub

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

Merged
christianlupus merged 15 commits into from
Mar 5, 2025
Merged

Fix a lot of psalm issues #2628

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
be7d3bc
feat(psalm): Enable finding unused baseline entries
provokateurin Jan 12, 2025
11e2250
fix(composer): Use correct OCP version
provokateurin Jan 3, 2025
138fd5c
fix(composer): Add missing ext-dom dependency
provokateurin Jan 3, 2025
03f03c0
fix(psalm): Check against correct PHP version
provokateurin Jan 3, 2025
7425861
fix(RecipeService): Stop unnecessarily checking for methods on File
provokateurin Jan 3, 2025
fcefabb
fix(RecipeService): Catch recipe folder not being a folder
provokateurin Jan 3, 2025
fcce749
fix(ImageService): Add missing exception imports
provokateurin Jan 3, 2025
d15af6a
fix(UserConfigHelper): Explicitly cast value to string
provokateurin Jan 3, 2025
82302a2
fix(RestParameterParser): Handle unsupported content-types
provokateurin Jan 3, 2025
b9b6513
fix(ThumbnailFileHelper): Add missing exception imports
provokateurin Jan 3, 2025
00c839d
fix(HttpMicrodataParser): Use concrete DOMNode sub-types to fix polym…
provokateurin Jan 3, 2025
8441594
fix(SchemaConformityFilter): Fix type coercions for boolean operators
provokateurin Jan 3, 2025
669ad12
fix(FixInstructionsFilter): Prevent accidentally calling array_merge(…
provokateurin Jan 3, 2025
b3af7b1
fix(RecipeDb): Remove unnecessary is_array() check
provokateurin Jan 3, 2025
b544cf5
Update Changelog
christianlupus Mar 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix(RecipeDb): Remove unnecessary is_array() check 
Signed-off-by: provokateurin <[email protected]>
  • Loading branch information
@provokateurin @christianlupus
provokateurin authored and christianlupus committed Mar 5, 2025
commit b3af7b14493020402eb32b922d5d1de17bdbe2a1
2 changes: 1 addition & 1 deletion lib/Db/RecipeDb.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
/**
* @throws \OCP\AppFramework\Db\DoesNotExistException if not found
* @deprecated
* @todo Why deprecated?

Check warning on line 36 in lib/Db/RecipeDb.php

View workflow job for this annotation

GitHub Actions / Check for added todo messages 

Found @todo Why deprecated?
*/
public function findRecipeById(int $id) {
$qb = $this->db->getQueryBuilder();
Expand Down Expand Up @@ -349,7 +349,7 @@
* @throws \OCP\AppFramework\Db\DoesNotExistException if not found
*/
public function findRecipes(array $keywords, string $user_id) {
$has_keywords = $keywords && is_array($keywords) && $keywords[0];
$has_keywords = $keywords && $keywords[0];

if (!$has_keywords) {
return $this->findAllRecipes($user_id);
Expand Down
6 changes: 0 additions & 6 deletions tests/psalm-baseline.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<files psalm-version="5.26.1@d747f6500b38ac4f7dfc5edbcae6e4b637d7add0">
<file src="lib/Db/RecipeDb.php">
<RedundantCondition>
<code><![CDATA[$keywords && is_array($keywords)]]></code>
<code><![CDATA[is_array($keywords)]]></code>
</RedundantCondition>
</file>
<file src="lib/Helper/UserFolderHelper.php">
<MissingDependency>
<code><![CDATA[$this->root]]></code>
Expand Down
Loading