D10791: Fixed the Wikimedia Commons Picture of the Day provider
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 REVISION DETAIL https://phabricator.kde.org/D10791 To: manuelc, #plasma Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D10791: Fixed the Wikimedia Commons Picture of the Day provider
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 https://phabricator.kde.org/D10791 AFFECTED FILES dataengines/potd/wcpotdprovider.cpp To: manuelc, #plasma Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D10791: Fixed the Wikimedia Commons Picture of the Day provider
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, whether this is the right way to handle the two necessary API calls in one slot, or if I should relocate the second one into another slot. REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10791?vs=27908=33813 REVISION DETAIL https://phabricator.kde.org/D10791 AFFECTED FILES dataengines/potd/wcpotdprovider.cpp To: manuelc, #plasma Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D10791: Fixed the Wikimedia Commons Picture of the Day provider
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
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
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, apol, mart
D10791: Fixed the Wikimedia Commons Picture of the Day provider
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 https://phabricator.kde.org/D10791?vs=27904=27908 REVISION DETAIL https://phabricator.kde.org/D10791 AFFECTED FILES dataengines/potd/wcpotdprovider.cpp 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
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
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 decided to rewrite all of it to make it use the Wikimedia API REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D10791 AFFECTED FILES dataengines/potd/PoTD-list.txt dataengines/potd/wcpotdprovider.cpp dataengines/potd/wcpotdprovider.h To: manuelc Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart