Remove useless parens.  ? : is a long and weird operator, if you want putting them around the whole expression to visually encapsulate it is acceptable, i.e.:
```js
value = (value ? value + ',' + newvalue : newvalue);
```