Fix bug that prevent the entry point from being added to the pubspec

pkg/polymer/bin/new_entry.dart

 ///
 /// Script to create boilerplate for a Polymer element.
 /// Produces new .html entry point for a polymer app and updates the
 /// pubspec.yaml to reflect it.
 ///
 /// Run this script with pub run:
 ///
 ///     pub run polymer:new_entry <html_file>
 ///
 import 'dart:io';
@@ skipping 66 matching lines @@
     <script type="application/dart">export 'package:polymer/init.dart';</script>
   </body>
 </html>
 ''';
 
   new File(path.join(pubspecDir, entryPoint)).writeAsStringSync(html);
 
   var pubspecPath = path.join(pubspecDir, 'pubspec.yaml');
   var pubspecText = new File(pubspecPath).readAsStringSync();
   var transformers = loadYaml(pubspecText)['transformers'];
-  var entryPoints;
+  var entryPoints = [];
 
   var insertionPoint;
   var textToInsert = '';
 
   if (transformers != null) {
     // If there are transformers in the pubspec, look for the polymer
     // transformers, get the entry points, and delete the old entry points.
     var transformersSourceSpan = transformers.span;
     SourceSpan sourceSpan;
 
     for (var e in transformers) {
       if (e != 'polymer' && (e is! YamlMap || e['polymer'] == null)) continue;
       if (e == 'polymer' || !e['polymer'].containsKey('entry_points')) {
         if (path.split(entryPoint)[0] != 'web') {
           print('WARNING: Did not add entry_point $entryPoint to pubspec.yaml'
               ' because of already-existing transformer|polymer section');
         }
         return false;
       } else if (e['polymer'].keys.length > 1) {
         // TODO(dgrove): handle the case where there are additional sections
         // in the polymer transformer.
         throw new UnimplementedError('Cannot handle non-entry_point entries '
             'for polymer transformer');
       } else {
         var existing = e['polymer']['entry_points'];
-        entryPoints = existing == null ? [] :
-            (existing is String ? [existing] : existing.toList());
+        entryPoints = (existing == null ? [] :
+            (existing is String ? [existing] : existing.toList()));
 
         if (entryPoints.contains(entryPoint)) return false;
         entryPoints.add(entryPoint);
 
         sourceSpan = e.span;
       }
     }
 
     if (sourceSpan == null) {
       // There were no polymer transformers.
       insertionPoint = transformersSourceSpan.start.offset;
       textToInsert = '- ';
     } else {
       insertionPoint = sourceSpan.start.offset;
       pubspecText = '${pubspecText.substring(0, insertionPoint)}'
           '${pubspecText.substring(sourceSpan.end.offset)}';
     }
   } else {
     // There were no transformers at all.
     insertionPoint = pubspecText.length;
     textToInsert = 'transformers:\n- ';
+    entryPoints.add(entryPoint);

[Siggi Cherem (dart-lang), 2014/08/13 21:56:30]

alternatively, you could remove the initialization and set it here to be =
[entryPoint]

   }
 
   // TODO(dgrove): Once dartbug.com/20409 is addressed, use that here.
   var entryPointsText = entryPoints.map((e) => '    - $e').join('\n');
 
   textToInsert =
 '''${textToInsert}polymer:
     entry_points:
 $entryPointsText''';
 
 
   if (insertionPoint == pubspecText.length) {
     pubspecText = '${pubspecText}${textToInsert}';
   } else {
     pubspecText = '${pubspecText.substring(0, insertionPoint)}'
         '${textToInsert}\n${pubspecText.substring(insertionPoint)}';
   }
 
   _writePubspec(pubspecPath, pubspecText);
   return true;
 }
 
 _writePubspec(String pubspecPath, String text) {
   new File(pubspecPath).writeAsStringSync(text);
 }