diff options
| author | Gábor Hojtsy <gabor@hojtsy.hu> | 2007-08-18 20:03:19 +0000 |
|---|---|---|
| committer | Gábor Hojtsy <gabor@hojtsy.hu> | 2007-08-18 20:03:19 +0000 |
| commit | 8971add8493d49ed0e170a2521a62f2f0f39137c (patch) | |
| tree | d22f43ae4213b84c1089e7c12aa31c9220e77dd3 /modules/system/system.install | |
| parent | 5d31f59573b8a559b06842b7f90cc2bd588025a2 (diff) | |
| download | brdo-8971add8493d49ed0e170a2521a62f2f0f39137c.tar.gz brdo-8971add8493d49ed0e170a2521a62f2f0f39137c.tar.bz2 | |
#163191 follow up patch by hswong3i and pwolanin: use a has_rows flag where we dont need a counter
Diffstat (limited to 'modules/system/system.install')
| -rw-r--r-- | modules/system/system.install | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/modules/system/system.install b/modules/system/system.install index e0aa4d35e..56205fdc9 100644 --- a/modules/system/system.install +++ b/modules/system/system.install @@ -2169,7 +2169,8 @@ function system_update_179() { $limit = 20; $args = array_merge(array($_SESSION['system_update_179_uid'], $_SESSION['system_update_179_fid'], $_SESSION['system_update_179_uid']), $_SESSION['system_update_179_fields']); $result = db_query_range("SELECT fid, uid, value FROM {profile_values} WHERE ((uid = %d AND fid > %d) OR uid > %d) AND fid IN ". $_SESSION['system_update_179_field_string'] .' ORDER BY uid ASC, fid ASC', $args, 0, $limit); - $num_rows = 0; + + $has_rows = FALSE; while ($field = db_fetch_object($result)) { $_SESSION['system_update_179_uid'] = $field->uid; $_SESSION['system_update_179_fid'] = $field->fid; @@ -2177,11 +2178,11 @@ function system_update_179() { if ($field->value !== FALSE) { db_query("UPDATE {profile_values} SET value = '%s' WHERE uid = %d AND fid = %d", $field->value, $field->uid, $field->fid); } - $num_rows++; + $has_rows = TRUE; } // Done? - if ($num_rows == 0) { + if (!$has_rows) { unset($_SESSION['system_update_179_uid']); unset($_SESSION['system_update_179_fid']); unset($_SESSION['system_update_179_max']); |