1209190 - Assigning ternary operator to innerHTML isn't flagged

Status: RESOLVED INCOMPLETE

(addons.mozilla.org Graveyard :: Add-on Validation, defect)

Description

[Wladimir Palant]

I noticed that innerHTML in some minified code wasn't flagged by the validator. After some reducing, the validator apparently doesn't consider the following dynamic:

  a.innerHTML=(true?x:x)+""

This might be the same as bug 1197206, impossible to tell without real info over there.

Comment 1

[Wladimir Palant]

Looking a bit into this, conditional expressions apparently aren't being processed at all, so that their value is considered to be None. While that isn't considered a literal by itself, the processing of the binary + operator will convert None into an empty string - so the result of adding a string to a ternary operator is always considered to be a string.

The same issue exists with the sequence expression, its value isn't being determined either - so the following won't trigger any warnings:

  a.innerHTML=(y,x)+""

Also logical expression:

  a.innerHTML=(true&&x)+""

And yield expression:

  function* foo()
  {
    a.innerHTML=(yield getHTML())+""
  }


And array comprehension:

  a.innerHTML=[for (x of [foo, bar]) x]+""

While update expressions and generator comprehensions aren't handled either, these don't seem important in this context.