[subtitles] removed only-sub option (--skip-download achieves the same
functionality)
This commit is contained in:
		@@ -38,7 +38,6 @@
 | 
				
			|||||||
    "writedescription": false, 
 | 
					    "writedescription": false, 
 | 
				
			||||||
    "writeinfojson": true, 
 | 
					    "writeinfojson": true, 
 | 
				
			||||||
    "writesubtitles": false,
 | 
					    "writesubtitles": false,
 | 
				
			||||||
    "onlysubtitles": false,
 | 
					 | 
				
			||||||
    "allsubtitles": false,
 | 
					    "allsubtitles": false,
 | 
				
			||||||
    "listssubtitles": false
 | 
					    "listssubtitles": false
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -46,14 +46,6 @@ class TestDailymotionSubtitles(unittest.TestCase):
 | 
				
			|||||||
        info_dict = IE.extract(TEST_URL)
 | 
					        info_dict = IE.extract(TEST_URL)
 | 
				
			||||||
        sub = info_dict[0]['subtitles']['fr']
 | 
					        sub = info_dict[0]['subtitles']['fr']
 | 
				
			||||||
        self.assertEqual(md5(sub), '594564ec7d588942e384e920e5341792')
 | 
					        self.assertEqual(md5(sub), '594564ec7d588942e384e920e5341792')
 | 
				
			||||||
    def test_onlysubtitles(self):
 | 
					 | 
				
			||||||
        DL = FakeYDL()
 | 
					 | 
				
			||||||
        DL.params['writesubtitles'] = True
 | 
					 | 
				
			||||||
        DL.params['onlysubtitles'] = True
 | 
					 | 
				
			||||||
        IE = DailymotionIE(DL)
 | 
					 | 
				
			||||||
        info_dict = IE.extract(TEST_URL)
 | 
					 | 
				
			||||||
        sub = info_dict[0]['subtitles']['en']
 | 
					 | 
				
			||||||
        self.assertEqual(md5(sub), '976553874490cba125086bbfea3ff76f')
 | 
					 | 
				
			||||||
    def test_allsubtitles(self):
 | 
					    def test_allsubtitles(self):
 | 
				
			||||||
        DL = FakeYDL()
 | 
					        DL = FakeYDL()
 | 
				
			||||||
        DL.params['allsubtitles'] = True
 | 
					        DL.params['allsubtitles'] = True
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -45,14 +45,6 @@ class TestYoutubeSubtitles(unittest.TestCase):
 | 
				
			|||||||
        info_dict = IE.extract('QRS8MkLhQmM')
 | 
					        info_dict = IE.extract('QRS8MkLhQmM')
 | 
				
			||||||
        sub = info_dict[0]['subtitles']['it']
 | 
					        sub = info_dict[0]['subtitles']['it']
 | 
				
			||||||
        self.assertEqual(md5(sub), '164a51f16f260476a05b50fe4c2f161d')
 | 
					        self.assertEqual(md5(sub), '164a51f16f260476a05b50fe4c2f161d')
 | 
				
			||||||
    def test_youtube_onlysubtitles(self):
 | 
					 | 
				
			||||||
        DL = FakeYDL()
 | 
					 | 
				
			||||||
        DL.params['writesubtitles'] = True
 | 
					 | 
				
			||||||
        DL.params['onlysubtitles'] = True
 | 
					 | 
				
			||||||
        IE = YoutubeIE(DL)
 | 
					 | 
				
			||||||
        info_dict = IE.extract('QRS8MkLhQmM')
 | 
					 | 
				
			||||||
        sub = info_dict[0]['subtitles']['en']
 | 
					 | 
				
			||||||
        self.assertEqual(md5(sub), '4cd9278a35ba2305f47354ee13472260')
 | 
					 | 
				
			||||||
    def test_youtube_allsubtitles(self):
 | 
					    def test_youtube_allsubtitles(self):
 | 
				
			||||||
        DL = FakeYDL()
 | 
					        DL = FakeYDL()
 | 
				
			||||||
        DL.params['allsubtitles'] = True
 | 
					        DL.params['allsubtitles'] = True
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -193,9 +193,6 @@ def parseOpts(overrideArguments=None):
 | 
				
			|||||||
    subtitles.add_option('--write-auto-sub', '--write-automatic-sub',
 | 
					    subtitles.add_option('--write-auto-sub', '--write-automatic-sub',
 | 
				
			||||||
            action='store_true', dest='writeautomaticsub',
 | 
					            action='store_true', dest='writeautomaticsub',
 | 
				
			||||||
            help='write automatic subtitle file (youtube only)', default=False)
 | 
					            help='write automatic subtitle file (youtube only)', default=False)
 | 
				
			||||||
    subtitles.add_option('--only-sub',
 | 
					 | 
				
			||||||
            action='store_true', dest='skip_download',
 | 
					 | 
				
			||||||
            help='[deprecated] alias of --skip-download', default=False)
 | 
					 | 
				
			||||||
    subtitles.add_option('--all-subs',
 | 
					    subtitles.add_option('--all-subs',
 | 
				
			||||||
            action='store_true', dest='allsubtitles',
 | 
					            action='store_true', dest='allsubtitles',
 | 
				
			||||||
            help='downloads all the available subtitles of the video', default=False)
 | 
					            help='downloads all the available subtitles of the video', default=False)
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user