Manager was getting pushed, debuged to this logic bug that 0 fails the test intended to check if the element exists. Replaced with equivalent test logic.
Fixed:
MovePush check incorrectly treating manager 0 value as nonexistent and taking default 10 causing unintended push.
Testing checklist:
[x] Changes are backward-compatible OR version migration code is included
[x] Codebase compiles with current tsconfig configuration
[x] Tested changes on PUBLIC server OR changes are trivial (e.g. typos)
Pull request summary
Description:
Manager was getting pushed, debuged to this logic bug that 0 fails the test intended to check if the element exists. Replaced with equivalent test logic.
Fixed:
MovePush check incorrectly treating manager 0 value as nonexistent and taking default 10 causing unintended push.
Testing checklist:
tsconfig
configuration