Skip to content

Commit 2f9244e

Browse files
committed
Small refactor to reuse the build variable and reduce 1 db query
1 parent 5f663d4 commit 2f9244e

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

readthedocs/builds/views.py

+2-3
Original file line numberDiff line numberDiff line change
@@ -65,12 +65,12 @@ def post(self, request, project_slug):
6565
project=project,
6666
)
6767

68-
build = get_object_or_404(
68+
build_to_retrigger = get_object_or_404(
6969
Build.objects.all(),
7070
pk=build_pk,
7171
version=version,
7272
)
73-
if build != Build.objects.filter(version=version).first():
73+
if build_to_retrigger != Build.objects.filter(version=version).first():
7474
messages.add_message(
7575
request,
7676
messages.ERROR,
@@ -79,7 +79,6 @@ def post(self, request, project_slug):
7979
return HttpResponseRedirect(request.path)
8080

8181
# Set either the build to re-trigger it or None
82-
build_to_retrigger = version.builds.filter(pk=build_pk).first()
8382
if build_to_retrigger:
8483
commit_to_retrigger = build_to_retrigger.commit
8584
log.info(

0 commit comments

Comments
 (0)