google/closure-compiler

Polymer pass crashes when "properties" is not an object literal

Open

#1950 aperta il 11 ago 2016

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Java (1197 fork)batch import
help wanted

Metriche repository

Star
 (7176 star)
Metriche merge PR
 (Merge medio 3g 4h) (1 PR mergiata in 30 g)

Descrizione

https://closure-compiler-debugger.appspot.com/#input0%3DPolymer(%257B%250A%2520%2520is%253A%2520'x-y'%252C%250A%2520%2520properties%253A%2520(function()%2520%257B%250A%2520%2520%2520%2520return%2520%257B%257D%253B%250A%2520%2520%257D)()%252C%250A%257D)%26input1%26conformanceConfig%26externs%3Dvar%2520Polymer%2520%253D%2520function(obj)%2520%257B%257D%250Avar%2520PolymerElement%2520%253D%2520function(obj)%2520%257B%257D%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26CLOSURE_PASS%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26POLYMER_PASS%3D1%26PRESERVE_TYPE_ANNOTATIONS%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1

We don't want to allow this, I don't think. But we should report an error rather than crashing :)

Guida contributor

Direzione di ricerca
Individua il codice del pass Polymer nel Closure Compiler e identifica il percorso del codice che assume che 'properties' sia un oggetto letterale. Modificalo per gestire valori non letterali di oggetti e lanciare un errore descrittivo invece di crashare.
Tech stack
java
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJava
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75