From 64481effbcd60b315433ec771d1bc82c48e9e910 Mon Sep 17 00:00:00 2001
From: Juanfran <juanfran.alcantara@kaleidos.net>
Date: Tue, 10 Mar 2015 09:11:01 +0100
Subject: [PATCH] prevent edit external user role name

---
 app/partials/admin/admin-roles.jade | 37 ++++++++++++++++-------------
 1 file changed, 21 insertions(+), 16 deletions(-)

diff --git a/app/partials/admin/admin-roles.jade b/app/partials/admin/admin-roles.jade
index 94982f98..2899d960 100644
--- a/app/partials/admin/admin-roles.jade
+++ b/app/partials/admin/admin-roles.jade
@@ -13,23 +13,28 @@ div.wrapper.roles(ng-controller="RolesController as ctrl",
                     span Delete
 
 
-        div(tg-edit-role)
-            .edit-role
-                input(type="text", value="{{ role.name }}")
-                a.save.icon.icon-floppy(href="", title="Save")
+        div(ng-if="!role.external_user")
+            div(tg-edit-role)
+                .edit-role
+                    input(type="text", value="{{ role.name }}")
+                    a.save.icon.icon-floppy(href="", title="Save")
 
+                p.total
+                    span.role-name(title="{{ role.members_count }} members with this role") {{ role.name }}
+                    a.edit-value.icon.icon-edit
+
+            div.any-computable-role(ng-hide="anyComputableRole") Be careful, no role in your project will be able to estimate the point value for user stories
+
+            div.general-category
+                | When enabled, members assigned to this role will be able to estimate the point value for user stories
+                div.check
+                    input(type="checkbox", ng-model="role.computable", ng-change="ctrl.setComputable()")
+                    div
+                    span.check-text.check-yes Yes
+                    span.check-text.check-no No
+
+        div(ng-if="role.external_user")
             p.total
-                span.role-name(title="{{ role.members_count }} members with this role") {{ role.name }}
-                a.edit-value.icon.icon-edit
-
-        div.any-computable-role(ng-hide="anyComputableRole") Be careful, no role in your project will be able to estimate the point value for user stories
-
-        div.general-category(ng-if="!role.external_user")
-            | When enabled, members assigned to this role will be able to estimate the point value for user stories
-            div.check
-                input(type="checkbox", ng-model="role.computable", ng-change="ctrl.setComputable()")
-                div
-                span.check-text.check-yes Yes
-                span.check-text.check-no No
+                span.role-name {{ role.name }}
 
         div(tg-role-permissions, ng-model="role")