SubtitleExtractor icon indicating copy to clipboard operation
SubtitleExtractor copied to clipboard

Common functions re-factoring

Open ghost opened this issue 8 years ago • 1 comments

createSoupObject is common to bbc, comedycentral, crackle, crunchyroll, fox, hulu, netflix, youtube and can be refactored. Despite it being in common already?

deleteUnnecessaryfiles is common to all, but already appears in common.

downloadXMLTranscript is common to bbc, crackle, and youtube

convertXMLToSrt is common to bbc, crunchyroll, and youtube

downloadDfxpTranscript is common to amazon, comedycentral and netflix

getContentID1 and getContentID2 are common to comedycentral and fox, and hulu, but hulu has a different version.

getShowJson, getShowDetails, getSubtitleUrl, and processShowName are common to comedycentral and fox. comedycentral and fox seem to share a lot of code.

BBC and crackle share the same getTitle code.

ghost avatar Nov 29 '16 04:11 ghost

downloadXMLTranscript for "BBC,Crackle" and Youtube are different. Would like to see how you merge them in the code.

getContentID1 and getContentID2 of Hulu might be completely different. You can ignore that.

Regarding getShowJson, getShowDetails check if the functions are really being used. Otherwise you can simply ignore them.

abhishek-vinjamoori avatar Nov 30 '16 22:11 abhishek-vinjamoori