Merge pull request #513 from corespring/feature/PE-452-refactor-depen…

…dencies

Feature/pe 452 refactor dependencies

modules/container-client/grunt/config/devEditor.coffee

@@ -5,23 +5,24 @@ _ = require 'lodash'
 
 editorSrcs = [
   'bower_components/angular-animate/angular-animate(.min).js',
+  'bower_components/angular-bootstrap/ui-bootstrap-tpls(.min).js',
   'bower_components/angular-route/angular-route(.min).js',
+  'bower_components/angular-ui-ace/ui-ace(.min).js',
   'bower_components/angular-ui-router/release/angular-ui-router(.min).js',
-  'bower_components/wiggi-wiz/dist/wiggi-wiz.js',
-  'bower_components/select2/select2(.min).js',
   'bower_components/angular-ui-select2/src/select2.js',
   'bower_components/angular-ui/build/angular-ui(.min).js',
-  'bower_components/angular-bootstrap/ui-bootstrap-tpls(.min).js',
-  'bower_components/angular-ui-ace/ui-ace(.min).js',
   'bower_components/jquery.browser/dist/jquery.browser(.min).js',
+  'bower_components/select2/select2(.min).js',
   'bower_components/undo.js/undo.js',
+  'bower_components/wiggi-wiz/dist/wiggi-wiz.js',
   'js/corespring/core-library.js',
   'js/corespring/server/init-core-library.js',
   'js/dev-editor/**/*.js',
+  'js/editing/**/*.js',
   'js/editor/**/*.js',
-  'js/render/services/**/*.js',
-  'js/render/directives/**/*.js',
   'js/render/controllers/**/*.js'
+  'js/render/directives/**/*.js',
+  'js/render/services/**/*.js',
 ]
 
 exports.js =
