D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-05-08 Thread Manuel Caldeira
manuelc abandoned this revision. manuelc added a comment. Okay, no, that clearly didn't work, I think I don't quite know how to resolve that... But, what I also just noticed, is that someone else took on this task anyway, so I'm just gonna abandon this. REPOSITORY R114 Plasma Addons

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-05-08 Thread Manuel Caldeira
manuelc updated this revision to Diff 33814. manuelc added a comment. And I think I totally messed up the diff just now, here I go again... REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10791?vs=33813=33814 REVISION DETAIL

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-05-08 Thread Manuel Caldeira
manuelc updated this revision to Diff 33813. manuelc added a comment. After some time and careful consideration of how the other providers work I rewrote the JSON parsing code. While doing that I noticed that I overlooked an object in the second return value... I don't know though,

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-02-27 Thread Manuel Caldeira
manuelc planned changes to this revision. REVISION DETAIL https://phabricator.kde.org/D10791 To: manuelc, #plasma Cc: ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-02-27 Thread Manuel Caldeira
manuelc reclaimed this revision. REVISION DETAIL https://phabricator.kde.org/D10791 To: manuelc, #plasma Cc: ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-02-24 Thread Manuel Caldeira
manuelc abandoned this revision. manuelc added a comment. I'm sorry, I realised I had to look over my changes once more, more closely REVISION DETAIL https://phabricator.kde.org/D10791 To: manuelc, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas,

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-02-24 Thread Manuel Caldeira
manuelc updated this revision to Diff 27908. manuelc added a comment. I didn't know how important this was and building took a bit long. Also, should these be initialised externally since they never change or is it fine to put them directly into the function? CHANGES SINCE LAST UPDATE

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-02-24 Thread Manuel Caldeira
manuelc added a reviewer: Plasma. REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D10791 To: manuelc, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10791: Fixed the Wikimedia Commons Picture of the Day provider

2018-02-24 Thread Manuel Caldeira
manuelc created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. manuelc requested review of this revision. REVISION SUMMARY Since the previously used method is no longer working because the page doesn't exist anymore I