hello use strict. and other code style changes. and gulpfile mods. an…

…d other goodness

Gulpfile.js

@@ -4,6 +4,8 @@ var gulp = require('gulp')
     , concat = require('gulp-concat')
     , uglify = require('gulp-uglify')
     , rename = require('gulp-rename')
+    , zip = require('gulp-zip')
+    , jshint = require('gulp-jshint')
 
     , pkg = require('./package.json');
 
@@ -20,10 +22,12 @@ gulp.task('localBuild', function(){
 
     gulp.src(['public/javascripts/app.js', './public/javascripts/components/**/*.js','!./public/javascripts/components/**/*_test.js'])
         .pipe(concat('app.js'))
+        .pipe(jshint())
+        .pipe(jshint.reporter('default'))
         .pipe(gulp.dest('public/javascripts/built/'));
 });
 
-gulp.task('publish', function(){
+gulp.task('generateForPublish', function(){
     'use strict';
 
     gulp.src('public/javascripts/action.js')
@@ -34,6 +38,14 @@ gulp.task('publish', function(){
         .pipe(gulp.dest('packages/' + pkg.version + '/'));
 });
 
+gulp.task('publish', ['generateForPublish'], function(){
+    'use strict';
+
+     // gulp.src(['packages/' + pkg.version + '/*.js'])
+     //    .pipe(zip('action.zip'))
+     //    .pipe(gulp.dest('packages/' + pkg.version + '/'));
+});
+
 // gulp.task('default', function () {
 //     watch({ glob: ['public/javascripts/app.js', 'public/javascripts/components/**/*.js'], emitOnGlob: false })
 //         .pipe(plumber())

app.js

@@ -19,6 +19,7 @@ app.use(express.logger('dev'));
 app.use(express.bodyParser());
 app.use(express.methodOverride());
 app.use(app.router);
+app.use(express.compress());
 app.use(express.static(path.join(__dirname, 'public')));
 
 // development only

package.json

@@ -15,6 +15,8 @@
     "gulp-uglify": "^0.2.1",
     "gulp-plumber": "^0.6.0",
     "gulp-watch": "^0.5.4",
-    "gulp-rename": "^1.2.0"
+    "gulp-rename": "^1.2.0",
+    "gulp-zip": "^0.3.4",
+    "gulp-jshint": "^1.5.5"
   }
 }

public/javascripts/action.js

@@ -40,7 +40,7 @@ var action = function(){
                         }
 
                         if(eventStack[i].once){
-                            that.silenceLocal(eventNameIn, eventStack[i].call, true, isLocal);
+                            that.silence(eventNameIn, eventStack[i].call, true, isLocal);
                         }
                     }
                 }

public/javascripts/app.js

@@ -4,6 +4,8 @@ angLee = action.modelMe({
     , role: 'Director'
 
     , init: function(){
+        'use strict';
+
         var that = this;
 
         that.listen('cast:movie', function () {
@@ -15,10 +17,14 @@ angLee = action.modelMe({
     }
 
     , castingCall: function(){
+        'use strict';
+
         this.emit('actor:cast');
     }
 
     , castingSelection: function(){
+        'use strict';
+
         this.emit('actor:choose');
     }
 });
@@ -27,6 +33,8 @@ currentActor = action.eventMe({
     actors: []
 
     , init: function(){
+        'use strict';
+
         var that = this;
 
         that.listen('actor:me', function(actor){
@@ -51,5 +59,7 @@ currentActor = action.eventMe({
 });
 
 action.listen('actor:change', function(actor){
+    'use strict';
+
     $('.currentActorName').text(actor.get('name'));
 });

public/javascripts/components/actorPicture.js

@@ -1,4 +1,6 @@
 $(function(){
+    'use strict';
+
     action.listen('actor:change', function(actor){
         $('#actorImage').attr('src', actor.get('image'));
     });