svn: /pear/pear-core/trunk/PEAR/ Downloader.php

dufuz                                    Mon, 13 Sep 2010 21:50:02 +0000

Revision: http://svn.php.net/viewvc?view=revision&revision=303331

Log:
Fixed Bug #17781: invalid argument warning on foreach due to an empty optional dependencie [dufuz]

Bug: http://pear.php.net/bugs/17781 (unknown) 
      
Changed paths:
    U   pear/pear-core/trunk/PEAR/Downloader.php

Modified: pear/pear-core/trunk/PEAR/Downloader.php
===================================================================
--- pear/pear-core/trunk/PEAR/Downloader.php	2010-09-13 20:14:18 UTC (rev 303330)
+++ pear/pear-core/trunk/PEAR/Downloader.php	2010-09-13 21:50:02 UTC (rev 303331)
@@ -553,7 +553,7 @@
                     }

                     $failed = false;
-                    if (isset($deps['required'])) {
+                    if (isset($deps['required']) && is_array($deps['required'])) {
                         foreach ($deps['required'] as $type => $dep) {
                             // note: Dependency2 will never return a PEAR_Error if ignore-errors
                             // is specified, so soft is needed to turn off logging
@@ -587,7 +587,7 @@
                             }
                         }

-                        if (isset($deps['optional'])) {
+                        if (isset($deps['optional']) && is_array($deps['optional'])) {
                             foreach ($deps['optional'] as $type => $dep) {
                                 if (!isset($dep[0])) {
                                     if (PEAR::isError($e =