Skip to content

Commit

Permalink
XFA - remove unnecessary check in the handleBreak function (PR 1368…
Browse files Browse the repository at this point in the history
…7 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.)
  • Loading branch information
Snuffleupagus committed Jul 7, 2021
1 parent fbd2f28 commit df61077
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/core/xfa/template.js
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ function handleBreak(node) {
if (!target) {
return false;
}
target = target ? target[0] : target;
target = target[0];
}

const { currentPageArea, currentContentArea } = root[$extra];
Expand Down

0 comments on commit df61077

Please sign in to comment.