Browse Source

Improved ldap summary layout.

Martin Edenhofer 8 years ago
parent
commit
75bd61ce91

+ 16 - 11
app/assets/javascripts/app/views/integration/ldap_last_import.jst.eco

@@ -23,18 +23,23 @@
         <div class="flex">
           <progress max="<%= @job.result.sum %>" value="<%= @countDone %>"></progress>
         </div>
-        <ul>
-          <li><%- @T('Users') %>: <%= @countDone %>/<%= @job.result.sum %> (<%= @job.result.created %> <%- @T('created') %>, <%= @job.result.updated %> <%- @T('updated') %>, <%= @job.result.unchanged %> <%- @T('untouched') %>, <%= @job.result.skipped %> <%- @T('skipped') %>, <%= @job.result.failed %> <%- @T('failed') %>)
-          <% if !_.isEmpty(@job.result.roles): %>
-            <li><%- @T('%s groups to %s roles assignments', 'LDAP', 'Zammad') %>:
-              <ul>
-                <% for role, result of @job.result.roles: %>
-                  <li> <%- @T(role) %>: <%= result.created %> <%- @T('created') %>, <%= result.updated %> <%- @T('updated') %>, <%= result.unchanged %> <%- @T('untouched') %>, <%= result.failed %> <%- @T('failed') %>
-                <% end %>
-              </ul>
-          <% end %>
-        </ul>
       <% end %>
     <% end %>
+    <% if !_.isEmpty(@job.result): %>
+      <ul>
+        <li><%- @T('%s user to %s user', 'LDAP', 'Zammad') %> (<%= @countDone %>/<%= @job.result.sum %>):
+          <ul>
+            <li><%- @T('Users') %>: <%= @job.result.created %> <%- @T('created') %>, <%= @job.result.updated %> <%- @T('updated') %>, <%= @job.result.unchanged %> <%- @T('untouched') %>, <%= @job.result.skipped %> <%- @T('skipped') %>, <%= @job.result.failed %> <%- @T('failed') %>
+          </ul>
+        <% if !_.isEmpty(@job.result.roles): %>
+          <li><%- @T('%s groups to %s roles assignments', 'LDAP', 'Zammad') %>:
+            <ul>
+              <% for role, result of @job.result.roles: %>
+                <li> <%- @T(role) %>: <%= result.created %> <%- @T('created') %>, <%= result.updated %> <%- @T('updated') %>, <%= result.unchanged %> <%- @T('untouched') %>, <%= result.failed %> <%- @T('failed') %>
+              <% end %>
+            </ul>
+        <% end %>
+      </ul>
+    <% end %>
   <% end %>
 </div>

+ 6 - 1
app/assets/javascripts/app/views/integration/ldap_summary.jst.eco

@@ -1,5 +1,9 @@
 <ul>
-  <li><%- @T('Users') %>: <%= @countDone %> (<%= @job.result.created %> <%- @T('created') %>, <%= @job.result.updated %> <%- @T('updated') %>, <%= @job.result.unchanged %> <%- @T('untouched') %>, <%= @job.result.skipped %> <%- @T('skipped') %>, <%= @job.result.failed %> <%- @T('failed') %>)
+  <li><%- @T('%s user to %s user', 'LDAP', 'Zammad') %> (<%= @countDone %>):
+    <ul>
+      <li><%- @T('Users') %>: <%= @job.result.created %> <%- @T('created') %>, <%= @job.result.updated %> <%- @T('updated') %>, <%= @job.result.unchanged %> <%- @T('untouched') %>, <%= @job.result.skipped %> <%- @T('skipped') %>, <%= @job.result.failed %> <%- @T('failed') %>
+    </ul>
+  </li>
   <% if !_.isEmpty(@job.result.roles): %>
     <li><%- @T('%s groups to %s roles assignments', 'LDAP', 'Zammad') %>:
       <ul>
@@ -7,5 +11,6 @@
           <li><%- @T(role) %>: <%= result.created %> <%- @T('created') %>, <%= result.updated %> <%- @T('updated') %>, <%= result.unchanged %> <%- @T('untouched') %>, <%= result.failed %> <%- @T('failed') %>
         <% end %>
       </ul>
+    </li>
   <% end %>
 </ul>