ezcommunity / ezupgrade

GNU General Public License v2.0
10 stars 3 forks source link

PHP Notice when defining mutiples siteaccesses #24

Closed pedroresende closed 12 years ago

pedroresende commented 12 years ago

When running php ezupgrade, if defined various siteaccesses the following error raises

Enter a comma separated list of siteaccesses with a unique database: ezflow_site,eng,ezflow_site_admin,iphone Enter the name of the MySQL database for siteaccess ezflow_site: ezp Enter the name of the MySQL username for siteaccess ezflow_site: ezp Enter the name of the MySQL password for siteaccess ezflow_site: ezp Logging in.. OK PHP Notice: Undefined variable: siteaccessList in /var/www/apache2php53/ezupgrade/classes/accountconfiguration.php on line 179 PHP Warning: Invalid argument supplied for foreach() in /var/www/apache2php53/ezupgrade/classes/accountconfiguration.php on line 179 E.

pedroresende commented 12 years ago

To correct this issue, simply edit the following file /classes/accountconfiguration.php and in line 179 change $siteaccessList to $this->siteaccessList and in line 181 change $accountName to $this->accountName.

cgjengedal commented 12 years ago

Fixed in https://github.com/ezsystems/ezupgrade/commit/22633402000eddd9eb07faceef4f02e7d9afa942