From: Zach Copley <zach@controlyourself.ca>
Date: Wed, 24 Sep 2008 06:04:55 +0000 (-0400)
Subject: Twitter bridge - Was using the wrong idiom to update foreign_link
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=845db5ec168e26759c79de4f8216d1b54f7c0d8a;p=quix0rs-gnu-social.git

Twitter bridge - Was using the wrong idiom to update foreign_link

objs, which was causing UI inconsistencies. Fixed.

darcs-hash:20080924060455-7b5ce-71ac4bb3a80ca44e6ae758d2287d50c918d1d908.gz
---

diff --git a/actions/twittersettings.php b/actions/twittersettings.php
index d8eb7aba08..d37c042acf 100644
--- a/actions/twittersettings.php
+++ b/actions/twittersettings.php
@@ -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;