Uniform interface to add multiple LDAP connections

app/views/admin/groups/_form.html.haml

@@ -57,4 +57,5 @@
       = link_to  'Cancel', admin_group_path(@group), class: "btn btn-cancel"
 
 - if @group.persisted?
-  = render 'groups/ldap_group_links/form', group: @group
+  %h3.page-title LDAP Synchronizations
+  = render 'ldap_group_links/form', group: @group

app/views/admin/groups/show.html.haml

@@ -33,7 +33,7 @@
 
     .panel.panel-default
       .panel-heading
-        LDAP Connections
+        LDAP Synchronizations
       %ul.well-list
         - if @group.ldap_group_links.any?
           - @group.ldap_group_links.each do |ldap_group_link|

app/views/groups/edit.html.haml

@@ -44,7 +44,7 @@
           .form-actions
             = f.submit 'Save group', class: "btn btn-save"
 
-        = render 'groups/ldap_group_links/form', group: @group
+        = render 'ldap_group_links/form', group: @group
     .panel.panel-danger
       .panel-heading Remove group
       .panel-body

app/views/groups/ldap_group_links/_form.html.haml → app/views/ldap_group_links/_form.html.haml

 %section.ldap-group-links
-  %fieldset
-    %legend Current LDAP links
-    - if group.ldap_group_links.any?
-      %ul.bordered-list
-        - group.ldap_group_links.each do |ldap_group_link|
-          %li
-            %h4
-              = ldap_group_link.cn
-              %small.light== as #{ldap_group_link.human_access}
-              .pull-right
-                = link_to group_ldap_group_link_path(group, ldap_group_link), method: :delete, class: 'btn btn-danger btn-small' do
-                  = fa_icon('unlink', text: 'unlink')
-    - else
-      %p No links yet
-  = form_for [group, group.ldap_group_links.new] do |f|
+  = form_for [group, LdapGroupLink.new] do |f|
     %fieldset
-      %legend Add new LDAP link
+      %legend
       %div.form-holder
         .form-group.clearfix
           = f.label :ldap_cn, class: 'control-label' do
@@ -38,4 +24,6 @@
               You can manage permission levels for individual group members in the Members tab.
 
       .form-actions
-        = f.submit 'Save changes', class: "btn btn-primary"
+        = f.submit 'Add Synchronization', class: "btn btn-create"
+
+  = render partial: 'ldap_group_links/ldap_group_links', locals: { group: group }
\ No newline at end of file

app/views/ldap_group_links/_ldap_group_link.html.haml

+%li
+  = ldap_group_link.cn
+  %small.light== as #{ldap_group_link.human_access}
+  .pull-right
+    = link_to group_ldap_group_link_path(group, ldap_group_link), method: :delete, class: 'btn btn-danger btn-small' do
+      = fa_icon('unlink', text: 'unlink')

app/views/ldap_group_links/_ldap_group_links.html.haml

+.panel.panel-default
+  .panel-heading
+    %h4.panel-title
+      LDAP Synchronizations
+      == (#{group.ldap_group_links.count})
+
+  - if group.ldap_group_links.any?
+    %ul.well-list
+      = render collection: group.ldap_group_links, partial: 'ldap_group_links/ldap_group_link', locals: { group: group }
+  - else
+    %p No synchronizations yet