Make ignore_data_tracks yes by default

This commit is contained in:
Filipe Fortes 2018-09-04 10:04:17 -04:00
parent ce03e5a2e4
commit 1701aefe5c
3 changed files with 6 additions and 7 deletions

View file

@ -129,7 +129,7 @@ match:
ignored: []
required: []
ignored_media: []
ignore_data_tracks: no
ignore_data_tracks: yes
ignore_video_tracks: yes
track_length_grace: 10
track_length_max: 30

View file

@ -815,10 +815,9 @@ ignore_data_tracks
~~~~~~~~~~~~~~~~~~~
By default, audio files contained in data tracks within a release are included
in the album's tracklist. If you do not want them to be included, set it to
``yes``.
in the album's tracklist. If you want them to be included, set it ``no``.
Default: ``no``.
Default: ``yes``.
.. _ignore_video_tracks:

View file

@ -371,7 +371,8 @@ class MBAlbumInfoTest(_common.TestCase):
self.assertEqual(d.tracks[0].title, 'TITLE ONE')
self.assertEqual(d.tracks[1].title, 'TITLE TWO')
def test_no_skip_audio_data_tracks(self):
def test_no_skip_audio_data_tracks_if_configured(self):
config['match']['ignore_data_tracks'] = False
tracks = [self._make_track('TITLE ONE', 'ID ONE', 100.0 * 1000.0),
self._make_track('TITLE TWO', 'ID TWO', 200.0 * 1000.0)]
data_tracks = [self._make_track('TITLE AUDIO DATA', 'ID DATA TRACK',
@ -383,8 +384,7 @@ class MBAlbumInfoTest(_common.TestCase):
self.assertEqual(d.tracks[1].title, 'TITLE TWO')
self.assertEqual(d.tracks[2].title, 'TITLE AUDIO DATA')
def test_skip_audio_data_tracks_if_configured(self):
config['match']['ignore_data_tracks'] = True
def test_skip_audio_data_tracks(self):
tracks = [self._make_track('TITLE ONE', 'ID ONE', 100.0 * 1000.0),
self._make_track('TITLE TWO', 'ID TWO', 200.0 * 1000.0)]
data_tracks = [self._make_track('TITLE AUDIO DATA', 'ID DATA TRACK',