Andrew,

The pull request has been updated with the changes you recommended. I also
merged with the latest version of master. I ran into a problem however with
the 'phantomjs' gem. When I ran the test suite, I received the following
error:

"PhantomJS has crashed. Please read the crash reporting guide at
https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
report at https://code.google.com/p/phantomjs/issues/entry with the crash
dump file attached: /tmp/DA902328-2F20-4825-842C-4BF1A8B305AD.dmp"

Removing the phantomjs gem made all tests green. My guess is that this gem
is somehow causing conflicts the version of PhantomJS installed on the
computers here at Epicodus.

Best Regards,
Steven


On Fri, Oct 25, 2013 at 4:27 PM, Andrew Lorente <notifications@github.com>wrote:

> In app/assets/javascripts/projects.js:
>
> > @@ -0,0 +1,10 @@
> > +$(function() {
> > +  $('form').on('click', '.add_fields', function(event) {
> > +    time = new Date().getTime();
> > +    regexp = new RegExp($(this).data('id'), 'g');
>
> time and regexp need var declarations!
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/code-scouts/green_mercury/pull/69/files#r7231214>
> .
>