Fixes profile edition errors due to null visibility.
authorStéphane Jacob <sj@m4x.org>
Thu, 22 Sep 2011 07:42:46 +0000 (09:42 +0200)
committerStéphane Jacob <sj@m4x.org>
Thu, 22 Sep 2011 07:42:46 +0000 (09:42 +0200)
Signed-off-by: Stéphane Jacob <sj@m4x.org>
classes/address.php
classes/phone.php

index 665d91d..871526a 100644 (file)
@@ -878,7 +878,7 @@ class Address
     }
 
     static public function iterate(array $pids = array(), array $types = array(),
-                                   array $jobids = array(), Visibility $visibility)
+                                   array $jobids = array(), $visibility = null)
     {
         return new AddressIterator($pids, $types, $jobids, $visibility);
     }
@@ -895,7 +895,7 @@ class AddressIterator implements PlIterator
 {
     private $dbiter;
 
-    public function __construct(array $pids, array $types, array $jobids, Visibility $visibility)
+    public function __construct(array $pids, array $types, array $jobids, $visibility)
     {
         $where = array();
         if (count($pids) != 0) {
@@ -907,7 +907,7 @@ class AddressIterator implements PlIterator
         if (count($jobids) != 0) {
             $where[] = XDB::format('(pa.jobid IN {?})', $jobids);
         }
-        if ($visibility == null) {
+        if ($visibility == null || !($visibility instanceof Visibility)) {
             $visibility = Visibility::defaultForRead();
         }
         $where[] = 'pve.best_display_level+0 <= pa.pub+0';
index 0cb0703..4ccab6b 100644 (file)
@@ -374,7 +374,7 @@ class Phone
     }
 
     static public function iterate(array $pids = array(), array $link_types = array(),
-                                   array $link_ids = array(), Visibility $visibility)
+                                   array $link_ids = array(), $visibility = null)
     {
         return new PhoneIterator($pids, $link_types, $link_ids, $visibility);
     }
@@ -391,7 +391,7 @@ class PhoneIterator implements PlIterator
 {
     private $dbiter;
 
-    public function __construct(array $pids, array $link_types, array $link_ids, Visibility $visibility)
+    public function __construct(array $pids, array $link_types, array $link_ids, $visibility)
     {
         $where = array();
         if (count($pids) != 0) {
@@ -403,7 +403,7 @@ class PhoneIterator implements PlIterator
         if (count($link_ids) != 0) {
             $where[] = XDB::format('(link_id IN {?})', $link_ids);
         }
-        if ($visibility == null) {
+        if ($visibility == null || !($visibility instanceof Visibility)) {
             $visibility = Visibility::defaultForRead();
         }
         $where[] = 'pve.best_display_level+0 <= pub+0';