Twitter bridge - Was using the wrong idiom to update foreign_link

objs, which was causing UI inconsistencies. Fixed.

darcs-hash:20080924060455-7b5ce-71ac4bb3a80ca44e6ae758d2287d50c918d1d908.gz
This commit is contained in:
Zach Copley 2008-09-24 02:04:55 -04:00
parent e8b6e394dc
commit 845db5ec16
1 changed files with 9 additions and 2 deletions

View File

@ -238,6 +238,10 @@ class TwittersettingsAction extends SettingsAction {
return;
}
$flink->query('BEGIN');
$original = clone($flink);
if ($noticesync) {
if ($replysync) {
$flink->noticesync = 3;
@ -250,14 +254,17 @@ class TwittersettingsAction extends SettingsAction {
$flink->friendsync = ($friendsync) ? 2 : 0;
// $flink->profilesync = 0; // XXX: leave as default?
$result = $flink->update();
if (!$result) {
$result = $flink->update($original);
if ($result === FALSE) {
common_log_db_error($flink, 'UPDATE', __FILE__);
$this->show_form(_('Couldn\'t save Twitter preferences.'));
return;
}
$flink->query('COMMIT');
$this->show_form(_('Twitter preferences saved.'));
return;