[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.

api/README.md

@@ -1,20 +0,0 @@
-# API
-
-This directory is for draft API documentation and design.  The API is organized into a few major components, which are documented in their own READMEs.  See the following:
-
-* Domain Objects
-    Capabilities
-    Events
-    Mutation
-    Etc
-
-* [Object API](object-api/README.md) (encapsulates persistence), should include roots
-* [Region API](region-api/README.md) 
-* [Telemetry API](telemetry-api/README.md)
-* [Type API](type-api/README.md)
-
-Not yet started:
-
-* [Action API](action-api/README.md)
-* [Indicators API](indicators-api/README.md) -- potentially compress into regions?
-* [Plugin API](plugin-api/README.md)

api/composition-api/CompositionAPI.js

@@ -1,93 +0,0 @@
-define([
-
-], function (
-
-) {
-
-
-    var PROVIDER_REGISTRY = [];
-
-    function getProvider (object) {
-        return PROVIDER_REGISTRY.filter(function (p) {
-            return p.appliesTo(object);
-        })[0];
-    };
-
-    function composition(object) {
-        var provider = getProvider(object);
-
-        if (!provider) {
-            return;
-        }
-
-        return new CompositionCollection(object, provider);
-    };
-
-    composition.addProvider = function (provider) {
-        PROVIDER_REGISTRY.unshift(provider);
-    };
-
-    window.MCT = window.MCT || {};
-    window.MCT.composition = composition;
-
-    function CompositionCollection(domainObject, provider) {
-        this.domainObject = domainObject;
-        this.provider = provider;
-    };
-
-    CompositionCollection.prototype.add = function (child, skipMutate) {
-        if (!this._children) {
-            throw new Error("Must load composition before you can add!");
-        }
-        // we probably should not add until we have loaded.
-        // todo: should we modify parent?
-        if (!skipMutate) {
-            this.provider.add(this.domainObject, child);
-        }
-        this.children.push(child);
-        this.emit('add', child);
-    };
-
-    CompositionCollection.prototype.load = function () {
-        return this.provider.load(this.domainObject)
-            .then(function (children) {
-                this._children = [];
-                children.map(function (c) {
-                    this.add(c, true);
-                }, this);
-                this.emit('load');
-                // Todo: set up listener for changes via provider?
-            }.bind(this));
-    };
-
-    CompositionCollection.prototype.remove = function (child) {
-        var index = this.children.indexOf(child);
-        if (index === -1) {
-            throw new Error("Unable to remove child: not found in composition");
-        }
-        this.provider.remove(this.domainObject, child);
-        this.children.splice(index, 1);
-        this.emit('remove', index, child);
-    };
-
-    var DefaultCompositionProvider = {
-        appliesTo: function (domainObject) {
-            return !!domainObject.composition;
-        },
-        load: function (domainObject) {
-            return Promise.all(domainObject.composition.map(MCT.objects.get));
-        },
-        add: function (domainObject, child) {
-            domainObject.composition.push(child.key);
-        }
-    };
-
-    composition.addProvider(DefaultCompositionProvider);
-
-    function Injector() {
-        console.log('composition api injected!');
-    }
-
-    return Injector;
-
-});

api/composition-api/README.md

@@ -1,35 +0,0 @@
-# Composition API - Overview
-
-The composition API is straightforward:
-
-MCT.composition(object) -- returns a `CompositionCollection` if the object has 
-composition, returns undefined if it doesn't.
-
-## CompositionCollection
-
-Has three events:
-* `load`: when the collection has completed loading.
-* `add`: when a new object has been added to the collection.
-* `remove` when an object has been removed from the collection.
-
-Has three methods:
-
-`Collection.load()` -- returns a promise that is fulfilled when the composition 
-    has loaded.
-`Collection.add(object)` -- add a domain object to the composition.
-`Collection.remove(object)` -- remove the object from the composition.
-
-## Composition providers
-composition providers are anything that meets the following interface:
-
-* `provider.appliesTo(domainObject)` -> return true if this provider can provide 
-    composition for a given domain object.
-* `provider.add(domainObject, childObject)` -> adds object 
-* `provider.remove(domainObject, childObject)` -> immediately removes objects
-* `provider.load(domainObject)` -> returns promise for array of children
-
-There is a default composition provider which handles loading composition for 
-any object with a `composition` property.  If you want specialized composition 
-loading behavior, implement your own composition provider and register it with
-
-`MCT.composition.addProvider(myProvider)`

api/composition-api/bundle.js

@@ -1,47 +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.
- *****************************************************************************/
-/*global define*/
-
-define([
-    './CompositionAPI',
-    'legacyRegistry'
-], function (
-    CompositionAPI,
-    legacyRegistry
-) {
-    legacyRegistry.register('api/composition-api', {
-        name: 'Composition API',
-        description: 'The public Composition API',
-        extensions: {
-            runs: [
-                {
-                    key: "CompositionAPI",
-                    priority: "mandatory",
-                    implementation: CompositionAPI,
-                    depends: [
-                    ]
-                }
-            ]
-        }
-    });
-
-});

api/object-api/ObjectAPI.js

@@ -1,200 +0,0 @@
-define([
-    'lodash'
-], function (
-    _
-) {
-
-    /**
-        Object API.  Intercepts the existing object API while also exposing
-        A new Object API.
-
-        MCT.objects.get('mine')
-            .then(function (root) {
-                console.log(root);
-                MCT.objects.getComposition(root)
-                    .then(function (composition) {
-                        console.log(composition)
-                    })
-            });
-    */
-
-    var Objects = {},
-        ROOT_REGISTRY = [],
-        PROVIDER_REGISTRY = {},
-        FALLBACK_PROVIDER;
-
-    window.MCT = window.MCT || {};
-    window.MCT.objects = Objects;
-
-    // take a key string and turn it into a key object
-    // 'scratch:root' ==> {namespace: 'scratch', identifier: 'root'}
-    function parseKeyString(key) {
-        if (typeof key === 'object') {
-            return key;
-        }
-        var namespace = '',
-            identifier = key;
-        for (var i = 0, escaped = false, len=key.length; i < len; i++) {
-            if (key[i] === ":" && !escaped) {
-                namespace = key.slice(0, i);
-                identifier = key.slice(i + 1);
-                break;
-            }
-        }
-        return {
-            namespace: namespace,
-            identifier: identifier
-        };
-    };
-
-    // take a key and turn it into a key string
-    // {namespace: 'scratch', identifier: 'root'} ==> 'scratch:root'
-    function makeKeyString(key) {
-        if (typeof key === 'string') {
-            return key;
-        }
-        if (!key.namespace) {
-            return key.identifier;
-        }
-        return [
-            key.namespace.replace(':', '\\:'),
-            key.identifier.replace(':', '\\:')
-        ].join(':');
-    };
-
-    // Converts composition to use key strings instead of keys
-    function toOldFormat(model) {
-        delete model.key;
-        if (model.composition) {
-            model.composition = model.composition.map(makeKeyString);
-        }
-        return model;
-    };
-
-    // converts composition to use keys instead of key strings
-    function toNewFormat(model, key) {
-        model.key = key;
-        if (model.composition) {
-            model.composition = model.composition.map(parseKeyString);
-        }
-        return model;
-    };
-
-    // Root provider is hardcoded in; can't be skipped.
-    var RootProvider = {
-        'get': function () {
-            return Promise.resolve({
-                name: 'The root object',
-                type: 'root',
-                composition: ROOT_REGISTRY
-            });
-        }
-    };
-
-    // Retrieve the provider for a given key.
-    function getProvider(key) {
-        if (key.identifier === 'ROOT') {
-            return RootProvider;
-        }
-        return PROVIDER_REGISTRY[key.namespace] || FALLBACK_PROVIDER;
-    };
-
-    Objects.addProvider = function (namespace, provider) {
-        PROVIDER_REGISTRY[namespace] = provider;
-    };
-
-    [
-        'save',
-        'delete',
-        'get'
-    ].forEach(function (method) {
-        Objects[method] = function () {
-            var key = arguments[0],
-                provider = getProvider(key);
-
-            if (!provider) {
-                throw new Error('No Provider Matched');
-            }
-
-            if (!provider[method]) {
-                throw new Error('Provider does not support [' + method + '].');
-            }
-
-            return provider[method].apply(provider, arguments);
-        };
-    });
-
-    Objects.addRoot = function (key) {
-        ROOT_REGISTRY.unshift(key);
-    };
-
-    Objects.removeRoot = function (key) {
-        ROOT_REGISTRY = ROOT_REGISTRY.filter(function (k) {
-            return (
-                k.identifier !== key.identifier ||
-                k.namespace !== key.namespace
-            );
-        });
-    };
-
-    function ObjectServiceProvider(objectService, instantiate) {
-        this.objectService = objectService;
-        this.instantiate = instantiate;
-    }
-
-    ObjectServiceProvider.prototype.save = function (object) {
-        var key = object.key,
-            keyString = makeKeyString(key),
-            newObject = this.instantiate(toOldFormat(object), keyString);
-
-        return object.getCapability('persistence')
-                .persist()
-                .then(function () {
-                    return toNewFormat(object, key);
-                });
-    };
-
-    ObjectServiceProvider.prototype.delete = function (object) {
-        // TODO!
-    };
-
-    ObjectServiceProvider.prototype.get = function (key) {
-        var keyString = makeKeyString(key);
-        return this.objectService.getObjects([keyString])
-            .then(function (results) {
-                var model = JSON.parse(JSON.stringify(results[keyString].getModel()));
-                return toNewFormat(model, key);
-            });
-    };
-
-    // Injects new object API as a decorator so that it hijacks all requests.
-    // Object providers implemented on new API should just work, old API should just work, many things may break.
-    function ObjectAPIInjector(ROOTS, instantiate, objectService) {
-        this.getObjects = function (keys) {
-            var results = {},
-                promises = keys.map(function (keyString) {
-                    var key = parseKeyString(keyString);
-                    return Objects.get(key)
-                        .then(function (object) {
-                            object = toOldFormat(object)
-                            results[keyString] = instantiate(object, keyString);
-                        });
-                });
-
-            return Promise.all(promises)
-                .then(function () {
-                    return results;
-                });
-        };
-
-        FALLBACK_PROVIDER = new ObjectServiceProvider(objectService, instantiate);
-
-        ROOTS.forEach(function (r) {
-            ROOT_REGISTRY.push(parseKeyString(r.id));
-        });
-
-        return this;
-    }
-
-    return ObjectAPIInjector;
-});

api/object-api/README.md

@@ -1,101 +0,0 @@
-# Object API - Overview
-
-The object API provides methods for fetching domain objects.
-
-# Keys
-Keys are a composite identifier that is used to create and persist objects.  Ex:
-```javascript
-{
-    namespace: 'elastic',
-    identifier: 'myIdentifier'
-}
-```
-
-In old MCT days, we called this an "id", and we encoded it in a single string.  
-The above key would encode into the identifier, `elastic:myIdentifier`.
-
-When interacting with the API you will be dealing with key objects.
-
-# Configuring the Object API
-
-The following methods should be used before calling run.  They allow you to 
-configure the persistence space of MCT.
-
-* `MCT.objects.addRoot(key)` -- add a "ROOT" to Open MCT by specifying it's 
-    key.
-* `MCT.objects.removeRoot(key)` -- Remove a "ROOT" from Open MCT by key.
-* `MCT.objects.addProvider(namespace, provider)` -- register an object provider
-    for a specific namespace.  See below for documentation on the provider 
-    interface.
-    
-# Using the object API
-
-The object API provides methods for getting, saving, and deleting objects.
-
-* MCT.objects.get(key) -> returns promise for an object
-* MCT.objects.save(object) -> returns promise that is resolved when object
-    has been saved
-* MCT.objects.delete(object) -> returns promise that is resolved when object has 
-    been deleted
-
-## Configuration Example: Adding a groot
-
-The following example adds a new root object for groot and populates it with
-some pieces of groot.
-
-```javascript
-
-var ROOT_KEY = {
-    namespace: 'groot',
-    identifier: 'groot'
-};
-
-var GROOT_ROOT = {
-    name: 'I am groot',
-    type: 'folder',
-    composition: [
-        {
-            namespace: 'groot',
-            identifier: 'arms'
-        },
-        {
-            namespace: 'groot',
-            identifier: 'legs'
-        },
-        {
-            namespace: 'groot',
-            identifier: 'torso'
-        }
-    ]
-};
-
-var GrootProvider = {
-    get: function (key) {
-        if (key.identifier === 'groot') {
-            return Promise.resolve(GROOT_ROOT);
-        }
-        return Promise.resolve({
-            name: 'Groot\'s ' + key.identifier
-        });
-    }
-};
-
-MCT.objects.addRoot(ROOT_KEY);
-
-MCT.objects.addProvider('groot', GrootProvider);
-
-MCT.run();
-```
-
-### Making a custom provider:
-
-All methods on the provider interface are optional, so you do not need
-to modify them.
-
-* `provider.get(key)` -> promise for a domain object.
-* `provider.save(domainObject)` -> returns promise that is fulfilled when object 
-    has been saved.
-* `provider.delete(domainObject)` -> returns promise that is fulfilled when 
-    object has been deleted.
-
-

api/object-api/bundle.js

@@ -1,50 +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.
- *****************************************************************************/
-/*global define*/
-
-define([
-    './ObjectAPI',
-    'legacyRegistry'
-], function (
-    ObjectAPI,
-    legacyRegistry
-) {
-    legacyRegistry.register('api/object-api', {
-        name: 'Object API',
-        description: 'The public Objects API',
-        extensions: {
-            components: [
-                {
-                    provides: "objectService",
-                    type: "decorator",
-                    priority: "mandatory",
-                    implementation: ObjectAPI,
-                    depends: [
-                        "roots[]",
-                        "instantiate"
-                    ]
-                }
-            ]
-        }
-    });
-
-});

api/region-api/README.md

@@ -1,64 +0,0 @@
-# Region API - Overview
-
-The region API provides a method for specifying which views should display in a given region for a given domain object.  As such, they also define the basic view interface that components must define.
-
-### MCT.region.Region
-
-The base region type, all regions implement this interface.
-
-`register(view)`
-
-`getViews(domainObject)`
-
-Additionally, Regions may have subregions for different modes of the application.  Specifying a view for a region 
-
-### MCT.region.View
-
-The basic type for views.  You can extend this to implement your own functionality, or you can create your own object so long as it meets the interface.
-
-`attribute` | `type` | `note`
---- | --- | ---
-`label` | `string` or `Function` | The name of the view.  Used in the view selector when more than one view exists for an object.
-`glyph` | `string` or `Function` | The glyph to associate with this view.  Used in the view selector when more than one view exists for an object.
-`instantiate` | `Function` | constructor for a view.  Will be invoked with two arguments, `container` and `domainObject`.  It should return an object with a `destroy` method that is called when the view is removed.
-`appliesTo` | `Function` | Determines if a view applies to a specific domain object.  Will be invoked with a domainObject.  Should return a number, `priority` if the view applies to a given object.  If multiple views return a truthy value for a given object, they will be ordered by priority, and the largest priority value will be the default view for the object.  Return `false` if a view should not apply to an object.
-
-Basic Hello World:
-
-```javascript
-
-function HelloWorldView(container, domainObject) {
-    container.innerHTML = 'Hello World!';
-}
-
-HelloWorldView.label = 'Hello World';
-HelloWorldView.glyph = 'whatever';
-
-HelloWorldView.appliesTo = function (domainObject) {
-    return 10;
-};
-
-HelloWorldView.prototype.destroy = function () {
-    // clean up outstanding handlers;
-};
-
-MCT.regions.Main.register(HelloWorldView);
-
-```
-
-## Region Hierarchy
-
-Regions are organized in a hierarchy, with the most specific region taking precedence over less specific regions.
-
-If you specify a view for the Main Region, it will be used for both Edit and View modes.  You can override the Main Region view for a specific mode by registering the view with that specific mode.
-
-### MCT.regions.Tree
-### MCT.regions.Main
-### MCT.regions.Main.View
-### MCT.regions.Main.Edit
-### MCT.regions.Inspector
-### MCT.regions.Inspector.View
-### MCT.regions.Inspector.Edit
-### MCT.regions.Toolbar
-### MCT.regions.Toolbar.View
-### MCT.regions.Toolbar.Edit

api/region-api/RegionAPI.js

@@ -1,11 +0,0 @@
-define([
-    
-], function () {
-    
-    function RegionAPI() {
-        window.MCT = window.MCT || {};
-        window.MCT.regions = {};
-    }
-    
-    return RegionAPI;
-})

api/region-api/bundle.js

@@ -1,45 +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([
-    './RegionAPI',
-    'legacyRegistry'
-], function (
-    RegionAPI,
-    legacyRegistry
-) {
-    legacyRegistry.register('api/region-api', {
-        name: 'Region API',
-        description: 'The public Region API',
-        extensions: {
-            runs: [
-                {
-                    key: "RegionAPI",
-                    implementation: RegionAPI,
-                    depends: [
-                    ]
-                }
-            ]
-        }
-    });
-
-});

api/telemetry-api/README.md

@@ -1,71 +0,0 @@
-# Telemetry API - Overview
-
-The Telemetry API provides basic methods for retrieving historical and realtime telemetry data, retrieving telemetry metadata, and registering additional telemetry providers.   
-
-The Telemetry API also provides a set of helpers built upon these basics-- TelemetryFormatters help you format telemetry values for display purposes, LimitEvaluators help you display evaluate and display alarm states, while TelemetryCollections provide a method for seamlessly combining historical and realtime data, while supporting more advanced client side filtering and interactivity.
-
-
-## Getting Telemetry Data
-
-
-### `MCT.telemetry.request(domainObject, options)`
-
-Request historical telemetry for a domain object.  Options allows you to specify filters (start, end, etc.), sort order, and strategies for retrieving telemetry (aggregation, latest available, etc.).
-
-Returns a `Promise` for an array of telemetry values.
-
-### `MCT.telemetry.subscribe(domainObject, callback, options)`
-
-Subscribe to realtime telemetry for a specific domain object.  callback will be called whenever data is received from a realtime provider.  Options allows you to specify ???
-
-## Understanding Telemetry
-
-### `MCT.telemetry.getMetadata(domainObject)`
-
-Retrieve telemetry metadata for a domain object.  Telemetry metadata helps you understand the sort of telemetry data a domain object can provide-- for instances, the possible enumerations or states, the units, and more.
-
-### `MCT.telemetry.Formatter`
-
-Telemetry formatters help you format telemetry values for display.  Under the covers, they use telemetry metadata to interpret your telemetry data, and then they use the format API to format that data for display.
-
-
-### `MCT.telemetry.LimitEvaluator`
-
-Limit Evaluators help you evaluate limit and alarm status of individual telemetry datums for display purposes without having to interact directly with the Limit API.  
-
-## Adding new telemetry sources
-
-### `MCT.telemetry.registerProvider(telemetryProvider)`
-
-Register a telemetry provider with the telemetry service.  This allows you to connect alternative telemetry sources to   For more information, see the `MCT.telemetry.BaseProvider`
-
-### `MCT.telemetry.BaseProvider`
-
-The base provider is a great starting point for developers who would like to implement their own telemetry provider.  At the same time, you can implement your own telemetry provider as long as it meets the TelemetryProvider (see other docs).
-
-## Other tools
-
-### `MCT.telemetry.TelemetryCollection`
-
-The TelemetryCollection is a useful tool for building advanced displays.  It helps you seamlessly handle both historical and realtime telemetry data, while making it easier to deal with large data sets and interactive displays that need to frequently requery data.
-
-
-
-# API Reference (TODO)
-
-* Telemetry Metadata
-* Request Options
-    -- start
-    -- end
-    -- sort
-    -- ???
-    -- strategies -- specify which strategies you want.  an array provides for fallback strategies without needing decoration.  Design fallbacks into API.
-
-### `MCT.telemetry.request(domainObject, options)`
-### `MCT.telemetry.subscribe(domainObject, callback, options)`
-### `MCT.telemetry.getMetadata(domainObject)`
-### `MCT.telemetry.Formatter`
-### `MCT.telemetry.LimitEvaluator`
-### `MCT.telemetry.registerProvider(telemetryProvider)`
-### `MCT.telemetry.BaseProvider`
-### `MCT.telemetry.TelemetryCollection`

api/telemetry-api/TelemetryAPI.js

@@ -1,239 +0,0 @@
-/*global define,window,console,MCT*/
-
-/**
-
-var key = '114ced6c-deb7-4169-ae71-68c571665514';
-MCT.objects.getObject([key])
-    .then(function (results) {
-        console.log('got results');
-        return results[key];
-    })
-    .then(function (domainObject) {
-        console.log('got object');
-        MCT.telemetry.subscribe(domainObject, function (datum) {
-            console.log('gotData!', datum);
-        });
-    });
-});
-
-
-*/
-
-define([
-    'lodash',
-    'eventemitter2'
-], function (
-    _,
-    EventEmitter
-) {
-
-    // format map is a placeholder until we figure out format service.
-    var FORMAT_MAP = {
-        generic: function (range) {
-            return function (datum) {
-                return datum[range.key];
-            };
-        },
-        enum: function (range) {
-            var enumMap = _.indexBy(range.enumerations, 'value');
-            return function (datum) {
-                try {
-                    return enumMap[datum[range.valueKey]].text;
-                } catch (e) {
-                    return datum[range.valueKey];
-                }
-            };
-        }
-    };
-
-    FORMAT_MAP.number =
-        FORMAT_MAP.float =
-        FORMAT_MAP.integer =
-        FORMAT_MAP.ascii =
-        FORMAT_MAP.generic;
-
-
-
-    function TelemetryAPI(
-        formatService
-    ) {
-
-        var FORMATTER_CACHE = new WeakMap(),
-            EVALUATOR_CACHE = new WeakMap();
-
-        function testAPI() {
-            var key = '114ced6c-deb7-4169-ae71-68c571665514';
-            window.MCT.objects.getObjects([key])
-                .then(function (results) {
-                    console.log('got results');
-                    return results[key];
-                })
-                .then(function (domainObject) {
-                    var formatter = new MCT.telemetry.Formatter(domainObject);
-                    console.log('got object');
-                    window.MCT.telemetry.subscribe(domainObject, function (datum) {
-                        var formattedValues = {};
-                        Object.keys(datum).forEach(function (key) {
-                            formattedValues[key] = formatter.format(datum, key);
-                        });
-                        console.log(
-                            'datum:',
-                            datum,
-                            'formatted:',
-                            formattedValues
-                        );
-                    });
-                });
-        }
-
-        function getFormatter(range) {
-            if (FORMAT_MAP[range.type]) {
-                return FORMAT_MAP[range.type](range);
-            }
-            try {
-                var format = formatService.getFormat(range.type).format.bind(
-                        formatService.getFormat(range.type)
-                    ),
-                    formatter = function (datum) {
-                        return format(datum[range.key]);
-                    };
-                return formatter;
-            } catch (e) {
-                console.log('could not retrieve format', range, e, e.message);
-                return FORMAT_MAP.generic(range);
-            }
-        }
-
-        function TelemetryFormatter(domainObject) {
-            this.metadata = domainObject.getCapability('telemetry').getMetadata();
-            this.formats = {};
-            var ranges = this.metadata.ranges.concat(this.metadata.domains);
-
-            ranges.forEach(function (range) {
-                this.formats[range.key] = getFormatter(range);
-            }, this);
-        }
-
-        /**
-         * Retrieve the 'key' from the datum and format it accordingly to
-         * telemetry metadata in domain object.
-         */
-        TelemetryFormatter.prototype.format = function (datum, key) {
-            return this.formats[key](datum);
-        };
-
-        function LimitEvaluator(domainObject) {
-            this.domainObject = domainObject;
-            this.evaluator = domainObject.getCapability('limit');
-            if (!this.evaluator) {
-                this.evalute = function () {
-                    return '';
-                }
-            }
-        }
-
-        /** TODO: Do we need a telemetry parser, or do we assume telemetry
-        is numeric by default? */
-
-        LimitEvaluator.prototype.evaluate = function (datum, key) {
-            return this.evaluator.evaluate(datum, key);
-        };
-
-        /** Basic telemetry collection, needs more magic. **/
-        function TelemetryCollection(domainObject) {
-            this.domainObject = domainObject;
-            this.data = [];
-        }
-
-        _.extend(TelemetryCollection.prototype, EventEmitter.prototype);
-
-        TelemetryCollection.prototype.request = function (options) {
-            request(this.domainObject, options).then(function (data) {
-                data.forEach(function (datum) {
-                    this.addDatum(datum);
-                }, this);
-            }.bind(this));
-        };
-
-        TelemetryCollection.prototype.addDatum = function (datum) {
-            this.data.push(datum);
-            this.emit('add', datum);
-        };
-
-        TelemetryCollection.prototype.subscribe = function (options) {
-            if (this.unsubscribe) {
-                this.unsubscribe();
-                delete this.unsubscribe;
-            }
-
-            this.unsubscribe = subscribe(
-                this.domainObject,
-                function (telemetrySeries) {
-                    telemetrySeries.getData().forEach(this.addDatum, this);
-                }.bind(this),
-                options
-            );
-        };
-
-        function registerProvider(provider) {
-            // Not yet implemented.
-            console.log('registering provider', provider);
-        }
-
-        function registerEvaluator(evaluator) {
-            // not yet implemented.
-            console.log('registering evaluator', evaluator);
-        }
-
-        function request(domainObject, options) {
-            return domainObject.getCapability('telemetry')
-                .requestData(options)
-                .then(function (telemetrySeries) {
-                    return telemetrySeries.getData();
-                });
-        }
-
-        function subscribe(domainObject, callback, options) {
-            return domainObject.getCapability('telemetry')
-                .subscribe(function (series) {
-                    series.getData().forEach(callback);
-                }, options);
-        }
-
-        var Telemetry = {
-            registerProvider: registerProvider,
-            registerEvaluator: registerEvaluator,
-            request: request,
-            subscribe: subscribe,
-            getMetadata: function (domainObject) {
-                return domainObject.getCapability('telemetry').getMetadata();
-            },
-            Formatter: function (domainObject) {
-                if (!FORMATTER_CACHE.has(domainObject)) {
-                    FORMATTER_CACHE.set(
-                        domainObject,
-                        new TelemetryFormatter(domainObject)
-                    );
-                }
-                return FORMATTER_CACHE.get(domainObject);
-            },
-            LimitEvaluator: function (domainObject) {
-                if (!EVALUATOR_CACHE.has(domainObject)) {
-                    EVALUATOR_CACHE.set(
-                        domainObject,
-                        new LimitEvaluator(domainObject)
-                    );
-                }
-                return EVALUATOR_CACHE.get(domainObject);
-            }
-        };
-
-        window.MCT = window.MCT || {};
-        window.MCT.telemetry = Telemetry;
-        window.testAPI = testAPI;
-
-        return Telemetry;
-    }
-
-    return TelemetryAPI;
-});

api/telemetry-api/bundle.js

@@ -1,46 +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.
- *****************************************************************************/
-/*global define*/
-
-define([
-    './TelemetryAPI',
-    'legacyRegistry'
-], function (
-    TelemetryAPI,
-    legacyRegistry
-) {
-    legacyRegistry.register('api/telemetry-api', {
-        name: 'Telemetry API',
-        description: 'The public Telemetry API',
-        extensions: {
-            runs: [
-                {
-                    key: "TelemetryAPI",
-                    implementation: TelemetryAPI,
-                    depends: [
-                        'formatService'
-                    ]
-                }
-            ]
-        }
-    });
-});

api/type-api/README.md

@@ -1,31 +0,0 @@
-# Type API - Overview
-
-The Type API allows you to register type information for domain objects, and allows you to retrieve type information about a given domain object.  Crucially, type information allows you to add new creatible object types.
-
-### MCT.types.Type
-
-The basic interface for a type.  You can extend this to implement your own type,
-or you can provide an object that implements the same interface.
-
-`attribute` | `type` | `note`
---- | --- | ---
-`label` | `String` | The human readible name of the type.
-`key` | `String` | The unique identifier for this type.  
-`glyph` | `String` | The glyph identifier for the type.  Displayed in trees, labels, and other locations.
-`description` | `String` | A basic description of the type visible in the create menu.
-`isCreatible` | `Boolean`, `Number`, or `Function` | If truthy, this type will be visible in the create menu.  Note that objects not in the create menu can be instantiated via other means.
-`namespace` | `String` | The object namespace that provides instances of this type.  This allows you to implement custom object providers for specific types while still utilizing other namespaces for persistence.
-`properties` | `Object` | Object defining properties of an instance of this class.  Properties are used for automatic form generation and automated metadata display.  For more information on the definition of this object, look at (some resource-- jsonschema?)
-`canContain` | `Function` | determins whether objects of this type can contain other objects.  Will be invoked with a domain object.  Return true to allow composition, return false to disallow composition.
-
-### MCT.types.register(type)
-
-Register a type with the type API.  Registering a type with the same key as another type will replace the original type definition.
-
-### MCT.types.getType(typeKey)
-
-Returns the type definition for a given typeKey.  returns undefined if type does not exist.
-
-### MCT.types.getType(domainObject)
-
-Return the type definition for a given domain object.

api/type-api/TypeAPI.js

@@ -1,10 +0,0 @@
-define([
-    
-], function () {
-    function TypeAPI() {
-        window.MCT = window.MCT || {};
-        window.MCT.types = {};
-    }
-    
-    return TypeAPI;
-});

bower.json

@@ -18,8 +18,6 @@
     "node-uuid": "^1.4.7",
     "comma-separated-values": "^3.6.4",
     "FileSaver.js": "^0.0.2",
-    "zepto": "^1.1.6",
-    "eventemitter2": "^1.0.0",
-    "lodash": "3.10.1"
+    "zepto": "^1.1.6"
   }
 }

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

