From ea24d72f01886f1f6deb767134b1dea0924455e1 Mon Sep 17 00:00:00 2001
From: Kyle Spearrin <kyle.spearrin@gmail.com>
Date: Wed, 10 May 2017 12:17:26 -0400
Subject: [PATCH] group accessall and readonly

---
 .../organizationCollectionsAddController.js   |  5 +-
 .../organizationCollectionsEditController.js  |  5 +-
 .../organizationGroupsAddController.js        | 11 ++-
 .../organizationGroupsEditController.js       | 19 +++--
 .../views/organizationCollectionsAdd.html     |  5 +-
 .../views/organizationCollectionsEdit.html    |  5 +-
 .../views/organizationGroupsAdd.html          | 80 +++++++++++--------
 .../views/organizationGroupsEdit.html         | 80 +++++++++++--------
 8 files changed, 130 insertions(+), 80 deletions(-)

diff --git a/src/app/organization/organizationCollectionsAddController.js b/src/app/organization/organizationCollectionsAddController.js
index 6ac9e00ad1..d220044ec3 100644
--- a/src/app/organization/organizationCollectionsAddController.js
+++ b/src/app/organization/organizationCollectionsAddController.js
@@ -32,7 +32,8 @@
             for (var i = 0; i < groups.Data.length; i++) {
                 groupsArr.push({
                     id: groups.Data[i].Id,
-                    name: groups.Data[i].Name
+                    name: groups.Data[i].Name,
+                    accessAll: groups.Data[i].AccessAll
                 });
             }
 
