Replace ./facets with reg.facet

https://fedorahosted.org/freeipa/ticket/3235

3 files changed, 27 insertions, 34 deletions

diff --git a/install/ui/src/freeipa/entity.js b/install/ui/src/freeipa/entity.js
index f132f325c..427d300e3 100644
--- a/install/ui/src/freeipa/entity.js
+++ b/install/ui/src/freeipa/entity.js
@@ -30,10 +30,9 @@ define([
         './jquery',
         './reg',
         './text',
-        './facets',
         './facet'],
     function(lang, metadata_provider, Singleton_registry, builder,
-             IPA, $, reg, text, facet_reg) {
+             IPA, $, reg, text) {
 
 var exp = {};
 
@@ -168,7 +167,7 @@ exp.entity = IPA.entity = function(spec) {
 
         //build all facets on the first time
         if(!that.facets_created) {
-            facets = facet_reg.builder.build(that.facet_specs, { entity: that });
+            facets = builder.build('facet', that.facet_specs, { entity: that });
             for (i=0; i<facets.length; i++) {
                 var facet = facets[i];
                 that.add_facet(facet);
diff --git a/install/ui/src/freeipa/facet.js b/install/ui/src/freeipa/facet.js
index ab897bff3..f9510e77a 100644
--- a/install/ui/src/freeipa/facet.js
+++ b/install/ui/src/freeipa/facet.js
@@ -29,8 +29,8 @@ define([
         'dojo/Stateful',
         'dojo/Evented',
         './_base/metadata_provider',
+        './_base/Singleton_registry',
         './builder',
-        './facets',
         './ipa',
         './jquery',
         './navigation',
@@ -42,7 +42,7 @@ define([
         './field',
         './widget'
        ], function(declare, lang, construct, on, Stateful, Evented, metadata_provider,
-                   builder, facets, IPA, $, navigation, phases, reg, su, text) {
+                   Singleton_registry, builder, IPA, $, navigation, phases, reg, su, text) {
 
 /**
  * Facet represents the content of currently displayed page.
@@ -1579,9 +1579,10 @@ exp.facet_preops = {
     }
 };
 
-phases.on('registration', function() {
+exp.register_facets = function() {
 
-    facets.register({
+    var f = reg.facet;
+    f.register({
         type: 'search',
         factory: IPA.search_facet,
         pre_ops: [
@@ -1589,7 +1590,7 @@ phases.on('registration', function() {
         ]
     });
 
-    facets.register({
+    f.register({
         type: 'nested_search',
         factory: IPA.nested_search_facet,
         pre_ops: [
@@ -1597,7 +1598,7 @@ phases.on('registration', function() {
         ]
     });
 
-    facets.register({
+    f.register({
         type: 'details',
         factory: IPA.details_facet,
         pre_ops: [
@@ -1605,7 +1606,7 @@ phases.on('registration', function() {
         ]
     });
 
-    facets.register({
+    f.register({
         type: 'association',
         factory: IPA.association_facet,
         pre_ops: [
@@ -1613,14 +1614,14 @@ phases.on('registration', function() {
         ]
     });
 
-    facets.register({
+    f.register({
         type: 'attribute',
         factory: IPA.attribute_facet,
         pre_ops: [
             exp.facet_preops.attribute
         ]
     });
-});
+};
 
 exp.action = IPA.action = function(spec) {
 
@@ -2421,6 +2422,10 @@ var FacetState = exp.FacetState = declare([Stateful, Evented], {
     }
 });
 
+// Facet builder and registry
+var registry = new Singleton_registry();
+reg.set('facet', registry);
+builder.set('facet', registry.builder);
 
 // Action builder and registry
 exp.action_builder = builder.get('action');
@@ -2441,6 +2446,7 @@ exp.register = function() {
 };
 
 phases.on('registration', exp.register);
+phases.on('registration', exp.register_facets);
 
 return exp;
 });
diff --git a/install/ui/src/freeipa/navigation/Router.js b/install/ui/src/freeipa/navigation/Router.js
index ae29ee177..09aed16aa 100644
--- a/install/ui/src/freeipa/navigation/Router.js
+++ b/install/ui/src/freeipa/navigation/Router.js
@@ -24,11 +24,10 @@ define(['dojo/_base/declare',
         'dojo/Evented',
         'dojo/io-query',
         'dojo/router',
-        '../entities',
-        '../facets',
-        '../ipa' //TODO: remove dependancy
+        '../ipa',
+        '../reg'
        ],
-       function(declare, lang, array, Evented, ioquery, router, entities, facets, IPA) {
+       function(declare, lang, array, Evented, ioquery, router, IPA, reg) {
 
     /**
     * Class navigation
@@ -132,8 +131,7 @@ define(['dojo/_base/declare',
             args.pkeys = pkeys;
 
             // set new facet state
-            //var entity = entities.get(entity_name);
-            var entity = IPA.get_entity(entity_name); // TODO: replace with prev line
+            var entity = reg.entity.get(entity_name);
             var facet = entity.get_facet(facet_name);
             facet.reset_state(args);
 
@@ -151,16 +149,8 @@ define(['dojo/_base/declare',
             var facet_name = event.params.page;
             var args = ioquery.queryToObject(event.params.args || '');
 
-//             // Find menu item
-//             var items = this.menu.items.query({ page: facet_name });
-//
-//             // Select menu item
-//             if (items.total > 0) {
-//                 this.menu.select(items[items.total-1]);
-//             }
-
             // set new facet state
-            var facet = facets.get(facet_name);
+            var facet = reg.facet.get(facet_name);
             facet.reset_state(args);
 
             this.show_facet(facet);
@@ -173,8 +163,7 @@ define(['dojo/_base/declare',
          */
         navigate_to_entity_facet: function(entity_name, facet_name, pkeys, args) {
 
-            //var entity = entities.get(entity_name);
-            var entity = IPA.get_entity(entity_name); // TODO: replace with prev line
+            var entity = reg.entity.get(entity_name);
             var facet = entity.get_facet(facet_name);
 
             if (!facet) return false; // TODO: maybe replace with exception
@@ -197,11 +186,10 @@ define(['dojo/_base/declare',
          */
         navigate_to_facet: function(facet_name, args) {
 
-            // TODO: uncoment when `facets` are implemented
-//             var facet = facets.get(facet_name);
-//             if (!args) args = facet.get_args();
-//             var hash = this._create_facet_hash(facet, { args: args });
-//             return this.navigate_to_hash(hash, facet);
+            var facet = reg.facet.get(facet_name);
+            if (!args) args = facet.get_state();
+            var hash = this._create_facet_hash(facet, args);
+            return this.navigate_to_hash(hash, facet);
         },
 
         /**