owncloud-archive/news

All but one starred message not displayed after upgrade to owncloud 9/news 8.7

gjanssens opened this issue · 12 comments

gjanssens commented

IMPORTANT

Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)

Explain the Problem

What problem did you encounter?
I have upgraded/migrated my owncloud installation from

  • owncloud 8.2.3
  • centos 6 32-bit
  • php 5.4.x

to

  • owncloud 9.0.1
  • centos 7 64-bit
  • php 5.6 (via the php56 software collection)

This mostly works fine. The only issue I have is that my previously starred articles in news are not displayed. I only get the first (or last) starred article (depending on the sort order I choose). The total number of starred articles is still correct (667). I just can't seem to display them any more. It doesn't matter whether I choose compact or normal view.

The list of unread articles on the other hand seems to load and display fine.

Yet when I enable "show all articles", I again see the same issue: only the first (or last) article is visible.

Steps to Reproduce

Explain what you did to encounter the issue

  1. Upgrade to owncloud 9.0/php 5.6/news 8.7.1
  2. Go to news page
  3. Select "Starred"

=> Only one item is visible (either the first or the last)

System Information

  • News app version: 8.7.1
  • ownCloud version: 9.0.1
  • PHP version: rh-php56-php-fpm-5.6.5-7.el7.x86_64
  • Database and version: Postgresql 9.3
  • Browser and version: Firefox 45.02
  • Distribution and version: Fedora 23 (64-bit)

Contents of owncloud/data/owncloud.log

{"reqId":"VyH7d3GwUo9ZQJC-3qEZLgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:56+02:00","method":"GET","url":"\/index.php\/apps\/news\/","user":"geert"}
{"reqId":"VyH7d3GwUo9ZQJC-3qEZLgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:56+02:00","method":"GET","url":"\/index.php\/apps\/news\/","user":"geert"}
{"reqId":"VyH7d3GwUo9ZQJC-3qEZLgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:56+02:00","method":"GET","url":"\/index.php\/apps\/news\/","user":"geert"}
{"reqId":"VyH7d3GwUo9ZQJC-3qEZLgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:56+02:00","method":"GET","url":"\/index.php\/apps\/news\/","user":"geert"}
{"reqId":"VyH7ejROvYS-9MDr9BzsfAAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:58+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds","user":"geert"}
{"reqId":"VyH7ejROvYS-9MDr9BzsfAAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:58+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds","user":"geert"}
{"reqId":"VyH7ejROvYS-9MDr9BzsfAAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:58+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds","user":"geert"}
{"reqId":"VyH7ejROvYS-9MDr9BzsfAAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:58+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds","user":"geert"}
{"reqId":"VyH7ehUG5O02-XhaWZ1KOQAAAAc","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/settings","user":"geert"}
{"reqId":"VyH7ehUG5O02-XhaWZ1KOQAAAAc","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/settings","user":"geert"}
{"reqId":"VyH7ehUG5O02-XhaWZ1KOQAAAAc","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/settings","user":"geert"}
{"reqId":"VyH7ehUG5O02-XhaWZ1KOQAAAAc","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/settings","user":"geert"}
{"reqId":"VyH7eoKX9jWlw@xSWQin4QAAAA0","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds\/active","user":"geert"}
{"reqId":"VyH7eoKX9jWlw@xSWQin4QAAAA0","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds\/active","user":"geert"}
{"reqId":"VyH7eoKX9jWlw@xSWQin4QAAAA0","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds\/active","user":"geert"}
{"reqId":"VyH7eoKX9jWlw@xSWQin4QAAAA0","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/feeds\/active","user":"geert"}
{"reqId":"VyH7esWqsLaBbtBWr@FaOgAAAAY","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/folders","user":"geert"}
{"reqId":"VyH7esWqsLaBbtBWr@FaOgAAAAY","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/folders","user":"geert"}
{"reqId":"VyH7esWqsLaBbtBWr@FaOgAAAAY","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/folders","user":"geert"}
{"reqId":"VyH7esWqsLaBbtBWr@FaOgAAAAY","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:00:59+02:00","method":"GET","url":"\/index.php\/apps\/news\/folders","user":"geert"}
{"reqId":"VyH7e3GwUo9ZQJC-3qEZMgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7e3GwUo9ZQJC-3qEZMgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7e3GwUo9ZQJC-3qEZMgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7e3GwUo9ZQJC-3qEZMgAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7fDROvYS-9MDr9BzsfQAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7fDROvYS-9MDr9BzsfQAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7fDROvYS-9MDr9BzsfQAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7fDROvYS-9MDr9BzsfQAAAAI","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:00+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=3","user":"geert"}
{"reqId":"VyH7fnGwUo9ZQJC-3qEZMwAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyHost\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:03+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=2","user":"geert"}
{"reqId":"VyH7fnGwUo9ZQJC-3qEZMwAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPort\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:03+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=2","user":"geert"}
{"reqId":"VyH7fnGwUo9ZQJC-3qEZMwAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyUser\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:03+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=2","user":"geert"}
{"reqId":"VyH7fnGwUo9ZQJC-3qEZMwAAAAQ","remoteAddr":"192.168.93.163","app":"news","message":"Configuration value \"proxyPassword\" does not exist. Ignored value.","level":2,"time":"2016-04-28T14:01:03+02:00","method":"GET","url":"\/index.php\/apps\/news\/items?limit=40&oldestFirst=true&search=&showAll=false&type=2","user":"geert"}

