From 34ab6e4ccc59a25811b4760024bc1315ef4d5c10 Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Mon, 15 Jun 2015 16:11:39 -0400
Subject: [PATCH] Default to the read role in the permissions table

Fixes #126
---
 .../ui/repository-permissions-table.js          | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/static/js/directives/ui/repository-permissions-table.js b/static/js/directives/ui/repository-permissions-table.js
index c15c8e8f0..a4e98b361 100644
--- a/static/js/directives/ui/repository-permissions-table.js
+++ b/static/js/directives/ui/repository-permissions-table.js
@@ -28,11 +28,16 @@ angular.module('quay').directive('repositoryPermissionsTable', function () {
       'repository': '=repository',
       'isEnabled': '=isEnabled'
     },
-    controller: function($scope, $element, ApiService, Restangular, UtilService) {
+    controller: function($scope, $element, ApiService, Restangular, UtilService, RolesService) {
       $scope.permissionResources = {'team': {}, 'user': {}};
       $scope.permissionCache = {};
       $scope.permissions = {};
-      $scope.addPermissionInfo = {};
+
+      var readRole = RolesService.repoRoles[0].id;
+
+      $scope.addPermissionInfo = {
+        'role': readRole
+      };
 
       var loadAllPermissions = function() {
         if (!$scope.repository || !$scope.isEnabled) { return; }
@@ -160,13 +165,17 @@ angular.module('quay').directive('repositoryPermissionsTable', function () {
 
         var errorHandler = ApiService.errorDisplay('Cannot change permission', function() {
           opt_callback && opt_callback(false);
-          $scope.addPermissionInfo = {};
+          $scope.addPermissionInfo = {
+            'role': readRole
+          };
         });
 
         var endpoint = getPermissionEndpoint(entityName, kind);
         endpoint.customPUT(permission).then(function(result) {
           $scope.permissions[kind][entityName] = result;
-          $scope.addPermissionInfo = {};
+          $scope.addPermissionInfo = {
+            'role': readRole
+          };
           opt_callback && opt_callback(true)
         }, errorHandler);
       };