COmanage Developers List

[Scott Koranda <>]

Hi Benn,

Can you please take a look at my comment below regarding
CO-239?

I am not able to reproduce the problem and I think my comment
demonstrates a fair test, i.e. the database contents are
appropriate for a COU admin and the return value from
calculateCMRoles() shows couadmin set.

Can you let me know if the value set for couadmin is correct
and if so how you want to proceed on CO-239?

Thanks,

Scott

----- Forwarded message from "Scott Koranda (JIRA)" 
<>
 -----

Date: Tue, 21 Feb 2012 12:43:54 -0500 (EST)
From: "Scott Koranda (JIRA)" 
<>
To: 

Subject: [JIRA] (CO-239) couadmin not properly calculated


    [ 
https://bugs.internet2.edu/jira/browse/CO-239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16457#comment-16457
 ] 

Scott Koranda commented on CO-239:
----------------------------------

I am using r231 for testing.

I have updated the state of the registry to add a COU named 'HannoverTheory' 
and added an admin with name 'Carsten Aubert' and a valid identifier for 
logging in. This SQL shows the state:


mysql> SELECT cm_co_groups.name,cm_names.given,cm_names.family FROM 
cm_co_group_members JOIN cm_co_groups ON cm_co_group_members.co_group_id = 
cm_co_groups.id JOIN cm_co_people ON cm_co_group_members.co_person_id = 
cm_co_people.id JOIN cm_names ON cm_co_people.id = cm_names.co_person_id 
WHERE cm_co_groups.id = 5;
+----------------------+---------+--------+
| name                 | given   | family |
+----------------------+---------+--------+
| admin:HannoverTheory | Carsten | Aubert |
+----------------------+---------+--------+
1 row in set (0.00 sec)

I then edited CoGroupsController.php and added this line to the 
isAuthorized() method:

$this->log("isAuthorized: cmr = " . var_export($cmr, true));'

where $cmr is set by this call

$cmr = $this->calculateCMRoles(); 

When I log in as this test user and load the URL

https://127.0.0.1/registry/co_groups/index/co:2

I see this in the log file:

2012-02-21 11:14:08 Error: isAuthorized: cmr = array (
  'cmadmin' => false,
  'coadmin' => false,
  'couadmin' => 
  array (
    0 => 'HannoverTheory',
  ),
  'comember' => true,
  'admin' => false,
  'subadmin' => true,
  'user' => true,
  'apiuser' => false,
  'orgidentityid' => false,
  'copersonid' => '2',
  'orgidentities' => 
  array (
    0 => 
    array (
      'org_id' => '2',
      'co_id' => '2',
    ),
  ),
)

So it appears that couadmin is being properly set in 
AppController::calculateCMRoles().

                
> couadmin not properly calculated
> --------------------------------
>
>                 Key: CO-239
>                 URL: https://bugs.internet2.edu/jira/browse/CO-239
>             Project: COmanage
>          Issue Type: Bug
>          Components: Registry
>    Affects Versions: COmanage Registry 0.4 (Reshaped Cake)
>            Reporter: Benn Oshrin
>            Assignee: Scott Koranda
>            Priority: Critical
>              Labels: sprint04
>             Fix For: COmanage Registry 0.4 (Reshaped Cake)
>
>
> AppController::calculateCMRoles is not properly calculating couadmin. It 
> returns empty for a person who is a COU admin.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA 
administrators: 
https://bugs.internet2.edu/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

----- End forwarded message -----