COmanage Developers List

[Benn Oshrin <>]

Strange, that's clearly a bug and yet it works fine for me. For a change 
like this (a small fix after a commit) you can just push it with a 
message like "additional fix for CO-123". Please push (assuming you're 
not going to commit the commented out line).

Thanks,

-Benn-

On 9/24/13 5:45 AM, Scott Koranda wrote:
> Hi,
>
> I pulled the latest code and to get it to work I had to make this
> change in CoPerson.php:
>
> $ git diff CoPerson.php
> diff --git a/app/Model/CoPerson.php b/app/Model/CoPerson.php
> index 2496e2f..27a5f41 100644
> --- a/app/Model/CoPerson.php
> +++ b/app/Model/CoPerson.php
> @@ -76,7 +76,8 @@ class CoPerson extends AppModel {
>         'foreignKey' => 'actor_co_person_id'
>       ),
>       "CoProvisioningExport" => array('dependent' => true),
> -    "CoTandCAgreement" => array('dependent' => true),
> +    //"CoTandCAgreement" => array('dependent' => true),
> +    "CoTAndCAgreement" => array('dependent' => true),
>       // A person can have one or more email address
>       "EmailAddress" => array('dependent' => true),
>       // We allow dependent=true for co_person_id but not for
> actor_co_person_id (see CO-404).
>
> Is that a bug?
>
> If so, shall I create a ticket or is this small enough that I should
> just push it?
>
> Thanks,
>
> Scott