COmanage Developers List

[Marie Huynh <>]

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