Only compiling once to prevent ng-repeats from growing on each infoWindow.open()

The original code decorated the infoWindow.open() and would compile() before calling the original infoWindow.open(). This causes ng-repeats to grow with each call. Compiling is needed or else things like ng-click won't work. But a single compile appears to be enough for all functionality to work as expected.

Author: Valorum

ui-map.js

@@ -69,12 +69,9 @@
            straight removing things from the dom can mess up angular) */
           elm.replaceWith('<div></div>');
 
-          //Decorate infoWindow.open to $compile contents before opening
-          var _open = infoWindow.open;
-          infoWindow.open = function open(a1, a2, a3, a4, a5, a6) {
-            $compile(elm.contents())(scope);
-            _open.call(infoWindow, a1, a2, a3, a4, a5, a6);
-          };
+          /* Need to compile once to make sure ng-clicks are hoooked up properly.
+           Don't compile more than once though, because that messes up ng-repeats! */
+          $compile(elm.contents())(scope);
         }
       };
     }]);