Skip to content

Commit 8ad259a

Browse files
authored
Merge pull request #4937 from stsewd/safe-symlink-on-version-deletion
Safe symlink on version deletion
2 parents ee2d784 + 214afbb commit 8ad259a

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

readthedocs/builds/models.py

+10-2
Original file line numberDiff line numberDiff line change
@@ -213,10 +213,18 @@ def save(self, *args, **kwargs): # pylint: disable=arguments-differ
213213
def delete(self, *args, **kwargs): # pylint: disable=arguments-differ
214214
from readthedocs.projects import tasks
215215
log.info('Removing files for version %s', self.slug)
216-
broadcast(type='app', task=tasks.clear_artifacts, args=[self.get_artifact_paths()])
217216
broadcast(
218-
type='app', task=tasks.symlink_project, args=[self.project.pk])
217+
type='app',
218+
task=tasks.clear_artifacts,
219+
args=[self.get_artifact_paths()],
220+
)
221+
project_pk = self.project.pk
219222
super(Version, self).delete(*args, **kwargs)
223+
broadcast(
224+
type='app',
225+
task=tasks.symlink_project,
226+
args=[project_pk],
227+
)
220228

221229
@property
222230
def identifier_friendly(self):

0 commit comments

Comments
 (0)