Skip to content

Commit e685c66

Browse files
refactor: removed a unnecessary if/else
1 parent f017a31 commit e685c66

File tree

1 file changed

+4
-13
lines changed

1 file changed

+4
-13
lines changed

lib/rules/ensure-matching-remove-event-listener.js

+4-13
Original file line numberDiff line numberDiff line change
@@ -95,19 +95,10 @@ module.exports = {
9595
}
9696
if (hasAddEventListener) {
9797
if (!hasRemoveEventListener) {
98-
if (!hasReturnStatement) {
99-
const messageId = 'requiredCleanup';
100-
101-
report(context, messages, messageId, {
102-
node,
103-
});
104-
} else {
105-
const messageId = 'requiredRemoveEventListener';
106-
107-
report(context, messages, messageId, {
108-
node,
109-
});
110-
}
98+
const messageID = hasReturnStatement ? 'requiredRemoveEventListener' : 'requiredCleanup';
99+
report(context, messages[messageID], messageID, {
100+
node,
101+
});
111102
}
112103
}
113104
}

0 commit comments

Comments
 (0)