Linter does not report undefined on += and similar operators #1671

Activity

1. Mikhail Arkhipov (MikhailArkhipov)

   added

   bugSomething isn't workingSomething isn't working

   feature: linter

   on Oct 11, 2019
2. jakebailey commented on Oct 11, 2019

   

   Jake Bailey (jakebailey)

   on Oct 11, 2019

   Last edited by jakebailey

   Member

   More actions

   We don't consider assignment operators to be actual assignments, since they'd need to have some sort of assignment beforehand to then add to. We could make this more loose but it might be strange.

   I misinterpreted when I wrote this comment; the title says that this is an issue (not that it should be okay and make dest defined).
3. bsandrow commented on Oct 23, 2019

   

   Brandy Sandrowicz (bsandrow)

   on Oct 23, 2019

   More actions

   Shouldn't dest be marked as an undefined variable on the += line if you want to keep that classification?
4. jakebailey commented on Oct 23, 2019

   

   Jake Bailey (jakebailey)

   on Oct 23, 2019

   Member

   More actions

   Yes, that's what this bug is about, that it should. The linter is likely just not walking the lefthand side of these augmented assignments.
5. Mikhail Arkhipov (MikhailArkhipov)

   self-assigned this

   on Nov 12, 2019
6. Mikhail Arkhipov (MikhailArkhipov)

   added this to the November 2019.1 milestone on Nov 12, 2019
7. Jake Bailey (jakebailey)

   closed this as completedin #1788on Nov 13, 2019