move use strict declarations to inside functions

Author: terite

.jshintrc

@@ -3,7 +3,6 @@
   "browser": true,
   "eqnull": true,
   "expr": true,
-  "globalstrict": true,
   "immed": true,
   "laxbreak": true,
   "loopfunc": true,
@@ -13,6 +12,7 @@
   "nonew": true,
   "quotmark": true,
   "smarttabs": true,
+  "strict": true,
   "sub": true,
   "trailing": true,
   "undef": true,

demo/demo.js

@@ -1,10 +1,10 @@
-'use strict';
-
 angular.module('doc.ui-utils', ['ui.utils', 'prettifyDirective' ]);
 
 angular.module('ui.scroll')
     .factory('datasource', [
     '$log', '$timeout', function(console, $timeout) {
+        'use strict';
+
         var get = function(index, count, success) {
             return $timeout(function() {
                 var i, result, _i, _ref;

modules/alias/alias.js

@@ -1,6 +1,6 @@
-'use strict';
-
 angular.module('ui.alias', []).config(['$compileProvider', 'uiAliasConfig', function($compileProvider, uiAliasConfig){
+  'use strict';
+
   uiAliasConfig = uiAliasConfig || {};
   angular.forEach(uiAliasConfig, function(config, alias){
     if (angular.isString(config)) {

modules/event/event.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * General-purpose Event binding. Bind any event not natively supported by Angular
  * Pass an object with keynames for events to ui-event
@@ -12,6 +10,8 @@
  */
 angular.module('ui.event',[]).directive('uiEvent', ['$parse',
   function ($parse) {
+    'use strict';
+
     return function ($scope, elm, attrs) {
       var events = $scope.$eval(attrs.uiEvent);
       angular.forEach(events, function (uiEvent, eventName) {

modules/format/format.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * A replacement utility for internationalization very similar to sprintf.
  *
@@ -14,6 +12,8 @@
  * @example: '$0 agrees to all mentions $0 makes in the event that $0 hits a tree while $0 is driving drunk'.format('Bob')
  */
 angular.module('ui.format',[]).filter('format', function(){
+  'use strict';
+
   return function(value, replace) {
     var target = value;
     if (angular.isString(target) && replace !== undefined) {

modules/highlight/highlight.js

@@ -1,12 +1,12 @@
-'use strict';
-
 /**
  * Wraps the
  * @param text {string} haystack to search through
  * @param search {string} needle to search for
  * @param [caseSensitive] {boolean} optional boolean to use case-sensitive searching
  */
 angular.module('ui.highlight',[]).filter('highlight', function () {
+  'use strict';
+
   return function (text, search, caseSensitive) {
     if (text && (search || angular.isNumber(search))) {
       text = text.toString();

modules/include/include.js

@@ -1,9 +1,9 @@
-'use strict';
-
 // modeled after: angular-1.0.7/src/ng/directive/ngInclude.js
 angular.module('ui.include',[])
 .directive('uiInclude', ['$http', '$templateCache', '$anchorScroll', '$compile',
                  function($http,   $templateCache,   $anchorScroll,   $compile) {
+  'use strict';
+
   return {
     restrict: 'ECA',
     terminal: true,

modules/indeterminate/indeterminate.js

@@ -1,12 +1,12 @@
-'use strict';
-
 /**
  * Provides an easy way to toggle a checkboxes indeterminate property
  *
  * @example <input type="checkbox" ui-indeterminate="isUnkown">
  */
 angular.module('ui.indeterminate',[]).directive('uiIndeterminate', [
   function () {
+    'use strict';
+
     return {
       compile: function(tElm, tAttrs) {
         if (!tAttrs.type || tAttrs.type.toLowerCase() !== 'checkbox') {

modules/inflector/inflector.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Converts variable-esque naming conventions to something presentational, capitalized words separated by space.
  * @param {String} value The value to be parsed and prettified.
@@ -10,6 +8,7 @@
  *          {{ 'Here Is my_phoneNumber' | inflector:'variable' }} => hereIsMyPhoneNumber
  */
 angular.module('ui.inflector',[]).filter('inflector', function () {
+  'use strict';
 
   function tokenize(text) {
     text = text.replace(/([A-Z])|([\-|\_])/g, function(_, $1) { return ' ' + ($1 || ''); });

modules/jq/jq.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * General-purpose jQuery wrapper. Simply pass the plugin name as the expression.
  *
@@ -18,6 +16,8 @@
 angular.module('ui.jq',[]).
   value('uiJqConfig',{}).
   directive('uiJq', ['uiJqConfig', '$timeout', function uiJqInjectingFunction(uiJqConfig, $timeout) {
+  'use strict';
+
 
   return {
     restrict: 'A',

modules/keypress/keypress.js

@@ -1,7 +1,7 @@
-'use strict';
-
 angular.module('ui.keypress',[]).
 factory('keypressHelper', ['$parse', function keypress($parse){
+  'use strict';
+
   var keysByCode = {
     8: 'backspace',
     9: 'tab',
@@ -92,6 +92,8 @@ factory('keypressHelper', ['$parse', function keypress($parse){
  * @example <input ui-keypress="{enter:'x = 1', 'ctrl-shift-space':'foo()', 'shift-13':'bar()'}" /> <input ui-keypress="foo = 2 on ctrl-13 and bar('hello') on shift-esc" />
  **/
 angular.module('ui.keypress').directive('uiKeydown', ['keypressHelper', function(keypressHelper){
+  'use strict';
+
   return {
     link: function (scope, elm, attrs) {
       keypressHelper('keydown', scope, elm, attrs);
@@ -100,6 +102,8 @@ angular.module('ui.keypress').directive('uiKeydown', ['keypressHelper', function
 }]);
 
 angular.module('ui.keypress').directive('uiKeypress', ['keypressHelper', function(keypressHelper){
+  'use strict';
+
   return {
     link: function (scope, elm, attrs) {
       keypressHelper('keypress', scope, elm, attrs);
@@ -108,6 +112,8 @@ angular.module('ui.keypress').directive('uiKeypress', ['keypressHelper', functio
 }]);
 
 angular.module('ui.keypress').directive('uiKeyup', ['keypressHelper', function(keypressHelper){
+  'use strict';
+
   return {
     link: function (scope, elm, attrs) {
       keypressHelper('keyup', scope, elm, attrs);

modules/mask/mask.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /*
  Attaches input mask onto input element
  */
@@ -12,6 +10,8 @@ angular.module('ui.mask', [])
     }
   })
   .directive('uiMask', ['uiMaskConfig', '$parse', function (maskConfig, $parse) {
+    'use strict';
+
     return {
       priority: 100,
       require: 'ngModel',

modules/reset/reset.js

@@ -1,9 +1,9 @@
-'use strict';
-
 /**
  * Add a clear button to form inputs to reset their value
  */
 angular.module('ui.reset',[]).value('uiResetConfig',null).directive('uiReset', ['uiResetConfig', function (uiResetConfig) {
+  'use strict';
+
   var resetValue = null;
   if (uiResetConfig !== undefined){
       resetValue = uiResetConfig;

modules/route/route.js

@@ -1,9 +1,9 @@
-'use strict';
-
 /**
  * Set a $uiRoute boolean to see if the current route matches
  */
 angular.module('ui.route', []).directive('uiRoute', ['$location', '$parse', function ($location, $parse) {
+  'use strict';
+
   return {
     restrict: 'AC',
     scope: true,

modules/scroll/scroll-jqlite.js

@@ -1,6 +1,7 @@
-'use strict';
 angular.module('ui.scroll.jqlite', ['ui.scroll']).service('jqLiteExtras', [
   '$log', '$window', function(console, window) {
+    'use strict';
+
     return {
       registerFor: function(element) {
         var convertToPx, css, getMeasurements, getStyle, getWidthHeight, isWindow, scrollTo;
@@ -217,6 +218,8 @@ angular.module('ui.scroll.jqlite', ['ui.scroll']).service('jqLiteExtras', [
   }
 ]).run([
   '$log', '$window', 'jqLiteExtras', function(console, window, jqLiteExtras) {
+    'use strict';
+
     if (!window.jQuery) {
       return jqLiteExtras.registerFor(angular.element);
     }

modules/scroll/scroll.js

@@ -1,4 +1,3 @@
-'use strict';
 /*
 globals: angular, window
 
@@ -14,6 +13,8 @@ globals: angular, window
 
 angular.module('ui.scroll', []).directive('uiScrollViewport', [
   '$log', function() {
+    'use strict';
+
     return {
       controller: [
         '$scope', '$element', function(scope, element) {
@@ -25,6 +26,8 @@ angular.module('ui.scroll', []).directive('uiScrollViewport', [
   }
 ]).directive('uiScroll', [
   '$log', '$injector', '$rootScope', '$timeout', function(console, $injector, $rootScope, $timeout) {
+    'use strict';
+
     return {
       require: ['?^uiScrollViewport'],
       transclude: 'element',

modules/scrollfix/scrollfix.js

@@ -1,11 +1,11 @@
-'use strict';
-
 /**
  * Adds a 'ui-scrollfix' class to the element when the page scrolls past it's position.
  * @param [offset] {int} optional Y-offset to override the detected offset.
  *   Takes 300 (absolute) or -300 or +300 (relative to detected)
  */
 angular.module('ui.scrollfix',[]).directive('uiScrollfix', ['$window', function ($window) {
+  'use strict';
+
   function getWindowScrollTop() {
     if (angular.isDefined($window.pageYOffset)) {
       return $window.pageYOffset;
@@ -50,6 +50,7 @@ angular.module('ui.scrollfix',[]).directive('uiScrollfix', ['$window', function
     }
   };
 }]).directive('uiScrollfixTarget', [function () {
+  'use strict';
   return {
     controller: ['$element', function($element) {
       this.$element = $element;

modules/showhide/showhide.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * uiShow Directive
  *
@@ -10,6 +8,8 @@
  */
 angular.module('ui.showhide',[])
 .directive('uiShow', [function () {
+  'use strict';
+
   return function (scope, elm, attrs) {
     scope.$watch(attrs.uiShow, function (newVal) {
       if (newVal) {
@@ -30,6 +30,8 @@ angular.module('ui.showhide',[])
  * @param expression {boolean} evaluated expression to determine if the class should be added
  */
 .directive('uiHide', [function () {
+  'use strict';
+
   return function (scope, elm, attrs) {
     scope.$watch(attrs.uiHide, function (newVal) {
       if (newVal) {
@@ -51,6 +53,8 @@ angular.module('ui.showhide',[])
  * @param expression {boolean} evaluated expression to determine if the class should be added
  */
 .directive('uiToggle', [function () {
+  'use strict';
+
   return function (scope, elm, attrs) {
     scope.$watch(attrs.uiToggle, function (newVal) {
       if (newVal) {

modules/unique/unique.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Filters out all duplicate items from an array by checking the specified key
  * @param [key] {string} the name of the attribute of each object to compare for uniqueness
@@ -8,6 +6,7 @@
  * @return {array}
  */
 angular.module('ui.unique',[]).filter('unique', ['$parse', function ($parse) {
+  'use strict';
 
   return function (items, filterOn) {
 

modules/uploader/uploader.js

@@ -1,4 +1,3 @@
-'use strict';
 /*
  * Author: Remy Alain Ticona Carbajal http://realtica.org
  * Description: The main objective of ng-uploader is to have a user control,
@@ -11,6 +10,8 @@ angular.module('ui.uploader', []).service('uiUploader', uiUploader);
 uiUploader.$inject = ['$log'];
 
 function uiUploader($log) {
+    'use strict';
+
     /*jshint validthis: true */
     var self = this;
     self.files = [];
@@ -117,4 +118,4 @@ function uiUploader($log) {
         return xhr;
     }
 
-}
+}

modules/validate/validate.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * General-purpose validator for ngModel.
  * angular.js comes with several built-in validation mechanism for input fields (ngRequired, ngPattern etc.) but using
@@ -17,6 +15,7 @@
  * In both cases validator function should take a value to validate as its argument and should return true/false indicating a validation result.
  */
 angular.module('ui.validate',[]).directive('uiValidate', function () {
+  'use strict';
 
   return {
     restrict: 'A',