@@ -6,13 +6,12 @@ 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
@@ -1601,61 +1600,6 @@ 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 Web platform provides a variety of services which can be retrieved 

docs/src/process/cycle.md

@@ -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

@@ -3,10 +3,8 @@
 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 Web.

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/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": {
@@ -100,46 +99,26 @@ define([
                         "domains": [
                             {
                                 "key": "time",
-                                "name": "Time",
-                                "type": "utc"
+                                "name": "Time"
                             },
                             {
                                 "key": "yesterday",
-                                "name": "Yesterday",
-                                "type": "utc"
+                                "name": "Yesterday"
                             },
                             {
                                 "key": "delta",
                                 "name": "Delta",
-                                "type": "example.delta"
+                                "format": "example.delta"
                             }
                         ],
                         "ranges": [
                             {
                                 "key": "sin",
-                                "name": "Sine",
-                                "type": "generic"
+                                "name": "Sine"
                             },
                             {
                                 "key": "cos",
-                                "name": "Cosine",
-                                "type": "generic"
-                            },
-                            {
-                                "key": "positive",
-                                "name": "Positive Sine?",
-                                "type": "enum",
-                                "valueKey": "positive",
-                                "enumerations": [
-                                    {
-                                        "value": 0,
-                                        "text": "FALSE"
-                                    },
-                                    {
-                                        "value": 1,
-                                        "text": "TRUE"
-                                    }
-                                ]
+                                "name": "Cosine"
                             }
                         ]
                     },

example/generator/src/SinewaveLimitCapability.js

@@ -62,9 +62,6 @@ define(
                 },
                 evaluate: function (datum, range) {
                     range = range || 'sin';
-                    if (['sin', 'cos'].indexOf(range) === -1) {
-                        return '';
-                    }
                     if (datum[range] > RED) {
                         return LIMITS.rh;
                     }
@@ -87,4 +84,4 @@ define(
 
         return SinewaveLimitCapability;
     }
-);
+);

example/generator/src/SinewaveTelemetryProvider.js

