fix(contentscript): avoid null access

fixes #335
sanemat · May 30, 2024 · 92a6ed2 · 92a6ed2
1 parent 0b92137
commit 92a6ed2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/scripts/contentscript.js b/app/scripts/contentscript.js
@@ -3,7 +3,7 @@
 
   function changeMergeButtonState() {
     let container = document.querySelector('#js-repo-pjax-container');
-    let issueTitle = container.querySelector('.js-issue-title').textContent;
+    let issueTitle = container.querySelector('.js-issue-title')?.textContent;
     let buttonMerges = container.querySelectorAll('.merge-message button[data-details-container]');
     let buttonMergeOptions = container.querySelectorAll('.merge-message button[data-details-container] + .select-menu-button');
     let disabled = false;
@@ -17,7 +17,7 @@
       const wipTagRegex = /(wip|do\s*not\s*merge|dnm)/i;
 
       const isWipTitle = wipTitleRegex.test(issueTitle);
-      const isWipTaskList = container.querySelector('.timeline-comment') && container.querySelector('.timeline-comment').querySelector('input[type="checkbox"]:not(:checked)') !== null;
+      const isWipTaskList = container.querySelector('.timeline-comment') && container.querySelector('.timeline-comment')?.querySelector('input[type="checkbox"]:not(:checked)') !== null;
       let isSquashCommits = false;
       for (const commitMessage of container.querySelectorAll('.commit-message')) {
         isSquashCommits = isSquashCommits || commitMessage.textContent.match(/(squash|fixup)!/);