Fix appconfig cleanup, #build
Changed: - fixed runtime error in appconfig cleanup - fixed reindex video index name lookup
This commit is contained in:
commit
9f3ab54dd3
|
|
@ -180,17 +180,17 @@ class AppConfig:
|
||||||
def clear_old_keys(self) -> list[str]:
|
def clear_old_keys(self) -> list[str]:
|
||||||
"""clear old unused keys"""
|
"""clear old unused keys"""
|
||||||
cleared = []
|
cleared = []
|
||||||
for key, value in self.config.items():
|
for key in list(self.config.keys()):
|
||||||
if key not in self.CONFIG_DEFAULTS:
|
if key not in self.CONFIG_DEFAULTS:
|
||||||
# complete key removed
|
# complete key removed
|
||||||
self.config.pop(key)
|
value = self.config.pop(key)
|
||||||
cleared.append(str({key: value}))
|
cleared.append(str({key: value}))
|
||||||
continue
|
continue
|
||||||
|
|
||||||
expected_keys = set(
|
expected_keys = set(
|
||||||
self.CONFIG_DEFAULTS[key].keys() # type: ignore
|
self.CONFIG_DEFAULTS[key].keys() # type: ignore
|
||||||
)
|
)
|
||||||
is_keys = set(self.config[key].keys())
|
is_keys = set(list(self.config[key].keys()))
|
||||||
|
|
||||||
for to_delete in is_keys - expected_keys:
|
for to_delete in is_keys - expected_keys:
|
||||||
self.config[key].pop(to_delete)
|
self.config[key].pop(to_delete)
|
||||||
|
|
|
||||||
|
|
@ -288,7 +288,7 @@ class Reindex(ReindexBase):
|
||||||
self._notify(name, total, idx)
|
self._notify(name, total, idx)
|
||||||
|
|
||||||
index_name = index_config["index_name"]
|
index_name = index_config["index_name"]
|
||||||
if index_name == "ta_vide":
|
if index_name == "ta_video":
|
||||||
video = self.reindex_single_video(youtube_id)
|
video = self.reindex_single_video(youtube_id)
|
||||||
if video:
|
if video:
|
||||||
self._reindex_video_related(video)
|
self._reindex_video_related(video)
|
||||||
|
|
|
||||||
|
|
@ -280,12 +280,11 @@ class Command(BaseCommand):
|
||||||
self.style.SUCCESS(f" added new default: {new_default}")
|
self.style.SUCCESS(f" added new default: {new_default}")
|
||||||
)
|
)
|
||||||
|
|
||||||
# FIXME
|
cleared = AppConfig().clear_old_keys()
|
||||||
# cleared = AppConfig().clear_old_keys()
|
for removed_key in cleared:
|
||||||
# for removed_key in cleared:
|
self.stdout.write(
|
||||||
# self.stdout.write(
|
self.style.SUCCESS(f" removed old key: {removed_key}")
|
||||||
# self.style.SUCCESS(f" removed old key: {removed_key}")
|
)
|
||||||
# )
|
|
||||||
|
|
||||||
return
|
return
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue