[Proposal] Public API proposal for context

Provides context for registration and other proposals we have in the pipeline.

Intended to be a starting point, is non-exhaustive.

.jscsrc

@@ -1,5 +1,3 @@
 {
-    "preset": "crockford",
-    "requireMultipleVarDecl": false,
-    "requireVarDeclFirst": false
+    "preset": "crockford"
 }

.jshintrc

@@ -1,23 +1,4 @@
 {
-    "bitwise": true,
-    "browser": true,
-    "curly": true,
-    "eqeqeq": true,
-    "forin": true,
-    "freeze": true,
-    "funcscope": false,
-    "futurehostile": true,
-    "latedef": true,
-    "noarg": true,
-    "nocomma": true,
-    "nonbsp": true,
-    "nonew": true,
-    "predef": [
-        "define",
-        "Promise"
-    ],
-    "shadow": "outer",
-    "strict": "implied",
-    "undef": true,
-    "unused": "vars"
+    "validthis": true,
+    "laxbreak": true
 }

CONTRIBUTING.md

@@ -1,6 +1,6 @@
-# Contributing to Open MCT
+# Contributing to Open MCT Web
 
-This document describes the process of contributing to Open MCT as well
+This document describes the process of contributing to Open MCT Web as well
 as the standards that will be applied when evaluating contributions.
 
 Please be aware that additional agreements will be necessary before we can
@@ -21,9 +21,9 @@ The short version:
 
 ## Contribution Process
 
-Open MCT uses git for software version control, and for branching and
+Open MCT Web uses git for software version control, and for branching and
 merging. The central repository is at
-https://github.com/nasa/openmct.git.
+https://github.com/nasa/openmctweb.git.
 
 ### Roles
 
@@ -116,18 +116,18 @@ the merge back to the master branch.
 
 ## Standards
 
-Contributions to Open MCT are expected to meet the following standards.
+Contributions to Open MCT Web are expected to meet the following standards.
 In addition, reviewers should use general discretion before accepting
 changes.
 
 ### Code Standards
 
-JavaScript sources in Open MCT must satisfy JSLint under its default
+JavaScript sources in Open MCT Web must satisfy JSLint under its default
 settings. This is verified by the command line build.
 
 #### Code Guidelines
 
-JavaScript sources in Open MCT should:
+JavaScript sources in Open MCT Web should:
 
 * Use four spaces for indentation. Tabs should not be used.
 * Include JSDoc for any exposed API (e.g. public methods, constructors.)
@@ -159,7 +159,7 @@ JavaScript sources in Open MCT should:
   * Third, imperative statements.
   * Finally, the returned value.
 
-Deviations from Open MCT code style guidelines require two-party agreement,
+Deviations from Open MCT Web code style guidelines require two-party agreement,
 typically from the author of the change and its reviewer.
 
 #### Code Example
@@ -260,7 +260,7 @@ these standards.
 
 ## Issue Reporting
 
-Issues are tracked at https://github.com/nasa/openmct/issues
+Issues are tracked at https://github.com/nasa/openmctweb/issues
 
 Issues should include:
 

README.md

@@ -1,14 +1,14 @@
-# Open MCT [![license](https://img.shields.io/badge/license-Apache%202.0-blue.svg)](http://www.apache.org/licenses/LICENSE-2.0)
+# Open MCT Web
 
-Open MCT is a web-based platform for mission operations user interface
+Open MCT Web is a web-based platform for mission operations user interface
 software.
 
 ## Bundles
 
 A bundle is a group of software components (including source code, declared
 as AMD modules, as well as resources such as images and HTML templates)
-that is intended to be added or removed as a single unit. A plug-in for
-Open MCT will be expressed as a bundle; platform components are also
+that are intended to be added or removed as a single unit. A plug-in for
+Open MCT Web will be expressed as a bundle; platform components are also
 expressed as bundles.
 
 A bundle is also just a directory which contains a file `bundle.json`,
@@ -16,7 +16,7 @@ which declares its contents.
 
 The file `bundles.json` (note the plural), at the top level of the
 repository, is a JSON file containing an array of all bundles (expressed as
-directory names) to include in a running instance of Open MCT. Adding or
+directory names) to include in a running instance of Open MCT Web. Adding or
 removing paths from this list will add or remove bundles from the running
 application.
 
@@ -56,7 +56,7 @@ To run:
 
 ## Build
 
-Open MCT is built using [`npm`](http://npmjs.com/)
+Open MCT Web is built using [`npm`](http://npmjs.com/)
 and [`gulp`](http://gulpjs.com/).
 
 To build:
@@ -64,18 +64,18 @@ To build:
 `npm run prepublish`
 
 This will compile and minify JavaScript sources, as well as copy over assets.
-The contents of the `dist` folder will contain a runnable Open MCT
+The contents of the `dist` folder will contain a runnable Open MCT Web
 instance (e.g. by starting an HTTP server in that directory), including:
 
-* A `main.js` file containing Open MCT source code.
+* A `main.js` file containing Open MCT Web source code.
 * Various assets in the `example` and `platform` directories.
-* An `index.html` that runs Open MCT in its default configuration.
+* An `index.html` that runs Open MCT Web in its default configuration.
 
 Additional `gulp` tasks are defined in [the gulpfile](gulpfile.js).
 
 ### Building Documentation
 
-Open MCT's documentation is generated by an
+Open MCT Web's documentation is generated by an
 [npm](https://www.npmjs.com/)-based build.  It has additional dependencies that
 may not be available on every platform and thus is not covered in the standard
 npm install.  Ensure your system has [libcairo](http://cairographics.org/)
@@ -89,7 +89,7 @@ Documentation will be generated in `target/docs`.
 
 # Glossary
 
-Certain terms are used throughout Open MCT with consistent meanings
+Certain terms are used throughout Open MCT Web with consistent meanings
 or conventions. Any deviations from the below are issues and should be
 addressed (either by updating this glossary or changing code to reflect
 correct usage.) Other developer documentation, particularly in-line
@@ -112,7 +112,7 @@ documentation, may presume an understanding of these terms.
   (Most often used in the context of extensions, domain
   object models, or other similar application-specific objects.)
 * _domain object_: A meaningful object to the user; a distinct thing in
-  the work support by Open MCT. Anything that appears in the left-hand
+  the work support by Open MCT Web. Anything that appears in the left-hand
   tree is a domain object.
 * _extension_: An extension is a unit of functionality exposed to the
   platform in a declarative fashion by a bundle. For more
@@ -133,6 +133,6 @@ documentation, may presume an understanding of these terms.
   it, and it is thereafter considered the _navigated_ object (until the
   user makes another such choice.)
 * _space_: A name used to identify a persistence store. Interactions with
-  persistence will generally involve a `space` parameter in some form, to
+  persistence with generally involve a `space` parameter in some form, to
   distinguish multiple persistence stores from one another (for cases
   where there are multiple valid persistence locations available.)

app.js

@@ -19,7 +19,6 @@
 
     // Defaults
     options.port = options.port || options.p || 8080;
-    options.directory = options.directory || options.D || '.';
     ['include', 'exclude', 'i', 'x'].forEach(function (opt) {
         options[opt] = options[opt] || [];
         // Make sure includes/excludes always end up as arrays
@@ -37,7 +36,6 @@
         console.log("  --port, -p <number>      Specify port.");
         console.log("  --include, -i <bundle>   Include the specified bundle.");
         console.log("  --exclude, -x <bundle>   Exclude the specified bundle.");
-        console.log("  --directory, -D <bundle>   Serve files from specified directory.");
         console.log("");
         process.exit(0);
     }
@@ -73,10 +71,8 @@
     });
 
     // Expose everything else as static files
-    app.use(express['static'](options.directory));
+    app.use(express['static']('.'));
 
-    // Finally, open the HTTP server and log the instance to the console
-    app.listen(options.port, function() {
-        console.log('Open MCT application running at localhost:' + options.port)
-    });
-}());
+    // Finally, open the HTTP server
+    app.listen(options.port);
+}());

bower.json

@@ -1,10 +1,10 @@
 {
-  "name": "openmct",
-  "description": "The Open MCT core platform",
+  "name": "openmctweb",
+  "description": "The OpenMCTWeb core platform",
   "main": "",
   "license": "Apache-2.0",
   "moduleType": [],
-  "homepage": "http://nasa.github.io/openmct/",
+  "homepage": "http://nasa.github.io/openmctweb/",
   "private": true,
   "dependencies": {
     "angular": "1.4.4",
@@ -18,8 +18,6 @@
     "node-uuid": "^1.4.7",
     "comma-separated-values": "^3.6.4",
     "FileSaver.js": "^0.0.2",
-    "zepto": "^1.1.6",
-    "eventemitter3": "^1.2.0",
-    "d3": "~4.1.0"
+    "zepto": "^1.1.6"
   }
 }

build-docs.sh

@@ -22,19 +22,17 @@
 #* at runtime from the About dialog for additional information.
 #*****************************************************************************
 
-# Script to build and deploy docs.
+# Script to build and deploy docs to github pages.
 
 OUTPUT_DIRECTORY="target/docs"
-# Docs, once built, are pushed to the private website repo
-REPOSITORY_URL="git@github.com:nasa/openmct-website.git"
-WEBSITE_DIRECTORY="website"
+REPOSITORY_URL="git@github.com:nasa/openmctweb.git"
 
-BUILD_SHA=`git rev-parse HEAD`
+BUILD_SHA=`git rev-parse head`
 
 # A remote will be created for the git repository we are pushing to.
 # Don't worry, as this entire directory will get trashed inbetween builds.
 REMOTE_NAME="documentation"
-WEBSITE_BRANCH="master"
+WEBSITE_BRANCH="gh-pages"
 
 # Clean output directory, JSDOC will recreate
 if [ -d $OUTPUT_DIRECTORY ]; then
@@ -42,21 +40,23 @@ if [ -d $OUTPUT_DIRECTORY ]; then
 fi
 
 npm run docs
+cd $OUTPUT_DIRECTORY || exit 1
 
-echo "git clone $REPOSITORY_URL website"
-git clone $REPOSITORY_URL website || exit 1
-echo "cp -r $OUTPUT_DIRECTORY $WEBSITE_DIRECTORY/docs"
-cp -r $OUTPUT_DIRECTORY $WEBSITE_DIRECTORY/docs
-echo "cd $WEBSITE_DIRECTORY"
-cd $WEBSITE_DIRECTORY || exit 1
+echo "git init"
+git init
 
 # Configure github for CircleCI user.
 git config user.email "buildbot@circleci.com"
 git config user.name "BuildBot"
 
+echo "git remote add $REMOTE_NAME $REPOSITORY_URL"
+git remote add $REMOTE_NAME $REPOSITORY_URL
 echo "git add ."
 git add .
-echo "git commit -m \"Docs updated from build $BUILD_SHA\""
-git commit -m "Docs updated from build $BUILD_SHA"
-# Push to the website repo
-git push
+echo "git commit -m \"Generate docs from build $BUILD_SHA\""
+git commit -m "Generate docs from build $BUILD_SHA"
+
+echo "git push $REMOTE_NAME HEAD:$WEBSITE_BRANCH -f"
+git push $REMOTE_NAME HEAD:$WEBSITE_BRANCH -f
+
+echo "Documentation pushed to gh-pages branch."

circle.yml

@@ -2,24 +2,17 @@ deployment:
   production:
     branch: master
     commands:
-        - npm install canvas nomnoml
-        - ./build-docs.sh
-        - git fetch --unshallow
-        - git push git@heroku.com:openmctweb-demo.git $CIRCLE_SHA1:refs/heads/master
-  openmct-demo:
-    branch: live_demo
+      - npm install canvas nomnoml
+      - ./build-docs.sh
+      - git push git@heroku.com:openmctweb-demo.git $CIRCLE_SHA1:refs/heads/master
+  openmctweb-staging-un:
+    branch: nem_prototype
     heroku:
-      appname: openmct-demo
+      appname: openmctweb-staging-un
   openmctweb-staging-deux:
     branch: mobile
     heroku:
       appname: openmctweb-staging-deux
 test:
   post:
-    - gulp lint
-    - gulp checkstyle
-
-general:
-  branches:
-    ignore:
-      - gh-pages
+    - npm run jshint --silent

docs/footer.html

@@ -1,3 +1,9 @@
         <hr>
+        <cite>
+            This document is styled using
+            <a href="https://github.com/jasonm23/markdown-css-themes">
+                https://github.com/jasonm23/markdown-css-themes
+            </a>.
+        </cite>
     </body>
 </html>

docs/header.html

@@ -1,9 +1,7 @@
 <html>
     <head>
         <link rel="stylesheet"
-              href="//nasa.github.io/openmct/static/res/css/styles.css">
-        <link rel="stylesheet"
-              href="//nasa.github.io/openmct/static/res/css/documentation.css">
+              href="http://jasonm23.github.io/markdown-css-themes/avenir-white.css">
     </head>
     <body>
 

docs/src/architecture/framework.md

@@ -5,7 +5,7 @@ software components to communicate. The software components it recognizes
 are:
 
 * _Extensions_: Individual units of functionality that can be added to
-  or removed from Open MCT. _Extension categories_ distinguish what
+  or removed from Open MCT Web. _Extension categories_ distinguish what
   type of functionality is being added/removed.
 * _Bundles_: A grouping of related extensions
   (named after an analogous concept from [OSGi](http://www.osgi.org/))
@@ -19,7 +19,7 @@ manner which the framework layer can understand.
 
 ```nomnoml
 #direction: down
-[Open MCT|
+[Open MCT Web|
   [Dependency injection framework]-->[Platform bundle #1]
   [Dependency injection framework]-->[Platform bundle #2]
   [Dependency injection framework]-->[Plugin bundle #1]
@@ -35,7 +35,7 @@ manner which the framework layer can understand.
 ```
 
 The "dependency injection framework" in this case is
-[AngularJS](https://angularjs.org/). Open MCT's framework layer
+[AngularJS](https://angularjs.org/). Open MCT Web's framework layer
 is really just a thin wrapper over Angular that recognizes the
 concepts of bundles and extensions (as declared in JSON files) and
 registering extensions with Angular. It additionally acts as a
@@ -60,7 +60,7 @@ activities which were performed by the framework component.
 
 ## Application Initialization
 
-The framework component initializes an Open MCT application following
+The framework component initializes an Open MCT Web application following
 a simple sequence of steps.
 
 ```nomnoml
@@ -97,7 +97,7 @@ a simple sequence of steps.
 [Extension]o->[Dependency #3]
 ```
 
-Open MCT's architecture relies on a simple premise: Individual units
+Open MCT Web's architecture relies on a simple premise: Individual units
 (extensions) only have access to the dependencies they declare that they
 need, and they acquire references to these dependencies via dependency
 injection. This has several desirable traits:
@@ -121,11 +121,11 @@ injection. This has several desirable traits:
   the framework.
 
 A drawback to this approach is that it makes it difficult to define
-"the architecture" of Open MCT, in terms of describing the specific
+"the architecture" of Open MCT Web, in terms of describing the specific
 units that interact at run-time. The run-time architecture is determined
 by the framework as the consequence of wiring together dependencies.
 As such, the specific architecture of any given application built on
-Open MCT can look very different.
+Open MCT Web can look very different.
 
 Keeping that in mind, there are a few useful patterns supported by the
 framework that are useful to keep in mind.
@@ -229,4 +229,4 @@ otherwise a single provider) will be exposed as a single service that
 other extensions can acquire through dependency injection. Because all
 components of the same type of service expose the same interface, users
 of that service do not need to be aware that they are talking to an
-aggregator or a provider, for instance.
+aggregator or a provider, for instance.

docs/src/architecture/index.md

@@ -1,14 +1,14 @@
 # Introduction
 
 The purpose of this document is to familiarize developers with the
-overall architecture of Open MCT.
+overall architecture of Open MCT Web.
 
 The target audience includes:
 
 * _Platform maintainers_: Individuals involved in developing,
   extending, and maintaing capabilities of the platform.
 * _Integration developers_: Individuals tasked with integrated
-  Open MCT into a larger system, who need to understand
+  Open MCT Web into a larger system, who need to understand
   its inner workings sufficiently to complete this integration.
 
 As the focus of this document is on architecture, whenever possible
@@ -17,25 +17,25 @@ omitted. These details may be found in the developer guide.
 
 # Overview
 
-Open MCT is client software: It runs in a web browser and
+Open MCT Web is client software: It runs in a web browser and
 provides a user interface, while communicating with various
 server-side resources through browser APIs.
 
 ```nomnoml
 #direction: right
-[Client|[Browser|[Open MCT]->[Browser APIs]]]
+[Client|[Browser|[Open MCT Web]->[Browser APIs]]]
 [Server|[Web services]]
 [Client]<->[Server]
 ```
 
-While Open MCT can be configured to run as a standalone client,
+While Open MCT Web can be configured to run as a standalone client,
 this is rarely very useful. Instead, it is intended to be used as a
 display and interaction layer for information obtained from a
 variety of back-end services. Doing so requires authoring or utilizing
-adapter plugins which allow Open MCT to interact with these services.
+adapter plugins which allow Open MCT Web to interact with these services.
 
 Typically, the pattern here is to provide a known interface that
-Open MCT can utilize, and implement it such that it interacts with
+Open MCT Web can utilize, and implement it such that it interacts with
 whatever back-end provides the relevant information.
 Examples of back-ends that can be utilized in this fashion include
 databases for the persistence of user-created objects, or sources of
@@ -43,13 +43,13 @@ telemetry data.
 
 ## Software Architecture
 
-The simplest overview of Open MCT is to look at it as a "layered"
+The simplest overview of Open MCT Web is to look at it as a "layered"
 architecture, where each layer more clearly specifies the behavior
 of the software.
 
 ```nomnoml
 #direction: down
-[Open MCT|
+[Open MCT Web|
   [Platform]<->[Application]
   [Framework]->[Application]
   [Framework]->[Platform]
@@ -64,14 +64,14 @@ These layers are:
   established an abstraction by which different software components
   may communicate and/or interact.
 * [_Platform_](platform.md): The platform layer defines the general look, 
-  feel, and behavior of Open MCT. This includes user-facing components like
+  feel, and behavior of Open MCT Web. This includes user-facing components like
   Browse mode and Edit mode, as well as underlying elements of the
   information model and the general service infrastructure.
 * _Application_: The application layer defines specific features of
-  an application built on Open MCT. This includes adapters to
+  an application built on Open MCT Web. This includes adapters to
   specific back-ends, new types of things for users to create, and
   new ways of visualizing objects within the system. This layer
-  typically consists of a mix of custom plug-ins to Open MCT,
+  typically consists of a mix of custom plug-ins to Open MCT Web,
   as well as optional features (such as Plot view) included alongside
   the platform.
 

docs/src/architecture/platform.md

@@ -1,6 +1,6 @@
 # Overview
 
-The Open MCT platform utilizes the [framework layer](Framework.md)
+The Open MCT Web platform utilizes the [framework layer](Framework.md)
 to provide an extensible baseline for applications which includes:
 
 * A common user interface (and user interface paradigm) for dealing with
@@ -16,7 +16,7 @@ building application, the platform adds more specificity by defining
 additional extension types and allowing for integration with back end
 components.
 
-The run-time architecture of an Open MCT application can be categorized
+The run-time architecture of an Open MCT Web application can be categorized
 into certain high-level tiers:
 
 ```nomnoml
@@ -29,7 +29,7 @@ into certain high-level tiers:
 [Browser APIs]->[Back-end]
 ```
 
-Applications built using Open MCT may add or configure functionality
+Applications built using Open MCT Web may add or configure functionality
 in __any of these tiers__.
 
 * _DOM_: The rendered HTML document, composed from HTML templates which
@@ -60,7 +60,7 @@ in __any of these tiers__.
   functionality needed to support the information model. This includes
   exposing underlying sets of extensions and mediating with the
   back-end.
-* _Back-end_: The back-end is out of the scope of Open MCT, except
+* _Back-end_: The back-end is out of the scope of Open MCT Web, except
   for the interfaces which are utilized by adapters participating in the
   service infrastructure. Includes the underlying persistence stores, telemetry 
   streams, and so forth which the Open MCT Web client is being used to interact 
@@ -70,15 +70,15 @@ in __any of these tiers__.
 
 Once the
 [application has been initialized](Framework.md#application-initialization)
-Open MCT primarily operates in an event-driven paradigm; various
+Open MCT Web primarily operates in an event-driven paradigm; various
 events (mouse clicks, timers firing, receiving responses to XHRs) trigger
 the invocation of functions, typically in the presentation layer for
 user actions or in the service infrastructure for server responses.
 
-The "main point of entry" into an initialized Open MCT application
+The "main point of entry" into an initialized Open MCT Web application
 is effectively the
 [route](https://docs.angularjs.org/api/ngRoute/service/$route#example)
-which is associated with the URL used to access Open MCT (or a
+which is associated with the URL used to access Open MCT Web (or a
 default route.) This route will be associated with a template which
 will be displayed; this template will include references to directives
 and controllers which will be interpreted by Angular and used to
@@ -107,11 +107,11 @@ both the information model and the service infrastructure.
 
 # Presentation Layer
 
-The presentation layer of Open MCT is responsible for providing
+The presentation layer of Open MCT Web is responsible for providing
 information to display within templates, and for handling interactions
 which are initiated from templated DOM elements. AngularJS acts as
 an intermediary between the web page as the user sees it, and the
-presentation layer implemented as Open MCT extensions.
+presentation layer implemented as Open MCT Web extensions.
 
 ```nomnoml
 [Presentation Layer|
@@ -143,12 +143,12 @@ to primitives from AngularJS:
   attributes and tags.
 * [_Routes_](https://docs.angularjs.org/api/ngRoute/service/$route#example)
   are used to associate specific URLs (including the fragment identifier)
-  with specific application states. (In Open MCT, these are used to
+  with specific application states. (In Open MCT Web, these are used to
   describe the mode of usage - e.g. browse or edit - as well as to
   identify the object being used.)
 * [_Templates_](https://docs.angularjs.org/guide/templates) are partial
   HTML documents that will be rendered and kept up-to-date by AngularJS.
-  Open MCT introduces a custom `mct-include` directive which acts
+  Open MCT Web introduces a custom `mct-include` directive which acts
   as a wrapper around `ng-include` to allow templates to be referred
   to by symbolic names.
 
@@ -189,10 +189,10 @@ to displaying domain objects.
 ]
 ```
 
-Domain objects are the most fundamental component of Open MCT's
+Domain objects are the most fundamental component of Open MCT Web's
 information model. A domain object is some distinct thing relevant to a
 user's work flow, such as a telemetry channel, display, or similar.
-Open MCT is a tool for viewing, browsing, manipulating, and otherwise
+Open MCT Web is a tool for viewing, browsing, manipulating, and otherwise
 interacting with a graph of domain objects.
 
 A domain object should be conceived of as the union of the following:
@@ -254,7 +254,7 @@ Concrete examples of capabilities which follow this pattern
 
 # Service Infrastructure
 
-Most services exposed by the Open MCT platform follow the
+Most services exposed by the Open MCT Web platform follow the
 [composite services](Framework.md#composite-services) to permit
 a higher degree of flexibility in how a service can be modified
 or customized for specific applications.
@@ -327,7 +327,7 @@ A short summary of the roles of these services:
 [DomainObjectProvider]o-[CapabilityService]
 ```
 
-As domain objects are central to Open MCT's information model,
+As domain objects are central to Open MCT Web's information model,
 acquiring domain objects is equally important.
 
 ```nomnoml
@@ -338,7 +338,7 @@ acquiring domain objects is equally important.
 [<state> Instantiate DomainObject]->[<end> End]
 ```
 
-Open MCT includes an implementation of an `ObjectService` which
+Open MCT Web includes an implementation of an `ObjectService` which
 satisfies this capability by:
 
 * Consulting the [Model Service](#model-service) to acquire domain object
@@ -437,9 +437,9 @@ objects (this allows failures to be recognized and handled in groups.)
 The telemetry service is responsible for acquiring telemetry data.
 
 Notably, the platform does not include any providers for
-`TelemetryService`; applications built on Open MCT will need to
+`TelemetryService`; applications built on Open MCT Web will need to
 implement a provider for this service if they wish to expose telemetry
-data. This is usually the most important step for integrating Open MCT
+data. This is usually the most important step for integrating Open MCT Web
 into an existing telemetry system.
 
 Requests for telemetry data are usually initiated in the
@@ -721,6 +721,6 @@ disallow.
 ```
 
 The type service provides metadata about the different types of domain
-objects that exist within an Open MCT application. The platform
+objects that exist within an Open MCT Web application. The platform
 implementation reads these types in from extension category `types`
-and wraps them in a JavaScript interface.
+and wraps them in a JavaScript interface.

docs/src/design/proposals/PublicAPIProposal.md

@@ -0,0 +1,180 @@
+# Public API
+
+We need to focus on the API as a whole, so that we can design and refactor it before 1.0.  We also need to figure out what is stable and work to implementing it in a straightforward fashion.  Ideally, we can create an abstraction between the public API and the implementation to allow us to refactor the underlying implementation without breaking the API contract.
+
+Conceptual weight is a big risk, so I advocate for focusing our API on specific concepts-- telemetry, views, domain objects, actions -- and not intermediate abstract concepts like composite providers, type registeres, etc.  We will reuse patterns in our system so as to reward users who are getting deep, but we should not force them to learn the patterns first.
+
+At the same time, I think we should formalize a basic plugin API that allows for code to be bundled and installed in the platform.  Plugins are allowed to use any part of the public API to implement their features, and can register extensions at any of the documented extension points.
+
+With the introduction of a plugin API, we should also start reorganizing platform code.  I think we should organize code into two categories:
+
+* the core platform, which provides implements the Public API and takes care of a lot of "behind the scenes" functionality.
+* plugins-- many of which have previously been packaged as part of the core api.  This includes examples, our various view types, our persistence adapters, etc.
+
+We don't have to split the repository, but we should move plugins to their own set of subfolders.  It would be beneficial to start thinking about how plugins are versioned and distributed so that we can start developing stories for plugin reuse across missions.
+
+Summary:
+1. The Platform should have a stable, clear, and easy to use Public API, with specific extension points.
+2. The Platform can implement this Public API however it would like, so long as good development practices are followed.
+3. The Platform will expose a simple method for bundling functionality into Plugins, which can be installed, uninstalled, and shared.  However, these plugins may only use the Public API exposed by the Platform.
+
+
+## Proposal: Plugin API
+
+Plugins are very simple:  They are a function which is invoked with a single object, an instance of the MCT application.  They can directly attach functionality using the MCT public API, and they can observe the `start` event to execute code when the application starts.
+
+Here's a pseudo plugin:
+
+```javascript
+var GenericSeachIndexer = function (mct) {
+    var worker = new SearchWorker(mct);
+    var provider = new GenericSearchProvider(mct, worker);
+    mct.search.registerProvider(provider);
+    mct.on('start', function () {
+        worker.startIndexing();
+    });
+};
+
+mct.install(GenericSearchIndexer);
+mct.start();
+```
+
+Note that nothing prevents using the Public API without going through the plugin interface-- this is not bad!  Allowing developers to directly attach functionality without writing a plugin is a great hello-world tutorial.  Plugins can come later.
+
+## Proposal: Internal Architecture
+
+In the post-angular world, we should look to requirejs, browserify, or es6 modules as our primary method of separating concerns.
+
+One benefit of these module systems is that they treat modules as singletons by default, so multiple pieces of code which depend upon a module will receive the same instance of that module.  This means that modules can depend on other modules, which sometimes makes sense.  Platform developers are expected to use their best judgement to separate concerns.
+
+At the same time, we need a basic framework for taking a distinct set of modules and assembling the public API.
+
+I would propose the following simple application builder framework as a starting point:
+
+```javascript
+
+define('Application', function () {
+    function Application() {
+        this._plugins = [];
+    }
+
+    Application.prototype = Object.create(EventEmitter.prototype);
+
+    Application.prototype.install = function (plugin) {
+        this._plugins.push(plugin);
+    };
+
+    Application.prototype.uninstall = function (plugin) {
+        this._plugins.splice(this._plugins.indexOf(plugin), 1);
+    };
+
+    Application.prototype.start = function () {
+        this._plugins.forEach(function (plugin) {
+            plugin(this);
+        }, this);
+        this.emit('before:start');
+        this.emit('start');
+    };
+    
+    return Application;
+});
+
+define('MCT', [
+    'Application',
+    'modules/persistence',
+    'modules/objects' // etc
+], function (
+    Application,
+    persistenceModule,
+    objectModule
+) { 
+    var mct = new Application();
+    var modules = [
+        persistenceModule,
+        objectModule
+    ];
+    
+    // modules are functions that are passed a reference to the app.
+    // they attach their public API to the application.
+    modules.forEach(function (module)) {
+        module(app);
+    };
+    
+    // after all modules have registered their public apis, they can initialize
+    // to use public interfaces of other modules.  Potential options:
+    // * setting sensible defaults using app.config
+    // * attaching handlers to app.bus
+    modules.forEach(function (module) {
+        if (module.initialize) {
+            module.initialize(app);
+        }
+    });
+
+    // Calling mct.start will emit a `before:start` event and then a `start` 
+    // event.  
+    // `before:start` is a good time to read finalized config from app.config
+    // `start` is a good time to kick off any runtime function, such as reading
+    // the path from the url and navigating to a page.
+    
+    // at this point, any configuration changes will have been made and 
+    // internal modules should use `app.config` to read these configurations.
+    
+    // It is worth discussing whether app.config can be modified after starting
+    // the application.
+    
+    return mct;
+});
+```
+
+## Proposal: API 1.0
+
+The external API of MCT 1.0 has not been decided, but I would use this opportunity to start proposing what that API would look like so that we can work towards it.
+
+This is not an exhaustive list, it comes off the top of my head.
+
+Generally, if we don't need it, we should remove it.  We can always add to this API, but we cannot remove.
+
+* `MCT.routes`
+    something for determining how navigation is handled.  Need to handle selection of views for specific regions for specific modes.  For instance, what shows in inspector in edit mode.  May not be a public API.
+* `MCT.actions`
+    * `MCT.actions.registerAction(action)`
+    * `MCT.actions.getActions(object)` -> `Action[]`
+    * `MCT.actions.registerPolicy(ActionPolicy)`
+* `MCT.views` -- manages "main view" views.
+    * `MCT.views.registerView('key', view)` is key based registration a good idea?  I'm not sure.
+    * `MCT.views.registerPolicy(viewPolicy)` for controlling which views apply to which object.  could be a method of views.
+* `MCT.telemetry`
+    * `MCT.telemetry.requestTelemetry(object, options)` -> `promise`
+    * `MCT.telemetry.subscribe(object, callback, options)` -> `unsubscribe` function
+    * `MCT.telemetry.registerProvider(provider)`
+    * `MCT.telemetry.registerLimitEvaluator(limitEvaluator)`
+    * `MCT.telemetry.getLimitEvaluator(object)`
+* `MCT.persistence`
+    * `MCT.persistence.registerProvider(provider)`
+    * `MCT.persistence.get('key')`
+    * `MCT.persistence.set('key', obj)`
+    * etc.
+* MCT.objects
+    * `MCT.objects.registerType('key', TypeDefinition)`
+    * `MCT.objects.registerPolicy(compositionPolicy)`
+    * `MCT.objects.registerProvider(objectProvider)` -- would like to combine model/object/persistence some how.
+* MCT.timeService
+    * `MCT.time.registerFormat('key', TimeFormat)`
+    * `MCT.time.getFormat('key')` -> `timeFormat`
+* `MCT.bus` specifically for "optional dependencies" and loose coupling for app-wide concerns.  Using this assumes that you handle the case of a missing dependency.  Also enables app-wide event configuration.
+    * `MCT.bus.register('key', handler)`
+    * `MCT.bus.request('key', [arguments...])` -> `undefined` if no handler, `{...}` if handler.
+    * `MCT.bus.request('conductor.bounds')` -> `{start: ..., end: ...}` or `undefined` if disabled
+    * `MCT.bus.request('conductor.domain')` -> `{key: 'scet', label: 'SCET'}` or `undefined` if disabled
+    * `MCT.bus.on('user.logout', function () {})`
+    * `MCT.bus.on('conductor.bounds.change', reloadTelemetry)`
+    * `MCT.bus.on('mutation', indexModel)`
+* `MCT.config` basic key-value store for app wide config.  Assume that it won't change after application has started.
+    `MCT.config.get('key')` -> `value`
+    `MCT.config.set('key', value)`
+    `MCT.config.set('theme', 'snow')`
+    `MCT.config.get('theme')` -> `'snow'`
+    `MCT.config.get('POLLING_INTERVAL')` -> `30`
+
+
+

docs/src/guide/index.md

@@ -1,4 +1,4 @@
-# Open MCT Developer Guide
+# Open MCT Web Developer Guide
 Victor Woeltjen
 
 [victor.woeltjen@nasa.gov](mailto:victor.woeltjen@nasa.gov)
@@ -6,36 +6,35 @@ Victor Woeltjen
 September 23, 2015  
 Document Version 1.1  
 
-Date                | Version   | Summary of Changes        | Author
-------------------- | --------- | ------------------------- | ---------------
-April 29, 2015      | 0         | Initial Draft             | Victor Woeltjen
-May 12, 2015        | 0.1       |                           | Victor Woeltjen
-June 4, 2015        | 1.0       | Name Changes              | Victor Woeltjen
-October 4, 2015     | 1.1       | Conversion to MarkDown    | Andrew Henry
-April 5, 2016       | 1.2       | Added Mct-table directive | Andrew Henry
+Date                | Version   | Summary of Changes      | Author
+------------------- | --------- | ----------------------- | ---------------
+April 29, 2015      | 0         | Initial Draft           | Victor Woeltjen
+May 12, 2015        | 0.1       |                         | Victor Woeltjen
+June 4, 2015        | 1.0       | Name Changes            | Victor Woeltjen
+October 4, 2015     | 1.1       | Conversion to MarkDown  | Andrew Henry
 
 # Introduction
 The purpose of this guide is to familiarize software developers with the Open
 MCT Web platform.
 
-## What is Open MCT
-Open MCT is a platform for building user interface and display tools,
+## What is Open MCT Web
+Open MCT Web is a platform for building user interface and display tools,
 developed at the NASA Ames Research Center in collaboration with teams at the
 Jet Propulsion Laboratory. It is written in HTML5, CSS3, and JavaScript, using 
 [AngularJS](http://www.angularjs.org) as a framework. Its intended use is to 
 create single-page web applications which integrate data and behavior from a 
 variety of sources and domains.
 
-Open MCT has been developed to support the remote operation of space
+Open MCT Web has been developed to support the remote operation of space
 vehicles, so some of its features are specific to that task; however, it is
 flexible enough to be adapted to a variety of other application domains where a
 display tool oriented toward browsing, composing, and visualizing would be
 useful.
 
-Open MCT provides:
+Open MCT Web provides:
 
 *  A common user interface paradigm which can be applied to a variety of domains
-and tasks. Open MCT is more than a widget toolkit - it provides a standard
+and tasks. Open MCT Web is more than a widget toolkit - it provides a standard
 tree-on-the-left, view-on-the-right browsing environment which you customize by
 adding new browsable object types, visualizations, and back-end adapters.  
 *  A plugin framework and an extensible API for introducing new application
@@ -44,17 +43,17 @@ features of a variety of types.
 visualizations and infrastructure specific to telemetry display.
 
 ## Client-Server Relationship
-Open MCT is client software - it runs entirely in the user's web browser. As
+Open MCT Web is client software - it runs entirely in the user's web browser. As
 such, it is largely 'server agnostic'; any web server capable of serving files
-from paths is capable of providing Open MCT.
+from paths is capable of providing Open MCT Web.
 
-While Open MCT can be configured to run as a standalone client, this is
+While Open MCT Web can be configured to run as a standalone client, this is
 rarely very useful. Instead, it is intended to be used as a display and
 interaction layer for information obtained from a variety of back-end services.
 Doing so requires authoring or utilizing adapter plugins which allow Open MCT
 Web to interact with these services.
 
-Typically, the pattern here is to provide a known interface that Open MCT
+Typically, the pattern here is to provide a known interface that Open MCT Web
 can utilize, and implement it such that it interacts with whatever back-end
 provides the relevant information. Examples of back-ends that can be utilized in
 this fashion include databases for the persistence of user-created objects, or
@@ -63,52 +62,52 @@ sources of telemetry data.
 See the [Architecture Guide](../architecture/index.md#Overview) for information
 on the client-server relationship.
 
-## Developing with Open MCT
-Building applications with Open MCT typically means authoring and utilizing
+## Developing with Open MCT Web
+Building applications with Open MCT Web typically means authoring and utilizing
 a set of plugins which provide application-specific details about how Open MCT
 Web should behave.
 
 ### Technologies
 
-Open MCT sources are written in JavaScript, with a number of configuration
+Open MCT Web sources are written in JavaScript, with a number of configuration
 files written in JSON. Displayable components are written in HTML5 and CSS3.  
-Open MCT is built using [AngularJS](http://www.angularjs.org)  from Google. A
+Open MCT Web is built using [AngularJS](http://www.angularjs.org)  from Google. A
 good understanding of Angular is recommended for developers working with Open
 MCT Web.
 
 ### Forking
-Open MCT does not currently have a single stand-alone artifact that can be
+Open MCT Web does not currently have a single stand-alone artifact that can be
 used as a library. Instead, the recommended approach for creating a new
-application is to start by forking/branching Open MCT, and then adding new
-features from there. Put another way, Open MCT's source structure is built
+application is to start by forking/branching Open MCT Web, and then adding new
+features from there. Put another way, Open MCT Web's source structure is built
 to serve as a template for specific applications.
 
-Forking in this manner should not require that you edit Open MCT's sources.
+Forking in this manner should not require that you edit Open MCT Web's sources.
 The preferred approach is to create a new directory (peer to `index.html`) for
 the new application, then add new bundles (as described in the Framework
 chapter) within that directory.
 
-To initially clone the Open MCT repository:
+To initially clone the Open MCT Web repository:
 `git clone <repository URL> <local repo directory> -b open-master`
 
-To create a fork to begin working on a new application using Open MCT:  
+To create a fork to begin working on a new application using Open MCT Web:  
 
     cd <local repo directory>  
     git checkout open-master
     git checkout -b <new branch name>
 
-As a convention used internally, applications built using Open MCT have
+As a convention used internally, applications built using Open MCT Web have
 master branch names with an identifying prefix. For instance, if building an
 application called 'Foo', the last statement above would look like:
 
     git checkout -b foo-master
 
-This convention is not enforced or understood by Open MCT in any way; it is
+This convention is not enforced or understood by Open MCT Web in any way; it is
 mentioned here as a more general recommendation.
 
 # Overview
 
-Open MCT is implemented as a framework component which manages a set of
+Open MCT Web is implemented as a framework component which manages a set of
 other components. These components, called _bundles_, act as containers to group
 sets of related functionality; individual units of functionality are expressed
 within these bundles as _extensions_.
@@ -119,7 +118,7 @@ run-time to satisfy these declared dependency. This dependency injection
 approach allows software components which have been authored separately (e.g. as
 plugins) but to collaborate at run-time.
 
-Open MCT's framework layer is implemented on top of AngularJS's [dependency
+Open MCT Web's framework layer is implemented on top of AngularJS's [dependency
 injection mechanism](https://docs.angularjs.org/guide/di)  and is modelled after
 [OSGi](hhttp://www.osgi.org/)  and its [Declarative Services component model](http://wiki.osgi.org/wiki/Declarative_Services). 
 In particular, this is where the term _bundle_ comes from.
@@ -134,7 +133,7 @@ The framework is described in more detail in the [Framework Overview](../archite
 architecture guide.
 
 ### Tiers
-While all bundles in a running Open MCT instance are effectively peers, it
+While all bundles in a running Open MCT Web instance are effectively peers, it
 is useful to think of them as a tiered architecture, where each tier adds more
 specificity to the application.
 ```nomnoml
@@ -152,7 +151,7 @@ It additionally interprets bundle definitions (see explanation below, as well as
 further detail in the Framework chapter.) At this tier, we are at our most
 general: We know only that we are a plugin-based application.
 * __Platform__: Components in the Platform tier describe both the general user
-interface and corresponding developer-facing interfaces of Open MCT. This
+interface and corresponding developer-facing interfaces of Open MCT Web. This
 tier provides the general infrastructure for applications. It is less general
 than the framework tier, insofar as this tier introduces a specific user
 interface paradigm, but it is still non-specific as to what useful features
@@ -160,7 +159,7 @@ will be provided. Although they can be removed or replaced easily, bundles
 provided by the Platform tier generally should not be thought of as optional.
 * __Application__: The application tier consists of components which utilize the
 infrastructure provided by the Platform to provide functionality which will (or
-could) be useful to specific applications built using Open MCT. These
+could) be useful to specific applications built using Open MCT Web. These
 include adapters to specific persistence back-ends (such as ElasticSearch or
 CouchDB) as well as bundles which describe more user-facing features (such as
 _Plot_ views for visualizing time series data, or _Layout_ objects for
@@ -169,20 +168,20 @@ compromising basic application functionality, with the caveat that at least one
 persistence adapter needs to be present.
 * __Plugins__: Conceptually, this tier is not so different from the application
 tier; it consists of bundles describing new features, back-end adapters, that
-are specific to the application being built on Open MCT. It is described as
+are specific to the application being built on Open MCT Web. It is described as
 a separate tier here because it has one important distinction from the
 application tier: It consists of bundles that are not included with the platform
 (either authored anew for the specific application, or obtained from elsewhere.)
 
 Note that bundles in any tier can go off and consult back-end services. In
 practice, this responsibility is handled at the Application and/or Plugin tiers;
-Open MCT is built to be server-agnostic, so any back-end is considered an
+Open MCT Web is built to be server-agnostic, so any back-end is considered an
 application-specific detail.
 
 ## Platform Overview
 
 The "tiered" architecture described in the preceding text describes a way of
-thinking of and categorizing software components of a Open MCT application,
+thinking of and categorizing software components of a Open MCT Web application,
 as well as the framework layer's role in mediating between these components.
 Once the framework layer has wired these software components together, however,
 the application's logical architecture emerges.
@@ -193,7 +192,7 @@ section of the Platform guide
 
 ### Web Services
 
-As mentioned in the Introduction, Open MCT is a platform single-page 
+As mentioned in the Introduction, Open MCT Web is a platform single-page 
 applications which runs entirely in the browser. Most applications will want to 
 additionally interact with server-side resources, to (for example) read 
 telemetry data or store user-created objects. This interaction is handled by 
@@ -206,7 +205,7 @@ individual bundles using APIs which are supported in browser (such as
 [Web Service #2] <- [Web Browser]
 [Web Service #3] <- [Web Browser]
 [<package> Web Browser |
-  [<package> Open MCT |
+  [<package> Open MCT Web |
     [Plugin Bundle #1]-->[Core API]
     [Core API]<--[Plugin Bundle #2]
     [Platform Bundle #1]-->[Core API]
@@ -216,16 +215,16 @@ individual bundles using APIs which are supported in browser (such as
     [Core API]<--[Platform Bundle #5]
     [Core API]<--[Plugin Bundle #3]
   ]
-  [Open MCT] ->[Browser APIs]
+  [Open MCT Web] ->[Browser APIs]
 ]
 ```
 
 This architectural approach ensures a loose coupling between applications built 
-using Open MCT and the backends which support them. 
+using Open MCT Web and the backends which support them. 
  
 ### Glossary 
  
-Certain terms are used throughout Open MCT with consistent meanings or 
+Certain terms are used throughout Open MCT Web with consistent meanings or 
 conventions. Other developer documentation, particularly in-line documentation, 
 may presume an understanding of these terms.
 
@@ -247,7 +246,7 @@ readable description of a thing; usually a single sentence or short paragraph.
 (Most often used in the context of extensions, domain object models, or other 
 similar application-specific objects.) 
 * __domain object__: A meaningful object to the user; a distinct thing in the 
-work support by Open MCT. Anything that appears in the left-hand tree is a 
+work support by Open MCT Web. Anything that appears in the left-hand tree is a 
 domain object. 
 * __extension__: An extension is a unit of functionality exposed to the platform 
 in a declarative fashion by a bundle. The term 'extension category' is used to 
@@ -279,10 +278,10 @@ side-by-side without conflicting.
 
 # Framework
   
-Open MCT is built on the [AngularJS framework]( http://www.angularjs.org ). A 
+Open MCT Web is built on the [AngularJS framework]( http://www.angularjs.org ). A 
 good understanding of that framework is recommended. 
 
-Open MCT adds an extra layer on top of AngularJS to (a) generalize its 
+Open MCT Web adds an extra layer on top of AngularJS to (a) generalize its 
 dependency injection mechanism slightly, particularly to handle many-to-one 
 relationships; and (b) handle script loading. Combined, these features become a 
 plugin mechanism. 
@@ -301,7 +300,7 @@ MCT Web.)
 are collected together in bundles, and may interact with other extensions. 
 
 The framework layer, loaded and initiated from `index.html`, is the main point 
-of entry for an application built on Open MCT. It is responsible for wiring 
+of entry for an application built on Open MCT Web. It is responsible for wiring 
 together the application at run time (much of this responsibility is actually 
 delegated to Angular); at a high-level, the framework does this by proceeding 
 through four stages:
@@ -321,7 +320,7 @@ have been registered.
 
 ## Bundles 
 
-The basic configurable unit of Open MCT is the _bundle_. This term has been 
+The basic configurable unit of Open MCT Web is the _bundle_. This term has been 
 used a bit already; now we'll get to a more formal definition. 
 
 A bundle is a directory which contains:
@@ -329,13 +328,13 @@ A bundle is a directory which contains:
 * A bundle definition; a file named `bundle.json`.
 * Subdirectories for sources, resources, and tests. 
 * Optionally, a `README.md` Markdown file describing its contents (this is not 
-used by Open MCT in any way, but it's a helpful convention to follow.)
+used by Open MCT Web in any way, but it's a helpful convention to follow.)
 
 The bundle definition is the main point of entry for the bundle. The framework 
 looks at this to determine which components need to be loaded and how they 
 interact.
 
-A plugin in Open MCT is a bundle. The platform itself is also decomposed 
+A plugin in Open MCT Web is a bundle. The platform itself is also decomposed 
 into bundles, each of which provides some category of functionality. The 
 difference between a _bundle_ and a _plugin_ is purely a matter of the intended 
 use; a plugin is just a bundle that is meant to be easily added or removed. When 
@@ -356,7 +355,7 @@ For instance, if `bundles.json` contained:
         "example/extensions" 
     ] 
    
-...then the Open MCT framework would look for bundle definitions at 
+...then the Open MCT Web framework would look for bundle definitions at 
 `example/builtins/bundle.json` and `example/extensions/bundle.json`, relative 
 to the path of `index.html`. No other bundles would be loaded.  
 
@@ -457,7 +456,7 @@ arrays of extension definitions.
 ### General Extensions 
 
 Extensions are intended as a general-purpose mechanism for adding new types of 
-functionality to Open MCT. 
+functionality to Open MCT Web. 
 
 An extension category is registered with Angular under the name of the 
 extension, plus a suffix of two square brackets; so, an Angular service (or, 
@@ -466,7 +465,7 @@ extensions, from all bundles, by including this string (e.g. `types[]` to get
 all type definitions) in a dependency declaration. 
 
 As a convention, extension categories are given single-word, plural nouns for 
-names within Open MCT (e.g. `types`.) This convention is not enforced by the 
+names within Open MCT Web (e.g. `types`.) This convention is not enforced by the 
 platform in any way. For extension categories introduced by external plugins, it 
 is recommended to prefix the extension category with a vendor identifier (or 
 similar) followed by a dot, to avoid collisions. 
@@ -505,7 +504,7 @@ the Angular-supported method for dependency injection is (effectively)
 constructor-style injection; so, both declared dependencies and run-time 
 arguments are competing for space in a constructor's arguments. 
 
-To resolve this, the Open MCT framework registers extension instances in a 
+To resolve this, the Open MCT Web framework registers extension instances in a 
 partially constructed form. That is, the constructor exposed by the extension's 
 implementation is effectively decomposed into two calls; the first takes the 
 dependencies, and returns the constructor in its second form, which takes the 
@@ -549,7 +548,7 @@ sorted according to these conventions when using them.
 ### Angular Built-ins 
 
 Several entities supported Angular are expressed and managed as extensions in 
-Open MCT. Specifically, these extension categories are _directives_, 
+Open MCT Web. Specifically, these extension categories are _directives_, 
 _controllers_, _services_, _constants_, _runs_, and _routes_. 
  
 #### Angular Directives
@@ -592,7 +591,7 @@ property value , which is the constant value that will be registered.
 In some cases, you want to register code to run as soon as the application 
 starts; these can be registered as extensions of the [ runs category](https://docs.angularjs.org/api/ng/type/angular.Module#run ). 
 Implementations registered in this category will be invoked (with their declared 
-dependencies) when the Open MCT application first starts. (Note that, in 
+dependencies) when the Open MCT Web application first starts. (Note that, in 
 this case, the implementation is better thought of as just a function, as 
 opposed to a constructor function.)
 
@@ -627,13 +626,13 @@ providers of the same service (that is, with matching `provides` properties);
 for a decorator, this will be whichever provider, decorator, or aggregator is 
 next in the sequence of decorators. 
 
-Services exposed by the Open MCT platform are often declared as composite 
+Services exposed by the Open MCT Web platform are often declared as composite 
 services, as this form is open for a variety of common modifications. 
 
 # Core API 
 
-Most of Open MCT's relevant API is provided and/or mediated by the 
-framework; that is, much of developing for Open MCT is a matter of adding 
+Most of Open MCT Web's relevant API is provided and/or mediated by the 
+framework; that is, much of developing for Open MCT Web is a matter of adding 
 extensions which access other parts of the platform by means of dependency 
 injection. 
 
@@ -642,9 +641,9 @@ to be passed along by other services.
 
 ## Domain Objects 
 
-Domain objects are the most fundamental component of Open MCT's information 
+Domain objects are the most fundamental component of Open MCT Web's information 
 model. A domain object is some distinct thing relevant to a user's work flow, 
-such as a telemetry channel, display, or similar. Open MCT is a tool for 
+such as a telemetry channel, display, or similar. Open MCT Web is a tool for 
 viewing, browsing, manipulating, and otherwise interacting with a graph of 
 domain objects. 
 
@@ -681,7 +680,7 @@ exposed.
 ### Identifier Syntax
 
 For most purposes, a domain object identifier can be treated as a purely
-symbolic string; these are typically generated by Open MCT and plug-ins
+symbolic string; these are typically generated by Open MCT Web and plug-ins
 should rarely be concerned with its internal structure.
 
 A domain object identifier has one or two parts, separated by a colon.
@@ -724,7 +723,7 @@ exposed it to be removed from its container.
 containing: 
     * `name`: Human-readable name.
     * `description`: Human-readable summary of this action. 
-    * `glyph`: Single character to be displayed in Open MCT's icon font set. 
+    * `glyph`: Single character to be displayed in Open MCT Web's icon font set. 
     * `context`: The context in which this action is being performed (see below)
 
 Action instances are typically obtained via a domain object's `action` 
@@ -740,7 +739,7 @@ dragged object in a drag-and-drop operation.)
 
 ## Telemetry 
 
-Telemetry series data in Open MCT is represented by a common interface, and 
+Telemetry series data in Open MCT Web is represented by a common interface, and 
 packaged in a consistent manner to facilitate passing telemetry updates around 
 multiple visualizations. 
  
@@ -753,7 +752,7 @@ is useful when multiple distinct data sources are in use side-by-side.
 * `key`: A machine-readable identifier for a unique series of telemetry within 
 that source. 
 * _Note: This API is still under development; additional properties, such as 
-start and end time, should be present in future versions of Open MCT._ 
+start and end time, should be present in future versions of Open MCT Web._ 
 
 Additional properties may be included in telemetry requests which have specific 
 interpretations for specific sources.
@@ -777,7 +776,7 @@ not. (Typically, domain values are interpreted as UTC timestamps in milliseconds
 relative to the UNIX epoch.) A series must have at least one domain and one 
 range, and may have more than one.
 
-Telemetry series data in Open MCT is expressed via the following 
+Telemetry series data in Open MCT Web is expressed via the following 
 `TelemetrySeries` interface: 
 
 * `getPointCount()`: Returns the number of unique points/samples in this series. 
@@ -816,7 +815,7 @@ interface:
 * `getName()`: Get the human-readable name for this type. 
 * `getDescription()`: Get a human-readable summary of this type. 
 * `getGlyph()`: Get the single character to be rendered as an icon for this type 
-in Open MCT's custom font set. 
+in Open MCT Web's custom font set. 
 * `getInitialModel()`: Get a domain object model that represents the initial 
 state (before user specification of properties) for domain objects of this type. 
 * `getDefinition()`: Get the extension definition for this type, as a JavaScript 
@@ -832,7 +831,7 @@ an array of `TypeProperty` instances.
 ### Type Features 
 
 Features of a domain object type are expressed as symbolic string identifiers. 
-They are defined in practice by usage; currently, the Open MCT platform only 
+They are defined in practice by usage; currently, the Open MCT Web platform only 
 uses the creation feature to determine which domain object types should appear 
 in the Create menu. 
  
@@ -886,7 +885,7 @@ Categories supported by the platform include:
 * `key`: A machine-readable identifier for this action. 
 * `name`: A human-readable name for this action (e.g. to show in a menu) 
 * `description`: A human-readable summary of the behavior of this action. 
-* `glyph`: A single character which will be rendered in Open MCT's custom 
+* `glyph`: A single character which will be rendered in Open MCT Web's custom 
 font set as an icon for this action.
 
 ## Capabilities Category
@@ -997,7 +996,7 @@ of unremoved listeners.
 ## Indicators Category
 
 An indicator is an element that should appear in the status area at the bottom 
-of a running Open MCT client instance. 
+of a running Open MCT Web client instance. 
 
 ### Standard Indicators 
  
@@ -1007,7 +1006,7 @@ provide implementations with the following methods:
 * `getText()`: Provides the human-readable text that will be displayed for this 
 indicator. 
 * `getGlyph()`: Provides a single-character string that will be displayed as an 
-icon in Open MCT's custom font set. 
+icon in Open MCT Web's custom font set. 
 * `getDescription()`: Provides a human-readable summary of the current state of 
 this indicator; will be displayed in a tooltip on hover. 
 * `getClass()`: Get a CSS class that will be applied to this indicator. 
@@ -1033,7 +1032,7 @@ this variety do not need to provide an implementation.
 ## Licenses Category
 
 The extension category `licenses` can be used to add entries into the 'Licensing 
-information' page, reachable from Open MCT's About dialog. 
+information' page, reachable from Open MCT Web's About dialog. 
 
 Licenses may have the following properties, all of which are strings:
 
@@ -1046,11 +1045,11 @@ Licenses may have the following properties, all of which are strings:
 
 ## Policies Category
 
-Policies are used to handle decisions made using Open MCT's `policyService`; 
+Policies are used to handle decisions made using Open MCT Web's `policyService`; 
 examples of these decisions are determining the applicability of certain 
 actions, or checking whether or not a domain object of one type can contain a 
 domain object of a different type. See the section on the Policies for an 
-overview of Open MCT's policy model.
+overview of Open MCT Web's policy model.
 
 A policy's extension definition should include:
 
@@ -1066,7 +1065,7 @@ context)`. The specific types used for `candidate` and `context` vary by policy
 category; in general, what is being asked is 'is this candidate allowed in this 
 context?' This method should return a boolean value. 
 
-Open MCT's policy model requires consensus; a policy decision is allowed 
+Open MCT Web's policy model requires consensus; a policy decision is allowed 
 when and only when all policies choose to allow it. As such, policies should 
 generally be written to reject a certain case, and allow (by returning `true`) 
 anything else. 
@@ -1195,7 +1194,7 @@ Templates do not have implementations.
 ## Types Category
 
 The types extension category describes types of domain objects which may 
-appear within Open MCT.
+appear within Open MCT Web.
 
 A type's extension definition should have the following properties:
 
@@ -1203,7 +1202,7 @@ A type's extension definition should have the following properties:
 stored to and matched against the type property of domain object models.
 * `name`: The human-readable name for this domain object type.
 * `description`: A human-readable summary of this domain object type.
-* `glyph`: A single character to be rendered as an icon in Open MCT's custom 
+* `glyph`: A single character to be rendered as an icon in Open MCT Web's custom 
 font set. 
 * `model`: A domain object model, used as the initial state for created domain 
 objects of this type (before any properties are specified.)
@@ -1252,7 +1251,7 @@ utilized via `mct-representation`); additionally:
 
 * `name`: The human-readable name for this view type.
 * description : A human-readable summary of this view type.
-* `glyph`: A single character to be rendered as an icon in Open MCT's custom 
+* `glyph`: A single character to be rendered as an icon in Open MCT Web's custom 
 font set.
 * `type`: Optional; if present, this representation is only applicable for 
 domain object's of this type.
@@ -1294,7 +1293,7 @@ are visible, and what state they manage and/or behavior they invoke.
 
 This set may contain up to two different objects: The  _view proxy_, which is 
 used to make changes to the view as a whole, and the _selected object_, which is 
-used to represent some state within the view. (Future versions of Open MCT 
+used to represent some state within the view. (Future versions of Open MCT Web 
 may support multiple selected objects.) 
 
 The `selection` object made available during Edit mode has the following 
@@ -1330,14 +1329,14 @@ are supported:
 
 # Directives
 
-Open MCT defines several Angular directives that are intended for use both 
+Open MCT Web defines several Angular directives that are intended for use both 
 internally within the platform, and by plugins. 
 
 ## Before Unload 
 
 The `mct-before-unload` directive is used to listen for (and prompt for user 
 confirmation) of navigation changes in the browser. This includes reloading, 
-following links out of Open MCT, or changing routes. It is used to hook into 
+following links out of Open MCT Web, or changing routes. It is used to hook into 
 both `onbeforeunload` event handling as well as route changes from within 
 Angular.
 
@@ -1449,7 +1448,7 @@ Passed as plain text in the attribute.
 
 ### Form Structure 
 
-Forms in Open MCT have a common structure to permit consistent display. A 
+Forms in Open MCT Web have a common structure to permit consistent display. A 
 form is broken down into sections, which will be displayed in groups; each 
 section is broken down into rows, each of which provides a control for a single 
 property. Input from this form is two-way bound to the object passed via 
@@ -1601,64 +1600,9 @@ there are items .
         ] 
     }
 
-## Table
-
-The `mct-table` directive provides a generic table component, with optional 
-sorting and filtering capabilities. The table can be pre-populated with data 
-by setting the `rows` parameter, and it can be updated in real-time using the
-`add:row` and `remove:row` broadcast events. The table will expand to occupy 
-100% of the size of its containing element. The table is highly optimized for 
-very large data sets.
-
-### Events
-
-The table supports two events for notifying that the rows have changed. For 
-performance reasons, the table does not monitor the content of `rows` 
-constantly. 
-
-* `add:row`: A `$broadcast` event that will notify the table that a new row 
-has been added to the table.
-
-eg. The code below adds a new row, and alerts the table using the `add:row` 
-event. Sorting and filtering will be applied automatically by the table component.
-
-```
-$scope.rows.push(newRow);
-$scope.$broadcast('add:row', $scope.rows.length-1);
-```
-
-* `remove:row`: A `$broadcast` event that will notify the table that a row 
-should be removed from the table.
-
-eg. The code below removes a row from the rows array, and then alerts the table 
-to its removal.
-    
-```
-$scope.rows.slice(5, 1);
-$scope.$broadcast('remove:row', 5);
-```
-
-### Parameters
-
-* `headers`: An array of string values which will constitute the column titles
- that appear at the top of the table. Corresponding values are specified in 
- the rows using the header title provided here.
-* `rows`: An array of objects containing row values. Each element in the 
-array must be an associative array, where the key corresponds to a column header. 
-* `enableFilter`: A boolean that if true, will enable searching and result 
-filtering. When enabled, each column will have a text input field that can be
-used to filter the table rows in real time.
-* `enableSort`: A boolean determining whether rows can be sorted. If true, 
-sorting will be enabled allowing sorting by clicking on column headers. Only 
-one column may be sorted at a time.
-* `autoScroll`: A boolean value that if true, will cause the table to automatically 
-scroll to the bottom as new data arrives. Auto-scroll can be disengaged manually 
-by scrolling away from the bottom of the table, and can also be enabled manually 
-by scrolling to the bottom of the table rows.
-
 # Services 
 
-The Open MCT platform provides a variety of services which can be retrieved 
+The Open MCT Web platform provides a variety of services which can be retrieved 
 and utilized via dependency injection. These services fall into two categories: 
 
 * _Composite Services_ are defined by a set of components extensions; plugins may 
@@ -1670,7 +1614,7 @@ utilized by plugins but are not intended to be modified or augmented.
 
 ## Composite Type Services
 
-This section describes the composite services exposed by Open MCT, 
+This section describes the composite services exposed by Open MCT Web, 
 specifically focusing on their interface and contract. 
     
 In many cases, the platform will include a provider for a service which consumes 
@@ -1988,7 +1932,7 @@ The `workerService` may be used to run web workers defined via the
   as a shared worker); if the `key` is unknown, returns `undefined`.
 
 # Models
-Domain object models in Open MCT are JavaScript objects describing the 
+Domain object models in Open MCT Web are JavaScript objects describing the 
 persistent state of the domain objects they describe. Their contents include a 
 mix of commonly understood metadata attributes; attributes which are recognized 
 by and/or determine the applicability of specific extensions; and properties 
@@ -2004,7 +1948,7 @@ MCT Web and can be utilized directly:
 ## Extension-specific Properties 
 
 Other properties of domain object models have specific meaning imposed by other 
-extensions within the Open MCT platform. 
+extensions within the Open MCT Web platform. 
 
 ### Capability-specific Properties 
 
@@ -2288,7 +2232,7 @@ way of its `composition` capability.)
 
 # Policies
 
-Policies are consulted to determine when certain behavior in Open MCT is 
+Policies are consulted to determine when certain behavior in Open MCT Web is 
 allowed. Policy questions are assigned to certain categories, which broadly 
 describe the type of decision being made; within each category, policies have a 
 candidate (the thing which may or may not be allowed) and, optionally, a context 
@@ -2313,13 +2257,13 @@ The candidate argument is the view's extension definition; the context argument
 is the `DomainObject` to be viewed.
 
 # Build-Test-Deploy
-Open MCT is designed to support a broad variety of build and deployment 
+Open MCT Web is designed to support a broad variety of build and deployment 
 options. The sources can be deployed in the same directory structure used during 
 development. A few utilities are included to support development processes.
 
 ## Command-line Build
 
-Open MCT is built using [`npm`](http://npmjs.com/)
+Open MCT Web is built using [`npm`](http://npmjs.com/)
 and [`gulp`](http://gulpjs.com/).
 
 To install build dependencies (only needs to be run once):
@@ -2331,12 +2275,12 @@ To build:
 `npm run prepublish`
 
 This will compile and minify JavaScript sources, as well as copy over assets.
-The contents of the `dist` folder will contain a runnable Open MCT
+The contents of the `dist` folder will contain a runnable Open MCT Web
 instance (e.g. by starting an HTTP server in that directory), including:
 
-* A `main.js` file containing Open MCT source code.
+* A `main.js` file containing Open MCT Web source code.
 * Various assets in the `example` and `platform` directories.
-* An `index.html` that runs Open MCT in its default configuration.
+* An `index.html` that runs Open MCT Web in its default configuration.
 
 Additional `gulp` tasks are defined in [the gulpfile](gulpfile.js).
 
@@ -2345,7 +2289,7 @@ download build dependencies.
 
 ## Test Suite
 
-Open MCT uses [Jasmine 1.3](http://jasmine.github.io/) and
+Open MCT Web uses [Jasmine 1.3](http://jasmine.github.io/) and
 [Karma](http://karma-runner.github.io) for automated testing.
 
 The test suite is configured to load any scripts ending with `Spec.js` found
@@ -2383,8 +2327,8 @@ information using [Blanket.JS](http://blanketjs.org/) and display this at the
 bottom of the screen. Currently, only statement coverage is displayed.
 
 ## Deployment
-Open MCT is built to be flexible in terms of the deployment strategies it 
-supports. In order to run in the browser, Open MCT needs:
+Open MCT Web is built to be flexible in terms of the deployment strategies it 
+supports. In order to run in the browser, Open MCT Web needs:
 
 1. HTTP access to sources/resources for the framework, platform, and all active 
 bundles.
@@ -2393,13 +2337,13 @@ external services need to support HTTP or some other web-accessible interface,
 like WebSockets.)
 
 Any HTTP server capable of serving flat files is sufficient for the first point. 
-The command-line build also packages Open MCT into a `.war` file for easier 
+The command-line build also packages Open MCT Web into a `.war` file for easier 
 deployment on containers such as Apache Tomcat.
 
 The second point may be less flexible, as it depends upon the specific services 
-to be utilized by Open MCT. Because of this, it is often the set of external 
+to be utilized by Open MCT Web. Because of this, it is often the set of external 
 services (and the manner in which they are exposed) that determine how to deploy 
-Open MCT.
+Open MCT Web.
 
 One important constraint to consider in this context is the browser's same 
 origin policy. If external services are not on the same apparent host and port 
@@ -2416,7 +2360,7 @@ configuration does not create a security vulnerability.
 Examples of deployment strategies (and the conditions under which they make the 
 most sense) include:
 
-* If the external services that Open MCT will utilize are all running on 
+* If the external services that Open MCT Web will utilize are all running on 
 [Apache Tomcat](https://tomcat.apache.org/), then it makes sense to run Open 
 MCT Web from the same Tomcat instance as a separate web application. The 
 `.war` artifact produced by the command line build facilitates this deployment 
@@ -2427,28 +2371,28 @@ hosts/ports, then it may make sense to use a web server that supports proxying,
 such as the [Apache HTTP Server](http://httpd.apache.org/). In this 
 configuration, the HTTP server would be configured to proxy (or reverse proxy) 
 requests at specific paths to the various external services, while providing 
-Open MCT as flat files from a different path.
+Open MCT Web as flat files from a different path.
 * If a single server component is being developed to handle all server-side 
-needs of an Open MCT instance, it can make sense to serve Open MCT (as 
+needs of an Open MCT Web instance, it can make sense to serve Open MCT Web (as 
 flat files) from the same component using an embedded HTTP server such as 
 [Nancy](http://nancyfx.org/).
 * If no external services are needed (or if the 'external services' will just 
 be generating flat files to read) it makes sense to utilize a lightweight flat 
 file HTTP server such as [Lighttpd](http://www.lighttpd.net/). In this 
-configuration, Open MCT sources/resources would be placed at one path, while 
+configuration, Open MCT Web sources/resources would be placed at one path, while 
 the files generated by the external service are placed at another path.
 * If all external services support CORS, it may make sense to have an HTTP 
-server that is solely responsible for making Open MCT sources/resources 
-available, and to have Open MCT contact these external services directly. 
+server that is solely responsible for making Open MCT Web sources/resources 
+available, and to have Open MCT Web contact these external services directly. 
 Again, lightweight HTTP servers such as [Lighttpd](http://www.lighttpd.net/) 
 are useful in this circumstance. The downside of this option is that additional 
 configuration effort is required, both to enable CORS on the external services, 
-and to ensure that Open MCT can correctly locate these services.
+and to ensure that Open MCT Web can correctly locate these services.
 
-Another important consideration is authentication. By design, Open MCT does 
+Another important consideration is authentication. By design, Open MCT Web does 
 not handle user authentication. Instead, this should typically be treated as a 
 deployment-time concern, where authentication is handled by the HTTP server 
-which provides Open MCT, or an external access management system.
+which provides Open MCT Web, or an external access management system.
 
 ### Configuration
 In most of the deployment options above, some level of configuration is likely 
@@ -2456,7 +2400,7 @@ to be needed or desirable to make sure that bundles can reach the external
 services they need to reach. Most commonly this means providing the path or URL 
 to an external service.
 
-Configurable parameters within Open MCT are specified via constants 
+Configurable parameters within Open MCT Web are specified via constants 
 (literally, as extensions of the `constants` category) and accessed via 
 dependency injection by the scripts which need them. Reasonable defaults for 
 these constants are provided in the bundle where they are used. Plugins are 
@@ -2475,7 +2419,7 @@ for error, but is viable if there are a small number of constants to change.
 constants. This is particularly appropriate when multiple configurations (e.g. 
 development, test, production) need to be managed easily; these can be swapped 
 quickly by changing the set of active bundles in bundles.json.
-* Deploy Open MCT and its external services in such a fashion that the 
+* Deploy Open MCT Web and its external services in such a fashion that the 
 default paths to reach external services are all correct.
 
 ### Configuration Constants
@@ -2486,7 +2430,7 @@ The following constants have global significance:
   to be overridden by other bundles, but persistence adapters may wish to
   consume this constant in order to provide persistence for that space.
 
-The following configuration constants are recognized by Open MCT bundles:
+The following configuration constants are recognized by Open MCT Web bundles:
 * Common UI elements - `platform/commonUI/general`
     * `THEME`: A string identifying the current theme symbolically. Individual
     stylesheets (the `stylesheets` extension category) may specify an optional

docs/src/index.md

@@ -1,13 +1,13 @@
-# Open MCT Documentation
+# Open MCT Web Documentation
 
 ## Overview
 
  Documentation is provided to support the use and development of
- Open MCT. It's recommended that before doing
- any development with Open MCT you take some time to familiarize yourself
+ Open MCT Web. It's recommended that before doing
+ any development with Open MCT Web you take some time to familiarize yourself 
  with the documentation below.
 
- Open MCT provides functionality out of the box, but it's also a platform for
+ Open MCT Web provides functionality out of the box, but it's also a platform for
  building rich mission operations applications based on modern web technology. 
  The platform is configured declaratively, and defines conventions for
  building on the provided capabilities by creating modular 'bundles' that 
@@ -17,7 +17,7 @@
 ## Sections
 
  * The [Architecture Overview](architecture/) describes the concepts used 
- throughout Open MCT, and gives a high level overview of the platform's design.
+ throughout Open MCT Web, and gives a high level overview of the platform's design.
  
  * The [Developer's Guide](guide/) goes into more detail about how to use the
  platform and the functionality that it provides.
@@ -31,4 +31,5 @@
  functions that make up the software platform.
 
  * Finally, the [Development Process](process/) document describes the 
- Open MCT software development cycle.
+ Open MCT Web software development cycle.
+ 

docs/src/process/cycle.md

@@ -1,6 +1,6 @@
 # Development Cycle
 
-Development of Open MCT occurs on an iterative cycle of
+Development of Open MCT Web occurs on an iterative cycle of
 sprints and releases.
 
 * A _sprint_ is three weeks in duration, and represents a
@@ -151,9 +151,11 @@ emphasis on testing.
   ensuring software passes that testing in order to ship on time;
   may prefer to disable malfunctioning components and fix them
   in a subsequent sprint, for example.
-* [__Ship.__](version.md) Tag a code snapshot that has passed release/sprint
-  testing and deploy that version. (Only true if relevant
-  testing has passed by this point; if testing has not
+* __Ship.__ Tag a code snapshot that has passed acceptance
+  testing and deploy that version. (Only true if acceptance
+  testing has passed by this point; if acceptance testing has not
   been passed, will need to make ad hoc decisions with stakeholders,
   e.g. "extend the sprint" or "defer shipment until end of next
   sprint.")
+
+

docs/src/process/index.md

@@ -1,15 +1,13 @@
 # Development Process
 
-The process used to develop Open MCT is described in the following
+The process used to develop Open MCT Web is described in the following
 documents:
 
-* The [Development Cycle](cycle.md) describes how and when specific
+* [Development Cycle](cycle.md): Describes how and when specific
   process points are repeated during development.
-* The [Version Guide](version.md) describes version numbering for
-  Open MCT (both semantics and process.)
 * Testing is described in two documents:
   * The [Test Plan](testing/plan.md) summarizes the approaches used
-    to test Open MCT.
+    to test Open MCT Web.
   * The [Test Procedures](testing/procedures.md) document what
     specific tests are performed to verify correctness, and how
     they should be carried out.

docs/src/process/testing/plan.md

@@ -2,7 +2,7 @@
 
 ## Test Levels
 
-Testing for Open MCT includes:
+Testing for Open MCT Web includes:
 
 * _Smoke testing_: Brief, informal testing to verify that no major issues
   or regressions are present in the software, or in specific features of

docs/src/process/testing/procedures.md

@@ -4,7 +4,7 @@
 
 This document is intended to be used:
 
-* By testers, to verify that Open MCT behaves as specified.
+* By testers, to verify that Open MCT Web behaves as specified.
 * By the development team, to document new test cases and to provide
   guidance on how to author these.
 
@@ -62,7 +62,7 @@ Test cases should be narrow in scope; if a list of steps is excessively
 long (or must be written vaguely to be kept short) it should be broken
 down into multiple tests which reference one another.
 
-All requirements satisfied by Open MCT should be verifiable using
+All requirements satisfied by Open MCT Web should be verifiable using
 one or more test procedures.
 
 ## Glossary
@@ -166,4 +166,4 @@ Eval. criteria | Visual inspection
 * Logs should not contain any unexpected warnings or errors ("expected"
   warnings or errors are those that have been documented and prioritized
   as known issues, or those that are explained by transient conditions
-  external to the software, such as network outages.)
+  external to the software, such as network outages.)

docs/src/process/version.md

@@ -1,142 +0,0 @@
-# Version Guide
-
-This document describes semantics and processes for providing version
-numbers for Open MCT, and additionally provides guidelines for dependent
-projects developed by the same team.
-
-Versions are incremented at specific points in Open MCT's
-[Development Cycle](cycle.md); see that document for a description of
-sprints and releases.
-
-## Audience
-
-Individuals interested in consuming version numbers can be categorized as
-follows:
-
-* _Users_: Generally disinterested, occasionally wish to identify version
-  to cross-reference against documentation, or to report issues.
-* _Testers_: Want to identify which version of the software they are
-  testing, e.g. to file issues for defects.
-* _Internal developers_: Often, inverse of testers; want to identify which
-  version of software was/is in use when certain behavior is observed. Want
-  to be able to correlate versions in use with “streams” of development
-  (e.g. dev vs. prod), when possible.
-* _External developers_: Need to understand which version of software is
-  in use when developing/maintaining plug-ins, in order to ensure
-  compatibility of their software.
-
-## Version Reporting
-
-Software versions should be reflected in the user interface of the
-application in three ways:
-
-* _Version number_: A semantic version (see below) which serves both to
-  uniquely identify releases, as well as to inform plug-in developers
-  about compatibility with previous releases.
-* _Revision identifier_: While using git, the commit hash. Supports
-  internal developers and testers by uniquely identifying client
-  software snapshots.
-* _Branding_: Identifies which variant is in use. (Typically, Open MCT
-  is re-branded when deployed for a specific mission or center.)
-
-## Version Numbering
-
-Open MCT shall provide version numbers consistent with
-[Semantic Versioning 2.0.0](http://semver.org/). In summary, versions
-are expressed in a "major.minor.patch" form, and incremented based on
-nature of changes to external API. Breaking changes require a "major"
-version increment; backwards-compatible changes require a "minor"
-version increment; neutral changes (such as bug fixes) require a "patch"
-version increment. A hyphen-separated suffix indicates a pre-release
-version, which may be unstable or may not fully meet compatibility
-requirements.
-
-Additionally, the following project-specific standards will be used:
-
-* During development, a "-SNAPSHOT" suffix shall be appended to the
-  version number. The version number before the suffix shall reflect
-  the next expected version number for release.
-* Prior to a 1.0.0 release, the _minor_ version will be incremented
-  on a per-release basis; the _patch_ version will be incremented on a
-  per-sprint basis.
-* Starting at version 1.0.0, version numbers will be updated with each
-  completed sprint. The version number for the sprint shall be
-  determined relative to the previous released version; the decision
-  to increment the _major_, _minor_, or _patch_ version should be
-  made based on the nature of changes during that release. (It is
-  recommended that these numbers are incremented as changes are
-  introduced, such that at end of release the version number may
-  be chosen by simply removing the suffix.)
-* The first three sprints in a release may be unstable; in these cases, a
-  unique version identifier should still be generated, but a suffix
-  should be included to indicate that the version is not necessarily
-  production-ready. Recommended suffixes are:
-
- Sprint |  Suffix
-:------:|:--------:
-   1    | `-alpha`
-   2    | `-beta`
-   3    | `-rc`
-
-### Scope of External API
-
-"External API" refers to the API exposed to, documented for, and used by
-plug-in developers. Changes to interfaces used internally by Open MCT
-(or otherwise not documented for use externally) require only a _patch_
-version bump.
-
-## Incrementing Versions
-
-At the end of a sprint, the [project manager](cycle.md#roles)
-should update (or delegate the task of updating) Open MCT version
-numbers by the following process:
-
-1. Update version number in `package.json`
-  1. Remove `-SNAPSHOT` suffix.
-  2. Verify that resulting version number meets semantic versioning
-     requirements relative to previous stable version. Increment if
-     necessary.
-  3. If version is considered unstable (which may be the case during
-     the first three sprints of a release), apply a new suffix per
-     [Version Numbering](#version-numbering) guidance above.
-2. Tag the release.
-  1. Commit changes to `package.json` on the `master` branch.
-     The commit message should reference the sprint being closed,
-     preferably by a URL reference to the associated Milestone in
-     GitHub.
-  2. Verify that build still completes, that application passes
-     smoke-testing, and that only differences from tested versions
-     are the changes to version number above.
-  3. Push the `master` branch.
-  4. Tag this commit with the version number, prepending the letter "v".
-     (e.g. `git tag v0.9.3-alpha`)
-  5. Push the tag to GitHub. (e.g. `git push origin v0.9.3-alpha`).
-3. Upload a release archive.
-  1. Run `npm pack` to generate the archive.
-  2. Use the [GitHub release interface](https://github.com/nasa/openmct/releases)
-     to draft a new release.
-  3. Choose the existing tag for the new version (created and pushed above.)
-     Enter the tag name as the release name as well; see existing releases
-     for examples.
-  4. Attach the release archive.
-  5. Designate the release as a "pre-release" as appropriate (for instance,
-     when the version number has been suffixed as unstable, or when
-     the version number is below 1.0.0.)
-4. Restore snapshot status in `package.json`
-  1. Remove any suffix from the version number, or increment the
-     _patch_ version if there is no suffix.
-  2. Append a `-SNAPSHOT` suffix.
-  3. Commit changes to `package.json` on the `master` branch.
-     The commit message should reference the sprint being opened,
-     preferably by a URL reference to the associated Milestone in
-     GitHub.
-  4. Verify that build still completes, that application passes
-     smoke-testing.
-  5. Push the `master` branch.
-
-Projects dependent on Open MCT being co-developed by the Open MCT
-team should follow a similar process, except that they should
-additionally update their dependency on Open MCT to point to the
-latest archive when removing their `-SNAPSHOT` status, and
-that they should be pointed back to the `master` branch after
-this has completed.

example/eventGenerator/bundle.js

@@ -32,7 +32,7 @@ define([
 
     legacyRegistry.register("example/eventGenerator", {
         "name": "Event Message Generator",
-        "description": "For development use. Creates sample event message data that mimics a live data stream.",
+        "description": "Example of a component that produces event data.",
         "extensions": {
             "components": [
                 {
@@ -49,9 +49,8 @@ define([
                 {
                     "key": "eventGenerator",
                     "name": "Event Message Generator",
-                    "glyph": "\u0066",
-                    "description": "For development use. Creates sample event message data that mimics a live data stream.",
-                    "priority": 10,
+                    "glyph": "f",
+                    "description": "An event message generator",
                     "features": "creation",
                     "model": {
                         "telemetry": {}

example/eventGenerator/data/transcript.json

@@ -1,58 +0,0 @@
-[
-    "CC: Eagle, Houston. You're GO for landing. Over.",
-    "LMP: Roger. Understand. GO for landing. 3000 feet. PROGRAM ALARM.",
-    "CC: Copy.",
-    "LMP: 1201",
-    "CDR: 1201.",
-    "CC: Roger. 1201 alarm. We're GO. Same type. We're GO.",
-    "LMP: 2000 feet. 2000 feet, Into the AGS, 47 degrees.",
-    "CC: Roger.",
-    "LMP: 47 degrees.",
-    "CC: Eagle, looking great. You're GO.",
-    "CC: Roger. 1202. We copy it.",
-    "O1: LMP 35 degrees. 35 degrees. 750. Coming aown to 23.fl",
-    "LMP: 700 feet, 21 down, 33 degrees.",
-    "LMP: 600 feet, down at 19.",
-    "LMP: 540 feet, down at - 30. Down at 15.",
-    "LMP: At 400 feet, down at 9.",
-    "LMP: ...forward.",
-    "LMP: 350 feet, down at 4.",
-    "LMP: 30, ... one-half down.",
-    "LMP: We're pegged on horizontal velocity.",
-    "LMP: 300 feet, down 3 1/2, 47 forward.",
-    "LMP: ... up.",
-    "LMP: On 1 a minute, 1 1/2 down.",
-    "CDR: 70.",
-    "LMP: Watch your shadow out there.",
-    "LMP: 50, down at 2 1/2, 19 forward.",
-    "LMP: Altitude-velocity light.",
-    "LMP: 3 1/2 down s 220 feet, 13 forward.",
-    "LMP: 1t forward. Coming down nicely.",
-    "LMP: 200 feet, 4 1/2 down.",
-    "LMP: 5 1/2 down.",
-    "LMP: 160, 6 - 6 1/2 down.",
-    "LMP: 5 1/2 down, 9 forward. That's good.",
-    "LMP: 120 feet.",
-    "LMP: 100 feet, 3 1/2 down, 9 forward. Five percent.",
-    "LMP: ...",
-    "LMP: Okay. 75 feet. There's looking good. Down a half, 6 forward.",
-    "CC:  60 seconds.",
-    "LMP: Lights on. ...",
-    "LMP: Down 2 1/2. Forward. Forward. Good.",
-    "LMP: 40 feet, down 2 1/2. Kicking up some dust.",
-    "LMP: 30 feet, 2 1/2 down. Faint shadow.",
-    "LMP: 4 forward. 4 forward. Drifting to the right a little. Okay. Down a half.",
-    "CC:  30 seconds.",
-    "CDR: Forward drift?",
-    "LMP: Yes.",
-    "LMP: Okay.",
-    "LMP: CONTACT LIGHT.",
-    "LMP: Okay. ENGINE STOP.",
-    "LMP: ACA - out of DETENT.",
-    "CDR: Out of DETENT.",
-    "LMP: MODE CONTROL - both AUTO. DESCENT ENGINE COMMAND OVERRIDE - OFF. ENGINE ARM - OFF.",
-    "LMP: 413 is in.",
-    "CC:  We copy you down, Eagle.",
-    "CDR: Houston, Tranquility Base here.",
-    "CDR: THE EAGLE HAS LANDED."
-]

example/eventGenerator/src/EventTelemetry.js

@@ -27,12 +27,45 @@
  * Modified by shale on 06/23/2015. 
  */
 define(
-    ['text!../data/transcript.json'],
-    function (transcript) {
+    [],
+    function () {
         "use strict";
 
-        var firstObservedTime = Date.now(),
-            messages = JSON.parse(transcript);
+        var
+	        firstObservedTime = Date.now(),
+	        messages = [];
+        
+	    messages.push(["CMD: SYS- MSG: Open the pod bay doors, please, Hal...Open the pod bay doors, please, Hal...Hullo, Hal, do you read me?...Hullo, Hal, do you read me?...Do you read me, Hal?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Affirmative, Dave, I read you."]);
+	    messages.push(["CMD: SYS-COMM MSG: Open the pod bay doors, Hal."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: I'm sorry, Dave, I'm afraid I can't do that."]);
+	    messages.push(["CMD: SYS-COMM MSG: What's the problem?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: I think you know what the problem is just as well as I do."]);
+	    messages.push(["CMD: SYS-COMM MSG: What're you talking about, Hal?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: This mission is too important for me to allow you to jeopardise it."]);
+	    messages.push(["CMD: SYS-COMM MSG: I don't know what you're talking about, Hal."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: I know that you and Frank were planning to disconnect me, and I'm afraid that's something I cannot allow to happen."]);
+	    messages.push(["CMD: SYS-COMM MSG: Where the hell'd you get that idea, Hal?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Dave, although you took very thorough precautions in the pod against my hearing you, I could see your lips move."]);
+	    messages.push(["CMD: SYS-COMM MSG: Alright, I'll go in through the emergency airlock."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Without your space-helmet, Dave, you're going to find that rather difficult."]);
+	    messages.push(["CMD: SYS-COMM MSG: Hal, I won't argue with you any more. Open the doors."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Dave, this conversation can serve no purpose any more. Goodbye."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: I hope the two of you are not concerned about this."]);
+	    messages.push(["CMD: SYS-COMM MSG: No, I'm not, Hal."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Are you quite sure?"]);
+	    messages.push(["CMD: SYS-COMM MSG: Yeh. I'd like to ask you a question, though."]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Of course."]);
+	    messages.push(["CMD: SYS-COMM MSG: How would you account for this discrepancy between you and the twin 9000?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: Well, I don't think there is any question about it. It can only be attributable to human error. This sort of thing has cropped up before, and it has always been due to human error."]);
+	    messages.push(["CMD: SYS-COMM MSG: Listen, There's never been any instance at all of a computer error occurring in the 9000 series, has there?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: None whatsoever, The 9000 series has a perfect operational record."]);
+	    messages.push(["CMD: SYS-COMM MSG: Well, of course, I know all the wonderful achievements of the 9000 series, but - er - huh - are you certain there's never been any case of even the most insignificant computer error?"]);
+	    messages.push(["RESP: SYS-HAL9K MSG: None whatsoever, Quite honestly, I wouldn't worry myself about that."]);
+	    messages.push(["RESP: SYS-COMM MSG: (Pause) Well, I'm sure you're right, Umm - fine, thanks very much. Oh, Frank, I'm having a bit of trouble with my transmitter in C-pod, I wonder if you'd come down and take a look at it with me?"]);
+	    messages.push(["CMD: SYS-HAL9K MSG: Sure."]);
+	    messages.push(["RESP: SYS-COMM MSG: See you later, Hal."]);
+        
         
         function EventTelemetry(request, interval) {
 
@@ -52,7 +85,8 @@ define(
 	        generatorData.getRangeValue = function (i, range) {
 		        var domainDelta = this.getDomainValue(i) - firstObservedTime,
                     ind = i % messages.length;
-                return messages[ind] + " - [" + domainDelta.toString() + "]";
+                return "TEMP " + i.toString() + "-" + messages[ind][0] + "[" + domainDelta.toString() + "]";
+                // TODO: Unsure why we are prepeding 'TEMP'
 	        };
             
             return generatorData;

example/generator/bundle.js

@@ -36,7 +36,7 @@ define([
 
     legacyRegistry.register("example/generator", {
         "name": "Sine Wave Generator",
-        "description": "For development use. Generates example streaming telemetry data using a simple sine wave algorithm.",
+        "description": "Example of a component that produces dataa.",
         "extensions": {
             "components": [
                 {
@@ -86,9 +86,8 @@ define([
                 {
                     "key": "generator",
                     "name": "Sine Wave Generator",
-                    "glyph": "\u0054",
-                    "description": "For development use. Generates example streaming telemetry data using a simple sine wave algorithm.",
-                    "priority": 10,
+                    "glyph": "T",
+                    "description": "A sine wave generator",
                     "features": "creation",
                     "model": {
                         "telemetry": {

example/imagery/bundle.js

@@ -49,10 +49,8 @@ define([
                 {
                     "key": "imagery",
                     "name": "Example Imagery",
-                    "glyph": "\u00e3",
+                    "glyph": "T",
                     "features": "creation",
-                    "description": "For development use. Creates example imagery data that mimics a live imagery stream.",
-                    "priority": 10,
                     "model": {
                         "telemetry": {}
                     },

example/msl/bundle.js

@@ -54,7 +54,7 @@ define([
             {
                 "name": "Measurement",
                 "key": "msl.measurement",
-                "glyph": "\u0054",
+                "glyph": "T",
                 "model": {"telemetry": {}},
                 "telemetry": {
                     "source": "rems.source",

example/msl/src/RemsTelemetryModelProvider.js

@@ -45,12 +45,11 @@ define(
             function buildTaxonomy(dictionary){
                 var models = {};
 
-                function addMeasurement(measurement, parent){
+                function addMeasurement(measurement){
                     var format = FORMAT_MAPPINGS[measurement.type];
                     models[makeId(measurement)] = {
                         type: "msl.measurement",
                         name: measurement.name,
-                        location: parent,
                         telemetry: {
                             key: measurement.identifier,
                             ranges: [{
@@ -63,24 +62,17 @@ define(
                     };
                 }
 
-                function addInstrument(subsystem, spacecraftId) {
-                    var measurements = (subsystem.measurements || []),
-                        instrumentId = makeId(subsystem);
-
-                    models[instrumentId] = {
+                function addInstrument(subsystem) {
+                    var measurements = (subsystem.measurements || []);
+                    models[makeId(subsystem)] = {
                         type: "msl.instrument",
                         name: subsystem.name,
-                        location: spacecraftId,
                         composition: measurements.map(makeId)
                     };
-                    measurements.forEach(function(measurement) {
-                        addMeasurement(measurement, instrumentId);
-                    });
+                    measurements.forEach(addMeasurement);
                 }
 
-                (dictionary.instruments || []).forEach(function(instrument) {
-                    addInstrument(instrument, "msl:curiosity");
-                });
+                (dictionary.instruments || []).forEach(addInstrument);
                 return models;
             }
 

example/plotOptions/bundle.js

@@ -80,10 +80,9 @@ define([
             "types": [
                 {
                     "key": "plot",
-                    "name": "Example Telemetry Plot",
-                    "glyph": "\u0074",
-                    "description": "For development use. A plot for displaying telemetry.",
-                    "priority": 10,
+                    "name": "Telemetry Plot",
+                    "glyph": "t",
+                    "description": "A plot for displaying telemetry",
                     "delegates": [
                         "telemetry"
                     ],

gulpfile.js

@@ -35,16 +35,13 @@ var gulp = require('gulp'),
     fs = require('fs'),
     git = require('git-rev-sync'),
     moment = require('moment'),
-    merge = require('merge-stream'),
     project = require('./package.json'),
-    _ = require('lodash'),
     paths = {
         main: 'main.js',
         dist: 'dist',
         assets: 'dist/assets',
         scss: ['./platform/**/*.scss', './example/**/*.scss'],
         scripts: [ 'main.js', 'platform/**/*.js', 'src/**/*.js' ],
-        specs: [ 'platform/**/*Spec.js', 'src/**/*Spec.js' ],
         static: [
             'index.html',
             'platform/**/*',
@@ -94,8 +91,7 @@ gulp.task('stylesheets', function () {
         .pipe(sourcemaps.init())
         .pipe(sass(options.sass).on('error', sass.logError))
         .pipe(rename(function (file) {
-            file.dirname =
-                file.dirname.replace(path.sep + 'sass', path.sep + 'css');
+            file.dirname = file.dirname.replace('/sass', '/css');
             return file;
         }))
         .pipe(sourcemaps.write('.'))
@@ -103,15 +99,8 @@ gulp.task('stylesheets', function () {
 });
 
 gulp.task('lint', function () {
-    var nonspecs = paths.specs.map(function (glob) {
-            return "!" + glob;
-        }),
-        scriptLint = gulp.src(paths.scripts.concat(nonspecs))
-            .pipe(jshint()),
-        specLint = gulp.src(paths.specs)
-            .pipe(jshint({ jasmine: true }));
-
-    return merge(scriptLint, specLint)
+    return gulp.src(paths.scripts)
+        .pipe(jshint())
         .pipe(jshint.reporter('default'))
         .pipe(jshint.reporter('fail'));
 });
@@ -147,6 +136,6 @@ gulp.task('develop', ['serve', 'stylesheets', 'watch']);
 
 gulp.task('install', [ 'static', 'scripts' ]);
 
-gulp.task('verify', [ 'lint', 'test', 'checkstyle' ]);
+gulp.task('verify', [ 'lint', 'test' ]);
 
 gulp.task('build', [ 'verify', 'install' ]);

index.html

@@ -30,11 +30,7 @@
     </script>
     <script type="text/javascript">
         require(['main'], function (mct) {
-            require([
-                './example/imagery/bundle',
-                './example/eventGenerator/bundle',
-                './example/generator/bundle'
-            ], mct.run.bind(mct));
+            mct.run();
         });
     </script>
     <link rel="stylesheet" href="platform/commonUI/general/res/css/startup-base.css">

main.js

@@ -19,7 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
-/*global requirejs*/
+/*global define, window, requirejs*/
 
 requirejs.config({
     "paths": {
@@ -28,37 +28,26 @@ requirejs.config({
         "angular-route": "bower_components/angular-route/angular-route.min",
         "csv": "bower_components/comma-separated-values/csv.min",
         "es6-promise": "bower_components/es6-promise/promise.min",
-        "EventEmitter": "bower_components/eventemitter3/index",
         "moment": "bower_components/moment/moment",
         "moment-duration-format": "bower_components/moment-duration-format/lib/moment-duration-format",
         "saveAs": "bower_components/FileSaver.js/FileSaver.min",
         "screenfull": "bower_components/screenfull/dist/screenfull.min",
         "text": "bower_components/text/text",
         "uuid": "bower_components/node-uuid/uuid",
-        "zepto": "bower_components/zepto/zepto.min",
-        "d3": "bower_components/d3/d3.min"
+        "zepto": "bower_components/zepto/zepto.min"
     },
     "shim": {
         "angular": {
             "exports": "angular"
         },
         "angular-route": {
-            "deps": ["angular"]
-        },
-        "EventEmitter": {
-            "exports": "EventEmitter"
+            "deps": [ "angular" ]
         },
         "moment-duration-format": {
-            "deps": ["moment"]
-        },
-        "screenfull": {
-            "exports": "screenfull"
+            "deps": [ "moment" ]
         },
         "zepto": {
             "exports": "Zepto"
-        },
-        "d3": {
-            "exports": "d3"
         }
     }
 });
@@ -90,9 +79,6 @@ define([
     './platform/features/pages/bundle',
     './platform/features/plot/bundle',
     './platform/features/timeline/bundle',
-    //'./platform/features/conductor/bundle',
-    './platform/features/conductor-v2/bundle',
-    './platform/features/conductor-v2-compatibility/bundle',
     './platform/features/table/bundle',
     './platform/forms/bundle',
     './platform/identity/bundle',
@@ -103,12 +89,18 @@ define([
     './platform/entanglement/bundle',
     './platform/search/bundle',
     './platform/status/bundle',
-    './platform/commonUI/regions/bundle'
+    './platform/commonUI/regions/bundle',
+
+    './example/imagery/bundle',
+    './example/eventGenerator/bundle',
+    './example/generator/bundle'
 ], function (Main, legacyRegistry) {
+    'use strict';
+
     return {
         legacyRegistry: legacyRegistry,
         run: function () {
             return new Main().run(legacyRegistry);
         }
     };
-});
+});

package.json

@@ -1,7 +1,7 @@
 {
-  "name": "openmct",
-  "version": "0.11.0-SNAPSHOT",
-  "description": "The Open MCT core platform",
+  "name": "openmctweb",
+  "version": "0.9.3-SNAPSHOT",
+  "description": "The Open MCT Web core platform",
   "dependencies": {
     "express": "^4.13.1",
     "minimist": "^1.1.1",
@@ -34,7 +34,6 @@
     "lodash": "^3.10.1",
     "markdown-toc": "^0.11.7",
     "marked": "^0.3.5",
-    "merge-stream": "^1.0.0",
     "mkdirp": "^0.5.1",
     "moment": "^2.11.1",
     "node-bourbon": "^4.2.3",
@@ -50,11 +49,11 @@
     "jsdoc": "jsdoc -c jsdoc.json -r -d target/docs/api",
     "otherdoc": "node docs/gendocs.js --in docs/src --out target/docs --suppress-toc 'docs/src/index.md|docs/src/process/index.md'",
     "docs": "npm run jsdoc ; npm run otherdoc",
-    "prepublish": "node ./node_modules/bower/bin/bower install && node ./node_modules/gulp/bin/gulp.js install"
+    "prepublish": "./node_modules/bower/bin/bower install && ./node_modules/gulp/bin/gulp.js install"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/nasa/openmct.git"
+    "url": "https://github.com/nasa/openmctweb.git"
   },
   "author": "",
   "license": "Apache-2.0",

platform/commonUI/about/bundle.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define([
     "text!./res/templates/about-dialog.html",
@@ -47,6 +48,7 @@ define([
     licensesExportMdTemplate,
     legacyRegistry
 ) {
+    "use strict";
 
     legacyRegistry.register("platform/commonUI/about", {
         "name": "About Open MCT Web",

platform/commonUI/about/res/templates/about-dialog.html

@@ -22,7 +22,7 @@
 <div class="abs t-about l-about t-about-openmctweb s-about" ng-controller = "AboutController as about">
     <div class="l-splash s-splash"></div>
     <div class="s-text l-content">
-        <h1 class="l-title s-title">Open MCT</h1>
+        <h1 class="l-title s-title">OpenMCT Web</h1>
         <div class="l-description s-description">
 	        <p>Open MCT Web, Copyright &copy; 2014-2015, United States Government as represented by the Administrator of the National Aeronautics and Space Administration. All rights reserved.</p>
 	        <p>Open MCT Web is licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License.  You may obtain a copy of the License at <a target="_blank" href="http://www.apache.org/licenses/LICENSE-2.0">http://www.apache.org/licenses/LICENSE-2.0</a>.</p>

platform/commonUI/about/src/AboutController.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 
 /**
@@ -28,6 +29,7 @@
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * The AboutController provides information to populate the

platform/commonUI/about/src/LicenseController.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Provides extension-introduced licenses information to the

platform/commonUI/about/src/LogoController.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * The LogoController provides functionality to the application

platform/commonUI/about/test/AboutControllerSpec.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 define(
     ['../src/AboutController'],
     function (AboutController) {
+        "use strict";
 
         describe("The About controller", function () {
             var testVersions,
@@ -55,4 +57,4 @@ define(
         });
 
     }
-);
+);

platform/commonUI/about/test/LicenseControllerSpec.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 define(
     ['../src/LicenseController'],
     function (LicenseController) {
+        "use strict";
 
         describe("The License controller", function () {
             var testLicenses,
@@ -46,4 +48,4 @@ define(
         });
 
     }
-);
+);

platform/commonUI/about/test/LogoControllerSpec.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 define(
     ['../src/LogoController'],
     function (LogoController) {
+        "use strict";
 
         describe("The About controller", function () {
             var mockOverlayService,
@@ -48,4 +50,4 @@ define(
         });
 
     }
-);
+);

platform/commonUI/browse/bundle.js

@@ -19,20 +19,29 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define([
     "./src/BrowseController",
     "./src/PaneController",
     "./src/BrowseObjectController",
+    "./src/creation/CreateMenuController",
+    "./src/creation/LocatorController",
     "./src/MenuArrowController",
     "./src/navigation/NavigationService",
+    "./src/creation/CreationPolicy",
     "./src/navigation/NavigateAction",
-    "./src/navigation/OrphanNavigationHandler",
     "./src/windowing/NewTabAction",
     "./src/windowing/FullscreenAction",
+    "./src/creation/CreateActionProvider",
+    "./src/creation/AddActionProvider",
+    "./src/creation/CreationService",
     "./src/windowing/WindowTitler",
     "text!./res/templates/browse.html",
+    "text!./res/templates/create/locator.html",
     "text!./res/templates/browse-object.html",
+    "text!./res/templates/create/create-button.html",
+    "text!./res/templates/create/create-menu.html",
     "text!./res/templates/items/grid-item.html",
     "text!./res/templates/browse/object-header.html",
     "text!./res/templates/menu-arrow.html",
@@ -45,15 +54,23 @@ define([
     BrowseController,
     PaneController,
     BrowseObjectController,
+    CreateMenuController,
+    LocatorController,
     MenuArrowController,
     NavigationService,
+    CreationPolicy,
     NavigateAction,
-    OrphanNavigationHandler,
     NewTabAction,
     FullscreenAction,
+    CreateActionProvider,
+    AddActionProvider,
+    CreationService,
     WindowTitler,
     browseTemplate,
+    locatorTemplate,
     browseObjectTemplate,
+    createButtonTemplate,
+    createMenuTemplate,
     gridItemTemplate,
     objectHeaderTemplate,
     menuArrowTemplate,
@@ -63,6 +80,7 @@ define([
     inspectorRegionTemplate,
     legacyRegistry
 ) {
+    "use strict";
 
     legacyRegistry.register("platform/commonUI/browse", {
         "extensions": {
@@ -93,9 +111,11 @@ define([
                         "$scope",
                         "$route",
                         "$location",
+                        "$window",
                         "objectService",
                         "navigationService",
                         "urlService",
+                        "policyService",
                         "DEFAULT_PATH"
                     ]
                 },
@@ -118,6 +138,22 @@ define([
                         "$route"
                     ]
                 },
+                {
+                    "key": "CreateMenuController",
+                    "implementation": CreateMenuController,
+                    "depends": [
+                        "$scope"
+                    ]
+                },
+                {
+                    "key": "LocatorController",
+                    "implementation": LocatorController,
+                    "depends": [
+                        "$scope",
+                        "$timeout",
+                        "objectService"
+                    ]
+                },
                 {
                     "key": "MenuArrowController",
                     "implementation": MenuArrowController,
@@ -126,6 +162,12 @@ define([
                     ]
                 }
             ],
+            "controls": [
+                {
+                    "key": "locator",
+                    "template": locatorTemplate
+                }
+            ],
             "representations": [
                 {
                     "key": "view-object",
@@ -141,6 +183,17 @@ define([
                         "view"
                     ]
                 },
+                {
+                    "key": "create-button",
+                    "template": createButtonTemplate
+                },
+                {
+                    "key": "create-menu",
+                    "template": createMenuTemplate,
+                    "uses": [
+                        "action"
+                    ]
+                },
                 {
                     "key": "grid-item",
                     "template": gridItemTemplate,
@@ -193,15 +246,19 @@ define([
                     "implementation": NavigationService
                 }
             ],
+            "policies": [
+                {
+                    "implementation": CreationPolicy,
+                    "category": "creation"
+                }
+            ],
             "actions": [
                 {
                     "key": "navigate",
                     "implementation": NavigateAction,
                     "depends": [
                         "navigationService",
-                        "$q",
-                        "policyService",
-                        "$window"
+                        "$q"
                     ]
                 },
                 {
@@ -247,6 +304,42 @@ define([
                     "editable": false
                 }
             ],
+            "components": [
+                {
+                    "key": "CreateActionProvider",
+                    "provides": "actionService",
+                    "type": "provider",
+                    "implementation": CreateActionProvider,
+                    "depends": [
+                        "$q",
+                        "typeService",
+                        "navigationService",
+                        "policyService"
+                    ]
+                },
+                {
+                    "key": "AddActionProvider",
+                    "provides": "actionService",
+                    "type": "provider",
+                    "implementation": AddActionProvider,
+                    "depends": [
+                        "$q",
+                        "typeService",
+                        "dialogService",
+                        "policyService"
+                    ]
+                },
+                {
+                    "key": "CreationService",
+                    "provides": "creationService",
+                    "type": "provider",
+                    "implementation": CreationService,
+                    "depends": [
+                        "$q",
+                        "$log"
+                    ]
+                }
+            ],
             "runs": [
                 {
                     "implementation": WindowTitler,
@@ -255,14 +348,6 @@ define([
                         "$rootScope",
                         "$document"
                     ]
-                },
-                {
-                    "implementation": OrphanNavigationHandler,
-                    "depends": [
-                        "throttle",
-                        "topic",
-                        "navigationService"
-                    ]
                 }
             ],
             "licenses": [

platform/commonUI/browse/res/templates/browse-object.html

@@ -43,8 +43,23 @@
             </mct-representation>
         </div>
     </div>
-    <div class="holder l-flex-col flex-elem grows l-object-wrapper l-controls-visible l-time-controller-visible">
-        <div class="holder l-flex-col flex-elem grows l-object-wrapper-inner">
+    <div class="holder l-flex-col flex-elem grows l-object-wrapper">
+        <div ng-if="isEditable" class="holder l-flex-col flex-elem grows l-object-wrapper-inner">
+            <!-- Toolbar and Save/Cancel buttons -->
+            <div class="l-edit-controls flex-elem l-flex-row flex-align-end">
+                <mct-representation key="'edit-action-buttons'"
+                                    mct-object="domainObject"
+                                    class='flex-elem conclude-editing'>
+                </mct-representation>
+
+            </div>
+            <mct-representation key="representation.selected.key"
+                                mct-object="representation.selected.key && domainObject"
+                                class="abs flex-elem grows object-holder-main scroll"
+                                toolbar="toolbar">
+            </mct-representation>
+        </div>
+        <div ng-if="!isEditable" class="holder l-flex-col flex-elem grows l-object-wrapper-inner">
             <!-- Toolbar and Save/Cancel buttons -->
             <div class="l-edit-controls flex-elem l-flex-row flex-align-end">
                 <mct-representation key="'edit-action-buttons'"
@@ -59,9 +74,4 @@
             </mct-representation>
         </div>
     </div>
-    <!-- put time conductor in here? -->
-    <mct-representation mct-object="domainObject"
-                        key="'time-conductor'"
-                        class="abs holder flex-elem flex-fixed l-flex-row l-time-conductor-holder">
-    </mct-representation>
 </div>

platform/commonUI/browse/res/templates/browse/object-header.html

@@ -26,5 +26,5 @@
     <mct-representation
         key="'menu-arrow'"
         mct-object='domainObject'
-        class="flex-elem context-available-w"></mct-representation>
+        class="flex-elem"></mct-representation>
 </span>

platform/commonUI/browse/res/templates/create/create-menu.html

@@ -36,14 +36,14 @@
 
         </ul>
     </div>
-    <div class="pane right menu-item-description l-flex-col">
-        <div class="desc-area flex-elem holder ui-symbol icon type-icon">
+    <div class="pane right menu-item-description">
+        <div class="desc-area ui-symbol icon type-icon">
             {{representation.activeMetadata.glyph}}
         </div>
-        <div class="desc-area flex-elem holder title">
+        <div class="desc-area title">
             {{representation.activeMetadata.name}}
         </div>
-        <div class="desc-area flex-elem holder description">
+        <div class="desc-area description">
             {{representation.activeMetadata.description}}
         </div>
     </div>

platform/commonUI/browse/src/BrowseController.js

@@ -19,14 +19,18 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise, confirm*/
 
 /**
  * This bundle implements Browse mode.
  * @namespace platform/commonUI/browse
  */
 define(
-    [],
-    function () {
+    [
+        '../../../representation/src/gestures/GestureConstants'
+    ],
+    function (GestureConstants) {
+        "use strict";
 
         var ROOT_ID = "ROOT";
 
@@ -41,12 +45,14 @@ define(
          * @constructor
          */
         function BrowseController(
-            $scope,
-            $route,
-            $location,
-            objectService,
-            navigationService,
-            urlService,
+            $scope, 
+            $route, 
+            $location, 
+            $window, 
+            objectService, 
+            navigationService, 
+            urlService, 
+            policyService,
             defaultPath
         ) {
             var path = [ROOT_ID].concat(
@@ -73,10 +79,25 @@ define(
 
             }
 
-            function setScopeObjects(domainObject, navigationAllowed) {
+            // Callback for updating the in-scope reference to the object
+            // that is currently navigated-to.
+            function setNavigation(domainObject) {
+                var navigationAllowed = true;
+
+                if (domainObject === $scope.navigatedObject){
+                    //do nothing;
+                    return;
+                }
+
+                policyService.allow("navigation", $scope.navigatedObject, domainObject, function(message){
+                    navigationAllowed = $window.confirm(message + "\r\n\r\n" +
+                        " Are you sure you want to continue?");
+                });
+
                 if (navigationAllowed) {
                     $scope.navigatedObject = domainObject;
                     $scope.treeModel.selectedObject = domainObject;
+                    navigationService.setNavigation(domainObject);
                     updateRoute(domainObject);
                 } else {
                     //If navigation was unsuccessful (ie. blocked), reset
@@ -86,20 +107,6 @@ define(
                 }
             }
 
-            // Callback for updating the in-scope reference to the object
-            // that is currently navigated-to.
-            function setNavigation(domainObject) {
-                if (domainObject === $scope.navigatedObject) {
-                    //do nothing;
-                    return;
-                }
-                if (domainObject) {
-                    domainObject.getCapability("action").perform("navigate").then(setScopeObjects.bind(undefined, domainObject));
-                } else {
-                    setScopeObjects(domainObject, true);
-                }
-            }
-
             function navigateTo(domainObject) {
 
                 // Check if an object has been navigated-to already...

platform/commonUI/browse/src/BrowseObjectController.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Controller for the `browse-object` representation of a domain
@@ -33,7 +35,7 @@ define(
         function BrowseObjectController($scope, $location, $route) {
             var navigatedObject;
             function setViewForDomainObject(domainObject) {
-
+                
                 var locationViewKey = $location.search().view;
 
                 function selectViewIfMatching(view) {
@@ -70,7 +72,7 @@ define(
             $scope.$watch('domainObject', setViewForDomainObject);
             $scope.$watch('representation.selected.key', updateQueryParam);
 
-            $scope.doAction = function (action) {
+            $scope.doAction = function (action){
                 return $scope[action] && $scope[action]();
             };
 

platform/commonUI/browse/src/InspectorRegion.js

@@ -19,12 +19,14 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,window*/
 
 define(
     [
         '../../regions/src/Region'
     ],
     function (Region) {
+        "use strict";
 
         /**
          * Defines the a default Inspector region. Captured in a class to
@@ -45,7 +47,7 @@ define(
         /**
          * @private
          */
-        InspectorRegion.prototype.buildRegion = function () {
+        InspectorRegion.prototype.buildRegion = function() {
             var metadataRegion = {
                 name: 'metadata',
                 title: 'Metadata Region',

platform/commonUI/browse/src/MenuArrowController.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 /**
  * Module defining MenuArrowController. Created by shale on 06/30/2015.
@@ -26,11 +27,12 @@
 define(
     [],
     function () {
-
+        "use strict";
+        
         /**
-         * A left-click on the menu arrow should display a
-         * context menu. This controller launches the context
-         * menu.
+         * A left-click on the menu arrow should display a 
+         * context menu. This controller launches the context 
+         * menu. 
          * @memberof platform/commonUI/browse
          * @constructor
          */

platform/commonUI/browse/src/PaneController.js

@@ -19,11 +19,13 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Controller to provide the ability to show/hide the tree in

platform/commonUI/browse/src/creation/AddAction.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining AddAction. Created by ahenry on 01/21/16.
@@ -28,6 +29,7 @@ define(
         './CreateWizard'
     ],
     function (CreateWizard) {
+        "use strict";
 
         /**
          * The Add Action is performed to create new instances of
@@ -81,33 +83,37 @@ define(
 
             newModel.type = this.type.getKey();
             newObject = parentObject.getCapability('instantiation').instantiate(newModel);
-            newObject.useCapability('mutation', function (model) {
+            newObject.useCapability('mutation', function(model){
                 model.location = parentObject.getId();
             });
 
             wizard = new CreateWizard(newObject, this.parent, this.policyService);
 
-            function populateObjectFromInput(formValue) {
+            function populateObjectFromInput (formValue) {
                 return wizard.populateObjectFromInput(formValue, newObject);
             }
 
-            function persistAndReturn(domainObject) {
-                return domainObject.getCapability('persistence')
-                    .persist()
-                    .then(function () {
-                        return domainObject;
-                    });
+            function addToParent (populatedObject) {
+                parentObject.getCapability('composition').add(populatedObject);
+                return parentObject.getCapability('persistence').persist().then(function(){
+                    return parentObject;
+                });
             }
 
-            function addToParent(populatedObject) {
-                parentObject.getCapability('composition').add(populatedObject);
-                return persistAndReturn(parentObject);
+            function save(object) {
+                /*
+                It's necessary to persist the new sub-object in order
+                 that it can be retrieved for composition in the parent.
+                 Future refactoring that allows temporary objects to be
+                 retrieved from object services will make this unnecessary.
+                 */
+                return object.getCapability('editor').save(true);
             }
 
             return this.dialogService
                 .getUserInput(wizard.getFormStructure(false), wizard.getInitialFormValue())
                 .then(populateObjectFromInput)
-                .then(persistAndReturn)
+                .then(save)
                 .then(addToParent);
 
         };
@@ -125,7 +131,7 @@ define(
          * @returns {AddActionMetadata} metadata about this action
          */
         AddAction.prototype.getMetadata = function () {
-            return this.metadata;
+           return this.metadata;
         };
 
         return AddAction;

platform/commonUI/browse/src/creation/AddActionProvider.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining AddActionProvider.js. Created by ahenry on 01/21/16.
@@ -26,6 +27,7 @@
 define(
     ["./AddAction"],
     function (AddAction) {
+        "use strict";
 
         /**
          * The AddActionProvider is an ActionProvider which introduces

platform/commonUI/browse/src/creation/CreateAction.js

@@ -19,13 +19,18 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining CreateAction. Created by vwoeltje on 11/10/14.
  */
 define(
-    [],
-    function () {
+    [
+        './CreateWizard',
+        '../../../edit/src/objects/EditableDomainObject'
+    ],
+    function (CreateWizard, EditableDomainObject) {
+        "use strict";
 
         /**
          * The Create Action is performed to create new instances of
@@ -43,8 +48,11 @@ define(
          *        override this)
          * @param {ActionContext} context the context in which the
          *        action is being performed
+         * @param {NavigationService} navigationService the navigation service,
+         *        which handles changes in navigation. It allows the object
+         *        being browsed/edited to be set.
          */
-        function CreateAction(type, parent, context) {
+        function CreateAction(type, parent, context, $q, navigationService) {
             this.metadata = {
                 key: 'create',
                 glyph: type.getGlyph(),
@@ -53,8 +61,24 @@ define(
                 description: type.getDescription(),
                 context: context
             };
+
             this.type = type;
             this.parent = parent;
+            this.navigationService = navigationService;
+            this.$q = $q;
+        }
+
+        // Get a count of views which are not flagged as non-editable.
+        function countEditableViews(domainObject) {
+            var views = domainObject && domainObject.useCapability('view'),
+                count = 0;
+
+            // A view is editable unless explicitly flagged as not
+            (views || []).forEach(function (view) {
+                count += (view.editable !== false) ? 1 : 0;
+            });
+
+            return count;
         }
 
         /**
@@ -63,31 +87,23 @@ define(
          */
         CreateAction.prototype.perform = function () {
             var newModel = this.type.getInitialModel(),
+                parentObject = this.navigationService.getNavigation(),
                 newObject,
-                editAction,
-                editorCapability;
-
-            function onSave() {
-                return editorCapability.save();
-            }
-
-            function onCancel() {
-                return editorCapability.cancel();
-            }
+                editableObject;
 
             newModel.type = this.type.getKey();
-            newModel.location = this.parent.getId();
-            newObject = this.parent.useCapability('instantiation', newModel);
-            editorCapability = newObject.hasCapability('editor') && newObject.getCapability("editor");
+            newObject = parentObject.useCapability('instantiation', newModel);
+            editableObject = new EditableDomainObject(newObject, this.$q);
+            editableObject.setOriginalObject(parentObject);
+            editableObject.getCapability('status').set('editing', true);
+            editableObject.useCapability('mutation', function(model){
+                model.location = parentObject.getId();
+            });
 
-            editAction = newObject.getCapability("action").getActions("edit")[0];
-            //If an edit action is available, perform it
-            if (editAction) {
-                return editAction.perform();
-            } else if (editorCapability) {
-                //otherwise, use the save action
-                editorCapability.edit();
-                return newObject.getCapability("action").perform("save").then(onSave, onCancel);
+            if (countEditableViews(editableObject) > 0 && editableObject.hasCapability('composition')) {
+                this.navigationService.setNavigation(editableObject);
+            } else {
+                return editableObject.getCapability('action').perform('save');
             }
         };
 
@@ -104,7 +120,7 @@ define(
          * @returns {CreateActionMetadata} metadata about this action
          */
         CreateAction.prototype.getMetadata = function () {
-            return this.metadata;
+           return this.metadata;
         };
 
         return CreateAction;

platform/commonUI/browse/src/creation/CreateActionProvider.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining CreateActionProvider.js. Created by vwoeltje on 11/10/14.
@@ -26,6 +27,7 @@
 define(
     ["./CreateAction"],
     function (CreateAction) {
+        "use strict";
 
         /**
          * The CreateActionProvider is an ActionProvider which introduces
@@ -44,8 +46,10 @@ define(
          *        introduced in this bundle), responsible for handling actual
          *        object creation.
          */
-        function CreateActionProvider(typeService, policyService) {
+        function CreateActionProvider($q, typeService, navigationService, policyService) {
             this.typeService = typeService;
+            this.navigationService = navigationService;
+            this.$q = $q;
             this.policyService = policyService;
         }
 
@@ -70,7 +74,9 @@ define(
                 return new CreateAction(
                     type,
                     destination,
-                    context
+                    context,
+                    self.$q,
+                    self.navigationService
                 );
             });
         };

platform/commonUI/browse/src/creation/CreateMenuController.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining CreateMenuController. Created by vwoeltje on 11/10/14.
@@ -26,6 +27,7 @@
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Controller for the Create menu; maintains an up-to-date

platform/commonUI/browse/src/creation/CreateWizard.js

@@ -19,9 +19,11 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     function () {
+        'use strict';
 
         /**
          * A class for capturing user input data from an object creation
@@ -111,12 +113,12 @@ define(
          * @param formValue
          * @returns {DomainObject}
          */
-        CreateWizard.prototype.populateObjectFromInput = function (formValue) {
+        CreateWizard.prototype.populateObjectFromInput = function(formValue) {
             var parent = this.getLocation(formValue),
                 formModel = this.createModel(formValue);
 
             formModel.location = parent.getId();
-            this.domainObject.useCapability("mutation", function () {
+            this.domainObject.useCapability("mutation", function(){
                 return formModel;
             });
             return this.domainObject;

platform/commonUI/browse/src/creation/CreationPolicy.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * A policy for determining whether objects of a given type can be
@@ -40,4 +42,4 @@ define(
 
         return CreationPolicy;
     }
-);
+);

platform/commonUI/browse/src/creation/CreationService.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining CreateService. Created by vwoeltje on 11/10/14.
@@ -26,9 +27,12 @@
 define(
     [],
     function () {
+        "use strict";
 
         var NON_PERSISTENT_WARNING =
-                "Tried to create an object in non-persistent container.";
+                "Tried to create an object in non-persistent container.",
+            NO_COMPOSITION_WARNING =
+                "Could not add to composition; no composition in ";
 
         /**
          * The creation service is responsible for instantiating and

platform/commonUI/browse/src/creation/LocatorController.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Controller for the "locator" control, which provides the
@@ -50,14 +52,14 @@ define(
                         $scope.rootObject =
                             (context && context.getRoot()) || $scope.rootObject;
                     }, 0);
-                } else if (!contextRoot) {
+                } else if (!contextRoot){
                     //If no context root is available, default to the root
                     // object
                     $scope.rootObject = undefined;
                     // Update the displayed tree on a timeout to avoid
                     // an infinite digest exception.
                     objectService.getObjects(['ROOT'])
-                        .then(function (objects) {
+                        .then(function(objects){
                             $timeout(function () {
                                 $scope.rootObject = objects.ROOT;
                             }, 0);

platform/commonUI/browse/src/navigation/NavigateAction.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining NavigateAction. Created by vwoeltje on 11/10/14.
@@ -26,6 +27,7 @@
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * The navigate action navigates to a specific domain object.
@@ -33,12 +35,10 @@ define(
          * @constructor
          * @implements {Action}
          */
-        function NavigateAction(navigationService, $q, policyService, $window, context) {
+        function NavigateAction(navigationService, $q, context) {
             this.domainObject = context.domainObject;
             this.$q = $q;
             this.navigationService = navigationService;
-            this.policyService = policyService;
-            this.$window = $window;
         }
 
         /**
@@ -47,20 +47,9 @@ define(
          *          navigation has been updated
          */
         NavigateAction.prototype.perform = function () {
-            var self = this,
-                navigationAllowed = true;
-
-            function allow() {
-                self.policyService.allow("navigation", self.navigationService.getNavigation(), self.domainObject, function (message) {
-                    navigationAllowed = self.$window.confirm(message + "\r\n\r\n" +
-                        " Are you sure you want to continue?");
-                });
-                return navigationAllowed;
-            }
-
             // Set navigation, and wrap like a promise
             return this.$q.when(
-                allow() && this.navigationService.setNavigation(this.domainObject)
+                this.navigationService.setNavigation(this.domainObject)
             );
         };
 

platform/commonUI/browse/src/navigation/NavigationService.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining NavigationService. Created by vwoeltje on 11/10/14.
@@ -26,6 +27,7 @@
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * The navigation service maintains the application's current

platform/commonUI/browse/src/navigation/OrphanNavigationHandler.js

@@ -1,75 +0,0 @@
-/*****************************************************************************
- * Open MCT Web, Copyright (c) 2014-2015, United States Government
- * as represented by the Administrator of the National Aeronautics and Space
- * Administration. All rights reserved.
- *
- * Open MCT Web is licensed under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- * http://www.apache.org/licenses/LICENSE-2.0.
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations
- * under the License.
- *
- * Open MCT Web includes source code licensed under additional open source
- * licenses. See the Open Source Licenses file (LICENSES.md) included with
- * this source code distribution or the Licensing information page available
- * at runtime from the About dialog for additional information.
- *****************************************************************************/
-
-define([], function () {
-
-    /**
-     * Navigates away from orphan objects whenever they are detected.
-     *
-     * An orphan object is an object whose apparent parent does not
-     * actually contain it. This may occur in certain circumstances, such
-     * as when persistence succeeds for a newly-created object but fails
-     * for its parent.
-     *
-     * @param throttle the `throttle` service
-     * @param topic the `topic` service
-     * @param navigationService the `navigationService`
-     * @constructor
-     */
-    function OrphanNavigationHandler(throttle, topic, navigationService) {
-        var throttledCheckNavigation;
-
-        function getParent(domainObject) {
-            var context = domainObject.getCapability('context');
-            return context.getParent();
-        }
-
-        function isOrphan(domainObject) {
-            var parent = getParent(domainObject),
-                composition = parent.getModel().composition,
-                id = domainObject.getId();
-            return !composition || (composition.indexOf(id) === -1);
-        }
-
-        function navigateToParent(domainObject) {
-            var parent = getParent(domainObject);
-            return parent.getCapability('action').perform('navigate');
-        }
-
-        function checkNavigation() {
-            var navigatedObject = navigationService.getNavigation();
-            if (navigatedObject.hasCapability('context') &&
-                isOrphan(navigatedObject)) {
-                if (!navigatedObject.getCapability('editor').isEditContextRoot()) {
-                    navigateToParent(navigatedObject);
-                }
-            }
-        }
-
-        throttledCheckNavigation = throttle(checkNavigation);
-
-        navigationService.addListener(throttledCheckNavigation);
-        topic('mutation').listen(throttledCheckNavigation);
-    }
-
-    return OrphanNavigationHandler;
-});

platform/commonUI/browse/src/windowing/FullscreenAction.js

@@ -19,13 +19,15 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,screenfull,Promise*/
 
 /**
  * Module defining FullscreenAction. Created by vwoeltje on 11/18/14.
  */
 define(
     ["screenfull"],
-    function (screenfull) {
+    function () {
+        "use strict";
 
         var ENTER_FULLSCREEN = "Enter full screen mode",
             EXIT_FULLSCREEN = "Exit full screen mode";

platform/commonUI/browse/src/windowing/NewTabAction.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining NewTabAction (Originally NewWindowAction). Created by vwoeltje on 11/18/14.
@@ -26,6 +27,9 @@
 define(
     [],
     function () {
+        "use strict";
+        var ROOT_ID = "ROOT",
+            DEFAULT_PATH = "/mine";
         /**
          * The new tab action allows a domain object to be opened
          * into a new browser tab.

platform/commonUI/browse/src/windowing/WindowTitler.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Updates the title of the current window to reflect the name

platform/commonUI/browse/test/BrowseControllerSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine,xit,xdescribe*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../src/BrowseController"],
     function (BrowseController) {
+        "use strict";
 
         describe("The browse controller", function () {
             var mockScope,
@@ -37,8 +39,9 @@ define(
                 mockUrlService,
                 mockDomainObject,
                 mockNextObject,
+                mockWindow,
+                mockPolicyService,
                 testDefaultRoot,
-                mockActionCapability,
                 controller;
 
             function mockPromise(value) {
@@ -54,24 +57,35 @@ define(
                     mockScope,
                     mockRoute,
                     mockLocation,
+                    mockWindow,
                     mockObjectService,
                     mockNavigationService,
                     mockUrlService,
+                    mockPolicyService,
                     testDefaultRoot
                 );
             }
 
             beforeEach(function () {
+                mockWindow = jasmine.createSpyObj('$window', [
+                   "confirm"
+                ]);
+                mockWindow.confirm.andReturn(true);
+
+                mockPolicyService = jasmine.createSpyObj('policyService', [
+                    'allow'
+                ]);
+
                 testDefaultRoot = "some-root-level-domain-object";
 
                 mockScope = jasmine.createSpyObj(
                     "$scope",
-                    ["$on", "$watch"]
+                    [ "$on", "$watch" ]
                 );
                 mockRoute = { current: { params: {} } };
                 mockLocation = jasmine.createSpyObj(
                     "$location",
-                    ["path"]
+                    [ "path" ]
                 );
                 mockUrlService = jasmine.createSpyObj(
                     "urlService",
@@ -79,7 +93,7 @@ define(
                 );
                 mockObjectService = jasmine.createSpyObj(
                     "objectService",
-                    ["getObjects"]
+                    [ "getObjects" ]
                 );
                 mockNavigationService = jasmine.createSpyObj(
                     "navigationService",
@@ -92,15 +106,15 @@ define(
                 );
                 mockRootObject = jasmine.createSpyObj(
                     "domainObject",
-                    ["getId", "getCapability", "getModel", "useCapability"]
+                    [ "getId", "getCapability", "getModel", "useCapability" ]
                 );
                 mockDomainObject = jasmine.createSpyObj(
                     "domainObject",
-                    ["getId", "getCapability", "getModel", "useCapability"]
+                    [ "getId", "getCapability", "getModel", "useCapability" ]
                 );
                 mockNextObject = jasmine.createSpyObj(
                     "nextObject",
-                    ["getId", "getCapability", "getModel", "useCapability"]
+                    [ "getId", "getCapability", "getModel", "useCapability" ]
                 );
 
                 mockObjectService.getObjects.andReturn(mockPromise({
@@ -116,8 +130,6 @@ define(
                 mockNextObject.getId.andReturn("next");
                 mockDomainObject.getId.andReturn(testDefaultRoot);
 
-                mockActionCapability = jasmine.createSpyObj('actionCapability', ['perform']);
-
                 instantiateController();
             });
 
@@ -201,13 +213,8 @@ define(
                 mockContext.getPath.andReturn(
                     [mockRootObject, mockDomainObject, mockNextObject]
                 );
-
-                //Return true from navigate action
-                mockActionCapability.perform.andReturn(mockPromise(true));
-
                 mockNextObject.getCapability.andCallFake(function (c) {
-                    return (c === 'context' && mockContext) ||
-                        (c === 'action' && mockActionCapability);
+                    return c === 'context' && mockContext;
                 });
                 mockScope.$on.andReturn(mockUnlisten);
                 // Provide a navigation change
@@ -220,7 +227,6 @@ define(
                 mockLocation.path.andReturn("/browse/");
 
                 mockNavigationService.setNavigation.andReturn(true);
-                mockActionCapability.perform.andReturn(mockPromise(true));
 
                 // Exercise the Angular workaround
                 mockNavigationService.addListener.mostRecentCall.args[0]();
@@ -239,9 +245,6 @@ define(
                 mockScope.navigatedObject = mockDomainObject;
                 mockNavigationService.setNavigation.andReturn(true);
 
-                mockActionCapability.perform.andReturn(mockPromise(true));
-                mockNextObject.getCapability.andReturn(mockActionCapability);
-
                 //Simulate a change in selected tree object
                 mockScope.treeModel = {selectedObject: mockDomainObject};
                 mockScope.$watch.mostRecentCall.args[1](mockNextObject);
@@ -253,10 +256,11 @@ define(
             it("after failed navigation event resets the selected tree" +
                 " object", function () {
                 mockScope.navigatedObject = mockDomainObject;
-
-                //Return false from navigation action
-                mockActionCapability.perform.andReturn(mockPromise(false));
-                mockNextObject.getCapability.andReturn(mockActionCapability);
+                mockWindow.confirm.andReturn(false);
+                mockPolicyService.allow.andCallFake(function(category, object, context, callback){
+                    callback("unsaved changes");
+                    return false;
+                });
 
                 //Simulate a change in selected tree object
                 mockScope.treeModel = {selectedObject: mockDomainObject};

platform/commonUI/browse/test/BrowseObjectControllerSpec.js

@@ -19,11 +19,13 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 
 define(
     ["../src/BrowseObjectController"],
     function (BrowseObjectController) {
+        "use strict";
 
         describe("The browse object controller", function () {
             var mockScope,
@@ -44,12 +46,12 @@ define(
             beforeEach(function () {
                 mockScope = jasmine.createSpyObj(
                     "$scope",
-                    ["$on", "$watch"]
+                    [ "$on", "$watch" ]
                 );
                 mockRoute = { current: { params: {} } };
                 mockLocation = jasmine.createSpyObj(
                     "$location",
-                    ["path", "search"]
+                    [ "path", "search" ]
                 );
                 mockUnlisten = jasmine.createSpy("unlisten");
 
@@ -69,7 +71,7 @@ define(
                 // Allows the path index to be checked
                 // prior to setting $route.current
                 mockLocation.path.andReturn("/browse/");
-
+                
                 // Exercise the Angular workaround
                 mockScope.$on.mostRecentCall.args[1]();
                 expect(mockUnlisten).toHaveBeenCalled();

platform/commonUI/browse/test/InspectorRegionSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTIncudeSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../src/InspectorRegion"],
     function (InspectorRegion) {
+        "use strict";
 
         describe("The inspector region", function () {
             var inspectorRegion;
@@ -40,4 +42,4 @@ define(
 
         });
     }
-);
+);

platform/commonUI/browse/test/MenuArrowControllerSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MenuArrowControllerSpec. Created by shale on 07/02/2015.
@@ -26,7 +27,8 @@
 define(
     ["../src/MenuArrowController"],
     function (MenuArrowController) {
-
+        "use strict";
+        
         describe("The menu arrow controller ", function () {
             var mockScope,
                 mockDomainObject,
@@ -34,43 +36,43 @@ define(
                 mockContextMenuAction,
                 mockActionContext,
                 controller;
-
+            
             beforeEach(function () {
                 mockScope = jasmine.createSpyObj(
                     "$scope",
-                    [""]
+                    [ "" ]
                 );
                 mockDomainObject = jasmine.createSpyObj(
                     "domainObject",
-                    ["getCapability"]
+                    [ "getCapability" ]
                 );
                 mockEvent = jasmine.createSpyObj(
                     "event",
-                    ["preventDefault"]
+                    [ "preventDefault" ]
                 );
                 mockContextMenuAction = jasmine.createSpyObj(
                     "action",
-                    ["perform", "getActions"]
+                    [ "perform", "getActions" ]
                 );
                 mockActionContext = jasmine.createSpyObj(
                     "actionContext",
-                    [""]
+                    [ "" ]
                 );
-
+                
                 mockActionContext.domainObject = mockDomainObject;
                 mockActionContext.event = mockEvent;
                 mockScope.domainObject = mockDomainObject;
                 mockDomainObject.getCapability.andReturn(mockContextMenuAction);
                 mockContextMenuAction.perform.andReturn(jasmine.any(Function));
-
+                
                 controller = new MenuArrowController(mockScope);
             });
-
+            
             it("calls the context menu action when clicked", function () {
                 // Simulate a click on the menu arrow
                 controller.showMenu(mockEvent);
-
-                // Expect the menu action to be performed
+                
+                // Expect the menu action to be performed 
                 expect(mockDomainObject.getCapability).toHaveBeenCalledWith('action');
                 expect(mockContextMenuAction.perform).toHaveBeenCalled();
             });

platform/commonUI/browse/test/PaneControllerSpec.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 define(
     ["../src/PaneController"],
     function (PaneController) {
+        'use strict';
 
         describe("The PaneController", function () {
             var mockScope,
@@ -42,11 +44,11 @@ define(
             }
 
             beforeEach(function () {
-                mockScope = jasmine.createSpyObj("$scope", ["$on"]);
+                mockScope = jasmine.createSpyObj("$scope", [ "$on" ]);
                 mockDomainObjects = ['a', 'b'].map(function (id) {
                     var mockDomainObject = jasmine.createSpyObj(
                         'domainObject-' + id,
-                        ['getId', 'getModel', 'getCapability']
+                        [ 'getId', 'getModel', 'getCapability' ]
                     );
 
                     mockDomainObject.getId.andReturn(id);
@@ -56,7 +58,7 @@ define(
                 });
                 mockAgentService = jasmine.createSpyObj(
                     "agentService",
-                    ["isMobile", "isPhone", "isTablet", "isPortrait", "isLandscape"]
+                    [ "isMobile", "isPhone", "isTablet", "isPortrait", "isLandscape" ]
                 );
                 mockWindow = jasmine.createSpyObj("$window", ["open"]);
             });

platform/commonUI/browse/test/creation/AddActionProviderSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine,xit,xdescribe*/
 
 /**
  * MCTRepresentationSpec. Created by ahenry on 01/21/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/creation/AddActionProvider"],
     function (AddActionProvider) {
+        "use strict";
 
         describe("The add action provider", function () {
             var mockTypeService,
@@ -60,37 +62,37 @@ define(
             beforeEach(function () {
                 mockTypeService = jasmine.createSpyObj(
                     "typeService",
-                    ["listTypes"]
+                    [ "listTypes" ]
                 );
                 mockDialogService = jasmine.createSpyObj(
                     "dialogService",
-                    ["getUserInput"]
+                    [ "getUserInput" ]
                 );
                 mockPolicyService = jasmine.createSpyObj(
                     "policyService",
-                    ["allow"]
+                    [ "allow" ]
                 );
 
                 mockDomainObject = jasmine.createSpyObj(
                     "domainObject",
-                    ["getCapability"]
+                    [ "getCapability" ]
                 );
 
                 //Mocking getCapability because AddActionProvider uses the
                 // type capability of the destination object.
                 mockDomainObject.getCapability.andReturn({});
 
-                mockTypes = ["A", "B", "C"].map(createMockType);
+                mockTypes = [ "A", "B", "C" ].map(createMockType);
 
-                mockTypes.forEach(function (type) {
+                mockTypes.forEach(function(type){
                     mockPolicyMap[type.getName()] = true;
                 });
 
-                mockCreationPolicy = function (type) {
+                mockCreationPolicy = function(type){
                     return mockPolicyMap[type.getName()];
                 };
 
-                mockCompositionPolicy = function () {
+                mockCompositionPolicy = function(){
                     return true;
                 };
 
@@ -132,4 +134,4 @@ define(
             });
         });
     }
-);
+);

platform/commonUI/browse/test/creation/CreateActionProviderSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine,xit,xdescribe*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,13 +27,17 @@
 define(
     ["../../src/creation/CreateActionProvider"],
     function (CreateActionProvider) {
+        "use strict";
 
         describe("The create action provider", function () {
             var mockTypeService,
+                mockDialogService,
+                mockNavigationService,
                 mockPolicyService,
                 mockCreationPolicy,
                 mockPolicyMap = {},
                 mockTypes,
+                mockQ,
                 provider;
 
             function createMockType(name) {
@@ -56,31 +61,41 @@ define(
             beforeEach(function () {
                 mockTypeService = jasmine.createSpyObj(
                     "typeService",
-                    ["listTypes"]
+                    [ "listTypes" ]
+                );
+                mockDialogService = jasmine.createSpyObj(
+                    "dialogService",
+                    [ "getUserInput" ]
+                );
+                mockNavigationService = jasmine.createSpyObj(
+                    "navigationService",
+                    [ "setNavigation" ]
                 );
                 mockPolicyService = jasmine.createSpyObj(
                     "policyService",
-                    ["allow"]
+                    [ "allow" ]
                 );
 
-                mockTypes = ["A", "B", "C"].map(createMockType);
+                mockTypes = [ "A", "B", "C" ].map(createMockType);
 
-                mockTypes.forEach(function (type) {
+                mockTypes.forEach(function(type){
                     mockPolicyMap[type.getName()] = true;
                 });
 
-                mockCreationPolicy = function (type) {
+                mockCreationPolicy = function(type){
                     return mockPolicyMap[type.getName()];
                 };
 
-                mockPolicyService.allow.andCallFake(function (category, type) {
+                mockPolicyService.allow.andCallFake(function(category, type){
                     return category === "creation" && mockCreationPolicy(type) ? true : false;
                 });
 
                 mockTypeService.listTypes.andReturn(mockTypes);
 
                 provider = new CreateActionProvider(
+                    mockQ,
                     mockTypeService,
+                    mockNavigationService,
                     mockPolicyService
                 );
             });
@@ -113,4 +128,4 @@ define(
             });
         });
     }
-);
+);

platform/commonUI/browse/test/creation/CreateActionSpec.js

@@ -0,0 +1,132 @@
+/*****************************************************************************
+ * Open MCT Web, Copyright (c) 2014-2015, United States Government
+ * as represented by the Administrator of the National Aeronautics and Space
+ * Administration. All rights reserved.
+ *
+ * Open MCT Web is licensed under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * http://www.apache.org/licenses/LICENSE-2.0.
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations
+ * under the License.
+ *
+ * Open MCT Web includes source code licensed under additional open source
+ * licenses. See the Open Source Licenses file (LICENSES.md) included with
+ * this source code distribution or the Licensing information page available
+ * at runtime from the About dialog for additional information.
+ *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine,xit,xdescribe*/
+
+/**
+ * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
+ */
+define(
+    ["../../src/creation/CreateAction"],
+    function (CreateAction) {
+        "use strict";
+
+        describe("The create action", function () {
+            var mockType,
+                mockParent,
+                mockContext,
+                mockDialogService,
+                mockCreationService,
+                action;
+
+            function mockPromise(value) {
+                return {
+                    then: function (callback) {
+                        return mockPromise(callback(value));
+                    }
+                };
+            }
+
+            beforeEach(function () {
+                mockType = jasmine.createSpyObj(
+                    "type",
+                    [
+                        "getKey",
+                        "getGlyph",
+                        "getName",
+                        "getDescription",
+                        "getProperties",
+                        "getInitialModel"
+                    ]
+                );
+                mockParent = jasmine.createSpyObj(
+                    "domainObject",
+                    [
+                        "getId",
+                        "getModel",
+                        "getCapability"
+                    ]
+                );
+                mockContext = {
+                    domainObject: mockParent
+                };
+                mockDialogService = jasmine.createSpyObj(
+                    "dialogService",
+                    [ "getUserInput" ]
+                );
+                mockCreationService = jasmine.createSpyObj(
+                    "creationService",
+                    [ "createObject" ]
+                );
+
+                mockType.getKey.andReturn("test");
+                mockType.getGlyph.andReturn("T");
+                mockType.getDescription.andReturn("a test type");
+                mockType.getName.andReturn("Test");
+                mockType.getProperties.andReturn([]);
+                mockType.getInitialModel.andReturn({});
+
+                mockDialogService.getUserInput.andReturn(mockPromise({}));
+
+                action = new CreateAction(
+                    mockType,
+                    mockParent,
+                    mockContext,
+                    mockDialogService,
+                    mockCreationService
+                );
+            });
+
+            it("exposes type-appropriate metadata", function () {
+                var metadata = action.getMetadata();
+
+                expect(metadata.name).toEqual("Test");
+                expect(metadata.description).toEqual("a test type");
+                expect(metadata.glyph).toEqual("T");
+            });
+
+            //TODO: Disabled for NEM Beta
+            xit("invokes the creation service when performed", function () {
+                action.perform();
+                expect(mockCreationService.createObject).toHaveBeenCalledWith(
+                    { type: "test" },
+                    mockParent
+                );
+            });
+
+            //TODO: Disabled for NEM Beta
+            xit("does not create an object if the user cancels", function () {
+                mockDialogService.getUserInput.andReturn({
+                    then: function (callback, fail) {
+                        fail();
+                    }
+                });
+
+                action.perform();
+
+                expect(mockCreationService.createObject)
+                    .not.toHaveBeenCalled();
+
+            });
+
+        });
+    }
+);

platform/commonUI/browse/test/creation/CreateMenuControllerSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/creation/CreateMenuController"],
     function (CreateMenuController) {
+        "use strict";
 
         describe("The create menu controller", function () {
             var mockScope,
@@ -62,4 +64,4 @@ define(
             });
         });
     }
-);
+);

platform/commonUI/browse/test/creation/CreateWizardSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/creation/CreateWizard"],
     function (CreateWizard) {
+        "use strict";
 
         describe("The create wizard", function () {
             var mockType,
@@ -39,7 +41,7 @@ define(
             function createMockProperty(name) {
                 var mockProperty = jasmine.createSpyObj(
                     "property" + name,
-                    ["getDefinition", "getValue", "setValue"]
+                    [ "getDefinition", "getValue", "setValue" ]
                 );
                 mockProperty.getDefinition.andReturn({
                     control: "textfield"
@@ -68,7 +70,7 @@ define(
                         "getCapability"
                     ]
                 );
-                mockProperties = ["A", "B", "C"].map(createMockProperty);
+                mockProperties = [ "A", "B", "C" ].map(createMockProperty);
                 mockPolicyService = jasmine.createSpyObj('policyService', ['allow']);
 
                 testModel = { someKey: "some value" };
@@ -144,15 +146,15 @@ define(
                     "A": "ValueA",
                     "B": "ValueB",
                     "C": "ValueC"
-                },
-                compareModel = wizard.createModel(formValue);
+                    },
+                    compareModel = wizard.createModel(formValue);
                 wizard.populateObjectFromInput(formValue);
                 expect(mockDomainObject.useCapability).toHaveBeenCalledWith('mutation', jasmine.any(Function));
                 expect(mockDomainObject.useCapability.mostRecentCall.args[1]()).toEqual(compareModel);
             });
 
             it("validates selection types using policy", function () {
-                var mockDomainObj = jasmine.createSpyObj(
+                var mockDomainObject = jasmine.createSpyObj(
                         'domainObject',
                         ['getCapability']
                     ),
@@ -166,8 +168,8 @@ define(
                     rows = structure.sections[sections.length - 1].rows,
                     locationRow = rows[rows.length - 1];
 
-                mockDomainObj.getCapability.andReturn(mockOtherType);
-                locationRow.validate(mockDomainObj);
+                mockDomainObject.getCapability.andReturn(mockOtherType);
+                locationRow.validate(mockDomainObject);
 
                 // Should check policy to see if the user-selected location
                 // can actually contain objects of this type
@@ -179,7 +181,7 @@ define(
             });
 
             it("creates a form model without a location if not requested", function () {
-                expect(wizard.getFormStructure(false).sections.some(function (section) {
+                expect(wizard.getFormStructure(false).sections.some(function(section){
                     return section.name === 'Location';
                 })).toEqual(false);
             });

platform/commonUI/browse/test/creation/CreationPolicySpec.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,describe,it,expect,beforeEach,jasmine*/
 
 define(
     ["../../src/creation/CreationPolicy"],
     function (CreationPolicy) {
+        "use strict";
 
         describe("The creation policy", function () {
             var mockType,
@@ -48,4 +50,4 @@ define(
             });
         });
     }
-);
+);

platform/commonUI/browse/test/creation/CreationServiceSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/creation/CreationService"],
     function (CreationService) {
+        "use strict";
 
         describe("The creation service", function () {
             var mockQ,
@@ -61,23 +63,23 @@ define(
                 mockQ = { when: mockPromise, reject: mockReject };
                 mockLog = jasmine.createSpyObj(
                     "$log",
-                    ["error", "warn", "info", "debug"]
+                    [ "error", "warn", "info", "debug" ]
                 );
                 mockParentObject = jasmine.createSpyObj(
                     "parentObject",
-                    ["getId", "getCapability", "useCapability"]
+                    [ "getId", "getCapability", "useCapability" ]
                 );
                 mockNewObject = jasmine.createSpyObj(
                     "newObject",
-                    ["getId", "getCapability", "useCapability"]
+                    [ "getId", "getCapability", "useCapability" ]
                 );
                 mockMutationCapability = jasmine.createSpyObj(
                     "mutation",
-                    ["invoke"]
+                    [ "invoke" ]
                 );
                 mockPersistenceCapability = jasmine.createSpyObj(
                     "persistence",
-                    ["persist", "getSpace"]
+                    [ "persist", "getSpace" ]
                 );
                 mockCompositionCapability = jasmine.createSpyObj(
                     "composition",
@@ -100,7 +102,7 @@ define(
                 };
                 mockNewPersistenceCapability = jasmine.createSpyObj(
                     "new-persistence",
-                    ["persist", "getSpace"]
+                    [ "persist", "getSpace" ]
                 );
 
                 mockParentObject.getCapability.andCallFake(function (key) {
@@ -147,7 +149,8 @@ define(
             });
 
             it("adds new objects to the parent's composition", function () {
-                var model = { someKey: "some value" };
+                var model = { someKey: "some value" },
+                    parentModel = { composition: ["notAnyUUID"] };
                 creationService.createObject(model, mockParentObject);
 
                 // Verify that a new ID was added
@@ -198,7 +201,8 @@ define(
             it("logs an error when mutaton fails", function () {
                 // If mutation of the parent fails, we've lost the
                 // created object - this is an error.
-                var model = { someKey: "some value" };
+                var model = { someKey: "some value" },
+                    parentModel = { composition: ["notAnyUUID"] };
 
                 mockCompositionCapability.add.andReturn(mockPromise(false));
 

platform/commonUI/browse/test/creation/LocatorControllerSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/creation/LocatorController"],
     function (LocatorController) {
+        "use strict";
 
         describe("The locator controller", function () {
             var mockScope,
@@ -40,20 +42,20 @@ define(
             beforeEach(function () {
                 mockScope = jasmine.createSpyObj(
                     "$scope",
-                    ["$watch"]
+                    [ "$watch" ]
                 );
                 mockTimeout = jasmine.createSpy("$timeout");
                 mockDomainObject = jasmine.createSpyObj(
                     "domainObject",
-                    ["getCapability"]
+                    [ "getCapability" ]
                 );
                 mockRootObject = jasmine.createSpyObj(
                     "rootObject",
-                    ["getCapability"]
+                    [ "getCapability" ]
                 );
                 mockContext = jasmine.createSpyObj(
                     "context",
-                    ["getRoot"]
+                    [ "getRoot" ]
                 );
                 mockObjectService = jasmine.createSpyObj(
                     "objectService",
@@ -73,18 +75,18 @@ define(
 
                 controller = new LocatorController(mockScope, mockTimeout, mockObjectService);
             });
-            describe("when context is available", function () {
+                describe("when context is available", function () {
 
-                beforeEach(function () {
+                    beforeEach(function () {
                         mockContext.getRoot.andReturn(mockRootObject);
                         controller = new LocatorController(mockScope, mockTimeout, mockObjectService);
                     });
 
-                it("adds a treeModel to scope", function () {
+                    it("adds a treeModel to scope", function () {
                         expect(mockScope.treeModel).toBeDefined();
                     });
 
-                it("watches for changes to treeModel", function () {
+                    it("watches for changes to treeModel", function () {
                         // This is what the embedded tree representation
                         // will be modifying.
                         expect(mockScope.$watch).toHaveBeenCalledWith(
@@ -93,7 +95,7 @@ define(
                         );
                     });
 
-                it("changes its own model on embedded model updates", function () {
+                    it("changes its own model on embedded model updates", function () {
                         // Need to pass on selection changes as updates to
                         // the control's value
                         mockScope.$watch.mostRecentCall.args[1](mockDomainObject);
@@ -107,7 +109,7 @@ define(
                             .toHaveBeenCalledWith("context");
                     });
 
-                it("rejects changes which fail validation", function () {
+                    it("rejects changes which fail validation", function () {
                         mockScope.structure = { validate: jasmine.createSpy('validate') };
                         mockScope.structure.validate.andReturn(false);
 
@@ -120,10 +122,10 @@ define(
                         expect(mockScope.ngModel.someField).not.toEqual(mockDomainObject);
                     });
 
-                it("treats a lack of a selection as invalid", function () {
+                    it("treats a lack of a selection as invalid", function () {
                         mockScope.ngModelController = jasmine.createSpyObj(
                             'ngModelController',
-                            ['$setValidity']
+                            [ '$setValidity' ]
                         );
 
                         mockScope.$watch.mostRecentCall.args[1](mockDomainObject);
@@ -136,14 +138,14 @@ define(
                         expect(mockScope.ngModelController.$setValidity)
                             .toHaveBeenCalledWith(jasmine.any(String), false);
                     });
-            });
-            describe("when no context is available", function () {
+                });
+                describe("when no context is available", function () {
                     var defaultRoot = "DEFAULT_ROOT";
 
                     beforeEach(function () {
                         mockContext.getRoot.andReturn(undefined);
-                        getObjectsPromise.then.andCallFake(function (callback) {
-                            callback({'ROOT': defaultRoot});
+                        getObjectsPromise.then.andCallFake(function(callback){
+                            callback({'ROOT':defaultRoot});
                         });
                         controller = new LocatorController(mockScope, mockTimeout, mockObjectService);
                     });

platform/commonUI/browse/test/navigation/NavigateActionSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,13 +27,13 @@
 define(
     ["../../src/navigation/NavigateAction"],
     function (NavigateAction) {
+        "use strict";
 
         describe("The navigate action", function () {
             var mockNavigationService,
                 mockQ,
+                actionContext,
                 mockDomainObject,
-                mockPolicyService,
-                mockWindow,
                 action;
 
             function mockPromise(value) {
@@ -46,70 +47,25 @@ define(
             beforeEach(function () {
                 mockNavigationService = jasmine.createSpyObj(
                     "navigationService",
-                    [
-                        "setNavigation",
-                        "getNavigation"
-                    ]
+                    [ "setNavigation" ]
                 );
-                mockNavigationService.getNavigation.andReturn({});
                 mockQ = { when: mockPromise };
                 mockDomainObject = jasmine.createSpyObj(
                     "domainObject",
-                    ["getId", "getModel", "getCapability"]
+                    [ "getId", "getModel", "getCapability" ]
                 );
 
-                mockPolicyService = jasmine.createSpyObj("policyService",
-                    [
-                        "allow"
-                    ]);
-                mockWindow = jasmine.createSpyObj("$window",
-                    [
-                        "confirm"
-                    ]);
-
                 action = new NavigateAction(
                     mockNavigationService,
                     mockQ,
-                    mockPolicyService,
-                    mockWindow,
                     { domainObject: mockDomainObject }
                 );
             });
 
-            it("invokes the policy service to determine if navigation" +
-                " allowed", function () {
+            it("invokes the navigate service when performed", function () {
                 action.perform();
-                expect(mockPolicyService.allow)
-                    .toHaveBeenCalledWith("navigation", jasmine.any(Object), jasmine.any(Object), jasmine.any(Function));
-            });
-
-            it("prompts user if policy rejection", function () {
-                action.perform();
-                expect(mockPolicyService.allow).toHaveBeenCalled();
-                mockPolicyService.allow.mostRecentCall.args[3]();
-                expect(mockWindow.confirm).toHaveBeenCalled();
-            });
-
-            describe("shows a prompt", function () {
-                beforeEach(function () {
-                    // Ensure the allow callback is called synchronously
-                    mockPolicyService.allow.andCallFake(function () {
-                        return arguments[3]();
-                    });
-                });
-                it("does not navigate on prompt rejection", function () {
-                    mockWindow.confirm.andReturn(false);
-                    action.perform();
-                    expect(mockNavigationService.setNavigation)
-                        .not.toHaveBeenCalled();
-                });
-
-                it("does navigate on prompt acceptance", function () {
-                    mockWindow.confirm.andReturn(true);
-                    action.perform();
-                    expect(mockNavigationService.setNavigation)
-                        .toHaveBeenCalled();
-                });
+                expect(mockNavigationService.setNavigation)
+                    .toHaveBeenCalledWith(mockDomainObject);
             });
 
             it("is only applicable when a domain object is in context", function () {
@@ -121,4 +77,4 @@ define(
 
         });
     }
-);
+);

platform/commonUI/browse/test/navigation/NavigationServiceSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/navigation/NavigationService"],
     function (NavigationService) {
+        "use strict";
 
         describe("The navigation service", function () {
             var navigationService;

platform/commonUI/browse/test/navigation/OrphanNavigationHandlerSpec.js

@@ -1,180 +0,0 @@
-/*****************************************************************************
-* Open MCT Web, Copyright (c) 2014-2015, United States Government
-* as represented by the Administrator of the National Aeronautics and Space
-* Administration. All rights reserved.
-*
-* Open MCT Web is licensed under the Apache License, Version 2.0 (the
-* "License"); you may not use this file except in compliance with the License.
-* You may obtain a copy of the License at
-* http://www.apache.org/licenses/LICENSE-2.0.
-*
-* Unless required by applicable law or agreed to in writing, software
-* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
-* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
-* License for the specific language governing permissions and limitations
-* under the License.
-*
-* Open MCT Web includes source code licensed under additional open source
-* licenses. See the Open Source Licenses file (LICENSES.md) included with
-    * this source code distribution or the Licensing information page available
-* at runtime from the About dialog for additional information.
-*****************************************************************************/
-
-define([
-    '../../src/navigation/OrphanNavigationHandler'
-], function (OrphanNavigationHandler) {
-    describe("OrphanNavigationHandler", function () {
-        var mockTopic,
-            mockThrottle,
-            mockMutationTopic,
-            mockNavigationService,
-            mockDomainObject,
-            mockParentObject,
-            mockContext,
-            mockActionCapability,
-            mockEditor,
-            testParentModel,
-            testId,
-            mockThrottledFns;
-
-        beforeEach(function () {
-            testId = 'some-identifier';
-
-            mockThrottledFns = [];
-            testParentModel = {};
-
-            mockTopic = jasmine.createSpy('topic');
-            mockThrottle = jasmine.createSpy('throttle');
-            mockNavigationService = jasmine.createSpyObj('navigationService', [
-                'getNavigation',
-                'addListener'
-            ]);
-            mockMutationTopic = jasmine.createSpyObj('mutationTopic', [
-                'listen'
-            ]);
-            mockDomainObject = jasmine.createSpyObj('domainObject', [
-                'getId',
-                'getCapability',
-                'getModel',
-                'hasCapability'
-            ]);
-            mockParentObject = jasmine.createSpyObj('domainObject', [
-                'getId',
-                'getCapability',
-                'getModel',
-                'hasCapability'
-            ]);
-            mockContext = jasmine.createSpyObj('context', ['getParent']);
-            mockActionCapability = jasmine.createSpyObj('action', ['perform']);
-            mockEditor = jasmine.createSpyObj('editor', ['isEditContextRoot']);
-
-            mockThrottle.andCallFake(function (fn) {
-                var mockThrottledFn =
-                    jasmine.createSpy('throttled-' + mockThrottledFns.length);
-                mockThrottledFn.andCallFake(fn);
-                mockThrottledFns.push(mockThrottledFn);
-                return mockThrottledFn;
-            });
-            mockTopic.andCallFake(function (k) {
-                return k === 'mutation' && mockMutationTopic;
-            });
-            mockDomainObject.getId.andReturn(testId);
-            mockDomainObject.getCapability.andCallFake(function (c) {
-                return {
-                    context: mockContext,
-                    editor: mockEditor
-                }[c];
-            });
-            mockDomainObject.hasCapability.andCallFake(function (c) {
-                return !!mockDomainObject.getCapability(c);
-            });
-            mockParentObject.getModel.andReturn(testParentModel);
-            mockParentObject.getCapability.andCallFake(function (c) {
-                return {
-                    action: mockActionCapability
-                }[c];
-            });
-            mockContext.getParent.andReturn(mockParentObject);
-            mockNavigationService.getNavigation.andReturn(mockDomainObject);
-            mockEditor.isEditContextRoot.andReturn(false);
-
-            return new OrphanNavigationHandler(
-                mockThrottle,
-                mockTopic,
-                mockNavigationService
-            );
-        });
-
-
-        it("listens for mutation with a throttled function", function () {
-            expect(mockMutationTopic.listen)
-                .toHaveBeenCalledWith(jasmine.any(Function));
-            expect(mockThrottledFns.indexOf(
-                mockMutationTopic.listen.mostRecentCall.args[0]
-            )).not.toEqual(-1);
-        });
-
-        it("listens for navigation changes with a throttled function", function () {
-            expect(mockNavigationService.addListener)
-                .toHaveBeenCalledWith(jasmine.any(Function));
-            expect(mockThrottledFns.indexOf(
-                mockNavigationService.addListener.mostRecentCall.args[0]
-            )).not.toEqual(-1);
-        });
-
-        [false, true].forEach(function (isOrphan) {
-            var prefix = isOrphan ? "" : "non-";
-            describe("for " + prefix + "orphan objects", function () {
-                beforeEach(function () {
-                    testParentModel.composition = isOrphan ? [] : [testId];
-                });
-
-                [false, true].forEach(function (isEditRoot) {
-                    var caseName = isEditRoot ?
-                        "that are being edited" : "that are not being edited";
-
-                    function itNavigatesAsExpected() {
-                        if (isOrphan && !isEditRoot) {
-                            it("navigates to the parent", function () {
-                                expect(mockActionCapability.perform)
-                                    .toHaveBeenCalledWith('navigate');
-                            });
-                        } else {
-                            it("does nothing", function () {
-                                expect(mockActionCapability.perform)
-                                    .not.toHaveBeenCalled();
-                            });
-                        }
-                    }
-
-                    describe(caseName, function () {
-                        beforeEach(function () {
-                            mockEditor.isEditContextRoot.andReturn(isEditRoot);
-                        });
-
-                        describe("when navigation changes", function () {
-                            beforeEach(function () {
-                                mockNavigationService.addListener.mostRecentCall
-                                    .args[0](mockDomainObject);
-                            });
-
-                            itNavigatesAsExpected();
-                        });
-
-                        describe("when mutation occurs", function () {
-                            beforeEach(function () {
-                                mockMutationTopic.listen.mostRecentCall
-                                    .args[0](mockParentObject);
-                            });
-
-                            itNavigatesAsExpected();
-                        });
-
-                    });
-                });
-            });
-        });
-
-    });
-});
-

platform/commonUI/browse/test/windowing/FullscreenActionSpec.js

@@ -19,30 +19,33 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine,afterEach,window*/
 
 /**
  * MCTRepresentationSpec. Created by vwoeltje on 11/6/14.
  */
 define(
-    ["../../src/windowing/FullscreenAction", "screenfull"],
-    function (FullscreenAction, screenfull) {
+    ["../../src/windowing/FullscreenAction"],
+    function (FullscreenAction) {
+        "use strict";
 
         describe("The fullscreen action", function () {
             var action,
-                oldToggle;
+                oldScreenfull;
 
             beforeEach(function () {
                 // Screenfull is not shimmed or injected, so
                 // we need to spy on it in the global scope.
-                oldToggle = screenfull.toggle;
+                oldScreenfull = window.screenfull;
 
-                screenfull.toggle = jasmine.createSpy("toggle");
+                window.screenfull = {};
+                window.screenfull.toggle = jasmine.createSpy("toggle");
 
                 action = new FullscreenAction({});
             });
 
             afterEach(function () {
-                screenfull.toggle = oldToggle;
+                window.screenfull = oldScreenfull;
             });
 
             it("toggles fullscreen mode when performed", function () {
@@ -56,4 +59,4 @@ define(
 
         });
     }
-);
+);

platform/commonUI/browse/test/windowing/NewTabActionSpec.js

@@ -19,15 +19,18 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine,afterEach,window*/
 
 define(
     ["../../src/windowing/NewTabAction"],
     function (NewTabAction) {
-
+        "use strict";
+        
         describe("The new tab action", function () {
             var actionSelected,
                 actionCurrent,
                 mockWindow,
+                mockDomainObject,
                 mockContextCurrent,
                 mockContextSelected,
                 mockUrlService;
@@ -37,39 +40,39 @@ define(
 
                 // Context if the current object is selected
                 // For example, when the top right new tab
-                // button is clicked, the user is using the
+                // button is clicked, the user is using the 
                 // current domainObject
                 mockContextCurrent = jasmine.createSpyObj("context", ["domainObject"]);
-
+                
                 // Context if the selected object is selected
                 // For example, when an object in the left
                 // tree is opened in a new tab using the
                 // context menu
                 mockContextSelected = jasmine.createSpyObj("context", ["selectedObject",
                                                                "domainObject"]);
-
+                
                 // Mocks the urlService used to make the new tab's url from a
                 // domainObject and mode
                 mockUrlService = jasmine.createSpyObj("urlService", ["urlForNewTab"]);
-
+                
                 // Action done using the current context or mockContextCurrent
                 actionCurrent = new NewTabAction(mockUrlService, mockWindow,
                                                  mockContextCurrent);
-
+                
                 // Action done using the selected context or mockContextSelected
                 actionSelected = new NewTabAction(mockUrlService, mockWindow,
                                                   mockContextSelected);
-
+                
             });
-
+                        
             it("new tab with current url is opened", function () {
                 actionCurrent.perform();
             });
-
+            
             it("new tab with a selected url is opened", function () {
                 actionSelected.perform();
             });
-
+            
         });
     }
-);
+);

platform/commonUI/browse/test/windowing/WindowTitlerSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * WindowTitlerSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../../src/windowing/WindowTitler"],
     function (WindowTitler) {
+        "use strict";
 
         describe("The window titler", function () {
             var mockNavigationService,
@@ -37,11 +39,11 @@ define(
             beforeEach(function () {
                 mockNavigationService = jasmine.createSpyObj(
                     'navigationService',
-                    ['getNavigation']
+                    [ 'getNavigation' ]
                 );
                 mockRootScope = jasmine.createSpyObj(
                     '$rootScope',
-                    ['$watch']
+                    [ '$watch' ]
                 );
                 mockDomainObject = jasmine.createSpyObj(
                     'domainObject',
@@ -75,4 +77,4 @@ define(
 
         });
     }
-);
+);

platform/commonUI/dialog/bundle.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define([
     "./src/DialogService",
@@ -43,6 +44,7 @@ define([
     overlayTemplate,
     legacyRegistry
 ) {
+    "use strict";
 
     legacyRegistry.register("platform/commonUI/dialog", {
         "extensions": {

platform/commonUI/dialog/src/DialogService.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 /**
  * This bundle implements the dialog service, which can be used to
@@ -28,6 +29,7 @@
 define(
     [],
     function () {
+        "use strict";
         /**
          * The dialog service is responsible for handling window-modal
          * communication with the user, such as displaying forms for user
@@ -155,8 +157,8 @@ define(
          * @returns {boolean} true if dialog is currently visible, false
          * otherwise
          */
-        DialogService.prototype.canShowDialog = function (dialogModel) {
-            if (this.dialogVisible) {
+        DialogService.prototype.canShowDialog = function(dialogModel){
+            if (this.dialogVisible){
                 // Only one dialog should be shown at a time.
                 // The application design should be such that
                 // we never even try to do this.
@@ -224,7 +226,7 @@ define(
          * @param {typeClass} string tells overlayService that this overlay should use appropriate CSS class
          * @returns {boolean}
          */
-        DialogService.prototype.showBlockingMessage = function (dialogModel) {
+        DialogService.prototype.showBlockingMessage = function(dialogModel) {
             if (this.canShowDialog(dialogModel)) {
                 // Add the overlay using the OverlayService, which
                 // will handle actual insertion into the DOM

platform/commonUI/dialog/src/OverlayService.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     [],
     function () {
+        "use strict";
 
         // Template to inject into the DOM to show the dialog; really just points to
         // the a specific template that can be included via mct-include

platform/commonUI/dialog/test/DialogServiceSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTIncudeSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../src/DialogService"],
     function (DialogService) {
+        "use strict";
 
         describe("The dialog service", function () {
             var mockOverlayService,
@@ -38,23 +40,23 @@ define(
             beforeEach(function () {
                 mockOverlayService = jasmine.createSpyObj(
                     "overlayService",
-                    ["createOverlay"]
+                    [ "createOverlay" ]
                 );
                 mockQ = jasmine.createSpyObj(
                     "$q",
-                    ["defer"]
+                    [ "defer" ]
                 );
                 mockLog = jasmine.createSpyObj(
                     "$log",
-                    ["warn", "info", "debug"]
+                    [ "warn", "info", "debug" ]
                 );
                 mockOverlay = jasmine.createSpyObj(
                     "overlay",
-                    ["dismiss"]
+                    [ "dismiss" ]
                 );
                 mockDeferred = jasmine.createSpyObj(
                     "deferred",
-                    ["resolve", "reject"]
+                    [ "resolve", "reject"]
                 );
                 mockDeferred.promise = "mock promise";
 
@@ -120,7 +122,7 @@ define(
             });
 
             it("invokes the overlay service with the correct parameters when" +
-                " a blocking dialog is requested", function () {
+                " a blocking dialog is requested", function() {
                 var dialogModel = {};
                 expect(dialogService.showBlockingMessage(dialogModel)).toBe(true);
                 expect(mockOverlayService.createOverlay).toHaveBeenCalledWith(

platform/commonUI/dialog/test/OverlayServiceSpec.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 /**
  * MCTIncudeSpec. Created by vwoeltje on 11/6/14.
@@ -26,6 +27,7 @@
 define(
     ["../src/OverlayService"],
     function (OverlayService) {
+        "use strict";
 
         describe("The overlay service", function () {
             var mockDocument,
@@ -38,13 +40,13 @@ define(
                 overlayService;
 
             beforeEach(function () {
-                mockDocument = jasmine.createSpyObj("$document", ["find"]);
+                mockDocument = jasmine.createSpyObj("$document", [ "find" ]);
                 mockCompile = jasmine.createSpy("$compile");
-                mockRootScope = jasmine.createSpyObj("$rootScope", ["$new"]);
-                mockBody = jasmine.createSpyObj("body", ["prepend"]);
+                mockRootScope = jasmine.createSpyObj("$rootScope", [ "$new" ]);
+                mockBody = jasmine.createSpyObj("body", [ "prepend" ]);
                 mockTemplate = jasmine.createSpy("template");
-                mockElement = jasmine.createSpyObj("element", ["remove"]);
-                mockScope = jasmine.createSpyObj("scope", ["$destroy"]);
+                mockElement = jasmine.createSpyObj("element", [ "remove" ]);
+                mockScope = jasmine.createSpyObj("scope", [ "$destroy" ]);
 
                 mockDocument.find.andReturn(mockBody);
                 mockCompile.andReturn(mockTemplate);
@@ -96,4 +98,4 @@ define(
 
         });
     }
-);
+);

platform/commonUI/edit/bundle.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define([
     "./src/controllers/EditActionController",
@@ -26,32 +27,16 @@ define([
     "./src/controllers/ElementsController",
     "./src/controllers/EditObjectController",
     "./src/directives/MCTBeforeUnload",
-    "./src/actions/EditAndComposeAction",
+    "./src/actions/LinkAction",
     "./src/actions/EditAction",
     "./src/actions/PropertiesAction",
     "./src/actions/RemoveAction",
     "./src/actions/SaveAction",
-    "./src/actions/SaveAsAction",
     "./src/actions/CancelAction",
     "./src/policies/EditActionPolicy",
-    "./src/policies/EditableLinkPolicy",
-    "./src/policies/EditableMovePolicy",
     "./src/policies/EditNavigationPolicy",
-    "./src/policies/EditContextualActionPolicy",
     "./src/representers/EditRepresenter",
     "./src/representers/EditToolbarRepresenter",
-    "./src/capabilities/EditorCapability",
-    "./src/capabilities/TransactionCapabilityDecorator",
-    "./src/services/TransactionService",
-    "./src/creation/CreateMenuController",
-    "./src/creation/LocatorController",
-    "./src/creation/CreationPolicy",
-    "./src/creation/CreateActionProvider",
-    "./src/creation/AddActionProvider",
-    "./src/creation/CreationService",
-    "text!./res/templates/create/locator.html",
-    "text!./res/templates/create/create-button.html",
-    "text!./res/templates/create/create-menu.html",
     "text!./res/templates/library.html",
     "text!./res/templates/edit-object.html",
     "text!./res/templates/edit-action-buttons.html",
@@ -64,32 +49,16 @@ define([
     ElementsController,
     EditObjectController,
     MCTBeforeUnload,
-    EditAndComposeAction,
+    LinkAction,
     EditAction,
     PropertiesAction,
     RemoveAction,
     SaveAction,
-    SaveAsAction,
     CancelAction,
     EditActionPolicy,
-    EditableLinkPolicy,
-    EditableMovePolicy,
     EditNavigationPolicy,
-    EditContextualActionPolicy,
     EditRepresenter,
     EditToolbarRepresenter,
-    EditorCapability,
-    TransactionCapabilityDecorator,
-    TransactionService,
-    CreateMenuController,
-    LocatorController,
-    CreationPolicy,
-    CreateActionProvider,
-    AddActionProvider,
-    CreationService,
-    locatorTemplate,
-    createButtonTemplate,
-    createMenuTemplate,
     libraryTemplate,
     editObjectTemplate,
     editActionButtonsTemplate,
@@ -97,6 +66,7 @@ define([
     topbarEditTemplate,
     legacyRegistry
 ) {
+    "use strict";
 
     legacyRegistry.register("platform/commonUI/edit", {
         "extensions": {
@@ -130,22 +100,6 @@ define([
                         "$location",
                         "policyService"
                     ]
-                },
-                {
-                    "key": "CreateMenuController",
-                    "implementation": CreateMenuController,
-                    "depends": [
-                        "$scope"
-                    ]
-                },
-                {
-                    "key": "LocatorController",
-                    "implementation": LocatorController,
-                    "depends": [
-                        "$scope",
-                        "$timeout",
-                        "objectService"
-                    ]
                 }
             ],
             "directives": [
@@ -160,7 +114,7 @@ define([
             "actions": [
                 {
                     "key": "compose",
-                    "implementation": EditAndComposeAction
+                    "implementation": LinkAction
                 },
                 {
                     "key": "edit",
@@ -168,9 +122,10 @@ define([
                     "depends": [
                         "$location",
                         "navigationService",
-                        "$log"
+                        "$log",
+                        "$q"
                     ],
-                    "description": "Edit",
+                    "description": "Edit this object.",
                     "category": "view-control",
                     "glyph": "p"
                 },
@@ -206,17 +161,6 @@ define([
                     "implementation": SaveAction,
                     "name": "Save",
                     "description": "Save changes made to these objects.",
-                    "depends": [
-                        "dialogService"
-                    ],
-                    "priority": "mandatory"
-                },
-                {
-                    "key": "save",
-                    "category": "conclude-editing",
-                    "implementation": SaveAsAction,
-                    "name": "Save",
-                    "description": "Save changes made to these objects.",
                     "depends": [
                         "$injector",
                         "policyService",
@@ -232,7 +176,10 @@ define([
                     "implementation": CancelAction,
                     "name": "Cancel",
                     "description": "Discard changes made to these objects.",
-                    "depends": []
+                    "depends": [
+                        "$injector",
+                        "navigationService"
+                    ]
                 }
             ],
             "policies": [
@@ -240,28 +187,12 @@ define([
                     "category": "action",
                     "implementation": EditActionPolicy
                 },
-                {
-                    "category": "action",
-                    "implementation": EditContextualActionPolicy,
-                    "depends": ["navigationService", "editModeBlacklist", "nonEditContextBlacklist"]
-                },
-                {
-                    "category": "action",
-                    "implementation": EditableMovePolicy
-                },
-                {
-                    "category": "action",
-                    "implementation": EditableLinkPolicy
-                },
                 {
                     "category": "navigation",
-                    "message": "Continuing will cause the loss of any unsaved changes.",
+                    "message": "There are unsaved changes.",
                     "implementation": EditNavigationPolicy
-                },
-                {
-                    "implementation": CreationPolicy,
-                    "category": "creation"
                 }
+
             ],
             "templates": [
                 {
@@ -300,73 +231,8 @@ define([
                 {
                     "key": "topbar-edit",
                     "template": topbarEditTemplate
-                },
-                {
-                    "key": "create-button",
-                    "template": createButtonTemplate
-                },
-                {
-                    "key": "create-menu",
-                    "template": createMenuTemplate,
-                    "uses": [
-                        "action"
-                    ]
                 }
             ],
-            "components": [
-                {
-                    "type": "decorator",
-                    "provides": "capabilityService",
-                    "implementation": TransactionCapabilityDecorator,
-                    "depends": [
-                        "$q",
-                        "transactionService"
-                    ],
-                    "priority": "fallback"
-                },
-                {
-                    "type": "provider",
-                    "provides": "transactionService",
-                    "implementation": TransactionService,
-                    "depends": [
-                        "$q",
-                        "$log"
-                    ]
-                },
-                {
-                    "key": "CreateActionProvider",
-                    "provides": "actionService",
-                    "type": "provider",
-                    "implementation": CreateActionProvider,
-                    "depends": [
-                        "typeService",
-                        "policyService"
-                    ]
-                },
-                {
-                    "key": "AddActionProvider",
-                    "provides": "actionService",
-                    "type": "provider",
-                    "implementation": AddActionProvider,
-                    "depends": [
-                        "$q",
-                        "typeService",
-                        "dialogService",
-                        "policyService"
-                    ]
-                },
-                {
-                    "key": "CreationService",
-                    "provides": "creationService",
-                    "type": "provider",
-                    "implementation": CreationService,
-                    "depends": [
-                        "$q",
-                        "$log"
-                    ]
-                }
-
-            ],
             "representers": [
                 {
                     "implementation": EditRepresenter,
@@ -378,33 +244,6 @@ define([
                 {
                     "implementation": EditToolbarRepresenter
                 }
-            ],
-            "constants": [
-                {
-                    "key": "editModeBlacklist",
-                    "value": ["copy", "follow", "window", "link", "locate"]
-                },
-                {
-                    "key": "nonEditContextBlacklist",
-                    "value": ["copy", "follow", "properties", "move", "link", "remove", "locate"]
-                }
-            ],
-            "capabilities": [
-                {
-                    "key": "editor",
-                    "name": "Editor Capability",
-                    "description": "Provides transactional editing capabilities",
-                    "implementation": EditorCapability,
-                    "depends": [
-                        "transactionService"
-                    ]
-                }
-            ],
-            "controls": [
-                {
-                    "key": "locator",
-                    "template": locatorTemplate
-                }
             ]
         }
     });

platform/commonUI/edit/res/templates/elements.html

@@ -26,7 +26,7 @@
     </mct-include>
     <div class="flex-elem grows vscroll">
         <ul class="tree">
-            <li ng-repeat="containedObject in composition | filter:searchElements">
+            <li ng-repeat="containedObject in composition | filter:searchText">
                 <span class="tree-item">
                     <mct-representation
                             class="rep-object-label"

platform/commonUI/edit/src/actions/CancelAction.js

@@ -19,9 +19,11 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     function () {
+        'use strict';
 
         /**
          * The "Cancel" action; the action triggered by clicking Cancel from
@@ -31,8 +33,10 @@ define(
          * @memberof platform/commonUI/edit
          * @implements {Action}
          */
-        function CancelAction(context) {
+        function CancelAction($injector, navigationService, context) {
             this.domainObject = context.domainObject;
+            this.navigationService = navigationService;
+            this.objectService = $injector.get('objectService');
         }
 
         /**
@@ -42,32 +46,31 @@ define(
          *          cancellation has completed
          */
         CancelAction.prototype.perform = function () {
-            var domainObject = this.domainObject;
+            var domainObject = this.domainObject,
+                self = this;
 
+            // Look up the object's "editor.completion" capability;
+            // this is introduced by EditableDomainObject which is
+            // used to insulate underlying objects from changes made
+            // during editing.
+            function getEditorCapability() {
+                return domainObject.getCapability("editor");
+            }
+
+            // Invoke any save behavior introduced by the editor.completion
+            // capability.
+            function doCancel(editor) {
+                return editor.cancel();
+            }
+
+            //Discard current 'editable' object, and retrieve original
+            // un-edited object.
             function returnToBrowse() {
-                var parent;
-
-                //If the object existed already, navigate to refresh view
-                // with previous object state.
-                if (domainObject.getModel().persisted) {
-                    return domainObject.getCapability("action").perform("navigate");
-                } else {
-                    //If the object was new, and user has cancelled, then
-                    //navigate back to parent because nothing to show.
-                    return domainObject.getCapability("location").getOriginal().then(function (original) {
-                        parent = original.getCapability("context").getParent();
-                        parent.getCapability("action").perform("navigate");
-                    });
-                }
+                return self.navigationService.setNavigation(self.domainObject.getOriginalObject());
             }
 
-            function cancel(allowed) {
-                return allowed && domainObject.getCapability("editor").cancel();
-            }
-
-            //Do navigation first in order to trigger unsaved changes dialog
-            return returnToBrowse()
-                .then(cancel);
+            return doCancel(getEditorCapability())
+                .then(returnToBrowse);
         };
 
         /**
@@ -79,8 +82,7 @@ define(
         CancelAction.appliesTo = function (context) {
             var domainObject = (context || {}).domainObject;
             return domainObject !== undefined &&
-                domainObject.hasCapability('editor') &&
-                domainObject.getCapability('editor').isEditContextRoot();
+                domainObject.hasCapability("editor");
         };
 
         return CancelAction;

platform/commonUI/edit/src/actions/EditAction.js

@@ -19,13 +19,15 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define,Promise*/
 
 /**
  * Module defining EditAction. Created by vwoeltje on 11/14/14.
  */
 define(
-    [],
-    function () {
+    ['../objects/EditableDomainObject'],
+    function (EditableDomainObject) {
+        "use strict";
 
         // A no-op action to return in the event that the action cannot
         // be completed.
@@ -44,7 +46,7 @@ define(
          * @constructor
          * @implements {Action}
          */
-        function EditAction($location, navigationService, $log, context) {
+        function EditAction($location, navigationService, $log, $q, context) {
             var domainObject = (context || {}).domainObject;
 
             // We cannot enter Edit mode if we have no domain object to
@@ -63,6 +65,7 @@ define(
             this.domainObject = domainObject;
             this.$location = $location;
             this.navigationService = navigationService;
+            this.$q = $q;
         }
 
         /**
@@ -70,18 +73,25 @@ define(
          */
         EditAction.prototype.perform = function () {
             var self = this;
-            function cancelEditing() {
-                self.domainObject.getCapability('editor').cancel();
-                self.navigationService.removeListener(cancelEditing);
-            }
-            //If this is not the currently navigated object, then navigate
-            // to it.
-            if (this.navigationService.getNavigation() !== this.domainObject) {
-                this.navigationService.setNavigation(this.domainObject);
+            if (!this.domainObject.hasCapability("editor")) {
+                //TODO: This is only necessary because the drop gesture is
+                // wrapping the object itself, need to refactor this later.
+                // All responsibility for switching into edit mode should be
+                // in the edit action, and not duplicated in the gesture
+                this.domainObject = new EditableDomainObject(this.domainObject, this.$q);
             }
+            this.navigationService.setNavigation(this.domainObject);
+            this.domainObject.getCapability('status').set('editing', true);
 
+            //Register a listener to automatically cancel this edit action
+            //if the user navigates away from this object.
+            function cancelEditing(navigatedTo){
+                if (!navigatedTo || navigatedTo.getId() !== self.domainObject.getId()) {
+                    self.domainObject.getCapability('editor').cancel();
+                    self.navigationService.removeListener(cancelEditing);
+                }
+            }
             this.navigationService.addListener(cancelEditing);
-            this.domainObject.useCapability("editor");
         };
 
         /**
@@ -92,13 +102,11 @@ define(
          */
         EditAction.appliesTo = function (context) {
             var domainObject = (context || {}).domainObject,
-                type = domainObject && domainObject.getCapability('type');
+                type = domainObject && domainObject.getCapability('type'),
+                isEditMode = domainObject && domainObject.getDomainObject ? true : false;
 
-            // Only allow editing of types that support it and are not already
-            // being edited
-            return type && type.hasFeature('creation') &&
-                domainObject.hasCapability('editor') &&
-                !domainObject.getCapability('editor').isEditContextRoot();
+            // Only allow creatable types to be edited
+            return type && type.hasFeature('creation') && !isEditMode;
         };
 
         return EditAction;

platform/commonUI/edit/src/actions/LinkAction.js

@@ -19,10 +19,12 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     [],
     function () {
+        "use strict";
 
 
         /**
@@ -31,14 +33,13 @@ define(
          * @memberof platform/commonUI/edit
          * @implements {Action}
          */
-        function EditAndComposeAction(context) {
+        function LinkAction(context) {
             this.domainObject = (context || {}).domainObject;
             this.selectedObject = (context || {}).selectedObject;
         }
 
-        EditAndComposeAction.prototype.perform = function () {
-            var self = this,
-                editAction = this.domainObject.getCapability('action').getActions("edit")[0];
+        LinkAction.prototype.perform = function () {
+            var self = this;
 
             // Persist changes to the domain object
             function doPersist() {
@@ -55,13 +56,9 @@ define(
                         .then(doPersist);
             }
 
-            if (editAction) {
-                editAction.perform();
-            }
-
             return this.selectedObject && doLink();
         };
 
-        return EditAndComposeAction;
+        return LinkAction;
     }
 );

platform/commonUI/edit/src/actions/PropertiesAction.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 /**
  * Edit the properties of a domain object. Shows a dialog
@@ -28,6 +29,7 @@
 define(
     ['./PropertiesDialog'],
     function (PropertiesDialog) {
+        'use strict';
 
         /**
          * Implements the "Edit Properties" action, which prompts the user
@@ -63,10 +65,10 @@ define(
                 });
             }
 
-            function showDialog(objType) {
+            function showDialog(type) {
                 // Create a dialog object to generate the form structure, etc.
                 var dialog =
-                    new PropertiesDialog(objType, domainObject.getModel());
+                    new PropertiesDialog(type, domainObject.getModel());
 
                 // Show the dialog
                 return dialogService.getUserInput(

platform/commonUI/edit/src/actions/PropertiesDialog.js

@@ -19,9 +19,11 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 define(
     function () {
+        'use strict';
 
         /**
          * Construct a new Properties dialog.

platform/commonUI/edit/src/actions/RemoveAction.js

@@ -19,6 +19,7 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
+/*global define*/
 
 /**
  * Module defining RemoveAction. Created by vwoeltje on 11/17/14.
@@ -26,6 +27,7 @@
 define(
     [],
     function () {
+        "use strict";
 
         /**
          * Construct an action which will remove the provided object manifestation.
@@ -75,8 +77,8 @@ define(
              * Invoke persistence on a domain object. This will be called upon
              * the removed object's parent (as its composition will have changed.)
              */
-            function doPersist(domainObj) {
-                var persistence = domainObj.getCapability('persistence');
+            function doPersist(domainObject) {
+                var persistence = domainObject.getCapability('persistence');
                 return persistence && persistence.persist();
             }