@@ -61,7 +62,7 @@
         };
 
         $scope.groupSelected = function (group) {
-            return group.id in $scope.selectedGroups;
+            return group.id in $scope.selectedGroups || group.accessAll;
         };
 
         $scope.allSelected = function () {
diff --git a/src/app/organization/organizationCollectionsEditController.js b/src/app/organization/organizationCollectionsEditController.js
index af01c3a3fb..064308110b 100644
--- a/src/app/organization/organizationCollectionsEditController.js
+++ b/src/app/organization/organizationCollectionsEditController.js
@@ -45,7 +45,8 @@
             for (var i = 0; i < groups.Data.length; i++) {
                 groupsArr.push({
                     id: groups.Data[i].Id,
-                    name: groups.Data[i].Name
+                    name: groups.Data[i].Name,
+                    accessAll: groups.Data[i].AccessAll
                 });
             }
 
@@ -74,7 +75,7 @@
         };
 
         $scope.groupSelected = function (group) {
-            return group.id in $scope.selectedGroups;
+            return group.id in $scope.selectedGroups || group.accessAll;
         };
 
         $scope.allSelected = function () {
diff --git a/src/app/organization/organizationGroupsAddController.js b/src/app/organization/organizationGroupsAddController.js
index ef78c369b3..4f20c126d6 100644
--- a/src/app/organization/organizationGroupsAddController.js
+++ b/src/app/organization/organizationGroupsAddController.js
@@ -46,12 +46,15 @@
         $scope.submit = function (model) {
             var group = {
                 name: model.name,
-                collectionIds: []
+                accessAll: !!model.accessAll
             };
 
-            for (var id in $scope.selectedCollections) {
-                if ($scope.selectedCollections.hasOwnProperty(id)) {
-                    group.collectionIds.push(id);
+            if (!group.accessAll) {
+                group.collectionIds = [];
+                for (var id in $scope.selectedCollections) {
+                    if ($scope.selectedCollections.hasOwnProperty(id)) {
+                        group.collectionIds.push(id);
+                    }
                 }
             }
 
diff --git a/src/app/organization/organizationGroupsEditController.js b/src/app/organization/organizationGroupsEditController.js
index 2a0b33862c..41e77d0f70 100644
--- a/src/app/organization/organizationGroupsEditController.js
+++ b/src/app/organization/organizationGroupsEditController.js
@@ -13,7 +13,8 @@
         }).then(function (group) {
             $scope.group = {
                 id: id,
-                name: group.Name
+                name: group.Name,
+                accessAll: group.AccessAll
             };
 
             var collections = {};
@@ -59,11 +60,17 @@
         };
 
         $scope.submit = function () {
-            var group = $scope.group;
-            group.collectionIds = [];
-            for (var id in $scope.selectedCollections) {
-                if ($scope.selectedCollections.hasOwnProperty(id)) {
-                    group.collectionIds.push(id);
+            var group = {
+                name: $scope.group.name,
+                accessAll: !!$scope.group.accessAll
+            };
+
+            if (!group.accessAll) {
+                group.collectionIds = [];
+                for (var id in $scope.selectedCollections) {
+                    if ($scope.selectedCollections.hasOwnProperty(id)) {
+                        group.collectionIds.push(id);
+                    }
                 }
             }
 
diff --git a/src/app/organization/views/organizationCollectionsAdd.html b/src/app/organization/views/organizationCollectionsAdd.html
index 005a030b54..d7d774b236 100644
--- a/src/app/organization/views/organizationCollectionsAdd.html
+++ b/src/app/organization/views/organizationCollectionsAdd.html
@@ -51,10 +51,13 @@
                                        name="selectedGroups[]"
                                        value="{{group.id}}"
                                        ng-checked="groupSelected(group)"
-                                       ng-click="toggleGroupSelection(group.id)">
+                                       ng-click="toggleGroupSelection(group.id)"
+                                       ng-disabled="group.accessAll">
                             </td>
                             <td valign="middle">
                                 {{group.name}}
+                                <i class="fa fa-unlock text-muted" ng-show="group.accessAll"
+                                   title="This group can access all items"></i>
                             </td>
                         </tr>
                     </tbody>
diff --git a/src/app/organization/views/organizationCollectionsEdit.html b/src/app/organization/views/organizationCollectionsEdit.html
index 7fdea03d3e..8f8179727d 100644
--- a/src/app/organization/views/organizationCollectionsEdit.html
+++ b/src/app/organization/views/organizationCollectionsEdit.html
@@ -52,10 +52,13 @@
                                        name="selectedGroups[]"
                                        value="{{group.id}}"
                                        ng-checked="groupSelected(group)"
-                                       ng-click="toggleGroupSelection(group.id)">
+                                       ng-click="toggleGroupSelection(group.id)"
+                                       ng-disabled="group.accessAll">
                             </td>
                             <td valign="middle">
                                 {{group.name}}
+                                <i class="fa fa-unlock text-muted" ng-show="group.accessAll"
+                                   title="This group can access all items"></i>
                             </td>
                         </tr>
                     </tbody>
diff --git a/src/app/organization/views/organizationGroupsAdd.html b/src/app/organization/views/organizationGroupsAdd.html
index 4f50aa8587..cf0d406f61 100644
--- a/src/app/organization/views/organizationGroupsAdd.html
+++ b/src/app/organization/views/organizationGroupsAdd.html
@@ -21,40 +21,56 @@
             <label for="email">Name</label>
             <input type="text" id="name" name="Name" ng-model="model.name" class="form-control" required api-field />
         </div>
-        <h4>Collections</h4>
-        <div ng-show="loading && !collections.length">
-            Loading collections...
+        <h4>Access</h4>
+        <div class="radio">
+            <label>
+                <input type="radio" ng-model="model.accessAll" name="AccessAll"
+                       ng-value="true" ng-checked="model.accessAll">
+                This group can access and modify <u>all items</u>.
+            </label>
         </div>
-        <div ng-show="!loading && !collections.length">
-            <p>No collections for your organization.</p>
+        <div class="radio">
+            <label>
+                <input type="radio" ng-model="model.accessAll" name="AccessAll"
+                       ng-value="false" ng-checked="!model.accessAll">
+                This group can access only the selected collections.
+            </label>
         </div>
-        <div class="table-responsive" ng-show="collections.length" style="margin: 0;">
-            <table class="table table-striped table-hover" style="margin: 0;">
-                <thead>
-                    <tr>
-                        <th style="width: 40px;">
-                            <input type="checkbox"
-                                   ng-checked="allSelected()"
-                                   ng-click="toggleCollectionSelectionAll($event)">
-                        </th>
-                        <th>Name</th>
-                    </tr>
-                </thead>
-                <tbody>
-                    <tr ng-repeat="collection in collections | orderBy: ['name']">
-                        <td valign="middle">
-                            <input type="checkbox"
-                                   name="selectedCollections[]"
-                                   value="{{collection.id}}"
-                                   ng-checked="collectionSelected(collection)"
-                                   ng-click="toggleCollectionSelection(collection.id)">
-                        </td>
-                        <td valign="middle">
-                            {{collection.name}}
-                        </td>
-                    </tr>
-                </tbody>
-            </table>
+        <div ng-show="!model.accessAll">
+            <div ng-show="loading && !collections.length">
+                Loading collections...
+            </div>
+            <div ng-show="!loading && !collections.length">
+                <p>No collections for your organization.</p>
+            </div>
+            <div class="table-responsive" ng-show="collections.length" style="margin: 0;">
+                <table class="table table-striped table-hover" style="margin: 0;">
+                    <thead>
+                        <tr>
+                            <th style="width: 40px;">
+                                <input type="checkbox"
+                                       ng-checked="allSelected()"
+                                       ng-click="toggleCollectionSelectionAll($event)">
+                            </th>
+                            <th>Name</th>
+                        </tr>
+                    </thead>
+                    <tbody>
+                        <tr ng-repeat="collection in collections | orderBy: ['name']">
+                            <td valign="middle">
+                                <input type="checkbox"
+                                       name="selectedCollections[]"
+                                       value="{{collection.id}}"
+                                       ng-checked="collectionSelected(collection)"
+                                       ng-click="toggleCollectionSelection(collection.id)">
+                            </td>
+                            <td valign="middle">
+                                {{collection.name}}
+                            </td>
+                        </tr>
+                    </tbody>
+                </table>
+            </div>
         </div>
     </div>
     <div class="modal-footer">
diff --git a/src/app/organization/views/organizationGroupsEdit.html b/src/app/organization/views/organizationGroupsEdit.html
index 64c7880c91..8ec71103df 100644
--- a/src/app/organization/views/organizationGroupsEdit.html
+++ b/src/app/organization/views/organizationGroupsEdit.html
@@ -21,40 +21,56 @@
             <label for="email">Name</label>
             <input type="text" id="name" name="Name" ng-model="group.name" class="form-control" required api-field />
         </div>
-        <h4>Collections</h4>
-        <div ng-show="loading && !collections.length">
-            Loading collections...
+        <h4>Access</h4>
+        <div class="radio">
+            <label>
+                <input type="radio" ng-model="group.accessAll" name="AccessAll"
+                       ng-value="true" ng-checked="group.accessAll">
+                This group can access and modify <u>all items</u>.
+            </label>
         </div>
-        <div ng-show="!loading && !collections.length">
-            <p>No collections for your organization.</p>
+        <div class="radio">
+            <label>
+                <input type="radio" ng-model="group.accessAll" name="AccessAll"
+                       ng-value="false" ng-checked="!group.accessAll">
+                This group can access only the selected collections.
+            </label>
         </div>
-        <div class="table-responsive" ng-show="collections.length" style="margin: 0;">
-            <table class="table table-striped table-hover" style="margin: 0;">
-                <thead>
-                    <tr>
-                        <th style="width: 40px;">
-                            <input type="checkbox"
-                                   ng-checked="allSelected()"
-                                   ng-click="toggleCollectionSelectionAll($event)">
-                        </th>
-                        <th>Name</th>
-                    </tr>
-                </thead>
-                <tbody>
-                    <tr ng-repeat="collection in collections | orderBy: ['name']">
-                        <td valign="middle">
-                            <input type="checkbox"
-                                   name="selectedCollections[]"
-                                   value="{{collection.id}}"
-                                   ng-checked="collectionSelected(collection)"
-                                   ng-click="toggleCollectionSelection(collection.id)">
-                        </td>
-                        <td valign="middle">
-                            {{collection.name}}
-                        </td>
-                    </tr>
-                </tbody>
-            </table>
+        <div ng-show="!group.accessAll">
+            <div ng-show="loading && !collections.length">
+                Loading collections...
+            </div>
+            <div ng-show="!loading && !collections.length">
+                <p>No collections for your organization.</p>
+            </div>
+            <div class="table-responsive" ng-show="collections.length" style="margin: 0;">
+                <table class="table table-striped table-hover" style="margin: 0;">
+                    <thead>
+                        <tr>
+                            <th style="width: 40px;">
+                                <input type="checkbox"
+                                       ng-checked="allSelected()"
+                                       ng-click="toggleCollectionSelectionAll($event)">
+                            </th>
+                            <th>Name</th>
+                        </tr>
+                    </thead>
+                    <tbody>
+                        <tr ng-repeat="collection in collections | orderBy: ['name']">
+                            <td valign="middle">
+                                <input type="checkbox"
+                                       name="selectedCollections[]"
+                                       value="{{collection.id}}"
+                                       ng-checked="collectionSelected(collection)"
+                                       ng-click="toggleCollectionSelection(collection.id)">
+                            </td>
+                            <td valign="middle">
+                                {{collection.name}}
+                            </td>
+                        </tr>
+                    </tbody>
+                </table>
+            </div>
         </div>
     </div>
     <div class="modal-footer">