Metropolis Development - Printable Version
+- XBMC Community Forum (http://forum.xbmc.org)
+-- Forum: Help and Support (/forumdisplay.php?fid=33)
+--- Forum: Skin Help and Support (/forumdisplay.php?fid=67)
+---- Forum: Metropolis (/forumdisplay.php?fid=162)
+---- Thread: Metropolis Development (/showthread.php?tid=126856)
RE: Metropolis Development - thewarm - 2012-06-25 13:34
(2012-06-21 02:46)jingai Wrote: Also fixed a couple of other issues with the TV Shows Next Aired script -- it wasn't running at startup like I think is desirable, and it was running in views where it didn't need to.
The TV Shows Next Aired script has not been working for over a week! I get an "empty" TV Guide!
RE: Metropolis Development - stoli - 2012-06-25 15:41
I just merged the fix for this - it should show up in tonight's Passion Repo update.
Jingai has been doing great work on getting a lot of these items resolved.
RE: Metropolis Development - jingai - 2012-06-25 18:12
FYI, the commits I made for the TV Show Next Aired script won't fix the current problem where it refuses to update -- that's an issue with either tvrage.com or the script itself. If you check around the other skin forums, a lot of people are complaining that it's not working, so it's not unique to Metropolis.
RE: Metropolis Development - thewarm - 2012-06-25 18:34
(2012-06-25 18:12)jingai Wrote: FYI, the commits I made for the TV Show Next Aired script won't fix the current problem where it refuses to update -- that's an issue with either tvrage.com or the script itself. If you check around the other skin forums, a lot of people are complaining that it's not working, so it's not unique to Metropolis.
RE: Metropolis Development - jingai - 2012-06-25 18:59
Stoli, I rebased the commits in pull request 98. It should merge fine now.
I'm also almost done with RandomItems support. Just need to test it a bit more.
RE: Metropolis Development - thewarm - 2012-06-26 01:29
TV Guide should be fixed soon. See my post at TV Show Next Aired forum
RE: Metropolis Development - jingai - 2012-06-27 02:20
RandomItems and Watchlist support is in. Currently refactoring things so the user can choose which add-on to use on a per-library basis (different script for Movies, TVShows, and Music).
If it all goes well I'll submit a pull request tonight, but it's kind of a big pull so I'll make sure it's thoroughly tested before I do so.
RE: Metropolis Development - jingai - 2012-06-27 19:36
Pull Request #100 adds in support for the RandomItems and Watchlist scripts for the Movies, TV Shows, and Music sections. As well, it adds support for RecentlyAdded albums in the Music section. This takes care of Issue #32.
I have not yet added in the option to select a different script for each of the sections. I'll get that done by the end of this week, but I wanted to go ahead and push up these changes so folks here could test it this week. I'm going away on vacation next week and will have limited internet access, so I'd like to squash any new issues my changes might have created ASAP.
I chose to require a restart of XBMC if the user changes between the three scripts (and added a notice saying as much in the Settings dialog). I decided on this because both the RandomItems and Watchlist scripts can be quite slow on large libraries (especially when using a centralized database via MySQL), so I didn't think it was a good idea to unconditionally run them at startup, as a lot of people run XBMC on resource-limited machines. If it turns out this behavior is undesirable, it's easily changed later, however.
Also, you may close Issue #17 if you're satisfied with my patch for it.
RE: Metropolis Development - stoli - 2012-06-27 20:29
Merged. Great work as always!
@jingai - I added you as a collaborator on Git - feel free to merge at will.
RE: Metropolis Development - jingai - 2012-06-28 16:16
(2012-06-27 20:29)stoli Wrote: Merged. Great work as always!
Ok, thanks. I'll try to see to it that you don't regret that