COmanage Developers List

[Benn Oshrin <>]

Yes, I think it's fixed. I'm now getting different errors, similar but 
not identical to the ones you posted in CO-252:

 Notice (8): Undefined index: CoPerson 
[APP/View/CoNsfDemographics/fields.inc, line 61]

 Notice (8): Undefined variable: race_options 
[APP/View/CoNsfDemographics/fields.inc, line 172]

 Notice (8): Undefined variable: disability_options 
[APP/View/CoNsfDemographics/fields.inc, line 190]

-Benn-

On 2/17/12 7:46 PM, Marie Huynh wrote:
> Benn,
>
> Sorry, I grabbed the newest email and didn't realize this one hadn't
> come in yet.  Can you check on this for me?
>
> Thanks,
>
> Marie
>
>
>
>
> ---------- Forwarded message ----------
> From: Marie Huynh 
> (JIRA)<>
> Date: Fri, Feb 17, 2012 at 4:41 PM
> Subject: [JIRA] (CO-233) View My Demographics errors
> To: 
>
>
>
>
>     [ 
> https://bugs.internet2.edu/jira/browse/CO-233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16427#comment-16427
> ]
>
> Marie Huynh commented on CO-233:
> --------------------------------
>
> Index: Model/Cou.php
> ===================================================================
> --- Model/Cou.php       (revision 230)
> +++ Model/Cou.php       (working copy)
> @@ -148,7 +148,9 @@
>      {
>        foreach($parentData as $parent)
>        {
> -        $allChildren = array_merge($allChildren,
> $this->children($parent, false, 'name'));
> +        $thisChildren = $this->children($parent, false, 'name')
> +        if($thisChildren != NULL)
> +          $allChildren = array_merge($allChildren, $thisChildren);
>        }
>      }
>      $allChildren = Set::extract($allChildren, '{n}.Cou.name');
>
>
>
> Index: Controller/AppController.php
> ===================================================================
> --- Controller/AppController.php        (revision 230)
> +++ Controller/AppController.php        (working copy)
> @@ -227,7 +227,8 @@
>              $this->loadModel('Cou');
>
>              $ret['couadmin'] = $this->Cou->childCous($ret['couadmin']);
> -            sort($ret['couadmin']);
> +            if($ret['couadmin'] != NULL)
> +              sort($ret['couadmin']);
>
>              // Promote the set of COUs so they are globally available
>              $this->cur_cous =   $ret['couadmin'];
>
>
> > View My Demographics errors
> > ---------------------------
> >
> >                  Key: CO-233
> >                  URL: https://bugs.internet2.edu/jira/browse/CO-233
> >              Project: COmanage
> >           Issue Type: Bug
> >           Components: Registry
> >     Affects Versions: COmanage Registry 0.4 (Reshaped Cake)
> >             Reporter: Benn Oshrin
> >             Assignee: Marie Huynh
> >               Labels: sprint04
> >              Fix For: COmanage Registry 0.4 (Reshaped Cake)
> >
> >
> > /co_nsf_demographics/add/co:#/co_person_id:# generates various PHP undefined 
> > errors.
> > Warning (2): array_merge() [function.array-merge]: Argument #2 is not an 
> > array [APP/Model/Cou.php, line 149]
> > Warning (2): array_merge() [function.array-merge]: Argument #2 is not an 
> > array [APP/Model/Cou.php, line 153]
> > Warning (2): sort() expects parameter 1 to be array, null given 
> > [APP/Controller/AppController.php, line 230]
> > Warning (2): array_merge() [function.array-merge]: Argument #2 is not an 
> > array [APP/Model/Cou.php, line 149]
> > Warning (2): array_merge() [function.array-merge]: Argument #2 is not an 
> > array [APP/Model/Cou.php, line 153]
> > Warning (2): sort() expects parameter 1 to be array, null given 
> > [APP/Controller/AppController.php, line 230]
>
> --
> 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