From 6972203109e03593cd2fd0280dbf546eaf4c6986 Mon Sep 17 00:00:00 2001 From: rWatcher Date: Wed, 20 Jun 2012 14:47:30 -0400 Subject: [PATCH] Bug Fix: Make sure arrays have something in them before looping through them. --- .../exif_gps/controllers/admin_exif_gps.php | 24 ++++++++++++------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/3.0/modules/exif_gps/controllers/admin_exif_gps.php b/3.0/modules/exif_gps/controllers/admin_exif_gps.php index 40ef47ba..c291937b 100644 --- a/3.0/modules/exif_gps/controllers/admin_exif_gps.php +++ b/3.0/modules/exif_gps/controllers/admin_exif_gps.php @@ -48,19 +48,25 @@ class Admin_EXIF_GPS_Controller extends Admin_Controller { $checkbox_album = false; $checkbox_user = false; $checkbox_restriction = false; - for ($i = 0; $i < count($form->Global->toolbar_map_album); $i++) { - if ($form->Global->toolbar_map_album->value[$i] == "checkbox_album") { - $checkbox_album = true; + if (count($form->Global->toolbar_map_album) > 0) { + for ($i = 0; $i < count($form->Global->toolbar_map_album); $i++) { + if ($form->Global->toolbar_map_album->value[$i] == "checkbox_album") { + $checkbox_album = true; + } } } - for ($i = 0; $i < count($form->Global->toolbar_map_user); $i++) { - if ($form->Global->toolbar_map_user->value[$i] == "checkbox_user") { - $checkbox_user = true; + if (count($form->Global->toolbar_map_user) > 0) { + for ($i = 0; $i < count($form->Global->toolbar_map_user); $i++) { + if ($form->Global->toolbar_map_user->value[$i] == "checkbox_user") { + $checkbox_user = true; + } } } - for ($i = 0; $i < count($form->Global->restrict_maps); $i++) { - if ($form->Global->restrict_maps->value[$i] == "checkbox_restriction") { - $checkbox_restriction = true; + if (count($form->Global->restrict_maps) > 0) { + for ($i = 0; $i < count($form->Global->restrict_maps); $i++) { + if ($form->Global->restrict_maps->value[$i] == "checkbox_restriction") { + $checkbox_restriction = true; + } } } module::set_var("exif_gps", "toolbar_map_album", $checkbox_album);