Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed, PHP Warning: Invalid argument supplied for foreach() in ... #62

Merged
merged 1 commit into from
Mar 5, 2019

Conversation

asig2016
Copy link
Contributor

@asig2016 asig2016 commented Mar 5, 2019

On standard install messages like the following show up in the Server log:

===== 1 =====
PHP Warning: Invalid argument supplied for foreach() in .../html/egw/api/src/Db/Schema.php on line 170
PHP message: #1 .../html/egw/api/src/Db/Schema.php(566): EGroupware\Api\Db\Schema->_in_index('pm_id', NULL, true)
PHP message: #2 /home/asig/dev/egw/egwdev2/html/egw/api/src/Db/Schema.php(275): EGroupware\Api\Db\Schema->CreateIndex('egw_pm_roles', 'pm_id', false, Array, 'egw_pm_roles_id')
PHP message: #3 /home/asig/dev/egw/egwdev2/html/egw/api/src/Db/Schema.php(828): EGroupware\Api\Db\Schema->CreateTable('egw_pm_roles', Array)
PHP message: #4 /home/asig/dev/egw/egwdev2/html/egw/setup/inc/class.setup_process.inc.php(753): EGroupware\Api\Db\Schema->ExecuteScripts(Array, NULL)
PHP message: #5 /home/asig/dev/egw/egwdev2/html/egw/setup/inc/class.setup_process.inc.php(422): setup_process->post_process(Array, NULL)
PHP message: #6 /home/asig/dev/egw/egwdev2/html/egw/setup/inc/class.setup_process.inc.php(147): setup_process->current(Array, NULL)
PHP message: #7 /home/asig/dev/egw/egwdev2/html/egw/setup/index.php(349): setup_process->pass(Array, 'new', NULL, NULL)
PHP message: #8 {main}

==== 2 =====

PHP message: PHP Warning: Invalid argument supplied for foreach() in .../html/egw/importexport/inc/class.importexport_definition.inc.php on line 184
PHP message: #..../html/egw/importexport/inc/class.importexport_definition.inc.php(257): importexport_definition->set_options(Array)
PHP message: #2.../html/egw/importexport/inc/class.importexport_definitions_bo.inc.php(254): importexport_definition->set_record(Array)
PHP message: #3 .../html/egw/importexport/setup/default_records.inc.php(34): importexport_definitions_bo::import('/home/asig/dev/...')
PHP message: #4 ..../html/egw/setup/inc/class.setup_process.inc.php(503): include('/home/asig/dev/...')
PHP message: #5 .../html/egw/setup/inc/class.setup_process.inc.php(149): setup_pr

@ralfbecker ralfbecker merged commit a154557 into EGroupware:17.1 Mar 5, 2019
@ralfbecker
Copy link
Member

Thanks :)

@asig2016 asig2016 deleted the for_each branch March 5, 2019 08:12
@ralfbecker
Copy link
Member

Hi Alexandros,

thous kind of fixes are very welcome :)

For the next time, please either
a) make them only in master or
b) make them in master and 17.1
Just making them in 17.1 means, they are lost with the next release, unless we spot that and merge them up ourself.

Ralf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants