Skip to content

Phpstan Level6 for View.php#5269

Merged
Alkarex merged 14 commits intoFreshRSS:edgefrom
ColonelMoutarde:feature/phpstan-view-file
Apr 7, 2023
Merged

Phpstan Level6 for View.php#5269
Alkarex merged 14 commits intoFreshRSS:edgefrom
ColonelMoutarde:feature/phpstan-view-file

Conversation

@ColonelMoutarde
Copy link
Contributor

Changes proposed in this pull request:

  • Phpstan Level6 for View.php

Pull request checklist:

  • clear commit messages
  • code manually tested

@Alkarex Alkarex added the System care Everything related to system care label Apr 6, 2023
@Alkarex Alkarex added this to the 1.22.0 milestone Apr 6, 2023
ColonelMoutarde and others added 3 commits April 6, 2023 13:59
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
@Alkarex
Copy link
Member

Alkarex commented Apr 6, 2023

I have made a few more improvements / fixes c0ec012
Please double-check.
If you feel like it, it would be nice to address ./app/Models/View.php, which is the natural next step after ./lib/Minz/View.php, either in this PR or a new one.

@ColonelMoutarde
Copy link
Contributor Author

I have made a few more improvements / fixes c0ec012 Please double-check. If you feel like it, it would be nice to address ./app/Models/View.php, which is the natural next step after ./lib/Minz/View.php, either in this PR or a new one.

I add ./app/Models/View.php quickly

@ColonelMoutarde
Copy link
Contributor Author

I have made a few more improvements / fixes c0ec012 Please double-check. If you feel like it, it would be nice to address ./app/Models/View.php, which is the natural next step after ./lib/Minz/View.php, either in this PR or a new one.

done

@ColonelMoutarde ColonelMoutarde marked this pull request as draft April 6, 2023 16:12
@ColonelMoutarde ColonelMoutarde marked this pull request as ready for review April 6, 2023 17:16
@Alkarex
Copy link
Member

Alkarex commented Apr 6, 2023

To get some of types right, some more types had to be added to related files.
I have made a number of fixes and improvements. Please double-check if possible

@ColonelMoutarde
Copy link
Contributor Author

To get some of types right, some more types had to be added to related files. I have made a number of fixes and improvements. Please double-check if possible

I think it's Ok

@Alkarex Alkarex merged commit d23d10b into FreshRSS:edge Apr 7, 2023
@ColonelMoutarde ColonelMoutarde deleted the feature/phpstan-view-file branch May 3, 2023 08:39
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request May 15, 2023
@Alkarex Alkarex mentioned this pull request May 15, 2023

<?php if (!empty($items)) { ?>
<?php $this->logsPaginator->render('logs_pagination.phtml', 'page'); ?>
<?php $this->logsPaginator->render('logs_pagination.phtml', 0); ?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was wrong, breaking the pagination. Fixed in #5403

Alkarex added a commit that referenced this pull request May 15, 2023
* Fix logs pagination
Regression from #5269

* Add better default
math-GH pushed a commit to math-GH/FreshRSS that referenced this pull request Jul 4, 2023
* Fix logs pagination
Regression from FreshRSS#5269

* Add better default
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Jan 18, 2024
* @param array<string,mixed> $listDAO
* @return array<int,FreshRSS_Category>
*/
private static function daoToCategoryPrepopulated(array $listDAO) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Native array return type forgotten: fixed in #6046

Alkarex added a commit that referenced this pull request Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

System care Everything related to system care

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants