Unite and move facet pre_ops to related modules

Facet pre_ops defined in ./facet module were moved to modules where facet are actually defined. Moved pre_ops were united with the ones defined for the facets in these modules.

The move simplifies module dependencies - there is no reason to have general facet module dependent on specialized facet modules.

Pre_ops uniting makes the code simpler.

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

1 files changed, 8 insertions, 2 deletions

diff --git a/install/ui/src/freeipa/details.js b/install/ui/src/freeipa/details.js
index 6811a1dfb..149a04e43 100644
--- a/install/ui/src/freeipa/details.js
+++ b/install/ui/src/freeipa/details.js
@@ -30,10 +30,11 @@ define([
         './jquery',
         './phases',
         './reg',
+        './spec_util',
         './text',
         './facet',
         './add'],
-    function(lang, builder, IPA, $, phases, reg, text, mod_facet) {
+    function(lang, builder, IPA, $, phases, reg, su, text) {
 
 var exp = {};
 
@@ -243,7 +244,13 @@ exp.facet_policies = IPA.facet_policies = function(spec) {
 
 exp.details_facet_pre_op = function(spec, context) {
 
+    var entity = context.entity;
+    su.context_entity(spec, context);
+
     spec.name = spec.name || 'details';
+    spec.title = spec.title || entity.metadata.label_singular;
+    spec.label = spec.label || entity.metadata.label_singular;
+    spec.tab_label = spec.tab_label || '@i18n:facets.details';
 
     spec.actions = spec.actions || [];
     spec.actions.unshift(
@@ -1241,7 +1248,6 @@ exp.register = function() {
         type: 'details',
         factory: IPA.details_facet,
         pre_ops: [
-            mod_facet.facet_preops.details,
             exp.details_facet_pre_op
         ]
     });