Hi,
Here is short review. Unfortunatelly, i was not able to review too much
because of traceback.
1. Why is the plugin called "custom_subtitles"? :) I think it should be
"subtitles".
mediagoblin/plugins/custom_subtitles/tools.py:
2.
20 def get_path(path):
21 path = eval(path) # Converting string to a tuple
22 return path
2. Please don't use eval. Find another way. json might be a good choice.
3. Indentation in open_subtitle and save_subtitle is broken. Now when i
click "add subtitles" it doesn't work and i get a traceback.
To prevent errors like this i suggest to write some unit tests. Please
write some basic ones, that would upload subtitles and delete them.
Have a look at how video and audio are tested.