Project

General

Profile

« Previous | Next » 

Revision 9f472202

Added by NewEraCracker about 9 years ago

Fix PHP Warning about invalid argument supplied for foreach

If _POST['members'] or _POST['groups'] is not set / none selected at GUI, it would give a warning on crash reporter (dev versions)

View differences:

src/usr/local/www/system_groupmanager.php
193 193
	}
194 194

  
195 195
	/* Check the POSTed members to ensure they are valid and exist */
196
	foreach ($_POST['members'] as $newmember) {
197
		if (!is_numeric($newmember) || empty(getUserEntryByUID($newmember))) {
198
			$input_errors[] = gettext("One or more invalid group members was submitted.");
196
	if(is_array($_POST['members'])) {
197
		foreach ($_POST['members'] as $newmember) {
198
			if (!is_numeric($newmember) || empty(getUserEntryByUID($newmember))) {
199
				$input_errors[] = gettext("One or more invalid group members was submitted.");
200
			}
199 201
		}
200 202
	}
201 203

  
src/usr/local/www/system_usermanager.php
214 214
	}
215 215

  
216 216
	/* Check the POSTed groups to ensure they are valid and exist */
217
	foreach ($_POST['groups'] as $newgroup) {
218
		if (empty(getGroupEntry($newgroup))) {
219
			$input_errors[] = gettext("One or more invalid groups was submitted.");
217
	if(is_array($_POST['groups'])) {
218
		foreach ($_POST['groups'] as $newgroup) {
219
			if (empty(getGroupEntry($newgroup))) {
220
				$input_errors[] = gettext("One or more invalid groups was submitted.");
221
			}
220 222
		}
221 223
	}
222 224

  

Also available in: Unified diff