From 9405e1a9db11294a11efa24a7a3c36ea76a42f31 Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Fri, 4 Nov 2011 20:43:39 -0500
Subject: Added paging on search facet.

The search facet has been modified to support paging on most entities
using the --pkey-only option to get the primary keys and a batch command
to get the complete records.

Paging on DNS records is not supported because a record may appear as
multiple rows. The following entities do not have --pkey-only option:
Automount Key, Self-Service Permissions, Delegation.

The search and association facet have been refactored to reuse the
common code from the table facet base class.

Ticket #981
---
 install/ui/dns.js | 57 +++++++++++++++++++++++++++++++++----------------------
 1 file changed, 34 insertions(+), 23 deletions(-)

(limited to 'install/ui/dns.js')

diff --git a/install/ui/dns.js b/install/ui/dns.js
index 2cf5cd4a..933366c0 100644
--- a/install/ui/dns.js
+++ b/install/ui/dns.js
@@ -88,12 +88,13 @@ IPA.dns.zone_entity = function(spec) {
             }]
         }).
         nested_search_facet({
+            factory: IPA.dns.record_search_facet,
             facet_group: 'dnsrecord',
             nested_entity : 'dnsrecord',
             name: 'records',
+            pagination: false,
             title: IPA.metadata.objects.dnszone.label_singular,
             label: IPA.metadata.objects.dnsrecord.label,
-            load: IPA.dns_record_search_load,
             get_values: IPA.dnsrecord_get_delete_values,
             columns: [
                 {
@@ -407,31 +408,41 @@ IPA.dnszone_adder_dialog = function(spec) {
     return that;
 };
 
-IPA.dns_record_search_load = function (result) {
-    this.table.empty();
-    var normalized_record;
-    var dns_record_types = IPA.dns_record_types();
-    for (var i = 0; i<result.length; i++) {
-        var record = result[i];
-        for (var j =0; j < dns_record_types.length; j += 1){
-            var record_type = dns_record_types[j].value;
-            if (record[record_type]){
-                var record_of_type = record[record_type];
-                for (var k =0;
-                     k < record_of_type.length;
-                     k+=1)
-                {
-                    normalized_record = {
-                        idnsname:record.idnsname,
-                        type:record_type,
-                        data:record_of_type[k]
-                    };
-                    this.table.add_record(normalized_record);
+IPA.dns.record_search_facet = function(spec) {
+
+    var that = IPA.nested_search_facet(spec);
+
+    var init = function() {
+
+        that.table.load = function(result) {
+
+            that.table.empty();
+
+            var types = IPA.dns_record_types();
+
+            for (var i=0; i<result.length; i++) {
+                var record = result[i];
+
+                for (var j=0; j<types.length; j++) {
+                    var type = types[j].value;
+                    if (!record[type]) continue;
 
+                    var data = record[type];
+                    for (var k=0; k<data.length; k++) {
+                        that.table.add_record({
+                            idnsname: record.idnsname,
+                            type: type,
+                            data: data[k]
+                        });
+                    }
                 }
             }
-        }
-    }
+        };
+    };
+
+    init();
+
+    return that;
 };
 
 IPA.dns.record_entity = function(spec) {
-- 
cgit