search

owncloud-archive / news

:newspaper: News app for ownCloud
GNU Affero General Public License v3.0
290 stars 106 forks source link

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

Closed gjanssens closed 8 years ago

gjanssens commented 8 years ago

IMPORTANT

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

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

to

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


{"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]
BernhardPosselt commented 8 years ago

Try https://github.com/owncloud/news/issues/981#issuecomment-213791803

gjanssens commented 8 years ago

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 8 years ago

(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 8 years ago

Did you use the Android News app to sync?

gjanssens commented 8 years ago

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 8 years ago

See https://github.com/owncloud/News-Android-App/issues/518

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

gjanssens commented 8 years ago

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 8 years ago

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 8 years ago

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 8 years ago

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 8 years ago

Ah, I misunderstood :(

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

Thanks a lot !

BernhardPosselt commented 8 years ago

Great :)