From df6107714a53124cd6edeac1f10bb45274b5ee47 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 7 Jul 2021 23:22:29 +0200 Subject: [PATCH] XFA - remove unnecessary check in the `handleBreak` function (PR 13687 follow-up) With the changes in PR 13687 we're now checking if `target` is defined *twice* in a row, which shouldn't be necessary :-) (I noticed this when glancing at the unofficial LGTM results; maybe we should re-evalute the decision to not integrate that into the CI.) --- src/core/xfa/template.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/xfa/template.js b/src/core/xfa/template.js index 1bdbf773bb0a1..093d7eb1a735c 100644 --- a/src/core/xfa/template.js +++ b/src/core/xfa/template.js @@ -176,7 +176,7 @@ function handleBreak(node) { if (!target) { return false; } - target = target ? target[0] : target; + target = target[0]; } const { currentPageArea, currentContentArea } = root[$extra];