search

moreginger / xbmc-plugin.video.ted.talks

GNU General Public License v2.0
18 stars 20 forks source link

Verify argument(s) to script #9

Closed moreginger closed 12 years ago

moreginger commented 12 years ago

At the moment we just split up the argument(s) and jump in. This means (for example) that >= 2.3.0 will barf in a nasty way on favorites for previous versions.

The arguments should be verified as suitable for some mode and, if not, a sensible message displayed.

moreginger commented 12 years ago

Fixed in fd3be6bf995b5c41448c3236ad834753f5fe5fc8 I need all videos to go through the plugin again (NewTalks items currently fire up the URI directly) for #10 it seemed prudent to fix this one first.

Leaving open to request pull.

moreginger commented 12 years ago

http://xbmc.git.sourceforge.net/git/gitweb.cgi?p=xbmc/plugins;a=commit;h=f4f7af96f6a0e2b07531027b6c6e13a6ec814823