diff options
| author | ArthurHoaro <arthur@hoa.ro> | 2020-10-02 17:50:59 +0200 |
|---|---|---|
| committer | ArthurHoaro <arthur@hoa.ro> | 2020-10-13 13:50:11 +0200 |
| commit | efb7d21b52eb033530e80e5e49d175e6e3b031f4 (patch) | |
| tree | 4f34052788a08be1a30cb88c3339ae14e0b7c4da /application/feed | |
| parent | 29c31b7ec6ca48ba37b7eb6da650931fd0cb7164 (diff) | |
| download | Shaarli-efb7d21b52eb033530e80e5e49d175e6e3b031f4.tar.gz Shaarli-efb7d21b52eb033530e80e5e49d175e6e3b031f4.tar.zst Shaarli-efb7d21b52eb033530e80e5e49d175e6e3b031f4.zip | |
Add strict types for bookmarks management
Parameters typing and using strict types overall increase the codebase
quality by enforcing the a given parameter will have the expected type.
It also removes the need to unnecessary unit tests checking methods
behavior with invalid input.
Diffstat (limited to 'application/feed')
| -rw-r--r-- | application/feed/FeedBuilder.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/application/feed/FeedBuilder.php b/application/feed/FeedBuilder.php index f6def630..f70fce4f 100644 --- a/application/feed/FeedBuilder.php +++ b/application/feed/FeedBuilder.php | |||
| @@ -102,7 +102,7 @@ class FeedBuilder | |||
| 102 | } | 102 | } |
| 103 | 103 | ||
| 104 | // Optionally filter the results: | 104 | // Optionally filter the results: |
| 105 | $linksToDisplay = $this->linkDB->search($userInput, null, false, false, true); | 105 | $linksToDisplay = $this->linkDB->search($userInput ?? [], null, false, false, true); |
| 106 | 106 | ||
| 107 | $nblinksToDisplay = $this->getNbLinks(count($linksToDisplay), $userInput); | 107 | $nblinksToDisplay = $this->getNbLinks(count($linksToDisplay), $userInput); |
| 108 | 108 | ||