@@ -51,6 +52,13 @@ exports.ngModules = _.union(player.ngModules, [
   'corespring-dev-editor.controllers',
   'corespring-dev-editor.directives',
   'corespring-dev-editor.services',
+  'corespring-editing.controllers',
+  'corespring-editing.directives',
+  'corespring-editing.services',
+  'corespring-editing.wiggi-wiz-features',
+  'corespring-editing.wiggi-wiz-features.cs-image',
+  'corespring-editing.wiggi-wiz-features.link',
+  'corespring-editing.wiggi-wiz-features.mathjax',
   'corespring-editor.controllers',
   'corespring-editor.directives',
   'corespring-editor.services',

modules/container-client/grunt/config/editor.coffee

@@ -5,27 +5,28 @@ _ = require 'lodash'
 
 editorSrcs = [
   'bower_components/angular-animate/angular-animate(.min).js',
+  'bower_components/angular-bootstrap/ui-bootstrap-tpls(.min).js',
+  'bower_components/angular-debounce/dist/angular-debounce(.min).js',
   'bower_components/angular-route/angular-route(.min).js',
+  'bower_components/angular-ui-ace/ui-ace(.min).js',
   'bower_components/angular-ui-router/release/angular-ui-router(.min).js',
-  'bower_components/wiggi-wiz/dist/wiggi-wiz.js',
-  'bower_components/select2/select2(.min).js',
   'bower_components/angular-ui-select2/src/select2.js',
   'bower_components/angular-ui/build/angular-ui(.min).js',
-  'bower_components/angular-ui-ace/ui-ace(.min).js',
-  'bower_components/angular-bootstrap/ui-bootstrap-tpls(.min).js',
-  'bower_components/angular-debounce/dist/angular-debounce(.min).js',
-  'bower_components/jquery.browser/dist/jquery.browser(.min).js',
-  'bower_components/undo.js/undo.js',
   'bower_components/corespring-checkbox/src/directive.js',
   'bower_components/corespring-radio/src/directive.js',
+  'bower_components/jquery.browser/dist/jquery.browser(.min).js',
+  'bower_components/select2/select2(.min).js',
+  'bower_components/undo.js/undo.js',
+  'bower_components/wiggi-wiz/dist/wiggi-wiz.js',
+  'js/catalog/**/*.js',
+  'js/common/**/*.js',
   'js/corespring/core-library.js',
   'js/corespring/server/init-core-library.js',
-  'js/common/**/*.js',
+  'js/editing/**/*.js',
   'js/editor/**/*.js',
-  'js/catalog/**/*.js',
-  'js/render/services/**/*.js',
-  'js/render/directives/**/*.js',
   'js/render/controllers/**/*.js'
+  'js/render/directives/**/*.js',
+  'js/render/services/**/*.js'
 ]
 
 exports.js =
@@ -53,6 +54,12 @@ exports.css =
 
 exports.ngModules = _.union(player.ngModules, [
   'corespring-common.supporting-materials',
+  'corespring-editing.controllers',
+  'corespring-editing.directives',
+  'corespring-editing.services',
+  'corespring-editing.wiggi-wiz-features.cs-image',
+  'corespring-editing.wiggi-wiz-features.link',
+  'corespring-editing.wiggi-wiz-features.mathjax',
   'corespring-editor.controllers',
   'corespring-editor.directives',
   'corespring-editor.profile.controllers',

modules/container-client/grunt/config/singleComponentEditor.coffee

@@ -7,27 +7,25 @@ baseName = "singleComponentEditor"
 
 editorSrcs = [
   'bower_components/angular-animate/angular-animate(.min).js',
-  'bower_components/wiggi-wiz/dist/wiggi-wiz.js',
-  'bower_components/undo.js/undo.js',
-  'bower_components/select2/select2(.min).js',
-  'bower_components/angular-ui-select2/src/select2.js',
-  'bower_components/angular-ui/build/angular-ui(.min).js',
-  'bower_components/angular-ui-ace/ui-ace(.min).js',
   'bower_components/angular-bootstrap/ui-bootstrap-tpls(.min).js',
   'bower_components/angular-debounce/dist/angular-debounce(.min).js',
-  'bower_components/jquery.browser/dist/jquery.browser(.min).js',
+  'bower_components/angular-ui-ace/ui-ace(.min).js',
+  'bower_components/angular-ui-select2/src/select2.js',
+  'bower_components/angular-ui/build/angular-ui(.min).js',
   'bower_components/corespring-checkbox/src/directive.js',
   'bower_components/corespring-radio/src/directive.js',
-  'js/corespring/core-library.js',
-  'js/corespring/server/init-core-library.js',
+  'bower_components/jquery.browser/dist/jquery.browser(.min).js',
+  'bower_components/select2/select2(.min).js',
+  'bower_components/undo.js/undo.js',
+  'bower_components/wiggi-wiz/dist/wiggi-wiz.js',
   'js/common/**/*.js',
-  # it would be better to pull out the js we need here instead
-  'js/rig/**/*.js',
-  'js/editor/**/*.js',
   'js/component-editor/**/*.js',
-  'js/render/services/**/*.js',
+  'js/corespring/core-library.js',
+  'js/corespring/server/init-core-library.js',
+  'js/editing/**/*.js',
+  'js/render/controllers/**/*.js',
   'js/render/directives/**/*.js',
-  'js/render/controllers/**/*.js'
+  'js/render/services/**/*.js'
 ]
 
 exports.js =
@@ -53,18 +51,18 @@ exports.css =
   report: "#{baseName}-css-report.json"
 
 exports.ngModules = _.union(player.ngModules, [
-  'corespring.wiggi-wiz',
-  'corespring-editor.directives',
-  'corespring-editor.controllers',
-  'corespring-rig.directives',
-  'corespring-editor.services',
-  'ui.bootstrap.tabs',
-  'ui.ace',
-  'wiggi-wiz.features.core',
-  'corespring.wiggi-wiz-features.mathjax',
-  'corespring.wiggi-wiz-features.link',
-  'corespring.wiggi-wiz-features.cs-image',
+  'corespring-editing.controllers',
+  'corespring-editing.directives',
+  'corespring-editing.services',
+  'corespring-editing.wiggi-wiz-features',
+  'corespring-editing.wiggi-wiz-features.cs-image',
+  'corespring-editing.wiggi-wiz-features.link',
+  'corespring-editing.wiggi-wiz-features.mathjax',
   'corespring-singleComponentEditor.controllers',
-  'corespring-singleComponentEditor.services',
   'corespring-singleComponentEditor.directives'
+  'corespring-singleComponentEditor.services',
+  'corespring.wiggi-wiz',
+  'ui.ace',
+  'ui.bootstrap.tabs',
+  'wiggi-wiz.features.core'
 ])

modules/container-client/src/js/editing/controllers/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.controllers', [ ]);

modules/container-client/src/js/editor/controllers/client-side-preview.js → modules/container-client/src/js/editing/controllers/client-side-preview.js

@@ -1,18 +1,19 @@
-angular.module('corespring-editor.controllers')
+angular.module('corespring-editing.controllers')
     .controller('ClientSidePreview', [
     '$log',
     '$scope',
-    'ComponentData',
     'ClientSidePlayerService',
-    'STATIC_PATHS',
+    'ComponentData',
     'EDITOR_EVENTS',
+    'STATIC_PATHS',
     function ClientSidePreview(
       $log,
       $scope,
+      ClientSidePlayerServiceDef,
       ComponentData,
-      ClientSidePlayerServiceDef, 
-      STATIC_PATHS,
-      EDITOR_EVENTS) {
+      EDITOR_EVENTS,
+      STATIC_PATHS
+    ) {
 
     $scope.playerLabelImg = STATIC_PATHS.assets + '/item-player-label.png';
 
@@ -114,6 +115,3 @@ angular.module('corespring-editor.controllers')
 
 }]);
 
-angular.module('corespring-editor.controllers').constant('EDITOR_EVENTS', {
-    CONTENT_ADDED_TO_EDITOR: 'content.added.to.editor'
-});

modules/container-client/src/js/editing/controllers/editor-events.js

@@ -0,0 +1,3 @@
+angular.module('corespring-editing.controllers').constant('EDITOR_EVENTS', {
+  CONTENT_ADDED_TO_EDITOR: 'content.added.to.editor'
+});

modules/container-client/src/js/editing/directives/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.directives', [ ]);

modules/container-client/src/js/editor/question/directives/preview-player-control-panel.js → modules/container-client/src/js/editing/directives/preview-player-control-panel.js

@@ -1,5 +1,6 @@
-angular.module('corespring-editor.directives')
-  .directive('previewPlayerControlPanel', ['$log',
+angular.module('corespring-editing.directives')
+  .directive('previewPlayerControlPanel', [
+    '$log',
     function($log){
 
       function link($scope, $element, $attrs){
@@ -29,7 +30,7 @@ angular.module('corespring-editor.directives')
         mode: '=',
         score: '='
       },
-      templateUrl: '/editor/question/directives/preview-player-control-panel.html'
+      templateUrl: '/editing/directives/preview-player-control-panel.html'
     };
   }
 ]);

modules/container-client/src/js/editor/question/directives/preview-player.js → modules/container-client/src/js/editing/directives/preview-player.js

@@ -2,7 +2,7 @@
   /**
    * Non optimized preview player for the editor
    */
-  angular.module('corespring-editor.directives')
+  angular.module('corespring-editing.directives')
     .directive('previewPlayer', ['$log', 'ComponentRegister', 'ComponentData',
       function($log, ComponentRegister, ComponentData) {
 

modules/container-client/src/js/editing/services/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.services', [ ]);

modules/container-client/src/js/editor/services/component-data.js → modules/container-client/src/js/editing/services/component-data.js

@@ -1,4 +1,4 @@
-angular.module('corespring-editor.services')
+angular.module('corespring-editing.services')
   .service('ComponentData', [
     '$timeout',
     'LogFactory',

modules/container-client/src/js/editor/services/component-default-data.js → modules/container-client/src/js/editing/services/component-default-data.js

@@ -1,4 +1,4 @@
-angular.module('corespring-editor.services')
+angular.module('corespring-editing.services')
   .service('ComponentDefaultData', [
     'LogFactory',
     function(LogFactory) {

modules/container-client/src/js/editor/services/editor-dialog-template.js → modules/container-client/src/js/editing/services/editor-dialog-template.js

@@ -1,4 +1,4 @@
-angular.module('corespring-editor.services')
+angular.module('corespring-editing.services')
   .service('EditorDialogTemplate', [function() {
 
     function EditorDialogTemplate() {

modules/container-client/src/js/editing/wiggi-wiz-features/_declaration.js

@@ -0,0 +1,6 @@
+angular.module('corespring-editing.wiggi-wiz-features', [
+  'corespring-editing.wiggi-wiz-features.cs-image',
+  'corespring-editing.wiggi-wiz-features.footnotes',
+  'corespring-editing.wiggi-wiz-features.link',
+  'corespring-editing.wiggi-wiz-features.mathjax'
+  ]);

modules/container-client/src/js/editing/wiggi-wiz-features/csimage/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.wiggi-wiz-features.cs-image', []);

modules/container-client/src/js/editor/question/wiggi-wiz-features/csimage/feature.js → modules/container-client/src/js/editing/wiggi-wiz-features/csimage/feature.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.cs-image').factory('ImageFeature', [
+angular.module('corespring-editing.wiggi-wiz-features.cs-image').factory('ImageFeature', [
   'Image',
   'TemplateUtils',
   function(ImageDef, TemplateUtils) {

modules/container-client/src/js/editor/question/wiggi-wiz-features/csimage/image-holder.js → modules/container-client/src/js/editing/wiggi-wiz-features/csimage/image-holder.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.cs-image').directive('imageHolder', ['$log', 'ImageFeature',
+angular.module('corespring-editing.wiggi-wiz-features.cs-image').directive('imageHolder', ['$log', 'ImageFeature',
 
   function($log, ImageFeature) {
 

modules/container-client/src/js/editing/wiggi-wiz-features/footnotes/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.wiggi-wiz-features.footnotes', []);

modules/container-client/src/js/editor/question/wiggi-wiz-features/footnotes/feature.js → modules/container-client/src/js/editing/wiggi-wiz-features/footnotes/feature.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.footnotes').factory('WiggiFootnotesFeatureDef', [
+angular.module('corespring-editing.wiggi-wiz-features.footnotes').factory('WiggiFootnotesFeatureDef', [
   'LogFactory',
   function(LogFactory) {
 

modules/container-client/src/js/editor/question/wiggi-wiz-features/footnotes/footnotes-dialog.js → modules/container-client/src/js/editing/wiggi-wiz-features/footnotes/footnotes-dialog.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.footnotes').directive('footnotesDialog', [
+angular.module('corespring-editing.wiggi-wiz-features.footnotes').directive('footnotesDialog', [
   'LogFactory',
   function(LogFactory) {
 

modules/container-client/src/js/editor/question/wiggi-wiz-features/footnotes/footnotes-holder.js → modules/container-client/src/js/editing/wiggi-wiz-features/footnotes/footnotes-holder.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.footnotes').directive('footnotesHolder', [
+angular.module('corespring-editing.wiggi-wiz-features.footnotes').directive('footnotesHolder', [
   'LogFactory',
   function(LogFactory) {
 

modules/container-client/src/js/editing/wiggi-wiz-features/link/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.wiggi-wiz-features.link', []);

modules/container-client/src/js/editor/question/wiggi-wiz-features/link/feature.js → modules/container-client/src/js/editing/wiggi-wiz-features/link/feature.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.link').factory('WiggiLinkFeatureDef', [ '$compile', '$timeout',
+angular.module('corespring-editing.wiggi-wiz-features.link').factory('WiggiLinkFeatureDef', [ '$compile', '$timeout',
 
   function($compile, $timeout) {
 

modules/container-client/src/js/editor/question/wiggi-wiz-features/link/link-holder.js → modules/container-client/src/js/editing/wiggi-wiz-features/link/link-holder.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.link').directive('linkHolder', ['$log',
+angular.module('corespring-editing.wiggi-wiz-features.link').directive('linkHolder', ['$log',
 
   function($log) {
 

modules/container-client/src/js/editing/wiggi-wiz-features/mathjax/_declaration.js

@@ -0,0 +1 @@
+angular.module('corespring-editing.wiggi-wiz-features.mathjax', []);

modules/container-client/src/js/editor/question/wiggi-wiz-features/mathjax/feature.js → modules/container-client/src/js/editing/wiggi-wiz-features/mathjax/feature.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.mathjax').factory('WiggiMathJaxFeatureDef', [
+angular.module('corespring-editing.wiggi-wiz-features.mathjax').factory('WiggiMathJaxFeatureDef', [
   'MathJaxService',
   'MathFormatUtils',
   '$rootScope', 

modules/container-client/src/js/editor/question/wiggi-wiz-features/mathjax/mathformat-utils.js → modules/container-client/src/js/editing/wiggi-wiz-features/mathjax/mathformat-utils.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.mathjax').service('MathFormatUtils', ['$log',
+angular.module('corespring-editing.wiggi-wiz-features.mathjax').service('MathFormatUtils', ['$log',
   function($log) {
 
     function MathFormatUtils() {

modules/container-client/src/js/editor/question/wiggi-wiz-features/mathjax/mathjax-dialog.js → modules/container-client/src/js/editing/wiggi-wiz-features/mathjax/mathjax-dialog.js

@@ -1,5 +1,5 @@
 /* global MathJax */
-angular.module('corespring.wiggi-wiz-features.mathjax').directive('mathjaxDialog', [
+angular.module('corespring-editing.wiggi-wiz-features.mathjax').directive('mathjaxDialog', [
   '$log',
   '$timeout',
   'MathFormatUtils',
@@ -84,7 +84,7 @@ angular.module('corespring.wiggi-wiz-features.mathjax').directive('mathjaxDialog
       link: link,
       require: 'ngModel',
       replace: true,
-      templateUrl: '/editor/question/wiggi-wiz-features/mathjax/mathjax-dialog.html',
+      templateUrl: '/editing/wiggi-wiz-features/mathjax/mathjax-dialog.html',
     };
   }
 ]);

modules/container-client/src/js/editor/question/wiggi-wiz-features/mathjax/mathjax-holder.js → modules/container-client/src/js/editing/wiggi-wiz-features/mathjax/mathjax-holder.js

@@ -1,4 +1,4 @@
-angular.module('corespring.wiggi-wiz-features.mathjax')
+angular.module('corespring-editing.wiggi-wiz-features.mathjax')
 .directive('mathjaxHolder', 
   ['$log',
   '$templateCache',
@@ -17,7 +17,7 @@ angular.module('corespring.wiggi-wiz-features.mathjax')
     function compile($element) {
       html = $element.html();
       $element.addClass('mathjax-holder');
-      var template = $templateCache.get('/editor/question/wiggi-wiz-features/mathjax/mathjax-holder.html');
+      var template = $templateCache.get('/editing/wiggi-wiz-features/mathjax/mathjax-holder.html');
       $element.html(template);
       return link;
     }

modules/container-client/src/js/editor/controllers/_declaration.js

@@ -1,5 +1,5 @@
 angular.module('corespring-editor.controllers', [
   'wiggi-wiz.services',
-  'corespring.wiggi-wiz-features',
+  'corespring-editing.wiggi-wiz-features',
   'corespring-common.directives'
 ]);

modules/container-client/test/js/editor/controllers/client-side-preview-test.js → modules/container-client/test/js/editing/controllers/client-side-preview-test.js

@@ -28,7 +28,7 @@ describe('ClientSidePreview', function() {
     ComponentData.reset.calls.reset();
   });
 
-  beforeEach(angular.mock.module('corespring-editor.controllers'));
+  beforeEach(angular.mock.module('corespring-editing.controllers'));
 
   beforeEach(module(function($provide) {
     $provide.value('ComponentData', ComponentData);

modules/container-client/test/js/editor/services/component-data-test.js → modules/container-client/test/js/editing/services/component-data-test.js

@@ -46,7 +46,7 @@ describe('component data', function() {
 
   beforeEach(angular.mock.module('corespring-player.services'));
   beforeEach(angular.mock.module('corespring-common.services'));
-  beforeEach(angular.mock.module('corespring-editor.services'));
+  beforeEach(angular.mock.module('corespring-editing.services'));
 
   beforeEach(module(function($provide){
     $provide.value('$timeout', function(fn){