danvics/ytdl-sub
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[BUGFIX] Fix entry missing __contains__ function (#392)

Browse source 
* [BUGFIX] Fix missing contains function when checking entries

* remove children func

* Update failing hash
This commit is contained in:
Jesse Bannon 2022-12-13 00:23:16 -08:00 committed by GitHub
parent e54bf7d467
commit beaa34a367
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/ytdl_sub/entries/entry_parent.py
View file

@ -63,11 +63,6 @@ class EntryParent(BaseEntry):
"""This parent's children that are entries"""
return self._entry_children
@property
def children(self) -> List[TBaseEntry]:
"""Children, both entries and parent entries, cast as BaseEntry"""
return self._parent_children + self._entry_children
def num_children(self) -> int:
"""
Returns
@ -210,7 +205,7 @@ class EntryParent(BaseEntry):
if not playlist_id:
return False
return self.uid == playlist_id or any(child.__contains__(item) for child in self.children)
return self.uid == playlist_id or any(item in child for child in self.parent_children())
@classmethod
def _get_disconnected_root_parent(
@ -224,7 +219,9 @@ class EntryParent(BaseEntry):
return webpage_url in url or url in webpage_url
top_level_parents = [
parent for parent in parents if not parent.children and _url_matches(parent.webpage_url)
parent
for parent in parents
if parent.num_children() == 0 and _url_matches(parent.webpage_url)
]
match len(top_level_parents):

4
tests/resources/expected_downloads_summaries/plugins/chapters/test_chapters_from_comments.json
View file

@ -1,7 +1,7 @@
{
".ytdl-sub-chapters_from_comments-download-archive.json": "122723ce8d257eebb05178daa26141f6",
"JMC/JMC - Move 78 - Automated Improvisation [Full Album]-thumb.jpg": "c12e6a6f242680d1096a1a99d74a62c6",
"JMC/JMC - Move 78 - Automated Improvisation [Full Album].info.json": "f6045c28cc588187167a3f42e6f37f0c",
"JMC/JMC - Move 78 - Automated Improvisation [Full Album].mp4": "8b8a9a731a19bc37feb091559f97ebbc",
"JMC/JMC - Move 78 - Automated Improvisation [Full Album].info.json": "228adc1dbc785852c8266457af0adadc",
"JMC/JMC - Move 78 - Automated Improvisation [Full Album].mp4": "68c1c4b20b89511b26a92b967e54891e",
"JMC/JMC - Move 78 - Automated Improvisation [Full Album].nfo": "7a65b184d24c68fc0ec5380432250f5b"
}