Ovirt devel - May 2009 - [PATCH server] fix for Bug 464396 - There are two ovirtadmin User displayed in the User Access of the Smart Pool

For inherited permissions, show which pool the user inherits access from.

Signed-off-by: Scott Seago <sseago at redhat.com>
---
 src/app/models/permission.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/app/models/permission.rb b/src/app/models/permission.rb
index f74a11a..6adf16a 100644
--- a/src/app/models/permission.rb
+++ b/src/app/models/permission.rb
@@ -45,7 +45,7 @@ class Permission < ActiveRecord::Base
     !is_primary?
   end
   def source
-    is_primary? ? "Direct" : "Inherited"
+    is_primary? ? "(Direct)" : parent_permission.pool.name
   end
   def grid_id
     id.to_s + "_" + (is_primary? ? "1" : "0")
-- 
1.6.0.6

ACK, behaves as expected

On Tue, May 26, 2009 at 3:14 PM, Scott Seago <sseago at redhat.com> wrote:
> For inherited permissions, show which pool the user inherits access from.
>
> Signed-off-by: Scott Seago <sseago at redhat.com>
> ---
>  src/app/models/permission.rb |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/app/models/permission.rb b/src/app/models/permission.rb
> index f74a11a..6adf16a 100644
> --- a/src/app/models/permission.rb
> +++ b/src/app/models/permission.rb
> @@ -45,7 +45,7 @@ class Permission < ActiveRecord::Base
>     !is_primary?
>   end
>   def source
> -    is_primary? ? "Direct" : "Inherited"
> +    is_primary? ? "(Direct)" : parent_permission.pool.name
>   end
>   def grid_id
>     id.to_s + "_" + (is_primary? ? "1" : "0")
> --
> 1.6.0.6
>
> _______________________________________________
> Ovirt-devel mailing list
> Ovirt-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/ovirt-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://listman.redhat.com/archives/ovirt-devel/attachments/20090528/3de80cac/attachment.htm>