mirror of
https://github.com/phpbb/phpbb.git
synced 2025-06-28 06:08:52 +00:00
oop, only update if we have data ...
git-svn-id: file:///svn/phpbb/trunk@4064 89ea8834-ac86-4346-8a33-228a782c2dd0
This commit is contained in:
parent
b6a0ce82d4
commit
208f1e59bf
1 changed files with 20 additions and 18 deletions
|
@ -149,7 +149,7 @@ class ucp_profile extends ucp
|
||||||
$validate = array(
|
$validate = array(
|
||||||
'match' => array(
|
'match' => array(
|
||||||
'icq' => ($data['icq']) ? '#^[0-9]+$#i' : '',
|
'icq' => ($data['icq']) ? '#^[0-9]+$#i' : '',
|
||||||
'website' => ($data['website']) ? '#^http[s]?://(.*?\.)*?[a-z0-9\-]+\.[a-z]+#i' : '',
|
'website' => ($data['website']) ? '#^http[s]?://(.*?\.)*?[a-z0-9\-]+\.[a-z]{2,4}#i' : '',
|
||||||
),
|
),
|
||||||
);
|
);
|
||||||
$this->validate_data($data, $validate);
|
$this->validate_data($data, $validate);
|
||||||
|
@ -384,22 +384,26 @@ class ucp_profile extends ucp
|
||||||
|
|
||||||
if (!sizeof($this->error))
|
if (!sizeof($this->error))
|
||||||
{
|
{
|
||||||
$sql_ary = array(
|
// Do we actually have any data to update?
|
||||||
'user_avatar' => $data['filename'],
|
if (sizeof($data))
|
||||||
'user_avatar_type' => $data['type'],
|
|
||||||
'user_avatar_width' => $data['width'],
|
|
||||||
'user_avatar_height' => $data['height'],
|
|
||||||
);
|
|
||||||
|
|
||||||
$sql = 'UPDATE ' . USERS_TABLE . '
|
|
||||||
SET ' . $db->sql_build_array('UPDATE', $sql_ary) . '
|
|
||||||
WHERE user_id = ' . $user->data['user_id'];
|
|
||||||
$db->sql_query($sql);
|
|
||||||
|
|
||||||
// Delete old avatar if present
|
|
||||||
if ($user->data['user_avatar'] != '' && $data['filename'] != $user->data['user_avatar'])
|
|
||||||
{
|
{
|
||||||
$this->avatar_delete();
|
$sql_ary = array(
|
||||||
|
'user_avatar' => $data['filename'],
|
||||||
|
'user_avatar_type' => $data['type'],
|
||||||
|
'user_avatar_width' => $data['width'],
|
||||||
|
'user_avatar_height' => $data['height'],
|
||||||
|
);
|
||||||
|
|
||||||
|
$sql = 'UPDATE ' . USERS_TABLE . '
|
||||||
|
SET ' . $db->sql_build_array('UPDATE', $sql_ary) . '
|
||||||
|
WHERE user_id = ' . $user->data['user_id'];
|
||||||
|
$db->sql_query($sql);
|
||||||
|
|
||||||
|
// Delete old avatar if present
|
||||||
|
if ($user->data['user_avatar'] != '' && $data['filename'] != $user->data['user_avatar'])
|
||||||
|
{
|
||||||
|
$this->avatar_delete();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
meta_refresh(3, "ucp.$phpEx$SID&i=$id&mode=$submode");
|
meta_refresh(3, "ucp.$phpEx$SID&i=$id&mode=$submode");
|
||||||
|
@ -485,8 +489,6 @@ class ucp_profile extends ucp
|
||||||
|
|
||||||
$this->display($user->lang['UCP_PROFILE'], 'ucp_profile.html');
|
$this->display($user->lang['UCP_PROFILE'], 'ucp_profile.html');
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
?>
|
?>
|
Loading…
Add table
Reference in a new issue