@@ -19,11 +19,10 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
-/*global define,setInterval,clearInterval*/
+/*global define,Promise*/
 
 /**
  * Module defining SinewaveTelemetryProvider. Created by vwoeltje on 11/12/14.
- * Rewritten by larkin on 05/06/2016.
  */
 define(
     ["./SinewaveTelemetrySeries"],
@@ -35,108 +34,85 @@ define(
          * @constructor
          */
         function SinewaveTelemetryProvider($q, $timeout) {
-            this.$q = $q;
-            this.$timeout = $timeout;
-        }
+            var subscriptions = [],
+                generating = false;
 
-        SinewaveTelemetryProvider.prototype.canHandleRequest = function (request) {
-            return request.source === 'generator';
-        };
+            //
+            function matchesSource(request) {
+                return request.source === "generator";
+            }
 
-        SinewaveTelemetryProvider.prototype.requestTelemetry = function (requests) {
-            var sinewaveRequests = requests.filter(this.canHandleRequest, this),
-                response = {
-                    generator: {}
+            // Used internally; this will be repacked by doPackage
+            function generateData(request) {
+                return {
+                    key: request.key,
+                    telemetry: new SinewaveTelemetrySeries(request)
                 };
+            }
 
-            sinewaveRequests.forEach(function (request) {
-                response.generator[request.key] = this.singleRequest(request);
-            }, this);
-
-            return response;
-        };
-
-        SinewaveTelemetryProvider.prototype.subscribe = function (callback, requests) {
-            var sinewaveRequests = requests.filter(this.canHandleRequest, this),
-                unsubscribers = sinewaveRequests.map(function (request) {
-                    return this.singleSubscribe(
-                        function (series) {
-                            var response = {
-                                generator: {}
-                            };
-                            response.generator[request.key] = series;
-                            callback(response);
-                        },
-                        request
-                    );
-                }, this);
-
-            return function () {
-                unsubscribers.forEach(function (unsubscribe) {
-                    unsubscribe();
+            //
+            function doPackage(results) {
+                var packaged = {};
+                results.forEach(function (result) {
+                    packaged[result.key] = result.telemetry;
                 });
-            };
-        };
+                // Format as expected (sources -> keys -> telemetry)
+                return { generator: packaged };
+            }
 
-        SinewaveTelemetryProvider.prototype.singleRequest = function (request) {
-            var start = Math.floor(request.start / 1000) * 1000,
-                end = Math.floor(request.end / 1000) * 1000,
-                period = request.period || 30,
-                data = [],
-                current,
-                i;
+            function requestTelemetry(requests) {
+                return $timeout(function () {
+                    return doPackage(requests.filter(matchesSource).map(generateData));
+                }, 0);
+            }
 
-            for (current = start; current <= end; current += 2000) {
-                i = Math.floor((current - start) / 1000);
-                data.push({
-                    sin: Math.sin(i * Math.PI * 2 / period),
-                    cos: Math.cos(i * Math.PI * 2 / period),
-                    positive: Math.sin(i * Math.PI * 2 / period) >= 0,
-                    time: current,
-                    yesterday: current - (60 * 60 * 24 * 1000),
-                    delta: current
+            function handleSubscriptions() {
+                subscriptions.forEach(function (subscription) {
+                    var requests = subscription.requests;
+                    subscription.callback(doPackage(
+                        requests.filter(matchesSource).map(generateData)
+                    ));
                 });
             }
-            return new SinewaveTelemetrySeries(data);
-        };
 
+            function startGenerating() {
+                generating = true;
+                $timeout(function () {
+                    handleSubscriptions();
+                    if (generating && subscriptions.length > 0) {
+                        startGenerating();
+                    } else {
+                        generating = false;
+                    }
+                }, 1000);
+            }
 
-        SinewaveTelemetryProvider.prototype.singleSubscribe = function (callback, options) {
-            // calculate interval position based on start - end; such that this data will line up with data generated by singleRequest.
-            var start = Math.floor((options.start || Date.now()) / 1000) * 1000,
-                currentTime = Math.floor((options.end || Date.now()) / 1000) * 1000,
-                period = options.period || 30,
-                unsubscribe,
-                generatePoint,
-                interval;
+            function subscribe(callback, requests) {
+                var subscription = {
+                    callback: callback,
+                    requests: requests
+                };
 
-            generatePoint = function () {
-                var i = Math.floor((currentTime - start) / 1000),
-                    point = {
-                        sin: Math.sin(i * Math.PI * 2 / period),
-                        cos: Math.cos(i * Math.PI * 2 / period),
-                        positive: Math.sin(i * Math.PI * 2 / period) >= 0,
-                        time: currentTime,
-                        yesterday: currentTime - (60 * 60 * 24 * 1000),
-                        delta: currentTime
-                    };
-                currentTime += 1000;
-                return point;
+                function unsubscribe() {
+                    subscriptions = subscriptions.filter(function (s) {
+                        return s !== subscription;
+                    });
+                }
+
+                subscriptions.push(subscription);
+
+                if (!generating) {
+                    startGenerating();
+                }
+
+                return unsubscribe;
+            }
+
+            return {
+                requestTelemetry: requestTelemetry,
+                subscribe: subscribe
             };
-
-            interval = setInterval(function () {
-                var series = new SinewaveTelemetrySeries(generatePoint());
-                callback(series);
-            }, 1000);
-
-            unsubscribe = function () {
-                clearInterval(interval);
-            };
-
-            return unsubscribe;
-        };
-
-
+        }
 
         return SinewaveTelemetryProvider;
     }

example/generator/src/SinewaveTelemetrySeries.js

@@ -32,37 +32,47 @@ define(
         var ONE_DAY = 60 * 60 * 24,
             firstObservedTime = Math.floor(SinewaveConstants.START_TIME / 1000);
 
-        function SinewaveTelemetrySeries(data) {
-            if (!Array.isArray(data)) {
-                data = [data];
+        /**
+         *
+         * @constructor
+         */
+        function SinewaveTelemetrySeries(request) {
+            var timeOffset = (request.domain === 'yesterday') ? ONE_DAY : 0,
+                latestTime = Math.floor(Date.now() / 1000) - timeOffset,
+                firstTime = firstObservedTime - timeOffset,
+                endTime = (request.end !== undefined) ?
+                        Math.floor(request.end / 1000) : latestTime,
+                count = Math.min(endTime, latestTime) - firstTime,
+                period = +request.period || 30,
+                generatorData = {},
+                requestStart = (request.start === undefined) ? firstTime :
+                        Math.max(Math.floor(request.start / 1000), firstTime),
+                offset = requestStart - firstTime;
+
+            if (request.size !== undefined) {
+                offset = Math.max(offset, count - request.size);
             }
-            this.data = data;
+
+            generatorData.getPointCount = function () {
+                return count - offset;
+            };
+
+            generatorData.getDomainValue = function (i, domain) {
+                // delta uses the same numeric values as the default domain,
+                // so it's not checked for here, just formatted for display
+                // differently.
+                return (i + offset) * 1000 + firstTime * 1000 -
+                    (domain === 'yesterday' ? (ONE_DAY * 1000) : 0);
+            };
+
+            generatorData.getRangeValue = function (i, range) {
+                range = range || "sin";
+                return Math[range]((i + offset) * Math.PI * 2 / period);
+            };
+
+            return generatorData;
         }
 
-        SinewaveTelemetrySeries.prototype.getPointCount = function () {
-            return this.data.length;
-        };
-
-        SinewaveTelemetrySeries.prototype.getDomainValue = function (i, domain) {
-            return this.getDatum(i)[domain];
-        };
-
-        SinewaveTelemetrySeries.prototype.getRangeValue = function (i, range) {
-            return this.getDatum(i)[range];
-        };
-
-        SinewaveTelemetrySeries.prototype.getDatum = function (i) {
-            if (i >= this.data.length || i < 0) {
-                throw new Error('IndexOutOfRange: index not available in series.');
-            }
-            return this.data[i];
-        };
-
-        SinewaveTelemetrySeries.prototype.getData = function () {
-            return this.data;
-        };
-
         return SinewaveTelemetrySeries;
-
     }
 );

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/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

@@ -91,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('.'))

main.js

@@ -34,9 +34,7 @@ requirejs.config({
         "screenfull": "bower_components/screenfull/dist/screenfull.min",
         "text": "bower_components/text/text",
         "uuid": "bower_components/node-uuid/uuid",
-        "zepto": "bower_components/zepto/zepto.min",
-        "eventemitter2": "bower_components/eventemitter2/lib/eventemitter2",
-        "lodash": "bower_components/lodash/lodash"
+        "zepto": "bower_components/zepto/zepto.min"
     },
     "shim": {
         "angular": {
@@ -92,10 +90,6 @@ define([
     './platform/search/bundle',
     './platform/status/bundle',
     './platform/commonUI/regions/bundle',
-    './api/telemetry-api/bundle',
-    './api/object-api/bundle',
-    './api/composition-api/bundle',
-    './example/scratchpad/bundle',
 
     './example/imagery/bundle',
     './example/eventGenerator/bundle',
@@ -109,4 +103,4 @@ define([
             return new Main().run(legacyRegistry);
         }
     };
-});
+});

package.json

@@ -1,7 +1,7 @@
 {
   "name": "openmctweb",
-  "version": "0.10.1-SNAPSHOT",
-  "description": "The Open MCT core platform",
+  "version": "0.9.3-SNAPSHOT",
+  "description": "The Open MCT Web core platform",
   "dependencies": {
     "express": "^4.13.1",
     "minimist": "^1.1.1",
@@ -49,7 +49,7 @@
     "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",

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

@@ -44,7 +44,22 @@
         </div>
     </div>
     <div class="holder l-flex-col flex-elem grows l-object-wrapper">
-        <div class="holder l-flex-col flex-elem grows l-object-wrapper-inner">
+        <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'"

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

@@ -93,23 +93,27 @@ define(
                 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 persistAndReturn(parentObject);
+                return parentObject.getCapability('persistence').persist().then(function(){
+                    return 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);
 
         };

platform/commonUI/edit/bundle.js

@@ -32,13 +32,9 @@ define([
     "./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",
     "text!./res/templates/library.html",
@@ -58,13 +54,9 @@ define([
     PropertiesAction,
     RemoveAction,
     SaveAction,
-    SaveAsAction,
     CancelAction,
     EditActionPolicy,
-    EditableLinkPolicy,
-    EditableMovePolicy,
     EditNavigationPolicy,
-    EditContextualActionPolicy,
     EditRepresenter,
     EditToolbarRepresenter,
     libraryTemplate,
@@ -169,15 +161,6 @@ define([
                     "implementation": SaveAction,
                     "name": "Save",
                     "description": "Save changes made to these objects.",
-                    "depends": [],
-                    "priority": "mandatory"
-                },
-                {
-                    "key": "save",
-                    "category": "conclude-editing",
-                    "implementation": SaveAsAction,
-                    "name": "Save",
-                    "description": "Save changes made to these objects.",
                     "depends": [
                         "$injector",
                         "policyService",
@@ -204,19 +187,6 @@ define([
                     "category": "action",
                     "implementation": EditActionPolicy
                 },
-                {
-                    "category": "action",
-                    "implementation": EditContextualActionPolicy,
-                    "depends": ["navigationService", "editModeBlacklist", "nonEditContextBlacklist"]
-                },
-                {
-                    "category": "action",
-                    "implementation": EditableMovePolicy
-                },
-                {
-                    "category": "action",
-                    "implementation": EditableLinkPolicy
-                },
                 {
                     "category": "navigation",
                     "message": "There are unsaved changes.",
@@ -274,16 +244,6 @@ define([
                 {
                     "implementation": EditToolbarRepresenter
                 }
-            ],
-            "constants": [
-                {
-                    "key":"editModeBlacklist",
-                    "value": ["copy", "follow", "window", "link", "locate"]
-                },
-                {
-                    "key": "nonEditContextBlacklist",
-                    "value": ["copy", "follow", "properties", "move", "link", "remove", "locate"]
-                }
             ]
         }
     });

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

@@ -24,8 +24,8 @@
 
 
 define(
-    [],
-    function () {
+    ['../../../browse/src/creation/CreateWizard'],
+    function (CreateWizard) {
         'use strict';
 
         /**
@@ -37,11 +37,31 @@ define(
          * @memberof platform/commonUI/edit
          */
         function SaveAction(
+            $injector,
+            policyService,
+            dialogService,
+            creationService,
+            copyService,
             context
         ) {
             this.domainObject = (context || {}).domainObject;
+            this.injectObjectService = function(){
+                this.objectService = $injector.get("objectService");
+            };
+            this.policyService = policyService;
+            this.dialogService = dialogService;
+            this.creationService = creationService;
+            this.copyService = copyService;
         }
 
+        SaveAction.prototype.getObjectService = function(){
+            // Lazily acquire object service (avoids cyclical dependency)
+            if (!this.objectService) {
+                this.injectObjectService();
+            }
+            return this.objectService;
+        };
+
         /**
          * Save changes and conclude editing.
          *
@@ -50,7 +70,9 @@ define(
          * @memberof platform/commonUI/edit.SaveAction#
          */
         SaveAction.prototype.perform = function () {
-            var domainObject = this.domainObject;
+            var domainObject = this.domainObject,
+                copyService = this.copyService,
+                self = this;
 
             function resolveWith(object){
                 return function () {
@@ -58,13 +80,64 @@ define(
                 };
             }
 
+            function doWizardSave(parent) {
+                var context = domainObject.getCapability("context"),
+                    wizard = new CreateWizard(
+                        domainObject,
+                        parent,
+                        self.policyService
+                    );
+
+                return self.dialogService
+                    .getUserInput(
+                        wizard.getFormStructure(true),
+                        wizard.getInitialFormValue()
+                    )
+                    .then(wizard.populateObjectFromInput.bind(wizard));
+            }
+
+            function fetchObject(objectId){
+                return self.getObjectService().getObjects([objectId]).then(function(objects){
+                    return objects[objectId];
+                });
+            }
+
+            function getParent(object){
+                return fetchObject(object.getModel().location);
+            }
+
+            function allowClone(objectToClone) {
+                return (objectToClone.getId() === domainObject.getId()) ||
+                    objectToClone.getCapability('location').isOriginal();
+            }
+
+            function cloneIntoParent(parent) {
+                return copyService.perform(domainObject, parent, allowClone);
+            }
+
+            function cancelEditingAfterClone(clonedObject) {
+                return domainObject.getCapability("editor").cancel()
+                    .then(resolveWith(clonedObject));
+            }
+
             // Invoke any save behavior introduced by the editor capability;
             // this is introduced by EditableDomainObject which is
             // used to insulate underlying objects from changes made
             // during editing.
             function doSave() {
-                return domainObject.getCapability("editor").save()
-                    .then(resolveWith(domainObject.getOriginalObject()));
+                //This is a new 'virtual object' that has not been persisted
+                // yet.
+                if (domainObject.getModel().persisted === undefined){
+                    return getParent(domainObject)
+                        .then(doWizardSave)
+                        .then(getParent)
+                        .then(cloneIntoParent)
+                        .then(cancelEditingAfterClone)
+                        .catch(resolveWith(false));
+                } else {
+                    return domainObject.getCapability("editor").save()
+                        .then(resolveWith(domainObject.getOriginalObject()));
+                }
             }
 
             // Discard the current root view (which will be the editing
@@ -89,8 +162,7 @@ define(
         SaveAction.appliesTo = function (context) {
             var domainObject = (context || {}).domainObject;
             return domainObject !== undefined &&
-                domainObject.hasCapability("editor") &&
-                domainObject.getModel().persisted !== undefined;
+                domainObject.hasCapability("editor");
         };
 
         return SaveAction;

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

@@ -1,169 +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.
- *****************************************************************************/
-/*global define*/
-/*jslint es5: true */
-
-
-define(
-    ['../../../browse/src/creation/CreateWizard'],
-    function (CreateWizard) {
-        'use strict';
-
-        /**
-         * The "Save" action; the action triggered by clicking Save from
-         * Edit Mode. Exits the editing user interface and invokes object
-         * capabilities to persist the changes that have been made.
-         * @constructor
-         * @implements {Action}
-         * @memberof platform/commonUI/edit
-         */
-        function SaveAsAction(
-            $injector,
-            policyService,
-            dialogService,
-            creationService,
-            copyService,
-            context
-        ) {
-            this.domainObject = (context || {}).domainObject;
-            this.injectObjectService = function(){
-                this.objectService = $injector.get("objectService");
-            };
-            this.policyService = policyService;
-            this.dialogService = dialogService;
-            this.creationService = creationService;
-            this.copyService = copyService;
-        }
-
-        /**
-         * @private
-         */
-        SaveAsAction.prototype.createWizard = function (parent) {
-            return new CreateWizard(
-                this.domainObject,
-                parent,
-                this.policyService
-            );
-        };
-
-        /**
-         * @private
-         */
-        SaveAsAction.prototype.getObjectService = function(){
-            // Lazily acquire object service (avoids cyclical dependency)
-            if (!this.objectService) {
-                this.injectObjectService();
-            }
-            return this.objectService;
-        };
-
-        function resolveWith(object){
-            return function () {
-                return object;
-            };
-        }
-
-        /**
-         * Save changes and conclude editing.
-         *
-         * @returns {Promise} a promise that will be fulfilled when
-         *          cancellation has completed
-         * @memberof platform/commonUI/edit.SaveAction#
-         */
-        SaveAsAction.prototype.perform = function () {
-            // Discard the current root view (which will be the editing
-            // UI, which will have been pushed atop the Browse UI.)
-            function returnToBrowse(object) {
-                if (object) {
-                    object.getCapability("action").perform("navigate");
-                }
-                return object;
-            }
-
-            return this.save().then(returnToBrowse);
-        };
-
-        /**
-         * @private
-         */
-        SaveAsAction.prototype.save = function () {
-            var self = this,
-                domainObject = this.domainObject,
-                copyService = this.copyService;
-
-            function doWizardSave(parent) {
-                var wizard = self.createWizard(parent);
-
-                return self.dialogService
-                    .getUserInput(wizard.getFormStructure(true),
-                        wizard.getInitialFormValue()
-                    ).then(wizard.populateObjectFromInput.bind(wizard));
-            }
-
-            function fetchObject(objectId){
-                return self.getObjectService().getObjects([objectId]).then(function(objects){
-                    return objects[objectId];
-                });
-            }
-
-            function getParent(object){
-                return fetchObject(object.getModel().location);
-            }
-
-            function allowClone(objectToClone) {
-                return (objectToClone.getId() === domainObject.getId()) ||
-                    objectToClone.getCapability('location').isOriginal();
-            }
-
-            function cloneIntoParent(parent) {
-                return copyService.perform(domainObject, parent, allowClone);
-            }
-
-            function cancelEditingAfterClone(clonedObject) {
-                return domainObject.getCapability("editor").cancel()
-                    .then(resolveWith(clonedObject));
-            }
-
-            return getParent(domainObject)
-                .then(doWizardSave)
-                .then(getParent)
-                .then(cloneIntoParent)
-                .then(cancelEditingAfterClone)
-                .catch(resolveWith(false));
-        };
-
-        /**
-         * Check if this action is applicable in a given context.
-         * This will ensure that a domain object is present in the context,
-         * and that this domain object is in Edit mode.
-         * @returns true if applicable
-         */
-        SaveAsAction.appliesTo = function (context) {
-            var domainObject = (context || {}).domainObject;
-            return domainObject !== undefined &&
-                domainObject.hasCapability("editor") &&
-                domainObject.getModel().persisted === undefined;
-        };
-
-        return SaveAsAction;
-    }
-);

platform/commonUI/edit/src/policies/EditActionPolicy.js

@@ -38,6 +38,14 @@ define(
             this.policyService = policyService;
         }
 
+        function applicableView(key){
+            return ['plot', 'scrolling'].indexOf(key) >= 0;
+        }
+
+        function editableType(key){
+            return key === 'telemetry.panel';
+        }
+
         /**
          * Get a count of views which are not flagged as non-editable.
          * @private
@@ -57,8 +65,7 @@ define(
 
             // A view is editable unless explicitly flagged as not
             (views || []).forEach(function (view) {
-                if (view.editable === true ||
-                    (view.key === 'plot' && type.getKey() === 'telemetry.panel')) {
+                if (view.editable === true || (applicableView(view.key) && editableType(type.getKey()))) {
                     count++;
                 }
             });

platform/commonUI/edit/src/policies/EditContextualActionPolicy.js

@@ -1,74 +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.
- *****************************************************************************/
-/*global define*/
-
-define(
-    [],
-    function () {
-        "use strict";
-
-        /**
-         * Policy controlling whether the context menu is visible when
-         * objects are being edited
-         * @param navigationService
-         * @param editModeBlacklist A blacklist of actions disallowed from
-         * context menu when navigated object is being edited
-         * @param nonEditContextBlacklist A blacklist of actions disallowed
-         * from context menu of non-editable objects, when navigated object
-         * is being edited
-         * @constructor
-         */
-        function EditContextualActionPolicy(navigationService, editModeBlacklist, nonEditContextBlacklist) {
-            this.navigationService = navigationService;
-
-            //The list of objects disallowed on target object when in edit mode
-            this.editModeBlacklist = editModeBlacklist;
-            //The list of objects disallowed on target object that is not in
-            // edit mode (ie. the context menu in the tree on the LHS).
-            this.nonEditContextBlacklist = nonEditContextBlacklist;
-        }
-
-        function isParentEditable(object) {
-            var parent = object.hasCapability("context") && object.getCapability("context").getParent();
-            return !!parent && parent.hasCapability("editor");
-        }
-
-        EditContextualActionPolicy.prototype.allow = function (action, context) {
-            var selectedObject = context.domainObject,
-                navigatedObject = this.navigationService.getNavigation(),
-                actionMetadata = action.getMetadata ? action.getMetadata() : {};
-
-            if (navigatedObject.hasCapability('editor')) {
-                if (selectedObject.hasCapability('editor') || isParentEditable(selectedObject)){
-                    return this.editModeBlacklist.indexOf(actionMetadata.key) === -1;
-                } else {
-                    //Target is in the context menu
-                    return this.nonEditContextBlacklist.indexOf(actionMetadata.key) === -1;
-                }
-            } else {
-                return true;
-            }
-        };
-
-        return EditContextualActionPolicy;
-    }
-);

platform/commonUI/edit/src/policies/EditableLinkPolicy.js

@@ -1,52 +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.
- *****************************************************************************/
-/*global define*/
-
-define([], function () {
-    "use strict";
-
-    /**
-     * Policy suppressing links when the linked-to domain object is in
-     * edit mode. Domain objects being edited may not have been persisted,
-     * so creating links to these can result in inconsistent state.
-     *
-     * @memberof platform/commonUI/edit
-     * @constructor
-     * @implements {Policy.<View, DomainObject>}
-     */
-    function EditableLinkPolicy() {
-    }
-
-    EditableLinkPolicy.prototype.allow = function (action, context) {
-        var key = action.getMetadata().key;
-
-        if (key === 'link') {
-            return !((context.selectedObject || context.domainObject)
-                .hasCapability('editor'));
-        }
-
-        // Like all policies, allow by default.
-        return true;
-    };
-
-    return EditableLinkPolicy;
-});

platform/commonUI/edit/src/policies/EditableMovePolicy.js

@@ -1,51 +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.
- *****************************************************************************/
-/*global define*/
-
-define([], function () {
-    "use strict";
-
-    /**
-     * Policy suppressing move actions among editable and non-editable
-     * domain objects.
-     * @memberof platform/commonUI/edit
-     * @constructor
-     * @implements {Policy.<View, DomainObject>}
-     */
-    function EditableMovePolicy() {
-    }
-
-    EditableMovePolicy.prototype.allow = function (action, context) {
-        var domainObject = context.domainObject,
-            selectedObject = context.selectedObject,
-            key = action.getMetadata().key;
-
-        if (key === 'move' && domainObject.hasCapability('editor')) {
-            return !!selectedObject && selectedObject.hasCapability('editor');
-        }
-
-        // Like all policies, allow by default.
-        return true;
-    };
-
-    return EditableMovePolicy;
-});

platform/commonUI/edit/src/representers/EditRepresenter.js

@@ -118,6 +118,8 @@ define(
             // Track the represented object
             this.domainObject = representedObject;
 
+            this.scope.isEditable = representedObject.getCapability('status').get('editing');
+
             // Ensure existing watches are released
             this.destroy();
 

platform/commonUI/edit/test/actions/SaveActionSpec.js

@@ -27,11 +27,11 @@ define(
         "use strict";
 
         describe("The Save action", function () {
-            var mockDomainObject,
+            var mockLocation,
+                mockDomainObject,
                 mockEditorCapability,
+                mockUrlService,
                 actionContext,
-                mockActionCapability,
-                capabilities = {},
                 action;
 
             function mockPromise(value) {
@@ -43,62 +43,65 @@ define(
             }
 
             beforeEach(function () {
+                mockLocation = jasmine.createSpyObj(
+                    "$location",
+                    [ "path" ]
+                );
                 mockDomainObject = jasmine.createSpyObj(
                     "domainObject",
-                    [
-                        "getCapability",
-                        "hasCapability",
-                        "getModel",
-                        "getOriginalObject"
-                    ]
+                    [ "getCapability", "hasCapability" ]
                 );
                 mockEditorCapability = jasmine.createSpyObj(
                     "editor",
                     [ "save", "cancel" ]
                 );
-                mockActionCapability = jasmine.createSpyObj(
-                    "actionCapability",
-                    [ "perform"]
+                mockUrlService = jasmine.createSpyObj(
+                    "urlService",
+                    ["urlForLocation"]
                 );
-                capabilities.editor = mockEditorCapability;
-                capabilities.action = mockActionCapability;
+
 
                 actionContext = {
                     domainObject: mockDomainObject
                 };
 
                 mockDomainObject.hasCapability.andReturn(true);
-                mockDomainObject.getCapability.andCallFake(function (capability) {
-                    return capabilities[capability];
-                });
-                mockDomainObject.getModel.andReturn({persisted: 0});
+                mockDomainObject.getCapability.andReturn(mockEditorCapability);
                 mockEditorCapability.save.andReturn(mockPromise(true));
-                mockDomainObject.getOriginalObject.andReturn(mockDomainObject);
 
-                action = new SaveAction(actionContext);
+                action = new SaveAction(mockLocation, mockUrlService, actionContext);
 
             });
 
             it("only applies to domain object with an editor capability", function () {
-                expect(SaveAction.appliesTo(actionContext)).toBe(true);
+                expect(SaveAction.appliesTo(actionContext)).toBeTruthy();
                 expect(mockDomainObject.hasCapability).toHaveBeenCalledWith("editor");
 
                 mockDomainObject.hasCapability.andReturn(false);
                 mockDomainObject.getCapability.andReturn(undefined);
-                expect(SaveAction.appliesTo(actionContext)).toBe(false);
+                expect(SaveAction.appliesTo(actionContext)).toBeFalsy();
             });
 
-            it("only applies to domain object that has already been persisted",
-                function () {
-                    mockDomainObject.getModel.andReturn({persisted: undefined});
-                    expect(SaveAction.appliesTo(actionContext)).toBe(false);
+            //TODO: Disabled for NEM Beta
+            xit("invokes the editor capability's save functionality when performed", function () {
+                // Verify precondition
+                expect(mockEditorCapability.save).not.toHaveBeenCalled();
+                action.perform();
+
+                // Should have called cancel
+                expect(mockEditorCapability.save).toHaveBeenCalled();
+
+                // Also shouldn't call cancel
+                expect(mockEditorCapability.cancel).not.toHaveBeenCalled();
             });
 
-            it("uses the editor capability to save the object",
-                function () {
-                    action.perform();
-                    expect(mockEditorCapability.save).toHaveBeenCalled();
-                });
+            //TODO: Disabled for NEM Beta
+            xit("returns to browse when performed", function () {
+                action.perform();
+                expect(mockLocation.path).toHaveBeenCalledWith(
+                    mockUrlService.urlForLocation("browse", mockDomainObject)
+                );
+            });
         });
     }
 );

platform/commonUI/edit/test/actions/SaveAsActionSpec.js

@@ -1,174 +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.
- *****************************************************************************/
-/*global define,describe,it,expect,beforeEach,jasmine,xit,xdescribe*/
-
-define(
-    ["../../src/actions/SaveAsAction"],
-    function (SaveAsAction) {
-        "use strict";
-
-        describe("The Save As action", function () {
-            var mockDomainObject,
-                mockEditorCapability,
-                mockActionCapability,
-                mockObjectService,
-                mockDialogService,
-                mockCopyService,
-                mockParent,
-                mockUrlService,
-                actionContext,
-                capabilities = {},
-                action;
-
-            function noop () {}
-
-            function mockPromise(value) {
-                return (value || {}).then ? value :
-                    {
-                        then: function (callback) {
-                            return mockPromise(callback(value));
-                        },
-                        catch: function (callback) {
-                            return mockPromise(callback(value));
-                        }
-                }   ;
-            }
-
-            beforeEach(function () {
-                mockDomainObject = jasmine.createSpyObj(
-                    "domainObject",
-                    [
-                        "getCapability",
-                        "hasCapability",
-                        "getModel"
-                    ]
-                );
-                mockDomainObject.hasCapability.andReturn(true);
-                mockDomainObject.getCapability.andCallFake(function (capability) {
-                    return capabilities[capability];
-                });
-                mockDomainObject.getModel.andReturn({location: 'a', persisted: undefined});
-
-                mockParent = jasmine.createSpyObj(
-                    "parentObject",
-                    [
-                        "getCapability",
-                        "hasCapability",
-                        "getModel"
-                    ]
-                );
-
-                mockEditorCapability = jasmine.createSpyObj(
-                    "editor",
-                    [ "save", "cancel" ]
-                );
-                mockEditorCapability.cancel.andReturn(mockPromise(undefined));
-                mockEditorCapability.save.andReturn(mockPromise(true));
-                capabilities.editor = mockEditorCapability;
-
-                mockActionCapability = jasmine.createSpyObj(
-                    "action",
-                    ["perform"]
-                );
-                capabilities.action = mockActionCapability;
-
-                mockObjectService = jasmine.createSpyObj(
-                    "objectService",
-                    ["getObjects"]
-                );
-                mockObjectService.getObjects.andReturn(mockPromise({'a': mockParent}));
-
-                mockDialogService = jasmine.createSpyObj(
-                    "dialogService",
-                    [
-                        "getUserInput"
-                    ]
-                );
-                mockDialogService.getUserInput.andReturn(mockPromise(undefined));
-
-                mockCopyService = jasmine.createSpyObj(
-                    "copyService",
-                    [
-                        "perform"
-                    ]
-                );
-
-                mockUrlService = jasmine.createSpyObj(
-                    "urlService",
-                    ["urlForLocation"]
-                );
-
-                actionContext = {
-                    domainObject: mockDomainObject
-                };
-
-                action = new SaveAsAction(undefined, undefined, mockDialogService, undefined, mockCopyService, actionContext);
-
-                spyOn(action, "getObjectService");
-                action.getObjectService.andReturn(mockObjectService);
-
-                spyOn(action, "createWizard");
-                action.createWizard.andReturn({
-                    getFormStructure: noop,
-                    getInitialFormValue: noop,
-                    populateObjectFromInput: function() {
-                        return mockDomainObject;
-                    }
-                });
-
-            });
-
-            it("only applies to domain object with an editor capability", function () {
-                expect(SaveAsAction.appliesTo(actionContext)).toBe(true);
-                expect(mockDomainObject.hasCapability).toHaveBeenCalledWith("editor");
-
-                mockDomainObject.hasCapability.andReturn(false);
-                mockDomainObject.getCapability.andReturn(undefined);
-                expect(SaveAsAction.appliesTo(actionContext)).toBe(false);
-            });
-
-            it("only applies to domain object that has not already been" +
-                " persisted", function () {
-                expect(SaveAsAction.appliesTo(actionContext)).toBe(true);
-                expect(mockDomainObject.hasCapability).toHaveBeenCalledWith("editor");
-
-                mockDomainObject.getModel.andReturn({persisted: 0});
-                expect(SaveAsAction.appliesTo(actionContext)).toBe(false);
-            });
-
-            it("returns to browse after save", function () {
-                spyOn(action, "save");
-                action.save.andReturn(mockPromise(mockDomainObject));
-                action.perform();
-                expect(mockActionCapability.perform).toHaveBeenCalledWith(
-                    "navigate"
-                );
-            });
-
-            it("prompts the user for object details", function () {
-                action.perform();
-                expect(mockDialogService.getUserInput).toHaveBeenCalled();
-            });
-
-        });
-    }
-);

platform/commonUI/edit/test/policies/EditActionPolicySpec.js

@@ -35,43 +35,19 @@ define(
                 mockDomainObject,
                 mockEditAction,
                 mockPropertiesAction,
-                mockTypeCapability,
-                mockStatusCapability,
-                capabilities,
-                plotView,
                 policy;
 
             beforeEach(function () {
                 mockDomainObject = jasmine.createSpyObj(
                     'domainObject',
-                    [
-                        'useCapability',
-                        'hasCapability',
-                        'getCapability'
-                    ]
+                    [ 'useCapability' ]
                 );
-                mockStatusCapability = jasmine.createSpyObj('statusCapability', ['get']);
-                mockStatusCapability.get.andReturn(false);
-                mockTypeCapability = jasmine.createSpyObj('type', ['getKey']);
-                capabilities = {
-                    'status': mockStatusCapability,
-                    'type': mockTypeCapability
-                };
-
                 mockEditAction = jasmine.createSpyObj('edit', ['getMetadata']);
                 mockPropertiesAction = jasmine.createSpyObj('edit', ['getMetadata']);
 
-                mockDomainObject.getCapability.andCallFake(function(capability){
-                    return capabilities[capability];
-                });
-                mockDomainObject.hasCapability.andCallFake(function(capability){
-                    return !!capabilities[capability];
-                });
-
                 editableView = { editable: true };
                 nonEditableView = { editable: false };
                 undefinedView = { someKey: "some value" };
-                plotView = { key: "plot", editable: false };
                 testViews = [];
 
                 mockDomainObject.useCapability.andCallFake(function (c) {
@@ -90,53 +66,38 @@ define(
                 policy = new EditActionPolicy();
             });
 
-            it("allows the edit action when there are editable views", function () {
+            //TODO: Disabled for NEM Beta
+            xit("allows the edit action when there are editable views", function () {
                 testViews = [ editableView ];
-                expect(policy.allow(mockEditAction, testContext)).toBe(true);
+                expect(policy.allow(mockEditAction, testContext)).toBeTruthy();
+                // No edit flag defined; should be treated as editable
+                testViews = [ undefinedView, undefinedView ];
+                expect(policy.allow(mockEditAction, testContext)).toBeTruthy();
             });
 
-            it("allows the edit properties action when there are no editable views", function () {
+            //TODO: Disabled for NEM Beta
+            xit("allows the edit properties action when there are no editable views", function () {
                 testViews = [ nonEditableView, nonEditableView ];
-                expect(policy.allow(mockPropertiesAction, testContext)).toBe(true);
+                expect(policy.allow(mockPropertiesAction, testContext)).toBeTruthy();
             });
 
-            it("disallows the edit action when there are no editable views", function () {
+            //TODO: Disabled for NEM Beta
+            xit("disallows the edit action when there are no editable views", function () {
                 testViews = [ nonEditableView, nonEditableView ];
-                expect(policy.allow(mockEditAction, testContext)).toBe(false);
+                expect(policy.allow(mockEditAction, testContext)).toBeFalsy();
             });
 
-            it("disallows the edit properties action when there are" +
+            //TODO: Disabled for NEM Beta
+            xit("disallows the edit properties action when there are" +
                 " editable views", function () {
                 testViews = [ editableView ];
-                expect(policy.allow(mockPropertiesAction, testContext)).toBe(false);
+                expect(policy.allow(mockPropertiesAction, testContext)).toBeFalsy();
             });
 
-            it("disallows the edit action when object is already being" +
-                " edited", function () {
-                testViews = [ editableView ];
-                mockStatusCapability.get.andReturn(true);
-                expect(policy.allow(mockEditAction, testContext)).toBe(false);
-            });
-
-            it("allows editing of panels in plot view", function () {
-                testViews = [ plotView ];
-                mockTypeCapability.getKey.andReturn('telemetry.panel');
-
-                expect(policy.allow(mockEditAction, testContext)).toBe(true);
-            });
-
-            it("disallows editing of plot view when object not a panel type", function () {
-                testViews = [ plotView ];
-                mockTypeCapability.getKey.andReturn('something.else');
-
-                expect(policy.allow(mockEditAction, testContext)).toBe(false);
-            });
-
-
             it("allows the edit properties outside of the 'view-control' category", function () {
                 testViews = [ nonEditableView ];
                 testContext.category = "something-else";
-                expect(policy.allow(mockPropertiesAction, testContext)).toBe(true);
+                expect(policy.allow(mockPropertiesAction, testContext)).toBeTruthy();
             });
         });
     }

platform/commonUI/edit/test/policies/EditContextualActionPolicySpec.js

@@ -1,109 +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.
- *****************************************************************************/
-/*global define,describe,it,expect,beforeEach,jasmine,xit,xdescribe*/
-
-define(
-    ["../../src/policies/EditContextualActionPolicy"],
-    function (EditContextualActionPolicy) {
-        "use strict";
-
-        describe("The Edit contextual action policy", function () {
-            var policy,
-                navigationService,
-                mockAction,
-                context,
-                navigatedObject,
-                mockDomainObject,
-                metadata,
-                editModeBlacklist = ["copy", "follow", "window", "link", "locate"],
-                nonEditContextBlacklist = ["copy", "follow", "properties", "move", "link", "remove", "locate"];
-
-            beforeEach(function () {
-                navigatedObject = jasmine.createSpyObj("navigatedObject", ["hasCapability"]);
-                navigatedObject.hasCapability.andReturn(false);
-
-                mockDomainObject = jasmine.createSpyObj("domainObject", ["hasCapability", "getCapability"]);
-                mockDomainObject.hasCapability.andReturn(false);
-
-                navigationService = jasmine.createSpyObj("navigationService", ["getNavigation"]);
-                navigationService.getNavigation.andReturn(navigatedObject);
-
-                metadata = {key: "move"};
-                mockAction = jasmine.createSpyObj("action", ["getMetadata"]);
-                mockAction.getMetadata.andReturn(metadata);
-
-                context = {domainObject: mockDomainObject};
-
-                policy = new EditContextualActionPolicy(navigationService, editModeBlacklist, nonEditContextBlacklist);
-            });
-
-            it('Allows all actions when navigated object not in edit mode', function() {
-                expect(policy.allow(mockAction, context)).toBe(true);
-            });
-
-            it('Allows "window" action when navigated object in edit mode,' +
-                ' but selected object not in edit mode ', function() {
-                navigatedObject.hasCapability.andReturn(true);
-                metadata.key = "window";
-                expect(policy.allow(mockAction, context)).toBe(true);
-            });
-
-            it('Allows "remove" action when navigated object in edit mode,' +
-                ' and selected object not editable, but its parent is.',
-                function() {
-                    var mockParent = jasmine.createSpyObj("parentObject", ["hasCapability"]),
-                        mockContextCapability = jasmine.createSpyObj("contextCapability", ["getParent"]);
-
-                    mockParent.hasCapability.andReturn(true);
-                    mockContextCapability.getParent.andReturn(mockParent);
-                    navigatedObject.hasCapability.andReturn(true);
-
-                    mockDomainObject.getCapability.andReturn(mockContextCapability);
-                    mockDomainObject.hasCapability.andCallFake(function (capability) {
-                        switch (capability) {
-                            case "editor": return false;
-                            case "context": return true;
-                        }
-                    });
-                    metadata.key = "remove";
-
-                    expect(policy.allow(mockAction, context)).toBe(true);
-            });
-
-            it('Disallows "move" action when navigated object in edit mode,' +
-                ' but selected object not in edit mode ', function() {
-                navigatedObject.hasCapability.andReturn(true);
-                metadata.key = "move";
-                expect(policy.allow(mockAction, context)).toBe(false);
-            });
-
-            it('Disallows copy action when navigated object and' +
-                ' selected object in edit mode', function() {
-                navigatedObject.hasCapability.andReturn(true);
-                mockDomainObject.hasCapability.andReturn(true);
-                metadata.key = "copy";
-                expect(policy.allow(mockAction, context)).toBe(false);
-            });
-
-        });
-    }
-);

platform/commonUI/formats/src/UTCTimeFormat.js

@@ -28,10 +28,9 @@ define([
 ) {
     "use strict";
 
-    var DATE_FORMAT = "YYYY-MM-DD HH:mm:ss.SSS",
+    var DATE_FORMAT = "YYYY-MM-DD HH:mm:ss",
         DATE_FORMATS = [
             DATE_FORMAT,
-            "YYYY-MM-DD HH:mm:ss",
             "YYYY-MM-DD HH:mm",
             "YYYY-MM-DD"
         ];
@@ -49,7 +48,7 @@ define([
     }
 
     UTCTimeFormat.prototype.format = function (value) {
-        return moment.utc(value).format(DATE_FORMAT) + "Z";
+        return moment.utc(value).format(DATE_FORMAT);
     };
 
     UTCTimeFormat.prototype.parse = function (text) {

platform/commonUI/formats/test/UTCTimeFormatSpec.js

@@ -40,12 +40,6 @@ define(
                 expect(moment.utc(formatted).valueOf()).toEqual(timestamp);
             });
 
-            it("displays with millisecond precision", function () {
-                var timestamp = 12345670789,
-                    formatted = format.format(timestamp);
-                expect(moment.utc(formatted).valueOf()).toEqual(timestamp);
-            });
-
             it("validates time inputs", function () {
                 expect(format.validate("1977-05-25 11:21:22")).toBe(true);
                 expect(format.validate("garbage text")).toBe(false);

platform/commonUI/general/res/fonts/symbols/wtdsymbols.svg

@@ -80,7 +80,7 @@
 <glyph unicode="&#xe601;" glyph-name="icon-datatable" d="M1024 768c0-106.039-229.23-192-512-192s-512 85.961-512 192c0 106.039 229.23 192 512 192s512-85.961 512-192zM512 448c-282.8 0-512 86-512 192v-512c0-106 229.2-192 512-192s512 86 512 192v512c0-106-229.2-192-512-192zM896 385v-256c-36.6-15.6-79.8-28.8-128-39.4v256c48.2 10.6 91.4 23.8 128 39.4zM256 345.6v-256c-48.2 10.4-91.4 23.8-128 39.4v256c36.6-15.6 79.8-28.8 128-39.4zM384 70v256c41-4 83.8-6 128-6s87 2.2 128 6v-256c-41-4-83.8-6-128-6s-87 2.2-128 6z" />
 <glyph unicode="&#xe602;" glyph-name="icon-tabular-scrolling" d="M64 960c-35.2 0-64-28.8-64-64v-192h448v256h-384zM1024 704v192c0 35.2-28.8 64-64 64h-384v-256h448zM0 576v-192c0-35.2 28.8-64 64-64h384v256h-448zM960 320c35.2 0 64 28.8 64 64v192h-448v-256h384zM512-64l-256 256h512z" />
 <glyph unicode="&#xe603;" glyph-name="icon-alert-triangle" d="M998.208 111.136l-422.702 739.728c-34.928 61.124-92.084 61.124-127.012 0l-422.702-739.728c-34.928-61.126-5.906-111.136 64.494-111.136h843.428c70.4 0 99.422 50.010 64.494 111.136zM512 128c-35.2 0-64 28.8-64 64s28.8 64 64 64 64-28.8 64-64c0-35.2-28.8-64-64-64zM627.448 577.242l-38.898-194.486c-6.902-34.516-41.35-62.756-76.55-62.756s-69.648 28.24-76.552 62.758l-38.898 194.486c-6.902 34.516 16.25 62.756 51.45 62.756h128c35.2 0 58.352-28.24 51.448-62.758z" />
-<glyph unicode="&#xe604;" glyph-name="icon-tabular" d="M896 960h-768c-70.4 0-128-57.6-128-128v-768c0-70.4 57.6-128 128-128h768c70.4 0 128 57.6 128 128v768c0 70.4-57.6 128-128 128zM640 512h-256v192h256v-192zM384 448h256v-192h-256v192zM320 256h-256v192h256v-192zM320 704v-192h-256v192h256zM128 0c-17 0-33 6.6-45.2 18.8s-18.8 28.2-18.8 45.2v128h256v-192h-192zM384 0v192h256v-192h-256zM960 64c0-17-6.6-33-18.8-45.2s-28.2-18.8-45.2-18.8h-192v192h256v-128zM960 256h-256v192h256v-192zM960 512h-256v192h256v-192z" />
+<glyph unicode="&#xe604;" glyph-name="icon-tabular" d="M0 896v-192h448v256h-384c-35.2 0-64-28.8-64-64zM960 960h-384v-256h448v192c0 35.2-28.8 64-64 64zM576 576h448v-256h-448v256zM0 576h448v-256h-448v256zM0 0c0-35.2 28.8-64 64-64h384v256h-448v-192zM576-64h384c35.2 0 64 28.8 64 64v192h-448v-256z" />
 <glyph unicode="&#xe605;" glyph-name="icon-calendar" d="M896 960h-768c-70.4 0-128-57.6-128-128v-768c0-70.4 57.6-128 128-128h768c70.4 0 128 57.6 128 128v768c0 70.4-57.6 128-128 128zM640 512h-256v192h256v-192zM384 448h256v-192h-256v192zM320 256h-256v192h256v-192zM320 704v-192h-256v192h256zM128 0c-17 0-33 6.6-45.2 18.8s-18.8 28.2-18.8 45.2v128h256v-192h-192zM384 0v192h256v-192h-256zM960 64c0-17-6.6-33-18.8-45.2s-28.2-18.8-45.2-18.8h-192v192h256v-128zM960 256h-256v192h256v-192zM960 512h-256v192h256v-192z" />
 <glyph unicode="&#xe606;" glyph-name="icon-paint-bucket" d="M544 736v-224c0-88.4-71.6-160-160-160s-160 71.6-160 160v97.2l-197.4-196.4c-50-50-12.4-215.2 112.4-340s290-162.4 340-112.4l417 423.6-352 352zM896-64c70.6 0 128 57.4 128 128 0 108.6-128 192-128 192s-128-83.4-128-192c0-70.6 57.4-128 128-128zM384 448c-35.4 0-64 28.6-64 64v384c0 35.4 28.6 64 64 64s64-28.6 64-64v-384c0-35.4-28.6-64-64-64z" />
 <glyph unicode="&#xe607;" glyph-name="icon-x-in-circle" d="M512 960c-282.8 0-512-229.2-512-512s229.2-512 512-512 512 229.2 512 512-229.2 512-512 512zM832 256l-128-128-192 192-192-192-128 128 192 192-192 192 128 128 192-192 192 192 128-128-192-192 192-192z" />
@@ -94,11 +94,8 @@
 <glyph unicode="&#xe614;" glyph-name="icon-collapse-pane-right" d="M768 960h256v-1024h-256c-105.6 0-192 86.4-192 192v640c0 105.6 86.4 192 192 192zM512 640l-512-320v640z" />
 <glyph unicode="&#xe615;" glyph-name="icon-eye-open" d="M512 896c-261 0-480.6-195.4-512-448 31.4-252.6 251-448 512-448s480.6 195.4 512 448c-31.4 252.6-251 448-512 448zM768.2 225.4c-71.4-62.8-162.8-97.4-257.6-97.4s-186.2 34.6-257.6 97.4c-66.6 58.6-110.6 137.2-125 222.6 0 0 0 0.2 0 0.2 76.8 154 220.8 257.6 384 257.6s307.2-103.8 384-257.6c0 0 0-0.2 0-0.2-14.4-85.4-61.2-164-127.8-222.6zM512 672c-123.8 0-224-100.2-224-224s100.2-224 224-224 224 100.2 224 224-100.2 224-224 224z" />
 <glyph unicode="&#xe616;" glyph-name="icon-eye-open-no-gleam" d="M512 896c-261 0-480.6-195.4-512-448 31.4-252.6 251-448 512-448s480.6 195.4 512 448c-31.4 252.6-251 448-512 448zM768.2 225.4c-71.4-62.8-162.8-97.4-257.6-97.4s-186.2 34.6-257.6 97.4c-66.6 58.6-110.6 137.2-125 222.6 0 0 0 0.2 0 0.2 76.8 154 220.8 257.6 384 257.6s307.2-103.8 384-257.6c0 0 0-0.2 0-0.2-14.4-85.4-61.2-164-127.8-222.6zM512 672c-123.8 0-224-100.2-224-224s100.2-224 224-224 224 100.2 224 224-100.2 224-224 224zM576 416c-53 0-96 43-96 96s43 96 96 96 96-43 96-96c0-53-43-96-96-96z" />
-<glyph unicode="&#xe617;" glyph-name="icon-topic" d="M454.36 483.36l86.3 86.3c9.088 8.965 21.577 14.502 35.36 14.502s26.272-5.537 35.366-14.507l86.294-86.294c19.328-19.358 42.832-34.541 69.047-44.082l1.313 171.722-57.64 57.64c-34.407 34.33-81.9 55.558-134.35 55.558s-99.943-21.228-134.354-55.562l-86.296-86.297c-9.088-8.965-21.577-14.502-35.36-14.502s-26.272 5.537-35.366 14.507l-28.674 28.654v-172.14c19.045-7.022 41.040-11.084 63.984-11.084 52.463 0 99.966 21.239 134.379 55.587zM505.64 412.64l-86.3-86.3c-9.088-8.965-21.577-14.502-35.36-14.502s-26.272 5.537-35.366 14.507l-86.294 86.294c-2 2-4.2 4-6.36 6v-197.36c33.664-30.72 78.65-49.537 128.031-49.537 52.44 0 99.923 21.22 134.333 55.541l86.296 86.296c9.088 8.965 21.577 14.502 35.36 14.502s26.272-5.537 35.366-14.507l86.294-86.294c2-2 4.2-4 6.36-6v197.36c-33.664 30.72-78.65 49.537-128.031 49.537-52.44 0-99.923-21.22-134.333-55.541zM832 960h-128v-192h127.66l0.34-0.34v-639.32l-0.34-0.34h-127.66v-192h128c105.6 0 192 86.4 192 192v640c0 105.6-86.4 192-192 192zM320 128h-127.66l-0.34 0.34v639.32l0.34 0.34h127.66v192h-128c-105.6 0-192-86.4-192-192v-640c0-105.6 86.4-192 192-192h128v192z" />
-<glyph unicode="&#xe618;" glyph-name="icon-session" d="M635.6 435.6c6.6-4.2 13.2-8.6 19.2-13.6l120.4-96.4c29.6-23.8 83.8-23.8 113.4 0l135.2 108c0.2 4.8 0.2 9.4 0.2 14.2 0 52.2-7.8 102.4-22.2 149.8l-154.8-123.6c-58.2-46.6-140.2-59.2-211.4-38.4zM248.6 325.8l120.4 96.4c58 46.4 140 59.2 211.2 38.4-6.6 4.2-13.2 8.6-19.2 13.6l-120.4 96.4c-29.6 23.8-83.8 23.8-113.4 0l-120.2-96.6c-40-32-91.4-48-143-48-21.6 0-43 2.8-63.8 8.4 0-0.6 0-1.2 0-1.6 5-3.4 10-6.8 14.6-10.6l120.4-96.4c29.8-23.8 83.8-23.8 113.4 0zM120.6 581.8l120.4 96.4c80.2 64.2 205.6 64.2 285.8 0l120.4-96.4c29.6-23.8 83.8-23.8 113.4 0l181 144.8c-91.2 140.4-249.6 233.4-429.6 233.4-238.6 0-439.2-163.2-496-384.2 30.8-17.6 77.8-15.6 104.6 6zM689 218l-120.4 96.4c-29.6 23.8-83.8 23.8-113.4 0l-120.2-96.4c-40-32-91.4-48-143-48-47.8 0-95.4 13.8-134.2 41.4 85.6-163.6 256.8-275.4 454.2-275.4s368.6 111.8 454.2 275.4c-80.4-57.4-199.8-55.2-277.2 6.6z" />
+<glyph unicode="&#xe617;" glyph-name="icon-topic" d="M546.4 528.8l32 24c31.6 23.8 91.6 23.8 123.2 0l32-24c10.8-8 22.2-15.2 34.4-21.4v201.2c-38 19.6-82.2 30-128 30-60.4 0-118.2-18.2-162.4-51.4l-32-24c-31.6-23.8-91.6-23.8-123.2 0l-32 24c-10.8 8-22.2 15.2-34.4 21.4v-201.2c38-19.6 82.2-30 128-30 60.4 0 118.2 18.2 162.4 51.4zM640 418.6c-60.4 0-118.2-18.2-162.4-51.4l-32-24c-31.6-23.8-91.6-23.8-123.2 0l-32 24c-10.8 8-22.2 15.2-34.4 21.4v-201.2c38-19.6 82.2-30 128-30 60.4 0 118.2 18.2 162.4 51.4l32 24c31.6 23.8 91.6 23.8 123.2 0l32-24c10.8-8 22.2-15.2 34.4-21.4v201.2c-38 19.6-82.2 30-128 30zM832 960h-128v-192h127.6c0.2 0 0.2-0.2 0.4-0.4v-639.4c0-0.2-0.2-0.2-0.4-0.4h-127.6v-192h128c105.6 0 192 86.4 192 192v640.2c0 105.6-86.4 192-192 192zM320 128h-127.6c-0.2 0-0.2 0.2-0.4 0.4v639.4c0 0.2 0.2 0.2 0.4 0.4h127.6v191.8h-128c-105.6 0-192-86.4-192-192v-640c0-105.6 86.4-192 192-192h128v192z" />
+<glyph unicode="&#xe618;" glyph-name="icon-session" d="M923 521.8l-151 100.6c-36 24-103.8 24-139.8 0l-151-100.6c-44.6-29.8-102.6-46.2-163-46.2s-118.4 16.4-163 46.2l-151.4 100.6c-1.8 1.2-3.8 2.4-5.8 3.6v-208c36.6-7.4 70.6-20.8 99-39.8l151-100.6c36-24 103.8-24 139.8 0l151 100.6c44.6 29.8 102.6 46.2 163 46.2s118.4-16.4 163-46.2l151-100.6c1.8-1.2 3.8-2.4 5.8-3.6v208c-36.2 7.2-70.2 20.8-98.6 39.8zM923 137.8l-151 100.6c-36 24-103.8 24-139.8 0l-151-100.6c-44.6-29.8-102.6-46.2-163-46.2s-118.4 16.4-163 46.2l-151.4 100.6c-1.8 1.2-3.8 2.4-5.8 3.6v-112c0-105.6 86.4-192 192-192h640c94.8 0 174.2 69.8 189.4 160.4-35.6 7.4-68.6 20.8-96.4 39.4zM97 762.2l151-100.6c36-24 103.8-24 139.8 0l151 100.6c44.8 29.8 102.6 46.2 163.2 46.2s118.4-16.4 163-46.2l151-100.6c1.8-1.2 3.8-2.4 5.8-3.6v112c0 105.6-86.4 192-192 192h-639.8c-94.8 0-174.2-69.8-189.4-160.4 35.6-7.4 68.6-20.8 96.4-39.4z" />
 <glyph unicode="&#xe619;" glyph-name="icon-bullet" d="M832 208c0-44-36-80-80-80h-480c-44 0-80 36-80 80v480c0 44 36 80 80 80h480c44 0 80-36 80-80v-480z" />
-<glyph unicode="&#xe620;" glyph-name="icon-tabular-realtime" d="M896 960h-768c-70.606-0.215-127.785-57.394-128-127.979v-768.021c0.215-70.606 57.394-127.785 127.979-128h768.021c70.606 0.215 127.785 57.394 128 127.979v768.021c-0.215 70.606-57.394 127.785-127.979 128zM448 668l25.060-25.32c7.916-7.922 18.856-12.822 30.94-12.822s23.024 4.9 30.94 12.822l75.5 76.3c29.97 30.338 71.571 49.128 117.56 49.128s87.59-18.79 117.544-49.112l50.456-50.997v-152.2c-24.111 8.83-44.678 22.255-61.542 39.342l-75.518 76.318c-7.916 7.922-18.856 12.822-30.94 12.822s-23.024-4.9-30.94-12.822l-75.5-76.3c-29.971-30.343-71.575-49.137-117.568-49.137-20.084 0-39.331 3.584-57.137 10.146l1.145 151.831zM320 0h-192c-35.26 0.214-63.786 28.74-64 63.98v128.020h256v-192zM320 256h-256v192h256v-192zM320 512h-256v192h256v-192zM640 0h-256v192h256v-192zM448 323.38v174.5c1.88-1.74 3.74-3.5 5.56-5.34l75.5-76.3c7.916-7.922 18.856-12.822 30.94-12.822s23.024 4.9 30.94 12.822l75.5 76.3c29.966 30.333 71.56 49.119 117.542 49.119 43.28 0 82.673-16.644 112.128-43.879l-0.11-174.399c-1.88 1.74-3.74 3.5-5.56 5.34l-75.5 76.3c-7.916 7.922-18.856 12.822-30.94 12.822s-23.024-4.9-30.94-12.822l-75.5-76.3c-29.966-30.333-71.56-49.119-117.542-49.119-43.28 0-82.673 16.644-112.128 43.879zM960 64c-0.214-35.26-28.74-63.786-63.98-64h-192.020v192h256v-128z" />
-<glyph unicode="&#xe621;" glyph-name="icon-tabular-lad" d="M896 960h-768c-70.606-0.215-127.785-57.394-128-127.979v-768.021c0.215-70.606 57.394-127.785 127.979-128h768.021c70.606 0.215 127.785 57.394 128 127.979v768.021c-0.215 70.606-57.394 127.785-127.979 128zM64 704h256v-192h-256v192zM64 448h256v-192h-256v192zM128 0c-35.26 0.214-63.786 28.74-64 63.98v128.020h256v-192h-192zM384 0v192h256v-192h-256zM960 64c-0.214-35.26-28.74-63.786-63.98-64h-192.020v192h256v-128zM960 448v-192h-576v192h64v64h-64v192h576v-192h-64v-64h64zM782.32 412.62l-110.32 55.16v172.22c0 17.673-14.327 32-32 32s-32-14.327-32-32v-211.78l145.68-72.84c4.172-2.133 9.1-3.383 14.32-3.383 17.675 0 32.003 14.328 32.003 32.003 0 12.454-7.114 23.247-17.501 28.536z" />
-<glyph unicode="&#xe622;" glyph-name="icon-tabular-lad-set" d="M128 192v576c-70.606-0.215-127.785-57.394-128-127.979v-576.021c0.215-70.606 57.394-127.785 127.979-128h576.021c70.606 0.215 127.785 57.394 128 127.979l-576 0.021c-70.606 0.215-127.785 57.394-128 127.979zM896 960h-576c-70.606-0.215-127.785-57.394-128-127.979v-576.021c0.215-70.606 57.394-127.785 127.979-128h576.021c70.606 0.215 127.785 57.394 128 127.979v576.021c-0.215 70.606-57.394 127.785-127.979 128zM256 768h192v-128h-192v128zM256 576h192v-192h-192v192zM320 192c-35.26 0.214-63.786 28.74-64 63.98v64.020h192v-128h-128zM512 192v128h192v-128h-192zM960 256c-0.214-35.26-28.74-63.786-63.98-64h-128.020v128h192v-64zM960 384h-448v384h448v-384zM832 480c0.002 0 0.005 0 0.007 0 17.673 0 32 14.327 32 32 0 14.055-9.062 25.994-21.662 30.293l-74.345 24.767v104.94c0 17.673-14.327 32-32 32s-32-14.327-32-32v-151.060l117.88-39.3c3.018-1.040 6.495-1.64 10.113-1.64 0.003 0 0.005 0 0.008 0z" />
 <glyph unicode="&#xe642;" glyph-name="icon-x" d="M384 448l-365.332-365.332c-24.89-24.89-24.89-65.62 0-90.51l37.49-37.49c24.89-24.89 65.62-24.89 90.51 0 0 0 365.332 365.332 365.332 365.332l365.332-365.332c24.89-24.89 65.62-24.89 90.51 0l37.49 37.49c24.89 24.89 24.89 65.62 0 90.51l-365.332 365.332c0 0 365.332 365.332 365.332 365.332 24.89 24.89 24.89 65.62 0 90.51l-37.49 37.49c-24.89 24.89-65.62 24.89-90.51 0 0 0-365.332-365.332-365.332-365.332l-365.332 365.332c-24.89 24.89-65.62 24.89-90.51 0l-37.49-37.49c-24.89-24.89-24.89-65.62 0-90.51 0 0 365.332-365.332 365.332-365.332z" />
 </font></defs></svg>

platform/commonUI/general/res/sass/_constants.scss

@@ -98,11 +98,6 @@ $bubbleMaxW: 300px;
 $reqSymbolW: 15px;
 $reqSymbolM: $interiorMargin * 2;
 $reqSymbolFontSize: 0.7em;
-// Wait Spinner Defaults
-$waitSpinnerD: 32px;
-$waitSpinnerTreeD: 20px;
-$waitSpinnerBorderW: 5px;
-$waitSpinnerTreeBorderW: 4px;
 
 /************************** CONTROLS */
 $controlDisabledOpacity: 0.3;

platform/commonUI/general/res/sass/_global.scss

@@ -84,11 +84,7 @@ p {
 	margin-bottom: $interiorMarginLg;
 }
 
-ol, ul {
-    list-style: none;
-    margin: 0;
-    padding-left: 0;
-}
+ol, ul { padding-left: 0; }
 
 mct-container {
 	display: block;

platform/commonUI/general/res/sass/_object-label.scss

@@ -35,15 +35,15 @@
     }
 }
 
-mct-representation,
-.rep-object-label {
+mct-representation {
     &.s-status-pending {
         .t-object-label {
             .t-item-icon {
                 &:before {
-                    @include spinner($waitSpinnerTreeBorderW,  $colorLoadingFg);
+                    $spinBW: 4px;
+                    @include spinner($spinBW);
                     content: "";
-                    height: $waitSpinnerTreeD; width: $waitSpinnerTreeD;
+                    padding: 30%;
                 }
                 .t-item-icon-glyph {
                     display: none;
@@ -57,10 +57,7 @@ mct-representation,
     }
 }
 
-.selected mct-representation,
-.selected .rep-object-label {
-    .s-status-pending .t-object-label .t-item-icon:before {
-        border-color: rgba($colorItemTreeSelectedFg, 0.25) !important;
-        border-top-color: rgba($colorItemTreeSelectedFg, 1.0) !important;
-    }
+.selected mct-representation.s-status-pending .t-object-label .t-item-icon:before {
+    border-color: rgba($colorItemTreeSelectedFg, 0.25) !important;
+    border-top-color: rgba($colorItemTreeSelectedFg, 1.0) !important;
 }

platform/commonUI/general/res/sass/controls/_buttons.scss

@@ -76,11 +76,6 @@ $pad: $interiorMargin * $baseRatio;
         font-family: symbolsfont;
         margin-right: $interiorMarginSm;
     }
-    &.t-save-as:before {
-        content:'\e612';
-        font-family: symbolsfont;
-        margin-right: $interiorMarginSm;
-    }
     &.t-cancel {
         .title-label { display: none; }
         &:before {

platform/commonUI/general/res/sass/controls/_menus.scss

@@ -116,7 +116,6 @@
 	ul li {
 		a {
 			color: $colorMenuFg;
-            display: block;
 		}
 		.icon {
 			color: $colorMenuIc;

platform/commonUI/general/res/sass/controls/_time-controller.scss

@@ -1,42 +1,54 @@
 @mixin toiLineHovEffects() {
+	//@include pulse(.25s);
 	&:before,
 	&:after {
 		background-color: $timeControllerToiLineColorHov;
 	}
 }
 
-.l-time-controller {
+mct-include.l-time-controller {
 	$minW: 500px;
 	$knobHOffset: 0px;
 	$knobM: ($sliderKnobW + $knobHOffset) * -1;
 	$rangeValPad: $interiorMargin;
 	$rangeValOffset: $sliderKnobW;
+	//$knobCr: $sliderKnobW;
 	$timeRangeSliderLROffset: 130px + $sliderKnobW + $rangeValOffset;
 	$r1H: nth($ueTimeControlH,1);
 	$r2H: nth($ueTimeControlH,2);
 	$r3H: nth($ueTimeControlH,3);
 
+    //@include absPosDefault();
+    //@include test();
     display: block;
+    //top: auto;
 	height: $r1H + $r2H + $r3H + ($interiorMargin * 2);
     min-width: $minW;
     font-size: 0.8rem;
 
+	.l-time-range-inputs-holder,
+	.l-time-range-slider {
+		//font-size: 0.8em;
+	}
 
 	.l-time-range-inputs-holder,
 	.l-time-range-slider-holder,
 	.l-time-range-ticks-holder
 	{
+		//@include test();
 		@include absPosDefault(0, visible);
 		box-sizing: border-box;
 		top: auto;
 	}
 	.l-time-range-slider,
 	.l-time-range-ticks {
+		//@include test(red, 0.1);
 		@include absPosDefault(0, visible);
 		left: $timeRangeSliderLROffset; right: $timeRangeSliderLROffset;
 	}
 
 	.l-time-range-inputs-holder {
+		//@include test(red);
 		height: $r1H; bottom: $r2H + $r3H + ($interiorMarginSm * 2);
 		padding-top: $interiorMargin;
 		border-top: 1px solid $colorInteriorBorder;
@@ -58,6 +70,7 @@
 	}
 
 	.l-time-range-slider-holder {
+		//@include test(green);
 		height: $r2H; bottom: $r3H + ($interiorMarginSm * 1);
 		.range-holder {
 			box-shadow: none;
@@ -69,6 +82,7 @@
 					$myW: 8px;
 					@include transform(translateX(50%));
 					position: absolute;
+					//@include test();
 					top: 0; right: 0; bottom: 0px; left: auto;
 					width: $myW;
 					height: auto;
@@ -83,6 +97,7 @@
 						// Vert line
 						top: 0; right: auto; bottom: -10px; left: floor($myW/2) - 1;
 						width: 2px;
+						//top: 0; right: 3px; bottom: 0; left: 3px;
 					}
 					&:after {
 						// Circle element
@@ -99,6 +114,7 @@
 			}
 		}
 		&:not(:active) {
+			//@include test(#ff00cc);
 			.knob,
 			.range {
 				@include transition-property(left, right);
@@ -139,6 +155,7 @@
 	.knob {
 		z-index: 2;
 		.range-value {
+			//@include test($sliderColorRange);
 			@include trans-prop-nice-fade(.25s);
 			padding: 0 $rangeValOffset;
 			position: absolute;
@@ -150,6 +167,7 @@
 			color: $sliderColorKnobHov;
 		}
 		&.knob-l {
+			//border-bottom-left-radius: $knobCr; // MOVED TO _CONTROLS.SCSS
 			margin-left: $knobM;
 			.range-value {
 				text-align: right;
@@ -157,6 +175,7 @@
 			}
 		}
 		&.knob-r {
+			//border-bottom-right-radius: $knobCr;
 			margin-right: $knobM;
 			.range-value {
 				left: $rangeValOffset;
@@ -166,189 +185,15 @@
 			}
 		}
 	}
-
-	.l-time-domain-selector {
-		position: absolute;
-		right: 0px;
-		bottom: 46px;
-	}
-
 }
 
+//.slot.range-holder {
+//	background-color: $sliderColorRangeHolder;
+//}
+
 .s-time-range-val {
+	//@include test();
 	border-radius: $controlCr;
 	background-color: $colorInputBg;
 	padding: 1px 1px 0 $interiorMargin;
-}
-
-@include phoneandtablet {
-	.l-time-controller, .l-time-range-inputs-holder {
-		min-width: 0px;
-	}
-
-	.l-time-controller {
-
-		.l-time-domain-selector {
-			select {
-				height: 25px;
-				margin-bottom: 0px;
-			}
-		}
-
-		.l-time-range-slider-holder, .l-time-range-ticks-holder {
-			display: none;
-		}
-
-		.time-range-start, .time-range-end, {
-			width: 100%;
-		}
-
-		.l-time-range-inputs-holder {
-			.l-time-range-input {
-				display: block;
-				.s-btn {
-					padding-right: 18px;
-					white-space: nowrap;
-					input {
-						width: 100%;
-					}
-				}
-			}
-			.l-time-range-inputs-elem {
-
-			}
-		}
-	}
-}
-
-@include phone {
-	.l-time-controller {
-		height: 48px;
-
-		.l-time-range-inputs-holder {
-			bottom: 24px;
-		}
-
-		.l-time-domain-selector {
-			width: 33%;
-			bottom: -9px;
-		}
-
-		.l-time-range-inputs-holder {
-			.l-time-range-input {
-				margin-bottom: 5px;
-				.s-btn {
-					width: 66%;
-				}
-			}
-			.l-time-range-inputs-elem {
-				&.ui-symbol {
-					display: none;
-				}
-
-				&.lbl {
-					width: 33%;
-					right: 0px;
-					top: 5px;
-					display: block;
-					height: 25px;
-					margin: 0;
-					line-height: 25px;
-					position: absolute;
-				}
-			}
-		}
-	}
-}
-
-
-@include tablet {
-	.l-time-controller {
-		height: 17px;
-
-		.l-time-range-inputs-holder {
-			bottom: -7px;
-			left: -5px;
-		}
-
-		.l-time-domain-selector {
-			width: 23%;
-			right: -4px;
-			bottom: -10px;
-		}
-
-		.l-time-range-inputs-holder {
-			.l-time-range-input {
-				float: left;
-				.s-btn {
-					width: 100%;
-					padding-left: 4px;
-				}
-			}
-		}
-	}
-}
-
-@include tabletLandscape {
-	.l-time-controller {
-		height: 17px;
-
-		.l-time-range-inputs-holder {
-			bottom: -7px;
-		}
-
-		.l-time-domain-selector {
-			width: 23%;
-			right: auto;
-			bottom: -10px;
-			left: 391px;
-		}
-
-		.l-time-range-inputs-holder {
-			.l-time-range-inputs-elem {
-				&.ui-symbol, &.lbl {
-					display: block;
-					float: left;
-					line-height: 25px;
-				}
-			}
-		}
-	}
-
-	.pane-tree-hidden .l-time-controller {
-		.l-time-domain-selector {
-			left: 667px;
-		}
-		.l-time-range-inputs-holder {
-			padding-left: 277px;
-		}
-	}
-}
-@include tabletPortrait {
-	.l-time-controller {
-		height: 17px;
-
-		.l-time-range-inputs-holder {
-			bottom: -7px;
-			left: -5px;
-		}
-
-		.l-time-domain-selector {
-			width: 23%;
-			right: -4px;
-			bottom: -10px;
-		}
-
-		.l-time-range-inputs-holder {
-			.l-time-range-input {
-				width: 38%;
-				float: left;
-			}
-			.l-time-range-inputs-elem {
-				&.ui-symbol, &.lbl {
-					display: none;
-				}
-			}
-		}
-	}
-}
+}

platform/commonUI/general/res/sass/forms/_elems.scss

@@ -87,7 +87,9 @@
 			}
 		}
 
-		.hint, .field-hints {  color: $colorFieldHint;  }
+		.field-hints {
+			color: darken($colorBodyFg, 20%);
+		}
 
 		.selector-list {
 			// Used in create overlay to display tree view
@@ -128,14 +130,6 @@
     }
 }
 
-.l-controls-under.l-flex-row {
-    // Change to use column layout
-    @include flex-direction(column);
-    .flex-elem {
-        margin-bottom: $interiorMarginLg;
-    }
-}
-
 .no-validate {
     .form .form-row >.label {
         padding-right: 0;

platform/commonUI/general/res/sass/helpers/_wait-spinner.scss

@@ -28,7 +28,7 @@
     100% { @include transform(translate(-50%, -50%) rotate(360deg)); }
 }
 
-@mixin  spinner($b: 5px, $c: $colorKey) {
+@mixin spinner($b: 5px, $c: $colorKey) {
     @include transform-origin(center);
     @include animation-name(rotation-centered);
     @include animation-duration(0.5s);
@@ -46,7 +46,10 @@
 }
 
 .wait-spinner {
-	@include spinner($waitSpinnerBorderW, $colorKey);
+	$d: 5%;
+	@include spinner(0.5em, $colorKey);
+	height: auto; width: auto;
+	padding: $d; // Will size object based on parent container WIDTH
 	pointer-events: none;
 	z-index: 2;
 	&.inline {
@@ -57,6 +60,15 @@
 	}
 }
 
+.treeview .wait-spinner {
+    // Only used in subtree.html, which I don't think this is actually being used
+	$d: 10px;
+	height: $d; width: $d;
+	margin: 0 !important;
+	padding: 0 !important;
+	top: 2px; left: 0;
+}
+
 .loading {
 	// Can be applied to any block element with height and width
     pointer-events: none;
@@ -65,8 +77,8 @@
         content: '';
     }
 	&:before {
-        @include spinner($waitSpinnerBorderW, $colorLoadingFg);
-        height: $waitSpinnerD; width: $waitSpinnerD;
+        @include spinner(5px, $colorLoadingFg);
+        padding: 5%;
         z-index: 10;
 	}
     &:after {
@@ -75,22 +87,8 @@
         display: block;
         z-index: 9;
     }
-    &.tree-item.t-wait-node {
-        $d: $waitSpinnerTreeD;
-        $spinnerL: $treeVCW + $interiorMargin + 3px + $d/2;
-        padding-left: $spinnerL + $d/2 + $interiorMargin;
-        .t-title-label {
-            font-style: italic;
-            opacity: 0.6;
-        }
-        &:before {
-            height: $d;
-            width: $d;
-            border-width: 4px;
-            left: $spinnerL;
-        }
-        &:after {
-            display: none;
-        }
+    &.tree-item:before {
+        padding: $menuLineH / 4;
+        border-width: 2px;
     }
-}
+}

platform/commonUI/general/res/sass/overlay/_overlay.scss

@@ -65,7 +65,9 @@
         margin-bottom: $interiorMargin;
     }
 
-    .hint, .field-hints { color: $colorFieldHintOverlay !important;  }
+    .hint {
+        color: pushBack($colorOvrFg, 20%);
+    }
 
     .abs.top-bar {
         height: $ovrTopBarH;

platform/commonUI/general/res/templates/controls/time-controller.html

@@ -19,18 +19,15 @@
  this source code distribution or the Licensing information page available
  at runtime from the About dialog for additional information.
 -->
-<div ng-controller="TimeRangeController as trCtrl">
+<div ng-controller="TimeRangeController">
     <form class="l-time-range-inputs-holder"
-          ng-submit="trCtrl.updateBoundsFromForm()">
+          ng-submit="updateBoundsFromForm()">
         <span class="l-time-range-inputs-elem ui-symbol type-icon">&#x43;</span>
         <span class="l-time-range-input">
             <mct-control key="'datetime-field'"
-                         structure="{
-                             format: parameters.format,
-                             validate: trCtrl.validateStart
-                         }"
+                         structure="{ format: parameters.format, validate: validateStart }"
                          ng-model="formModel"
-                         ng-blur="trCtrl.updateBoundsFromForm()"
+                         ng-blur="updateBoundsFromForm()"
                          field="'start'"
                          class="time-range-start">
             </mct-control>
@@ -40,12 +37,9 @@
 
         <span class="l-time-range-input" ng-controller="ToggleController as t2">
             <mct-control key="'datetime-field'"
-                         structure="{
-                             format: parameters.format,
-                             validate: trCtrl.validateEnd
-                         }"
+                         structure="{ format: parameters.format, validate: validateEnd }"
                          ng-model="formModel"
-                         ng-blur="trCtrl.updateBoundsFromForm()"
+                         ng-blur="updateBoundsFromForm()"
                          field="'end'"
                          class="time-range-end">
             </mct-control>&nbsp;
@@ -59,25 +53,22 @@
             <div class="slider"
                  mct-resize="spanWidth = bounds.width">
                 <div class="knob knob-l"
-                     mct-drag-down="trCtrl.startLeftDrag()"
-                     mct-drag="trCtrl.leftDrag(delta[0])"
+                     mct-drag-down="startLeftDrag()"
+                     mct-drag="leftDrag(delta[0])"
                      ng-style="{ left: startInnerPct }">
                     <div class="range-value">{{startInnerText}}</div>
                 </div>
                 <div class="knob knob-r"
-                     mct-drag-down="trCtrl.startRightDrag()"
-                     mct-drag="trCtrl.rightDrag(delta[0])"
+                     mct-drag-down="startRightDrag()"
+                     mct-drag="rightDrag(delta[0])"
                      ng-style="{ right: endInnerPct }">
                     <div class="range-value">{{endInnerText}}</div>
                 </div>
                 <div class="slot range-holder">
                     <div class="range"
-                         mct-drag-down="trCtrl.startMiddleDrag()"
-                         mct-drag="trCtrl.middleDrag(delta[0])"
-                         ng-style="{
-                             left: startInnerPct,
-                             right: endInnerPct
-                         }">
+                         mct-drag-down="startMiddleDrag()"
+                         mct-drag="middleDrag(delta[0])"
+                         ng-style="{ left: startInnerPct, right: endInnerPct}">
                         <div class="toi-line"></div>
                     </div>
                 </div>

platform/commonUI/general/res/templates/tree/wait-node.html

@@ -19,6 +19,9 @@
  this source code distribution or the Licensing information page available
  at runtime from the About dialog for additional information.
 -->
-<li class='tree-item t-wait-node loading'>
-    <span class="t-title-label">Loading...</span>
+<li>
+    <span class="tree-item">
+        <span class="icon wait-spinner"></span>
+        <span class="title-label">Loading...</span>
+    </span>
 </li>

platform/commonUI/general/src/controllers/TimeRangeController.js

@@ -19,289 +19,247 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
-/*global define*/
+/*global define,Promise*/
 
-define([
+define(
+    ['moment'],
+    function (moment) {
+        "use strict";
 
-], function () {
-    "use strict";
+        var TICK_SPACING_PX = 150;
 
-    var TICK_SPACING_PX = 150;
 
-    /* format number as percent; 0.0-1.0 to "0%"-"100%" */
-    function toPercent(p) {
-        return (100 * p) + "%";
+        /**
+         * Controller used by the `time-controller` template.
+         * @memberof platform/commonUI/general
+         * @constructor
+         * @param $scope the Angular scope for this controller
+         * @param {FormatService} formatService the service to user to format
+         *        domain values
+         * @param {string} defaultFormat the format to request when no
+         *        format has been otherwise specified
+         * @param {Function} now a function to return current system time
+         */
+        function TimeRangeController($scope, formatService, defaultFormat, now) {
+            var tickCount = 2,
+                innerMinimumSpan = 1000, // 1 second
+                outerMinimumSpan = 1000, // 1 second
+                initialDragValue,
+                formatter = formatService.getFormat(defaultFormat);
+
+            function formatTimestamp(ts) {
+                return formatter.format(ts);
+            }
+
+            // From 0.0-1.0 to "0%"-"100%"
+            function toPercent(p) {
+                return (100 * p) + "%";
+            }
+
+            function updateTicks() {
+                var i, p, ts, start, end, span;
+                end = $scope.ngModel.outer.end;
+                start = $scope.ngModel.outer.start;
+                span = end - start;
+                $scope.ticks = [];
+                for (i = 0; i < tickCount; i += 1) {
+                    p = i / (tickCount - 1);
+                    ts = p * span + start;
+                    $scope.ticks.push(formatTimestamp(ts));
+                }
+            }
+
+            function updateSpanWidth(w) {
+                tickCount = Math.max(Math.floor(w / TICK_SPACING_PX), 2);
+                updateTicks();
+            }
+
+            function updateViewForInnerSpanFromModel(ngModel) {
+                var span = ngModel.outer.end - ngModel.outer.start;
+
+                // Expose readable dates for the knobs
+                $scope.startInnerText = formatTimestamp(ngModel.inner.start);
+                $scope.endInnerText = formatTimestamp(ngModel.inner.end);
+
+                // And positions for the knobs
+                $scope.startInnerPct =
+                    toPercent((ngModel.inner.start - ngModel.outer.start) / span);
+                $scope.endInnerPct =
+                    toPercent((ngModel.outer.end - ngModel.inner.end) / span);
+            }
+
+            function defaultBounds() {
+                var t = now();
+                return {
+                    start: t - 24 * 3600 * 1000, // One day
+                    end: t
+                };
+            }
+
+            function copyBounds(bounds) {
+                return { start: bounds.start, end: bounds.end };
+            }
+
+            function updateViewFromModel(ngModel) {
+                ngModel = ngModel || {};
+                ngModel.outer = ngModel.outer || defaultBounds();
+                ngModel.inner = ngModel.inner || copyBounds(ngModel.outer);
+
+                // Stick it back is scope (in case we just set defaults)
+                $scope.ngModel = ngModel;
+
+                updateViewForInnerSpanFromModel(ngModel);
+                updateTicks();
+            }
+
+            function startLeftDrag() {
+                initialDragValue = $scope.ngModel.inner.start;
+            }
+
+            function startRightDrag() {
+                initialDragValue = $scope.ngModel.inner.end;
+            }
+
+            function startMiddleDrag() {
+                initialDragValue = {
+                    start: $scope.ngModel.inner.start,
+                    end: $scope.ngModel.inner.end
+                };
+            }
+
+            function toMillis(pixels) {
+                var span =
+                    $scope.ngModel.outer.end - $scope.ngModel.outer.start;
+                return (pixels / $scope.spanWidth) * span;
+            }
+
+            function clamp(value, low, high) {
+                return Math.max(low, Math.min(high, value));
+            }
+
+            function leftDrag(pixels) {
+                var delta = toMillis(pixels);
+                $scope.ngModel.inner.start = clamp(
+                    initialDragValue + delta,
+                    $scope.ngModel.outer.start,
+                    $scope.ngModel.inner.end - innerMinimumSpan
+                );
+                updateViewFromModel($scope.ngModel);
+            }
+
+            function rightDrag(pixels) {
+                var delta = toMillis(pixels);
+                $scope.ngModel.inner.end = clamp(
+                    initialDragValue + delta,
+                    $scope.ngModel.inner.start + innerMinimumSpan,
+                    $scope.ngModel.outer.end
+                );
+                updateViewFromModel($scope.ngModel);
+            }
+
+            function middleDrag(pixels) {
+                var delta = toMillis(pixels),
+                    edge = delta < 0 ? 'start' : 'end',
+                    opposite = delta < 0 ? 'end' : 'start';
+
+                // Adjust the position of the edge in the direction of drag
+                $scope.ngModel.inner[edge] = clamp(
+                    initialDragValue[edge] + delta,
+                    $scope.ngModel.outer.start,
+                    $scope.ngModel.outer.end
+                );
+                // Adjust opposite knob to maintain span
+                $scope.ngModel.inner[opposite] = $scope.ngModel.inner[edge] +
+                    initialDragValue[opposite] - initialDragValue[edge];
+
+                updateViewFromModel($scope.ngModel);
+            }
+
+            function updateFormModel() {
+                $scope.formModel = {
+                    start: (($scope.ngModel || {}).outer || {}).start,
+                    end: (($scope.ngModel || {}).outer || {}).end
+                };
+            }
+
+            function updateOuterStart(t) {
+                var ngModel = $scope.ngModel;
+
+                ngModel.inner.start =
+                    Math.max(ngModel.outer.start, ngModel.inner.start);
+                ngModel.inner.end = Math.max(
+                    ngModel.inner.start + innerMinimumSpan,
+                    ngModel.inner.end
+                );
+
+                updateFormModel();
+                updateViewForInnerSpanFromModel(ngModel);
+                updateTicks();
+            }
+
+            function updateOuterEnd(t) {
+                var ngModel = $scope.ngModel;
+
+                ngModel.inner.end =
+                    Math.min(ngModel.outer.end, ngModel.inner.end);
+                ngModel.inner.start = Math.min(
+                    ngModel.inner.end - innerMinimumSpan,
+                    ngModel.inner.start
+                );
+
+                updateFormModel();
+                updateViewForInnerSpanFromModel(ngModel);
+                updateTicks();
+            }
+
+            function updateFormat(key) {
+                formatter = formatService.getFormat(key || defaultFormat);
+                updateViewForInnerSpanFromModel($scope.ngModel);
+                updateTicks();
+            }
+
+            function updateBoundsFromForm() {
+                var start = $scope.formModel.start,
+                    end = $scope.formModel.end;
+                if (end >= start + outerMinimumSpan) {
+                    $scope.ngModel = $scope.ngModel || {};
+                    $scope.ngModel.outer = { start: start, end: end };
+                }
+            }
+
+            function validateStart(startValue) {
+                return startValue <= $scope.formModel.end - outerMinimumSpan;
+            }
+
+            function validateEnd(endValue) {
+                return endValue >= $scope.formModel.start + outerMinimumSpan;
+            }
+
+            $scope.startLeftDrag = startLeftDrag;
+            $scope.startRightDrag = startRightDrag;
+            $scope.startMiddleDrag = startMiddleDrag;
+            $scope.leftDrag = leftDrag;
+            $scope.rightDrag = rightDrag;
+            $scope.middleDrag = middleDrag;
+
+            $scope.updateBoundsFromForm = updateBoundsFromForm;
+
+            $scope.validateStart = validateStart;
+            $scope.validateEnd = validateEnd;
+
+            $scope.ticks = [];
+
+            // Initialize scope to defaults
+            updateViewFromModel($scope.ngModel);
+            updateFormModel();
+
+            $scope.$watchCollection("ngModel", updateViewFromModel);
+            $scope.$watch("spanWidth", updateSpanWidth);
+            $scope.$watch("ngModel.outer.start", updateOuterStart);
+            $scope.$watch("ngModel.outer.end", updateOuterEnd);
+            $scope.$watch("parameters.format", updateFormat);
+        }
+
+        return TimeRangeController;
     }
-
-    function clamp(value, low, high) {
-        return Math.max(low, Math.min(high, value));
-    }
-
-    function copyBounds(bounds) {
-        return {
-            start: bounds.start,
-            end: bounds.end
-        };
-    }
-
-    /**
-     * Controller used by the `time-controller` template.
-     * @memberof platform/commonUI/general
-     * @constructor
-     * @param $scope the Angular scope for this controller
-     * @param {FormatService} formatService the service to user to format
-     *        domain values
-     * @param {string} defaultFormat the format to request when no
-     *        format has been otherwise specified
-     * @param {Function} now a function to return current system time
-     */
-    function TimeRangeController($scope, formatService, defaultFormat, now) {
-        this.$scope = $scope;
-        this.formatService = formatService;
-        this.defaultFormat = defaultFormat;
-        this.now = now;
-
-        this.tickCount = 2;
-        this.innerMinimumSpan = 1000; // 1 second
-        this.outerMinimumSpan = 1000; // 1 second
-        this.initialDragValue = undefined;
-        this.formatter = formatService.getFormat(defaultFormat);
-        this.formStartChanged = false;
-        this.formEndChanged = false;
-
-        this.$scope.ticks = [];
-
-        this.updateViewFromModel(this.$scope.ngModel);
-        this.updateFormModel();
-
-        [
-            'updateViewFromModel',
-            'updateSpanWidth',
-            'updateOuterStart',
-            'updateOuterEnd',
-            'updateFormat',
-            'validateStart',
-            'validateEnd',
-            'onFormStartChange',
-            'onFormEndChange'
-        ].forEach(function (boundFn) {
-            this[boundFn] = this[boundFn].bind(this);
-        }, this);
-
-        this.$scope.$watchCollection("ngModel", this.updateViewFromModel);
-        this.$scope.$watch("spanWidth", this.updateSpanWidth);
-        this.$scope.$watch("ngModel.outer.start", this.updateOuterStart);
-        this.$scope.$watch("ngModel.outer.end", this.updateOuterEnd);
-        this.$scope.$watch("parameters.format", this.updateFormat);
-        this.$scope.$watch("formModel.start", this.onFormStartChange);
-        this.$scope.$watch("formModel.end", this.onFormEndChange);
-    }
-
-    TimeRangeController.prototype.formatTimestamp = function (ts) {
-        return this.formatter.format(ts);
-    };
-
-    TimeRangeController.prototype.updateTicks = function () {
-        var i, p, ts, start, end, span;
-        end = this.$scope.ngModel.outer.end;
-        start = this.$scope.ngModel.outer.start;
-        span = end - start;
-        this.$scope.ticks = [];
-        for (i = 0; i < this.tickCount; i += 1) {
-            p = i / (this.tickCount - 1);
-            ts = p * span + start;
-            this.$scope.ticks.push(this.formatTimestamp(ts));
-        }
-    };
-
-    TimeRangeController.prototype.updateSpanWidth = function (w) {
-        this.tickCount = Math.max(Math.floor(w / TICK_SPACING_PX), 2);
-        this.updateTicks();
-    };
-
-    TimeRangeController.prototype.updateViewForInnerSpanFromModel = function (
-        ngModel
-    ) {
-        var span = ngModel.outer.end - ngModel.outer.start;
-
-        // Expose readable dates for the knobs
-        this.$scope.startInnerText = this.formatTimestamp(ngModel.inner.start);
-        this.$scope.endInnerText = this.formatTimestamp(ngModel.inner.end);
-
-        // And positions for the knobs
-        this.$scope.startInnerPct =
-            toPercent((ngModel.inner.start - ngModel.outer.start) / span);
-        this.$scope.endInnerPct =
-            toPercent((ngModel.outer.end - ngModel.inner.end) / span);
-    };
-
-    TimeRangeController.prototype.defaultBounds = function () {
-        var t = this.now();
-        return {
-            start: t - 24 * 3600 * 1000, // One day
-            end: t
-        };
-    };
-
-
-    TimeRangeController.prototype.updateViewFromModel = function (ngModel) {
-        ngModel = ngModel || {};
-        ngModel.outer = ngModel.outer || this.defaultBounds();
-        ngModel.inner = ngModel.inner || copyBounds(ngModel.outer);
-
-        // Stick it back is scope (in case we just set defaults)
-        this.$scope.ngModel = ngModel;
-
-        this.updateViewForInnerSpanFromModel(ngModel);
-        this.updateTicks();
-    };
-
-    TimeRangeController.prototype.startLeftDrag = function () {
-        this.initialDragValue = this.$scope.ngModel.inner.start;
-    };
-
-    TimeRangeController.prototype.startRightDrag = function () {
-        this.initialDragValue = this.$scope.ngModel.inner.end;
-    };
-
-    TimeRangeController.prototype.startMiddleDrag = function () {
-        this.initialDragValue = {
-            start: this.$scope.ngModel.inner.start,
-            end: this.$scope.ngModel.inner.end
-        };
-    };
-
-    TimeRangeController.prototype.toMillis = function (pixels) {
-        var span =
-            this.$scope.ngModel.outer.end - this.$scope.ngModel.outer.start;
-        return (pixels / this.$scope.spanWidth) * span;
-    };
-
-    TimeRangeController.prototype.leftDrag = function (pixels) {
-        var delta = this.toMillis(pixels);
-        this.$scope.ngModel.inner.start = clamp(
-            this.initialDragValue + delta,
-            this.$scope.ngModel.outer.start,
-            this.$scope.ngModel.inner.end - this.innerMinimumSpan
-        );
-        this.updateViewFromModel(this.$scope.ngModel);
-    };
-
-    TimeRangeController.prototype.rightDrag = function (pixels) {
-        var delta = this.toMillis(pixels);
-        this.$scope.ngModel.inner.end = clamp(
-            this.initialDragValue + delta,
-            this.$scope.ngModel.inner.start + this.innerMinimumSpan,
-            this.$scope.ngModel.outer.end
-        );
-        this.updateViewFromModel(this.$scope.ngModel);
-    };
-
-    TimeRangeController.prototype.middleDrag = function (pixels) {
-        var delta = this.toMillis(pixels),
-            edge = delta < 0 ? 'start' : 'end',
-            opposite = delta < 0 ? 'end' : 'start';
-
-        // Adjust the position of the edge in the direction of drag
-        this.$scope.ngModel.inner[edge] = clamp(
-            this.initialDragValue[edge] + delta,
-            this.$scope.ngModel.outer.start,
-            this.$scope.ngModel.outer.end
-        );
-        // Adjust opposite knob to maintain span
-        this.$scope.ngModel.inner[opposite] =
-            this.$scope.ngModel.inner[edge] +
-            this.initialDragValue[opposite] -
-            this.initialDragValue[edge];
-
-        this.updateViewFromModel(this.$scope.ngModel);
-    };
-
-    TimeRangeController.prototype.updateFormModel = function () {
-        this.$scope.formModel = {
-            start: ((this.$scope.ngModel || {}).outer || {}).start,
-            end: ((this.$scope.ngModel || {}).outer || {}).end
-        };
-    };
-
-    TimeRangeController.prototype.updateOuterStart = function () {
-        var ngModel = this.$scope.ngModel;
-
-        ngModel.inner.start =
-            Math.max(ngModel.outer.start, ngModel.inner.start);
-        ngModel.inner.end = Math.max(
-            ngModel.inner.start + this.innerMinimumSpan,
-            ngModel.inner.end
-        );
-
-        this.updateFormModel();
-        this.updateViewForInnerSpanFromModel(ngModel);
-        this.updateTicks();
-    };
-
-    TimeRangeController.prototype.updateOuterEnd = function () {
-        var ngModel = this.$scope.ngModel;
-
-        ngModel.inner.end =
-            Math.min(ngModel.outer.end, ngModel.inner.end);
-        ngModel.inner.start = Math.min(
-            ngModel.inner.end - this.innerMinimumSpan,
-            ngModel.inner.start
-        );
-
-        this.updateFormModel();
-        this.updateViewForInnerSpanFromModel(ngModel);
-        this.updateTicks();
-    };
-
-    TimeRangeController.prototype.updateFormat = function (key) {
-        this.formatter = this.formatService.getFormat(key || this.defaultFormat);
-        this.updateViewForInnerSpanFromModel(this.$scope.ngModel);
-        this.updateTicks();
-    };
-
-    TimeRangeController.prototype.updateBoundsFromForm = function () {
-        if (this.formStartChanged) {
-            this.$scope.ngModel.outer.start =
-                this.$scope.ngModel.inner.start =
-                this.$scope.formModel.start;
-            this.formStartChanged = false;
-        }
-        if (this.formEndChanged) {
-            this.$scope.ngModel.outer.end =
-                this.$scope.ngModel.inner.end =
-                this.$scope.formModel.end;
-            this.formEndChanged = false;
-        }
-    };
-
-    TimeRangeController.prototype.onFormStartChange = function (
-        newValue,
-        oldValue
-    ) {
-        if (!this.formStartChanged && newValue !== oldValue) {
-            this.formStartChanged = true;
-        }
-    };
-
-    TimeRangeController.prototype.onFormEndChange = function (
-        newValue,
-        oldValue
-    ) {
-        if (!this.formEndChanged && newValue !== oldValue) {
-            this.formEndChanged = true;
-        }
-    };
-
-    TimeRangeController.prototype.validateStart = function (startValue) {
-        return startValue <=
-            this.$scope.formModel.end - this.outerMinimumSpan;
-    };
-
-    TimeRangeController.prototype.validateEnd = function (endValue) {
-        return endValue >=
-            this.$scope.formModel.start + this.outerMinimumSpan;
-    };
-
-    return TimeRangeController;
-});
+);

platform/commonUI/general/test/controllers/TimeRangeControllerSpec.js

@@ -94,18 +94,18 @@ define(
             it("exposes start time validator", function () {
                 var testValue = 42000000;
                 mockScope.formModel = { end: testValue };
-                expect(controller.validateStart(testValue + 1))
+                expect(mockScope.validateStart(testValue + 1))
                     .toBe(false);
-                expect(controller.validateStart(testValue - 60 * 60 * 1000 - 1))
+                expect(mockScope.validateStart(testValue - 60 * 60 * 1000 - 1))
                     .toBe(true);
             });
 
             it("exposes end time validator", function () {
                 var testValue = 42000000;
                 mockScope.formModel = { start:  testValue };
-                expect(controller.validateEnd(testValue - 1))
+                expect(mockScope.validateEnd(testValue - 1))
                     .toBe(false);
-                expect(controller.validateEnd(testValue + 60 * 60 * 1000 + 1))
+                expect(mockScope.validateEnd(testValue + 60 * 60 * 1000 + 1))
                     .toBe(true);
             });
 
@@ -119,87 +119,25 @@ define(
                         start: DAY * 10000,
                         end: DAY * 11000
                     };
-                });
-
-                it('updates all changed bounds when requested', function () {
+                    // These watches may not exist, but Angular would fire
+                    // them if they did.
                     fireWatchCollection("formModel", mockScope.formModel);
                     fireWatch("formModel.start", mockScope.formModel.start);
                     fireWatch("formModel.end", mockScope.formModel.end);
-
-                    expect(mockScope.ngModel.outer.start)
-                        .not.toEqual(mockScope.formModel.start);
-                    expect(mockScope.ngModel.inner.start)
-                        .not.toEqual(mockScope.formModel.start);
-
-                    expect(mockScope.ngModel.outer.end)
-                        .not.toEqual(mockScope.formModel.end);
-                    expect(mockScope.ngModel.inner.end)
-                        .not.toEqual(mockScope.formModel.end);
-
-                    controller.updateBoundsFromForm();
-
-                    expect(mockScope.ngModel.outer.start)
-                        .toEqual(mockScope.formModel.start);
-                    expect(mockScope.ngModel.inner.start)
-                        .toEqual(mockScope.formModel.start);
-
-                    expect(mockScope.ngModel.outer.end)
-                        .toEqual(mockScope.formModel.end);
-                    expect(mockScope.ngModel.inner.end)
-                        .toEqual(mockScope.formModel.end);
                 });
 
-                it('updates changed start bound when requested', function () {
-                    fireWatchCollection("formModel", mockScope.formModel);
-                    fireWatch("formModel.start", mockScope.formModel.start);
-
+                it("does not immediately make changes to the model", function () {
                     expect(mockScope.ngModel.outer.start)
                         .not.toEqual(mockScope.formModel.start);
-                    expect(mockScope.ngModel.inner.start)
-                        .not.toEqual(mockScope.formModel.start);
-
                     expect(mockScope.ngModel.outer.end)
                         .not.toEqual(mockScope.formModel.end);
-                    expect(mockScope.ngModel.inner.end)
-                        .not.toEqual(mockScope.formModel.end);
-
-                    controller.updateBoundsFromForm();
-
-                    expect(mockScope.ngModel.outer.start)
-                        .toEqual(mockScope.formModel.start);
-                    expect(mockScope.ngModel.inner.start)
-                        .toEqual(mockScope.formModel.start);
-
-                    expect(mockScope.ngModel.outer.end)
-                        .not.toEqual(mockScope.formModel.end);
-                    expect(mockScope.ngModel.inner.end)
-                        .not.toEqual(mockScope.formModel.end);
                 });
 
-                it('updates changed end bound when requested', function () {
-                    fireWatchCollection("formModel", mockScope.formModel);
-                    fireWatch("formModel.end", mockScope.formModel.end);
-
+                it("updates model bounds on request", function () {
+                    mockScope.updateBoundsFromForm();
                     expect(mockScope.ngModel.outer.start)
-                        .not.toEqual(mockScope.formModel.start);
-                    expect(mockScope.ngModel.inner.start)
-                        .not.toEqual(mockScope.formModel.start);
-
+                        .toEqual(mockScope.formModel.start);
                     expect(mockScope.ngModel.outer.end)
-                        .not.toEqual(mockScope.formModel.end);
-                    expect(mockScope.ngModel.inner.end)
-                        .not.toEqual(mockScope.formModel.end);
-
-                    controller.updateBoundsFromForm();
-
-                    expect(mockScope.ngModel.outer.start)
-                        .not.toEqual(mockScope.formModel.start);
-                    expect(mockScope.ngModel.inner.start)
-                        .not.toEqual(mockScope.formModel.start);
-
-                    expect(mockScope.ngModel.outer.end)
-                        .toEqual(mockScope.formModel.end);
-                    expect(mockScope.ngModel.inner.end)
                         .toEqual(mockScope.formModel.end);
                 });
             });
@@ -222,27 +160,27 @@ define(
                 });
 
                 it("updates the start time for left drags", function () {
-                    controller.startLeftDrag();
-                    controller.leftDrag(250);
+                    mockScope.startLeftDrag();
+                    mockScope.leftDrag(250);
                     expect(mockScope.ngModel.inner.start)
                         .toEqual(DAY * 1000 + HOUR * 9);
                 });
 
                 it("updates the end time for right drags", function () {
-                    controller.startRightDrag();
-                    controller.rightDrag(-250);
+                    mockScope.startRightDrag();
+                    mockScope.rightDrag(-250);
                     expect(mockScope.ngModel.inner.end)
                         .toEqual(DAY * 1000 + HOUR * 15);
                 });
 
                 it("updates both start and end for middle drags", function () {
-                    controller.startMiddleDrag();
-                    controller.middleDrag(-125);
+                    mockScope.startMiddleDrag();
+                    mockScope.middleDrag(-125);
                     expect(mockScope.ngModel.inner).toEqual({
                         start: DAY * 1000,
                         end: DAY * 1000 + HOUR * 18
                     });
-                    controller.middleDrag(250);
+                    mockScope.middleDrag(250);
                     expect(mockScope.ngModel.inner).toEqual({
                         start: DAY * 1000 + HOUR * 6,
                         end: DAY * 1001
@@ -250,8 +188,8 @@ define(
                 });
 
                 it("enforces a minimum inner span", function () {
-                    controller.startRightDrag();
-                    controller.rightDrag(-9999999);
+                    mockScope.startRightDrag();
+                    mockScope.rightDrag(-9999999);
                     expect(mockScope.ngModel.inner.end)
                         .toBeGreaterThan(mockScope.ngModel.inner.start);
                 });

platform/commonUI/themes/espresso/res/sass/_constants.scss

@@ -77,7 +77,6 @@ $colorInputBg: rgba(#000, 0.1);
 $colorInputFg: pullForward($colorBodyFg, 20%);
 $colorFormText: rgba(#fff, 0.5);
 $colorInputIcon: pushBack($colorBodyFg, 15%);
-$colorFieldHint: pullForward($colorBodyFg, 20%);
 
 // Inspector
 $colorInspectorBg: pullForward($colorBodyBg, 3%);
@@ -126,7 +125,6 @@ $colorOvrBg: pullForward($colorBodyBg, 10%);
 $colorOvrFg: pullForward($colorBodyFg, 30%);
 $colorOvrBtnBg: pullForward($colorOvrBg, 20%);
 $colorOvrBtnFg: #fff;
-$colorFieldHintOverlay: pullForward($colorOvrBg, 30%);
 
 // Items
 $colorItemBg: lighten($colorBodyBg, 5%);

platform/commonUI/themes/snow/res/sass/_constants.scss

@@ -77,7 +77,6 @@ $colorInputBg: $colorGenBg;
 $colorInputFg: $colorBodyFg;
 $colorFormText: pushBack($colorBodyFg, 10%);
 $colorInputIcon: pushBack($colorBodyFg, 25%);
-$colorFieldHint: pullForward($colorBodyFg, 40%);
 
 // Inspector
 $colorInspectorBg: pullForward($colorBodyBg, 5%);
@@ -126,7 +125,6 @@ $colorOvrBg: $colorBodyBg;
 $colorOvrFg: $colorBodyFg;
 $colorOvrBtnBg: pullForward($colorOvrBg, 40%);
 $colorOvrBtnFg: #fff;
-$colorFieldHintOverlay: pullForward($colorOvrBg, 40%);
 
 // Items
 $colorItemBg: #ddd;

platform/core/bundle.js

@@ -27,7 +27,6 @@ define([
     "./src/models/StaticModelProvider",
     "./src/models/RootModelProvider",
     "./src/models/ModelAggregator",
-    "./src/models/ModelCacheService",
     "./src/models/PersistedModelProvider",
     "./src/models/CachingModelDecorator",
     "./src/models/MissingModelDecorator",
@@ -59,7 +58,6 @@ define([
     StaticModelProvider,
     RootModelProvider,
     ModelAggregator,
-    ModelCacheService,
     PersistedModelProvider,
     CachingModelDecorator,
     MissingModelDecorator,
@@ -184,10 +182,7 @@ define([
                 {
                     "provides": "modelService",
                     "type": "decorator",
-                    "implementation": CachingModelDecorator,
-                    "depends": [
-                        "cacheService"
-                    ]
+                    "implementation": CachingModelDecorator
                 },
                 {
                     "provides": "modelService",
@@ -260,15 +255,14 @@ define([
                 {
                     "key": "root",
                     "name": "Root",
-                    "glyph": "\u0046"
+                    "glyph": "F"
                 },
                 {
                     "key": "folder",
                     "name": "Folder",
-                    "glyph": "\u0046",
+                    "glyph": "F",
                     "features": "creation",
-                    "description": "Create folders to organize other objects or links to objects.",
-                    "priority": 1000,
+                    "description": "Useful for storing and organizing domain objects.",
                     "model": {
                         "composition": []
                     }
@@ -276,11 +270,11 @@ define([
                 {
                     "key": "unknown",
                     "name": "Unknown Type",
-                    "glyph": "\u003f"
+                    "glyph": "?"
                 },
                 {
                     "name": "Unknown Type",
-                    "glyph": "\u003f"
+                    "glyph": "?"
                 }
             ],
             "capabilities": [
@@ -325,7 +319,6 @@ define([
                     "key": "persistence",
                     "implementation": PersistenceCapability,
                     "depends": [
-                        "cacheService",
                         "persistenceService",
                         "identifierService",
                         "notificationService",
@@ -356,16 +349,11 @@ define([
                     "implementation": InstantiationCapability,
                     "depends": [
                         "$injector",
-                        "identifierService",
-                        "now"
+                        "identifierService"
                     ]
                 }
             ],
             "services": [
-                {
-                    "key": "cacheService",
-                    "implementation": ModelCacheService
-                },
                 {
                     "key": "now",
                     "implementation": Now
@@ -396,8 +384,7 @@ define([
                     "implementation": Instantiate,
                     "depends": [
                         "capabilityService",
-                        "identifierService",
-                        "cacheService"
+                        "identifierService"
                     ]
                 }
             ],

platform/core/src/actions/ActionCapability.js

@@ -28,7 +28,7 @@ define(
     [],
     function () {
         "use strict";
-
+        var DISALLOWED_ACTIONS = ["move", "copy", "link", "window", "follow"];
         /**
          * The ActionCapability allows applicable Actions to be retrieved and
          * performed for specific domain objects, e.g.:
@@ -54,22 +54,37 @@ define(
             this.domainObject = domainObject;
         }
 
+        function isEditable(domainObject){
+            return domainObject.getCapability('status').get('editing');
+        }
+
+        function hasEditableAncestor(domainObject){
+            return domainObject.hasCapability('context') &&
+                domainObject
+                    .getCapability('context')
+                    .getPath()
+                    .some(function isEditable (ancestor){
+                        return ancestor.getCapability('status').get('editing');
+                    });
+        }
+
         /**
-         * Perform an action. This will find and perform the
-         * first matching action available for the specified
-         * context or key.
+         * Retrieve the actions applicable to the domain object in the given
+         * context.
          *
          * @param {ActionContext|string} context the context in which
-         *       to perform the action; this is passed along to
-         *       the action service to match against available
+         *       to assess the applicability of the available actions; this is
+         *       passed along to the action service to match against available
          *       actions. The "domainObject" field will automatically
          *       be populated with the domain object that exposed
          *       this capability. If given as a string, this will
          *       be taken as the "key" field to match against
          *       specific actions.
-         * @returns {Promise} the result of the action that was
-         *       performed, or undefined if no matching action
-         *       was found.
+         *
+         *       Additionally, this function will limit the actions
+         *       available for an object in Edit Mode
+         * @returns {Array<Action>} The actions applicable to this domain
+         * object in the given context
          * @memberof platform/core.ActionCapability#
          */
         ActionCapability.prototype.getActions = function (context) {
@@ -78,11 +93,19 @@ define(
             // but additionally adds a domainObject field.
             var baseContext = typeof context === 'string' ?
                         { key: context } : (context || {}),
-                actionContext = Object.create(baseContext);
+                actionContext = Object.create(baseContext),
+                actions;
 
             actionContext.domainObject = this.domainObject;
 
-            return this.actionService.getActions(actionContext);
+            actions = this.actionService.getActions(actionContext) || [];
+            if (isEditable(this.domainObject) || hasEditableAncestor(this.domainObject)){
+                return actions.filter(function(action){
+                    return DISALLOWED_ACTIONS.indexOf(action.getMetadata().key) === -1;
+                });
+            } else {
+                return actions;
+            }
         };
 
         /**

platform/core/src/capabilities/InstantiationCapability.js

@@ -35,16 +35,10 @@ define(
          * @param $injector Angular's `$injector`
          * @implements {Capability}
          */
-        function InstantiationCapability(
-            $injector,
-            identifierService,
-            now,
-            domainObject
-        ) {
+        function InstantiationCapability($injector, identifierService, domainObject) {
             this.$injector = $injector;
             this.identifierService = identifierService;
             this.domainObject = domainObject;
-            this.now = now;
         }
 
         /**
@@ -63,8 +57,6 @@ define(
                 space = parsedId.getDefinedSpace(),
                 id = this.identifierService.generate(space);
 
-            model.modified = this.now();
-
             // Lazily initialize; instantiate depends on capabilityService,
             // which depends on all capabilities, including this one.
             this.instantiateFn = this.instantiateFn ||

platform/core/src/capabilities/PersistenceCapability.js

@@ -46,7 +46,6 @@ define(
          * @implements {Capability}
          */
         function PersistenceCapability(
-            cacheService,
             persistenceService,
             identifierService,
             notificationService,
@@ -57,7 +56,6 @@ define(
             this.modified = domainObject.getModel().modified;
 
             this.domainObject = domainObject;
-            this.cacheService = cacheService;
             this.identifierService = identifierService;
             this.persistenceService = persistenceService;
             this.notificationService = notificationService;
@@ -132,7 +130,6 @@ define(
                 domainObject = this.domainObject,
                 model = domainObject.getModel(),
                 modified = model.modified,
-                cacheService = this.cacheService,
                 persistenceService = this.persistenceService,
                 persistenceFn = model.persisted !== undefined ?
                     this.persistenceService.updateObject :

platform/core/src/models/CachingModelDecorator.js

@@ -35,8 +35,9 @@ define(
          * @param {ModelService} modelService this service to decorate
          * @implements {ModelService}
          */
-        function CachingModelDecorator(cacheService, modelService) {
-            this.cacheService = cacheService;
+        function CachingModelDecorator(modelService) {
+            this.cache = {};
+            this.cached = {};
             this.modelService = modelService;
         }
 
@@ -50,16 +51,17 @@ define(
         }
 
         CachingModelDecorator.prototype.getModels = function (ids) {
-            var cacheService = this.cacheService,
+            var cache = this.cache,
+                cached = this.cached,
                 neededIds = ids.filter(function notCached(id) {
-                    return !cacheService.has(id);
+                    return !cached[id];
                 });
 
             // Update the cached instance of a model to a new value.
             // We update in-place to ensure there is only ever one instance
             // of any given model exposed by the modelService as a whole.
             function updateModel(id, model) {
-                var oldModel = cacheService.get(id);
+                var oldModel = cache[id];
 
                 // Same object instance is a possibility, so don't copy
                 if (oldModel === model) {
@@ -69,7 +71,7 @@ define(
                 // If we'd previously cached an undefined value, or are now
                 // seeing undefined, replace the item in the cache entirely.
                 if (oldModel === undefined || model === undefined) {
-                    cacheService.put(id, model);
+                    cache[id] = model;
                     return model;
                 }
 
@@ -89,15 +91,15 @@ define(
             // Store the provided models in our cache
             function cacheAll(models) {
                 Object.keys(models).forEach(function (id) {
-                    var model = cacheService.has(id) ?
+                    cache[id] = cached[id] ?
                         updateModel(id, models[id]) : models[id];
-                    cacheService.put(id, model);
+                    cached[id] = true;
                 });
             }
 
             // Expose the cache (for promise chaining)
             function giveCache() {
-                return cacheService.all();
+                return cache;
             }
 
             // Look up if we have unknown IDs
@@ -108,7 +110,7 @@ define(
             }
 
             // Otherwise, just expose the cache directly
-            return fastPromise(cacheService.all());
+            return fastPromise(cache);
         };
 
         return CachingModelDecorator;

platform/core/src/models/ModelCacheService.js

@@ -1,83 +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.
- *****************************************************************************/
-/*global define*/
-
-define([], function () {
-    'use strict';
-
-    /**
-     * Provides a cache for domain object models which exist in memory,
-     * but may or may not exist in backing persistene stores.
-     * @constructor
-     * @memberof platform/core
-     */
-    function ModelCacheService() {
-        this.cache = {};
-    }
-
-    /**
-     * Put a domain object model in the cache.
-     * @param {string} id the domain object's identifier
-     * @param {object} model the domain object's model
-     */
-    ModelCacheService.prototype.put = function (id, model) {
-        this.cache[id] = model;
-    };
-
-    /**
-     * Retrieve a domain object model from the cache.
-     * @param {string} id the domain object's identifier
-     * @returns {object} the domain object's model
-     */
-    ModelCacheService.prototype.get = function (id) {
-        return this.cache[id];
-    };
-
-    /**
-     * Check if a domain object model is in the cache.
-     * @param {string} id the domain object's identifier
-     * @returns {boolean} true if present; false if not
-     */
-    ModelCacheService.prototype.has = function (id) {
-        return this.cache.hasOwnProperty(id);
-    };
-
-    /**
-     * Remove a domain object model from the cache.
-     * @param {string} id the domain object's identifier
-     */
-    ModelCacheService.prototype.remove = function (id) {
-        delete this.cache[id];
-    };
-
-    /**
-     * Retrieve all cached domain object models. These are given
-     * as an object containing key-value pairs, where keys are
-     * domain object identifiers and values are domain object models.
-     * @returns {object} all domain object models
-     */
-    ModelCacheService.prototype.all = function () {
-        return this.cache;
-    };
-
-    return ModelCacheService;
-});

platform/core/src/services/Instantiate.js

@@ -44,15 +44,10 @@ define(
          * @param {IdentifierService} identifierService service to generate
          *        new identifiers
          */
-        function Instantiate(
-            capabilityService,
-            identifierService,
-            cacheService
-        ) {
+        function Instantiate(capabilityService, identifierService) {
             return function (model, id) {
                 var capabilities = capabilityService.getCapabilities(model);
                 id = id || identifierService.generate();
-                cacheService.put(id, model);
                 return new DomainObjectImpl(id, model, capabilities);
             };
         }

platform/core/test/actions/ActionCapabilitySpec.js

@@ -19,7 +19,8 @@
  * 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*/
+/*global define,Promise,describe,xdescribe,it,expect,beforeEach,waitsFor,
+ jasmine*/
 
 /**
  * ActionCapabilitySpec. Created by vwoeltje on 11/6/14.
@@ -28,8 +29,8 @@ define(
     ["../../src/actions/ActionCapability"],
     function (ActionCapability) {
         "use strict";
-
-        describe("The action capability", function () {
+        //TODO: Disabled for NEM beta
+        xdescribe("The action capability", function () {
             var mockQ,
                 mockAction,
                 mockActionService,

platform/core/test/capabilities/InstantiationCapabilitySpec.js

@@ -31,7 +31,6 @@ define(
                 mockIdentifierService,
                 mockInstantiate,
                 mockIdentifier,
-                mockNow,
                 mockDomainObject,
                 instantiation;
 
@@ -58,13 +57,9 @@ define(
                 mockIdentifierService.parse.andReturn(mockIdentifier);
                 mockIdentifierService.generate.andReturn("some-id");
 
-                mockNow = jasmine.createSpy();
-                mockNow.andReturn(1234321);
-
                 instantiation = new InstantiationCapability(
                     mockInjector,
                     mockIdentifierService,
-                    mockNow,
                     mockDomainObject
                 );
             });
@@ -86,10 +81,7 @@ define(
                 expect(instantiation.instantiate(testModel))
                     .toBe(mockDomainObject);
                 expect(mockInstantiate)
-                    .toHaveBeenCalledWith({
-                        someKey: "some value",
-                        modified: mockNow()
-                    }, jasmine.any(String));
+                    .toHaveBeenCalledWith(testModel, jasmine.any(String));
             });
 
         });

platform/core/test/capabilities/PersistenceCapabilitySpec.js

@@ -36,7 +36,6 @@ define(
                 mockDomainObject,
                 mockIdentifier,
                 mockNofificationService,
-                mockCacheService,
                 mockQ,
                 id = "object id",
                 model,
@@ -82,10 +81,6 @@ define(
                     "notificationService",
                     ["error"]
                 );
-                mockCacheService = jasmine.createSpyObj(
-                    "cacheService",
-                    [ "get", "put", "remove", "all" ]
-                );
 
                 mockDomainObject = {
                     getId: function () { return id; },
@@ -101,7 +96,6 @@ define(
                 mockIdentifierService.parse.andReturn(mockIdentifier);
                 mockIdentifier.getSpace.andReturn(SPACE);
                 persistence = new PersistenceCapability(
-                    mockCacheService,
                     mockPersistenceService,
                     mockIdentifierService,
                     mockNofificationService,
@@ -177,7 +171,6 @@ define(
                     expect(mockNofificationService.error).not.toHaveBeenCalled();
                 });
             });
-
             describe("unsuccessful persistence", function() {
                 var sadPromise = {
                         then: function(callback){

platform/core/test/models/CachingModelDecoratorSpec.js

@@ -22,11 +22,8 @@
 /*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
 
 define(
-    [
-        "../../src/models/CachingModelDecorator",
-        "../../src/models/ModelCacheService"
-    ],
-    function (CachingModelDecorator, ModelCacheService) {
+    ["../../src/models/CachingModelDecorator"],
+    function (CachingModelDecorator) {
         "use strict";
 
         describe("The caching model decorator", function () {
@@ -70,10 +67,7 @@ define(
                     b: { someOtherKey: "some other value" }
                 };
                 mockModelService.getModels.andReturn(asPromise(testModels));
-                decorator = new CachingModelDecorator(
-                    new ModelCacheService(),
-                    mockModelService
-                );
+                decorator = new CachingModelDecorator(mockModelService);
             });
 
             it("loads models from its wrapped model service", function () {
@@ -156,4 +150,4 @@ define(
 
         });
     }
-);
+);

platform/core/test/models/ModelCacheServiceSpec.js

@@ -1,69 +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.
- *****************************************************************************/
-/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
-
-define(['../../src/models/ModelCacheService'], function (ModelCacheService) {
-    'use strict';
-    describe("ModelCacheService", function () {
-        var testIds,
-            testModels,
-            cacheService;
-
-        beforeEach(function () {
-            testIds = [ 'a', 'b', 'c', 'd' ];
-            testModels = testIds.reduce(function (models, id) {
-                models[id] = { someKey: "some value for " + id };
-                return models;
-            }, {});
-            cacheService = new ModelCacheService();
-        });
-
-        describe("when populated with models", function () {
-            beforeEach(function () {
-                testIds.forEach(function (id) {
-                    cacheService.put(id, testModels[id]);
-                });
-            });
-
-            it("indicates that it has these models", function () {
-                testIds.forEach(function (id) {
-                    expect(cacheService.has(id)).toBe(true);
-                });
-            });
-
-            it("provides all of these models", function () {
-                expect(cacheService.all()).toEqual(testModels);
-            });
-
-            it("allows models to be retrieved", function () {
-                testIds.forEach(function (id) {
-                    expect(cacheService.get(id)).toEqual(testModels[id]);
-                });
-            });
-
-            it("allows models to be removed", function () {
-                cacheService.remove('a');
-                expect(cacheService.has('a')).toBe(false);
-            });
-        });
-    });
-});

platform/core/test/services/InstantiateSpec.js

@@ -32,7 +32,8 @@ define(
                 mockIdentifierService,
                 mockCapabilityConstructor,
                 mockCapabilityInstance,
-                mockCacheService,
+                mockCapabilities,
+                mockIdentifier,
                 idCounter,
                 testModel,
                 instantiate,
@@ -61,17 +62,11 @@ define(
                             "some-id-" + (idCounter += 1);
                 });
 
-                mockCacheService = jasmine.createSpyObj(
-                    'cacheService',
-                    [ 'get', 'put', 'remove', 'all' ]
-                );
-
                 testModel = { someKey: "some value" };
 
                 instantiate = new Instantiate(
                     mockCapabilityService,
-                    mockIdentifierService,
-                    mockCacheService
+                    mockIdentifierService
                 );
                 domainObject = instantiate(testModel);
             });
@@ -97,13 +92,6 @@ define(
                 expect(instantiate(testModel).getId())
                     .not.toEqual(domainObject.getId());
             });
-
-            it("caches the instantiated model", function () {
-                expect(mockCacheService.put).toHaveBeenCalledWith(
-                    domainObject.getId(),
-                    testModel
-                );
-            });
         });
 
     }

platform/entanglement/bundle.js

@@ -29,9 +29,7 @@ define([
     "./src/actions/SetPrimaryLocationAction",
     "./src/services/LocatingCreationDecorator",
     "./src/services/LocatingObjectDecorator",
-    "./src/policies/CopyPolicy",
     "./src/policies/CrossSpacePolicy",
-    "./src/policies/MovePolicy",
     "./src/capabilities/LocationCapability",
     "./src/services/MoveService",
     "./src/services/LinkService",
@@ -46,9 +44,7 @@ define([
     SetPrimaryLocationAction,
     LocatingCreationDecorator,
     LocatingObjectDecorator,
-    CopyPolicy,
     CrossSpacePolicy,
-    MovePolicy,
     LocationCapability,
     MoveService,
     LinkService,
@@ -144,14 +140,6 @@ define([
                 {
                     "category": "action",
                     "implementation": CrossSpacePolicy
-                },
-                {
-                    "category": "action",
-                    "implementation": CopyPolicy
-                },
-                {
-                    "category": "action",
-                    "implementation": MovePolicy
                 }
             ],
             "capabilities": [

platform/entanglement/src/policies/CopyPolicy.js

@@ -1,57 +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.
- *****************************************************************************/
-
-/*global define */
-define([], function () {
-    'use strict';
-
-    /**
-     * Disallow duplication when the object to be duplicated is not
-     * creatable.
-     * @constructor
-     * @implements {Policy}
-     * @memberof platform/entanglement
-     */
-    function CopyPolicy() {
-    }
-
-    function allowCreation(domainObject) {
-        var type = domainObject && domainObject.getCapability('type');
-        return !!(type && type.hasFeature('creation'));
-    }
-
-    function selectedObject(context) {
-        return context.selectedObject || context.domainObject;
-    }
-
-    CopyPolicy.prototype.allow = function (action, context) {
-        var key = action.getMetadata().key;
-
-        if (key === 'copy') {
-            return allowCreation(selectedObject(context));
-        }
-
-        return true;
-    };
-
-    return CopyPolicy;
-});

platform/entanglement/src/policies/MovePolicy.js

@@ -1,63 +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.
- *****************************************************************************/
-
-/*global define */
-define([], function () {
-    'use strict';
-
-    /**
-     * Disallow moves when either the parent or the child are not
-     * modifiable by users.
-     * @constructor
-     * @implements {Policy}
-     * @memberof platform/entanglement
-     */
-    function MovePolicy() {
-    }
-
-    function parentOf(domainObject) {
-        var context = domainObject.getCapability('context');
-        return context && context.getParent();
-    }
-
-    function allowMutation(domainObject) {
-        var type = domainObject && domainObject.getCapability('type');
-        return !!(type && type.hasFeature('creation'));
-    }
-
-    function selectedObject(context) {
-        return context.selectedObject || context.domainObject;
-    }
-
-    MovePolicy.prototype.allow = function (action, context) {
-        var key = action.getMetadata().key;
-
-        if (key === 'move') {
-            return allowMutation(selectedObject(context)) &&
-                allowMutation(parentOf(selectedObject(context)));
-        }
-
-        return true;
-    };
-
-    return MovePolicy;
-});

platform/entanglement/test/policies/CopyPolicySpec.js

@@ -1,94 +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.
- *****************************************************************************/
-
-/*global define,describe,beforeEach,it,jasmine,expect,spyOn */
-define([
-    '../../src/policies/CopyPolicy',
-    '../DomainObjectFactory'
-], function (CopyPolicy, domainObjectFactory) {
-    'use strict';
-
-    describe("CopyPolicy", function () {
-        var testMetadata,
-            testContext,
-            mockDomainObject,
-            mockType,
-            mockAction,
-            policy;
-
-        beforeEach(function () {
-            mockType =
-                jasmine.createSpyObj('type', ['hasFeature']);
-
-            testMetadata = {};
-
-            mockDomainObject = domainObjectFactory({
-                capabilities: { type: mockType }
-            });
-
-            mockType.hasFeature.andCallFake(function (feature) {
-                return feature === 'creation';
-            });
-
-            mockAction = jasmine.createSpyObj('action', ['getMetadata']);
-            mockAction.getMetadata.andReturn(testMetadata);
-
-            testContext = { domainObject: mockDomainObject };
-
-            policy = new CopyPolicy();
-        });
-
-        describe("for copy actions", function () {
-            beforeEach(function () {
-                testMetadata.key = 'copy';
-            });
-
-            describe("when an object is non-creatable", function () {
-                beforeEach(function () {
-                    mockType.hasFeature.andReturn(false);
-                });
-
-                it("disallows the action", function () {
-                    expect(policy.allow(mockAction, testContext)).toBe(false);
-                });
-            });
-
-            describe("when an object is creatable", function () {
-                it("allows the action", function () {
-                    expect(policy.allow(mockAction, testContext)).toBe(true);
-                });
-            });
-        });
-
-        describe("for other actions", function () {
-            beforeEach(function () {
-                testMetadata.key = 'foo';
-            });
-
-            it("simply allows the action", function () {
-                expect(policy.allow(mockAction, testContext)).toBe(true);
-                mockType.hasFeature.andReturn(false);
-                expect(policy.allow(mockAction, testContext)).toBe(true);
-            });
-        });
-    });
-});

platform/entanglement/test/policies/MovePolicySpec.js

@@ -1,127 +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.
- *****************************************************************************/
-
-/*global define,describe,beforeEach,it,jasmine,expect,spyOn */
-define([
-    '../../src/policies/MovePolicy',
-    '../DomainObjectFactory'
-], function (MovePolicy, domainObjectFactory) {
-    'use strict';
-
-    describe("MovePolicy", function () {
-        var testMetadata,
-            testContext,
-            mockDomainObject,
-            mockParent,
-            mockParentType,
-            mockType,
-            mockAction,
-            policy;
-
-        beforeEach(function () {
-            var mockContextCapability =
-                    jasmine.createSpyObj('context', ['getParent']);
-
-            mockType =
-                jasmine.createSpyObj('type', ['hasFeature']);
-            mockParentType =
-                jasmine.createSpyObj('parent-type', ['hasFeature']);
-
-
-            testMetadata = {};
-
-            mockDomainObject = domainObjectFactory({
-                capabilities: {
-                    context: mockContextCapability,
-                    type: mockType
-                }
-            });
-            mockParent = domainObjectFactory({
-                capabilities: {
-                    type: mockParentType
-                }
-            });
-
-            mockContextCapability.getParent.andReturn(mockParent);
-
-            mockType.hasFeature.andCallFake(function (feature) {
-                return feature === 'creation';
-            });
-            mockParentType.hasFeature.andCallFake(function (feature) {
-                return feature === 'creation';
-            });
-
-            mockAction = jasmine.createSpyObj('action', ['getMetadata']);
-            mockAction.getMetadata.andReturn(testMetadata);
-
-            testContext = { domainObject: mockDomainObject };
-
-            policy = new MovePolicy();
-        });
-
-        describe("for move actions", function () {
-            beforeEach(function () {
-                testMetadata.key = 'move';
-            });
-
-            describe("when an object is non-modifiable", function () {
-                beforeEach(function () {
-                    mockType.hasFeature.andReturn(false);
-                });
-
-                it("disallows the action", function () {
-                    expect(policy.allow(mockAction, testContext)).toBe(false);
-                });
-            });
-
-            describe("when a parent is non-modifiable", function () {
-                beforeEach(function () {
-                    mockParentType.hasFeature.andReturn(false);
-                });
-
-                it("disallows the action", function () {
-                    expect(policy.allow(mockAction, testContext)).toBe(false);
-                });
-            });
-
-            describe("when an object and its parent are modifiable", function () {
-                it("allows the action", function () {
-                    expect(policy.allow(mockAction, testContext)).toBe(true);
-                });
-            });
-        });
-
-        describe("for other actions", function () {
-            beforeEach(function () {
-                testMetadata.key = 'foo';
-            });
-
-            it("simply allows the action", function () {
-                expect(policy.allow(mockAction, testContext)).toBe(true);
-                mockType.hasFeature.andReturn(false);
-                expect(policy.allow(mockAction, testContext)).toBe(true);
-                mockParentType.hasFeature.andReturn(false);
-                expect(policy.allow(mockAction, testContext)).toBe(true);
-            });
-        });
-    });
-});

platform/features/clock/bundle.js

@@ -157,9 +157,7 @@ define([
                 {
                     "key": "clock",
                     "name": "Clock",
-                    "glyph": "\u0043",
-                    "description": "A UTC-based clock that supports a variety of display formats. Clocks can be added to Display Layouts.",
-                    "priority": 101,
+                    "glyph": "C",
                     "features": [
                         "creation"
                     ],
@@ -214,9 +212,7 @@ define([
                 {
                     "key": "timer",
                     "name": "Timer",
-                    "glyph": "\u00f5",
-                    "description": "A timer that counts up or down to a datetime. Timers can be started, stopped and reset whenever needed, and support a variety of display formats. Each Timer displays the same value to all users. Timers can be added to Display Layouts.",
-                    "priority": 100,
+                    "glyph": "õ",
                     "features": [
                         "creation"
                     ],

platform/features/conductor/res/templates/time-conductor.html

@@ -6,6 +6,6 @@
              ng-model='ngModel'
              field="'domain'"
              options="ngModel.options"
-             class="l-time-domain-selector"
+             style="position: absolute; right: 0px; bottom: 46px;"
              >
 </mct-control>

platform/features/layout/bundle.js

@@ -58,7 +58,7 @@ define([
                 {
                     "key": "layout",
                     "name": "Display Layout",
-                    "glyph": "\u004c",
+                    "glyph": "L",
                     "type": "layout",
                     "template": layoutTemplate,
                     "editable": true,
@@ -82,28 +82,28 @@ define([
                                 "items": [
                                     {
                                         "method": "add",
-                                        "glyph": "\u002b",
+                                        "glyph": "+",
                                         "control": "menu-button",
                                         "text": "Add",
                                         "options": [
                                             {
                                                 "name": "Box",
-                                                "glyph": "\u00e0",
+                                                "glyph": "à",
                                                 "key": "fixed.box"
                                             },
                                             {
                                                 "name": "Line",
-                                                "glyph": "\u00e2",
+                                                "glyph": "â",
                                                 "key": "fixed.line"
                                             },
                                             {
                                                 "name": "Text",
-                                                "glyph": "\u00e4",
+                                                "glyph": "ä",
                                                 "key": "fixed.text"
                                             },
                                             {
                                                 "name": "Image",
-                                                "glyph": "\u00e3",
+                                                "glyph": "ã",
                                                 "key": "fixed.image"
                                             }
                                         ]
@@ -119,22 +119,22 @@ define([
                                         "options": [
                                             {
                                                 "name": "Move to Top",
-                                                "glyph": "\u00eb",
+                                                "glyph": "^",
                                                 "key": "top"
                                             },
                                             {
                                                 "name": "Move Up",
-                                                "glyph": "\u005e",
+                                                "glyph": "^",
                                                 "key": "up"
                                             },
                                             {
                                                 "name": "Move Down",
-                                                "glyph": "\u0076",
+                                                "glyph": "v",
                                                 "key": "down"
                                             },
                                             {
                                                 "name": "Move to Bottom",
-                                                "glyph": "\u00ee",
+                                                "glyph": "v",
                                                 "key": "bottom"
                                             }
                                         ]
@@ -263,9 +263,8 @@ define([
                 {
                     "key": "layout",
                     "name": "Display Layout",
-                    "glyph": "\u004c",
-                    "description": "Assemble other objects and components together into a reusable screen layout. Working in a simple canvas workspace, simply drag in the objects you want, position and size them. Save your design and view or edit it at any time.",
-                    "priority": 900,
+                    "glyph": "L",
+                    "description": "A layout in which multiple telemetry panels may be displayed.",
                     "features": "creation",
                     "model": {
                         "composition": []
@@ -297,7 +296,6 @@ define([
                     "name": "Telemetry Panel",
                     "glyph": "t",
                     "description": "A panel for collecting telemetry elements.",
-                    "priority": 899,
                     "delegates": [
                         "telemetry"
                     ],

platform/features/pages/bundle.js

@@ -38,9 +38,8 @@ define([
                 {
                     "key": "example.page",
                     "name": "Web Page",
-                    "glyph": "\u00ea",
-                    "description": "Embed a web page or web-based image in a resizeable window component. Can be added to Display Layouts. Note that the URL being embedded must allow iframing.",
-                    "priority": 50,
+                    "glyph": "ê",
+                    "description": "A component to display a web page or image with a valid URL. Can be added to a Display Layout.",
                     "features": [
                         "creation"
                     ],

platform/features/plot/src/elements/PlotTelemetryFormatter.js

@@ -26,8 +26,6 @@ define(
     function () {
         'use strict';
 
-        var DIGITS = 3;
-
         /**
          * Wraps a `TelemetryFormatter` to provide formats for domain and
          * range values; provides a single place to track domain/range
@@ -65,10 +63,6 @@ define(
         };
 
         PlotTelemetryFormatter.prototype.formatRangeValue = function (value) {
-            if (typeof value === 'number') {
-                return value.toFixed(DIGITS);
-            }
-
             return this.telemetryFormatter
                 .formatRangeValue(value, this.rangeFormat);
         };

platform/features/plot/test/elements/PlotTelemetryFormatterSpec.js

@@ -55,17 +55,14 @@ define(
                         .toHaveBeenCalledWith(12321, domainFormat);
                 });
 
-                it("includes format in formatRangeValue calls for strings", function () {
+                it("includes format in formatRangeValue calls", function () {
                     mockFormatter.formatRangeValue.andReturn("formatted!");
-                    expect(formatter.formatRangeValue('foo'))
+                    expect(formatter.formatRangeValue(12321))
                         .toEqual("formatted!");
                     expect(mockFormatter.formatRangeValue)
-                        .toHaveBeenCalledWith('foo', rangeFormat);
+                        .toHaveBeenCalledWith(12321, rangeFormat);
                 });
 
-                it("formats numeric values with three fixed digits", function () {
-                    expect(formatter.formatRangeValue(10)).toEqual("10.000");
-                });
             });
 
         });

platform/features/table/bundle.js

@@ -23,16 +23,16 @@
 
 define([
     "./src/directives/MCTTable",
-    "./src/controllers/RealtimeTableController",
-    "./src/controllers/HistoricalTableController",
+    "./src/controllers/RTTelemetryTableController",
+    "./src/controllers/TelemetryTableController",
     "./src/controllers/TableOptionsController",
     '../../commonUI/regions/src/Region',
     '../../commonUI/browse/src/InspectorRegion',
     "legacyRegistry"
 ], function (
     MCTTable,
-    RealtimeTableController,
-    HistoricalTableController,
+    RTTelemetryTableController,
+    TelemetryTableController,
     TableOptionsController,
     Region,
     InspectorRegion,
@@ -62,9 +62,8 @@ define([
                 {
                     "key": "table",
                     "name": "Historical Telemetry Table",
-                    "glyph": "\ue604",
-                    "description": "A static table of all values over time for all included telemetry elements. Rows are timestamped data values for each telemetry element; columns are data fields. The number of rows is based on the range of your query. New incoming data must be manually re-queried for.",
-                    "priority": 861,
+                    "glyph": "\ue605",
+                    "description": "A table for displaying telemetry data",
                     "features": "creation",
                     "delegates": [
                         "telemetry"
@@ -85,9 +84,9 @@ define([
                 {
                     "key": "rttable",
                     "name": "Real-time Telemetry Table",
-                    "glyph": "\ue620",
-                    "description": "A scrolling table of latest values for all included telemetry elements. Rows are timestamped data values for each telemetry element; columns are data fields. New incoming data is automatically added to the view.",
-                    "priority": 860,
+                    "glyph": "\ue605",
+                    "description": "A table for displaying realtime telemetry" +
+                    " data",
                     "features": "creation",
                     "delegates": [
                         "telemetry"
@@ -109,13 +108,13 @@ define([
             ],
             "controllers": [
                 {
-                    "key": "HistoricalTableController",
-                    "implementation": HistoricalTableController,
+                    "key": "TelemetryTableController",
+                    "implementation": TelemetryTableController,
                     "depends": ["$scope", "telemetryHandler", "telemetryFormatter"]
                 },
                 {
-                    "key": "RealtimeTableController",
-                    "implementation": RealtimeTableController,
+                    "key": "RTTelemetryTableController",
+                    "implementation": RTTelemetryTableController,
                     "depends": ["$scope", "telemetryHandler", "telemetryFormatter"]
                 },
                 {
@@ -129,8 +128,8 @@ define([
                 {
                     "name": "Historical Table",
                     "key": "table",
-                    "glyph": "\ue604",
-                    "templateUrl": "templates/historical-table.html",
+                    "glyph": "\ue605",
+                    "templateUrl": "templates/table.html",
                     "needs": [
                         "telemetry"
                     ],
@@ -140,7 +139,7 @@ define([
                 {
                     "name": "Real-time Table",
                     "key": "rt-table",
-                    "glyph": "\ue620",
+                    "glyph": "\ue605",
                     "templateUrl": "templates/rt-table.html",
                     "needs": [
                         "telemetry"
@@ -161,12 +160,6 @@ define([
                     "key": "table-options-edit",
                     "templateUrl": "templates/table-options-edit.html"
                 }
-            ],
-            "stylesheets": [
-                {
-                    "stylesheetUrl": "css/table.css",
-                    "priority": "mandatory"
-                }
             ]
         }
     });

platform/features/table/res/sass/table.scss

@@ -1,50 +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.
- *****************************************************************************/
-.sizing-table {
-    min-width: 100%;
-    z-index: -1;
-    visibility: hidden;
-    position: absolute;
-
-    //Add some padding to allow for decorations such as limits indicator
-    td {
-        padding-right: 15px;
-        padding-left: 10px;
-        white-space: nowrap;
-    }
-}
-.mct-table {
-    table-layout: fixed;
-    th {
-        box-sizing: border-box;
-    }
-    tbody {
-        tr {
-            position: absolute;
-        }
-        td {
-            white-space: nowrap;
-            overflow: hidden;
-            box-sizing: border-box;
-        }
-    }
-}

platform/features/table/res/templates/mct-table.html

@@ -1,25 +1,22 @@
-<div class="l-view-section scrolling" style="overflow: auto;">
-    <table class="sizing-table">
-        <tbody>
-            <tr>
-                <td ng-repeat="header in displayHeaders">{{header}}</td>
-            </tr>
-            <tr><td ng-repeat="header in displayHeaders" >
-                {{sizingRow[header].text}}
-            </td></tr>
-        </tbody>
-    </table>
-    <table class="filterable mct-table"
-           ng-style="{
+<div class="l-view-section scrolling"
+     ng-style="overrideRowPositioning ?
+         {'overflow': 'auto'} :
+         {'overflow': 'scroll'}"
+     >
+    <table class="filterable"
+           ng-style="overrideRowPositioning && {
             height: totalHeight + 'px',
+            'table-layout': overrideRowPositioning ? 'fixed' : 'auto',
             'max-width': totalWidth
-            }">
+        }">
         <thead>
             <tr>
                 <th ng-repeat="header in displayHeaders"
-                    ng-style="{
+                    ng-style="overrideRowPositioning && {
                         width: columnWidths[$index] + 'px',
                         'max-width': columnWidths[$index] + 'px',
+                        overflow: 'none',
+                        'box-sizing': 'border-box'
                     }"
                     ng-class="[
                         enableSort ? 'sortable' : '',
@@ -32,9 +29,11 @@
             </tr>
             <tr ng-if="enableFilter" class="s-filters">
                 <th ng-repeat="header in displayHeaders"
-                    ng-style="{
+                    ng-style="overrideRowPositioning && {
                         width: columnWidths[$index] + 'px',
                         'max-width': columnWidths[$index] + 'px',
+                        overflow: 'none',
+                        'box-sizing': 'border-box'
                     }">
                     <input type="text"
                            ng-model="filters[header]"/>
@@ -42,15 +41,21 @@
             </tr>
         </thead>
 
-        <tbody>
+        <tbody ng-style="overrideRowPositioning ? '' : {
+            'opacity': '0.0'
+        }">
             <tr ng-repeat="visibleRow in visibleRows track by visibleRow.rowIndex"
-                ng-style="{
+                ng-style="overrideRowPositioning && {
+                    position: 'absolute',
                     top: visibleRow.offsetY + 'px',
                 }">
                 <td ng-repeat="header in displayHeaders"
-                    ng-style=" {
+                    ng-style="overrideRowPositioning && {
                         width: columnWidths[$index] + 'px',
+                        'white-space': 'nowrap',
                         'max-width': columnWidths[$index] + 'px',
+                        overflow: 'hidden',
+                        'box-sizing': 'border-box'
                     }"
                     class="{{visibleRow.contents[header].cssClass}}">
                     {{ visibleRow.contents[header].text }}

platform/features/table/res/templates/rt-table.html

@@ -1,4 +1,4 @@
-<div ng-controller="RealtimeTableController">
+<div ng-controller="RTTelemetryTableController">
     <mct-table
         headers="headers"
         rows="rows"

platform/features/table/res/templates/table.html

@@ -1,4 +1,4 @@
-<div ng-controller="HistoricalTableController">
+<div ng-controller="TelemetryTableController">
     <mct-table
         headers="headers"
         rows="rows"

platform/features/table/src/DomainColumn.js

@@ -0,0 +1,64 @@
+/*****************************************************************************
+ * 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,moment*/
+
+/**
+ * Module defining DomainColumn.
+ */
+define(
+    [],
+    function () {
+        "use strict";
+
+        /**
+         * A column which will report telemetry domain values
+         * (typically, timestamps.) Used by the ScrollingListController.
+         *
+         * @memberof platform/features/table
+         * @constructor
+         * @param domainMetadata an object with the machine- and human-
+         *        readable names for this domain (in `key` and `name`
+         *        fields, respectively.)
+         * @param {TelemetryFormatter} telemetryFormatter the telemetry
+         *        formatting service, for making values human-readable.
+         */
+        function DomainColumn(domainMetadata, telemetryFormatter) {
+            this.domainMetadata = domainMetadata;
+            this.telemetryFormatter = telemetryFormatter;
+        }
+
+        DomainColumn.prototype.getTitle = function () {
+            return this.domainMetadata.name;
+        };
+
+        DomainColumn.prototype.getValue = function (domainObject, datum) {
+            return {
+                text: this.telemetryFormatter.formatDomainValue(
+                    datum[this.domainMetadata.key],
+                    this.domainMetadata.format
+                )
+            };
+        };
+
+        return DomainColumn;
+    }
+);

platform/features/table/src/RangeColumn.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 define*/
+/*global define,Promise*/
 
 /**
  * Module defining DomainColumn. Created by vwoeltje on 11/18/14.
@@ -41,23 +41,24 @@ define(
          * @param {TelemetryFormatter} telemetryFormatter the telemetry
          *        formatting service, for making values human-readable.
          */
-        function RangeColumn(columnMetadata) {
-            this.title = columnMetadata.name || '';
-            this.key = columnMetadata.key;
+        function RangeColumn(rangeMetadata, telemetryFormatter) {
+            this.rangeMetadata = rangeMetadata;
+            this.telemetryFormatter = telemetryFormatter;
         }
 
         RangeColumn.prototype.getTitle = function () {
-            return this.title;
+            return this.rangeMetadata.name;
         };
 
         RangeColumn.prototype.getValue = function (domainObject, datum) {
-            var formatter = MCT.telemetry.Formatter(domainObject),
-                evaluator = MCT.telemetry.LimitEvaluator(domainObject),
-                alarm = evaluator.evaluate(datum, this.key);
+            var range = this.rangeMetadata.key,
+                limit = domainObject.getCapability('limit'),
+                value = isNaN(datum[range]) ? datum[range] : parseFloat(datum[range]),
+                alarm = limit && limit.evaluate(datum, range);
 
             return {
                 cssClass: alarm && alarm.cssClass,
-                text: formatter.format(datum, this.key)
+                text: typeof(value) === 'undefined' ? undefined : this.telemetryFormatter.formatRangeValue(value)
             };
         };
 

platform/features/table/src/TableConfiguration.js

@@ -19,14 +19,15 @@
  * this source code distribution or the Licensing information page available
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
-/*global define*/
+/*global define,moment*/
 
 define(
     [
+        './DomainColumn',
         './RangeColumn',
         './NameColumn'
     ],
-    function (RangeColumn, NameColumn) {
+    function (DomainColumn, RangeColumn, NameColumn) {
         "use strict";
 
         /**
@@ -35,9 +36,10 @@ define(
          * @param domainObject
          * @constructor
          */
-        function TableConfiguration(domainObject) {
+        function TableConfiguration(domainObject, telemetryFormatter) {
             this.domainObject = domainObject;
             this.columns = [];
+            this.telemetryFormatter = telemetryFormatter;
         }
 
         /**
@@ -45,7 +47,7 @@ define(
          * @param metadata Metadata describing the domains and ranges available
          * @returns {TableConfiguration} This object
          */
-        TableConfiguration.prototype.populateColumns = function (metadata) {
+        TableConfiguration.prototype.buildColumns = function (metadata) {
             var self = this;
 
             this.columns = [];
@@ -55,7 +57,7 @@ define(
                 metadata.forEach(function (metadatum) {
                     //Push domains first
                     (metadatum.domains || []).forEach(function (domainMetadata) {
-                        self.addColumn(new RangeColumn(domainMetadata,
+                        self.addColumn(new DomainColumn(domainMetadata,
                             self.telemetryFormatter));
                     });
                     (metadatum.ranges || []).forEach(function (rangeMetadata) {
@@ -73,7 +75,7 @@ define(
 
         /**
          * Add a column definition to this Table
-         * @param {RangeColumn | NameColumn} column
+         * @param {RangeColumn | DomainColumn | NameColumn} column
          * @param {Number} [index] Where the column should appear (will be
          * affected by column filtering)
          */
@@ -85,25 +87,26 @@ define(
             }
         };
 
+        /**
+         * @private
+         * @param column
+         * @returns {*|string}
+         */
+        TableConfiguration.prototype.getColumnTitle = function (column) {
+                return column.getTitle();
+        };
+
         /**
          * Get a simple list of column titles
          * @returns {Array} The titles of the columns
          */
         TableConfiguration.prototype.getHeaders = function () {
-            return this.columns.map(this.getColumnTitle, this);
+            var self = this;
+            return this.columns.map(function (column, i){
+                return self.getColumnTitle(column) || 'Column ' + (i + 1);
+            });
         };
 
-         /**
-         * @private
-         * @param column
-         * @param i column index
-         * @returns {*|string}
-         */
-        TableConfiguration.prototype.getColumnTitle = function (column, i) {
-            return column.getTitle() || 'Column ' + (i + 1);
-        };
-
-
         /**
          * Retrieve and format values for a given telemetry datum.
          * @param telemetryObject The object that the telemetry data is
@@ -113,20 +116,32 @@ define(
          * title, and the value is the formatted value from the provided datum.
          */
         TableConfiguration.prototype.getRowValues = function (telemetryObject, datum) {
-            return this.columns.reduce(function (row, column, i){
-                var columnTitle = this.getColumnTitle(column, i),
+            var self = this;
+            return this.columns.reduce(function (rowObject, column, i){
+                var columnTitle = self.getColumnTitle(column) || 'Column ' + (i + 1),
                     columnValue = column.getValue(telemetryObject, datum);
 
-                row[columnTitle] = columnValue;
-                return row;
-            }.bind(this), {});
+                if (columnValue !== undefined && columnValue.text === undefined){
+                    columnValue.text = '';
+                }
+                // Don't replace something with nothing.
+                // This occurs when there are multiple columns with the
+                // column title
+                if (rowObject[columnTitle] === undefined ||
+                    rowObject[columnTitle].text === undefined ||
+                    rowObject[columnTitle].text.length === 0) {
+                    rowObject[columnTitle] = columnValue;
+                }
+                return rowObject;
+            }, {});
         };
 
         /**
          * @private
          */
         TableConfiguration.prototype.defaultColumnConfiguration = function () {
-            return ((this.domainObject.getModel().configuration || {}).table || {}).columns || {};
+            return ((this.domainObject.getModel().configuration || {}).table ||
+                {}).columns || {};
         };
 
         /**
@@ -141,16 +156,6 @@ define(
             });
         };
 
-        function configChanged(config1, config2) {
-            var config1Keys = Object.keys(config1),
-                config2Keys = Object.keys(config2);
-
-            return (config1Keys.length !== config2Keys.length) ||
-                config1Keys.some(function(key){
-                    return config1[key] !== config2[key];
-                });
-        }
-
         /**
          * As part of the process of building the table definition, extract
          * configuration from column definitions.
@@ -158,7 +163,7 @@ define(
          * pairs where the key is the column title, and the value is a
          * boolean indicating whether the column should be shown.
          */
-        TableConfiguration.prototype.buildColumnConfiguration = function () {
+        TableConfiguration.prototype.getColumnConfiguration = function () {
             var configuration = {},
                 //Use existing persisted config, or default it
                 defaultConfig = this.defaultColumnConfiguration();
@@ -174,11 +179,6 @@ define(
                         defaultConfig[columnTitle];
             });
 
-            //Synchronize column configuration with model
-            if (configChanged(configuration, defaultConfig)) {
-                this.saveColumnConfiguration(configuration);
-            }
-
             return configuration;
         };