From 8b85ef36ebc56237e2620ffba4c3f51046dc10bb Mon Sep 17 00:00:00 2001
From: Miguel Serrano <mserranom@users.noreply.github.com>
Date: Mon, 8 Nov 2021 12:15:24 +0100
Subject: [PATCH] added check of compilation result status before setting
 pdfUrl/pdfDownloadUrl (#5697)

GitOrigin-RevId: ebf78bc2f98a952411875740fe912e275f83b224
---
 services/web/frontend/js/shared/context/compile-context.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/services/web/frontend/js/shared/context/compile-context.js b/services/web/frontend/js/shared/context/compile-context.js
index e5d461000a..4e68dab585 100644
--- a/services/web/frontend/js/shared/context/compile-context.js
+++ b/services/web/frontend/js/shared/context/compile-context.js
@@ -229,10 +229,12 @@ export function CompileProvider({ children }) {
           setLogEntryAnnotations(
             buildLogEntryAnnotations(result.logEntries.all, ide.fileTreeManager)
           )
+          if (data.status === 'success') {
+            setPdfDownloadUrl(result.pdfDownloadUrl)
+            setPdfUrl(result.pdfUrl)
+          }
           setLogEntries(result.logEntries)
           setFileList(result.fileList)
-          setPdfDownloadUrl(result.pdfDownloadUrl)
-          setPdfUrl(result.pdfUrl)
           setRawLog(result.log)
 
           // sample compile stats for real users