search

dmunozv04 / iSponsorBlockTV

SponsorBlock client for all YouTube TV clients.
GNU General Public License v3.0
2.21k stars 92 forks source link

Fix segment UUID concatenation #64

Closed JoshCooley closed 9 months ago

JoshCooley commented 9 months ago

Fixes #66

Before, the whole UUID list would get added to the UUID list instead of just its members, resulting in nested lists, e.g.

segments = [{'start': 53.495, 'end': 61.544, 'UUID': ['3ff36ef516888600471f1a03338b2faa5972f1bc26a2da038983cb549f4e9af96']}, {'start': 598.649, 'end': 321.091, 'UUID': ['f5dac489d7bff2bbbc8b1cde89c8d2485a3979b227542cf34de133a7ab1963fb6', ['97489095d68cc9448b3a8a7e8379cf746b2ac9fdf94e917c9f24b9162f02d5b96', ['9e7cdc02c4495f7480f0792e005d29374e47d1fc99d7e3a57b16b53d958a8d166', ['9e1217491467df90bc592be7133dc2549b7cd55d4d280c4241c86dcd7e3e36cb6']]]]}]

With this change, just the UUID values themselves are added to the list, e.g.

segments = [{'start': 53.495, 'end': 61.544, 'UUID': ['3ff36ef516888600471f1a03338b2faa5972f1bc26a2da038983cb549f4e9af96']}, {'start': 598.649, 'end': 321.091, 'UUID': ['f5dac489d7bff2bbbc8b1cde89c8d2485a3979b227542cf34de133a7ab1963fb6', '97489095d68cc9448b3a8a7e8379cf746b2ac9fdf94e917c9f24b9162f02d5b96', '9e7cdc02c4495f7480f0792e005d29374e47d1fc99d7e3a57b16b53d958a8d166', '9e1217491467df90bc592be7133dc2549b7cd55d4d280c4241c86dcd7e3e36cb6']}]
dmunozv04 commented 9 months ago

Thanks for opening a PR and fixing segment concatenation!