GNUsocial/gnu-social
4
0
Fork 1
Code Issues Releases Wiki Activity

add better debugging

Browse Source 
darcs-hash:20080622140941-34904-2a0eda21f6a374a9d26107a4bc627fc6de2a7063.gz
This commit is contained in:
Evan Prodromou 2008-06-22 10:09:41 -04:00
parent 676bbebe1a
commit 745a145f0d
3 changed files with 35 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
actions/confirmemail.php
View File

@ -47,21 +47,27 @@ class ConfirmemailAction extends Action {
$this->client_error(_t('That email address is already confirmed.')); $this->client_error(_t('That email address is already confirmed.'));
return; return;
} }
$cur->query('BEGIN'); $cur->query('BEGIN');
$orig_user = clone($cur); $orig_user = clone($cur);
$cur->email = $confirm_email->email; $cur->email = $confirm_email->email;
common_debug('cur email = "' . $cur->email . '"', __FILE__);
$result = $cur->update($orig_user); $result = $cur->update($orig_user);
if (!$result) { if (!$result) {
$this->server_error(_t('Error setting email address.')); common_log_db_error($cur, 'UPDATE', __FILE__);
return; return;
} }
$result = $confirm_email->delete(); $result = $confirm_email->delete();
if (!$result) { if (!$result) {
$this->server_error(_t('Error deleting code.')); common_log_db_error($confirm_email, 'DELETE', __FILE__);
return; return;
} }
$cur->query('COMMIT'); $cur->query('COMMIT');
common_show_header(_t('Confirm E-mail Address')); common_show_header(_t('Confirm E-mail Address'));
common_element('p', NULL, common_element('p', NULL,
_t('The email address "') . $cur->email . _t('The email address "') . $cur->email .

6
actions/register.php
View File

@ -99,7 +99,9 @@ class RegisterAction extends Action {
$profile->created = DB_DataObject_Cast::dateTime(); # current time $profile->created = DB_DataObject_Cast::dateTime(); # current time
$id = $profile->insert(); $id = $profile->insert();
if (!$id) { if (!$id) {
common_log_db_error($profile, 'INSERT', __FILE__);
return FALSE; return FALSE;
} }
$user = new User(); $user = new User();
@ -110,11 +112,14 @@ class RegisterAction extends Action {
$user->uri = common_user_uri($user); $user->uri = common_user_uri($user);
$result = $user->insert(); $result = $user->insert();
if (!$result) { if (!$result) {
common_log_db_error($user, 'INSERT', __FILE__);
return FALSE; return FALSE;
} }
if ($email) { if ($email) {
$confirm = new Confirm_email(); $confirm = new Confirm_email();
$confirm->code = common_good_rand(16); $confirm->code = common_good_rand(16);
$confirm->user_id = $user->id; $confirm->user_id = $user->id;
@ -122,6 +127,7 @@ class RegisterAction extends Action {
$result = $confirm->insert(); $result = $confirm->insert();
if (!$result) { if (!$result) {
common_log_db_error($confirm, 'INSERT', __FILE__);
return FALSE; return FALSE;
} }
} }

19
lib/util.php
View File

@ -762,6 +762,25 @@ function common_debug($msg, $filename=NULL) {
} }
} }
function common_log_db_error($object, $verb, $filename=NULL) {
$objstr = common_log_objstring($ojbect);
$last_error = &PEAR::getStaticProperty('DB_DataObject','lastError');
common_log(LOG_ERROR, $last_error->message . '(' . $verb . ' on ' . $objstr . ')', $filename);
}
function common_log_objstring($object) {
if (is_null($object)) {
return "NULL";
}
$arr = $object->toArray();
$fields = array();
foreach ($arr as $k => $v) {
$fields[] = "$k='$v'";
}
$ojbstring = $object->tableName() . '[' . implode(',', $fields) . ']';
return $objstring;
}
function common_valid_http_url($url) { function common_valid_http_url($url) {
return Validate::uri($url, array('allowed_schemes' => array('http', 'https'))); return Validate::uri($url, array('allowed_schemes' => array('http', 'https')));
} }