Contents of Browser Error Console

Read http://ggnome.com/wiki/Using_The_Browser_Error_Console if you are unsure what to put here


Content Security Policy: Instructie ‘frame-src’ wordt niet meer ondersteund. Gebruik in plaats daarvan instructie ‘child-src’. <onbekend>
Using //@ to indicate sourceMappingURL pragmas is deprecated. Use //# instead jquery.min.js:1:0
Content Security Policy: De instellingen van de pagina blokkeerden het laden van een bron op self (‘script-src https://cloud.kobaltwit.be:22443 'unsafe-eval'’). news
mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create es6-shim.min.js:10:23991
Gebruik van getPreventDefault() wordt niet meer ondersteund. Gebruik in plaats daarvan defaultPrevented. jquery.min.js:5:17117
Laden van gemengde (onveilige) vensterinhoud ‘http://0.gravatar.com/blavatar/a172ab9144c841f3c4d78c9a31657887?s=16’ op een beveiligde pagina[Meer info] angular-animate.min.js:11:139
Laden van gemengde (onveilige) vensterinhoud ‘http://fedoraproject.org/favicon.ico’ op een beveiligde pagina[Meer info] angular-animate.min.js:11:139
Laden van gemengde (onveilige) vensterinhoud ‘http://www.juicebox.net/img/favicons/h/apple-touch-icon.png’ op een beveiligde pagina[Meer info] angular-animate.min.js:11:139
Laden van gemengde (onveilige) vensterinhoud ‘https://www.juicebox.net/img/favicons/h/apple-touch-icon.png’ op een beveiligde pagina[Meer info] angular-animate.min.js:11:139
Laden van gemengde (onveilige) vensterinhoud ‘http://planet.gnome.org/img/gnome-16.png’ op een beveiligde pagina[Meer info] angular-animate.min.js:11:139
Laden van gemengde (onveilige) vensterinhoud ‘http://planetkde.org/favicon.ico’ op een beveiligde pagina[Meer info]
gjanssens commented

Thanks for the pointer. It gives me a slight improvement: I now have 13 messages in the starred list. The other 654 are still not appearing.

gjanssens commented

(The previous comment got posted too soon. Not sure what happened...)

In an earlier version of the news app I could scroll all the way down and the next (undisplayed) batch of messages would automatically get added to the page. This no longer seems to happen.

BernhardPosselt commented

Did you use the Android News app to sync?

gjanssens commented

I have used it and just tried again. That app shows me 60 messages in the starred list. However the webinterface tells me I have 667. Not sure what this means.

BernhardPosselt commented

See nextcloud/news-android#518

Also starred articles with the same fingerprint (same text, url, title and enclosure) are reduced to one entry

gjanssens commented

I'm still on Android news app 0.9.8.2 so I shouldn't be affected by that bug ?

Also I'm seeing the issue on the news web interface (left bar says I have 667, but displays only 13).

BernhardPosselt commented

Hm, can you remove the try catch in this method: https://github.com/owncloud/news/blob/master/db/itemmapper.php#L368

And rerun the sql command that I linked above?

gjanssens commented

I commented out the code block (including the foreach, but not the function declaration), run the UPDATE command from issue 981, followed by an occ upgrade.

I see no visible difference on the webinterface. Still only 13 items displayed when I click on starred.

I'll add that the oc_news_items table has 464 rows for which the fingerprint is NULL. Perhaps that's useful information.

BernhardPosselt commented

If you comment out the full block, then fingerprints wont be generated :)

Just remove the try and opening closing brackets and the complete catch block so exceptions wont be swallowed (no idea why i've added that tbh)

gjanssens commented

Ah, I misunderstood :(

Removing the try/catch construct only it does properly generate fingerprints and all articles appear again.

Thanks a lot !

BernhardPosselt commented

Great :)