Skip to content
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.

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

[BUG] after update: not all unread entries displayed #5975

Closed
ParoTheParrot opened this issue Dec 25, 2023 · 9 comments · Fixed by #5978
Closed

[BUG] after update: not all unread entries displayed #5975

ParoTheParrot opened this issue Dec 25, 2023 · 9 comments · Fixed by #5978
Labels
Bug (unconfirmed) issues thar could not reproduced yeat
Milestone

Comments

@ParoTheParrot
Copy link
Contributor

ParoTheParrot commented Dec 25, 2023

After update to 1.23.0 I see only one unread article instead all!
This behavior is in main view and in the catagories.

The bottom bar "mark as read" is missing (see screenshot).

Clicking on the unread article does not open the article, just marked an read.

To Reproduce
Steps to reproduce the behavior: install 1.23.0 ;-)

Expected behavior
All unread entries should be shown in the list.

Screenshots
See attachment

Environment information:

  • Device: iPhone, iPad
  • OS: newest ios
  • Browser: newest Firefox ios
  • FreshRSS version: 1.23.0
  • Database version: sqlite
  • PHP version: 8.1.26
@ParoTheParrot
Copy link
Contributor Author

ParoTheParrot commented Dec 25, 2023

image

@ParoTheParrot
Copy link
Contributor Author

not all feeds are effected. Some work nornal
image

Very strange behavior!

@squromiv
Copy link

squromiv commented Dec 25, 2023

Have the same problem. See some errors in console of browsers (Firefox and Chrome).

image

image

@TopoDiFogna
Copy link

Have the same problem. See some errors in console of browsers (Firefox and Chrome).

image

image

I have only the error in the second image

@Alkarex
Copy link
Member

Alkarex commented Dec 25, 2023

Could you please check your server-side logs (Web server logs, FreshRSS logs)?

@Alkarex Alkarex added this to the 1.23.1 milestone Dec 25, 2023
@Alkarex Alkarex added the Bug (unconfirmed) issues thar could not reproduced yeat label Dec 25, 2023
@squromiv
Copy link

squromiv commented Dec 26, 2023

@Alkarex
PHP 8.0

[26-Dec-2023] PHP Fatal error: 
Uncaught TypeError: array_slice(): Argument #3 ($length) must be of type ?int, string given in W:\domains\localhost\fresh\app\views\index\normal.phtml:89
Stack trace:
#0 W:\domains\localhost\fresh\app\views\index\normal.phtml(89): array_slice()
#1 W:\domains\localhost\fresh\lib\Minz\View.php(93): include('W:\\domains\\loca...')
#2 W:\domains\localhost\fresh\lib\Minz\View.php(115): Minz_View->includeFile()
#3 W:\domains\localhost\fresh\app\layout\layout.phtml(73): Minz_View->render()
#4 W:\domains\localhost\fresh\lib\Minz\View.php(93): include('W:\\domains\\loca...')
#5 W:\domains\localhost\fresh\lib\Minz\View.php(106): Minz_View->includeFile()
#6 W:\domains\localhost\fresh\lib\Minz\View.php(73): Minz_View->buildLayout()
#7 W:\domains\localhost\fresh\lib\Minz\Dispatcher.php(59): Minz_View->build()
#8 W:\domains\localhost\fresh\lib\Minz\FrontController.php(61): Minz_Dispatcher->run()
#9 W:\domains\localhost\fresh\p\i\index.php(59): Minz_FrontController->run()
#10 {main}
  thrown in W:\domains\localhost\fresh\app\views\index\normal.phtml on line 89

Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Dec 26, 2023
@Alkarex
Copy link
Member

Alkarex commented Dec 26, 2023

Thanks @squromiv
Would you be able to try this patch? #5978

@squromiv
Copy link

squromiv commented Dec 26, 2023

this patch? #5978

Have checked. It looks like the issue is gone now.

Alkarex added a commit that referenced this issue Dec 26, 2023
@Alkarex
Copy link
Member

Alkarex commented Dec 26, 2023

Great! Thanks for the feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug (unconfirmed) issues thar could not reproduced yeat
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants