Skip to content

Commit 9ca62db

Browse files
committed
Fix local file delete logic with include_file=False
1 parent 440567a commit 9ca62db

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

readthedocs/projects/tasks.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1006,11 +1006,14 @@ def move_files(
10061006
# We set `epub=False` for example so data doesn't get re-uploaded on each web,
10071007
# so we need this to protect against deleting in those cases
10081008
if delete_unsynced_media:
1009+
10091010
if not pdf:
1011+
10101012
remove_dirs([
10111013
version.project.get_production_media_path(
10121014
type_='pdf',
10131015
version_slug=version.slug,
1016+
include_file=False,
10141017
),
10151018
])
10161019

@@ -1023,11 +1026,14 @@ def move_files(
10231026
if storage.exists(storage_path):
10241027
log.info('Removing %s from media storage', storage_path)
10251028
storage.delete(storage_path)
1029+
10261030
if not epub:
1031+
10271032
remove_dirs([
10281033
version.project.get_production_media_path(
10291034
type_='epub',
10301035
version_slug=version.slug,
1036+
include_file=False,
10311037
),
10321038
])
10331039

@@ -1042,10 +1048,12 @@ def move_files(
10421048
storage.delete(storage_path)
10431049

10441050
if not localmedia:
1051+
10451052
remove_dirs([
10461053
version.project.get_production_media_path(
10471054
type_='htmlzip',
10481055
version_slug=version.slug,
1056+
include_file=False,
10491057
),
10501058
])
10511059

0 commit comments

Comments
 (0)