View Issue Details

IDProjectCategoryView StatusLast Update
0002391AJAX/JSBug Report - Interfacepublic2016-02-09 20:06
Reporterchortos-2 Assigned ToDerIdiot  
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Summary0002391: fileadd.js syntax error: invalid identifiers with leading digits
DescriptionEver since fileadd.js was updated (by worf, I believe) to disable all video-related checkboxes when “no video track” is checked, it has contained references to `this.streams[0].10bit` and `this.streams[0].3dvideo`. This produces syntax errors, as identifiers cannot start with a digit and this is parsed as an integer followed by an identifier, which makes no sense. As a result, at least in Firefox, the entire script is discarded or something to the same effect.

Two fixes are possible:

1. Replace the dot syntax with the subscript syntax: `this.streams[0]['10bit']`. The dot syntax is merely syntactic sugar for subscripting.

2. Rename the `10bit` and `3dvideo` fields to start with a letter or an underscore.
TagsNo tags attached.

Activities

There are no notes attached to this issue.

Issue History

Date Modified Username Field Change
2016-02-05 16:27 chortos-2 New Issue
2016-02-09 20:06 DerIdiot Status new => resolved
2016-02-09 20:06 DerIdiot Resolution open => fixed
2016-02-09 20:06 DerIdiot Assigned To => DerIdiot