From 85bf20d95afeadd682584e595df9157ad853a3ed Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Mon, 20 Sep 2010 16:57:07 -0700 Subject: [PATCH] Fix error when editing OAuth app registration and only changing the icon. --- actions/editapplication.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/actions/editapplication.php b/actions/editapplication.php index b25457069c..477bcd6f0b 100644 --- a/actions/editapplication.php +++ b/actions/editapplication.php @@ -253,7 +253,10 @@ class EditApplicationAction extends OwnerDesignAction $result = $this->app->update($orig); - if (!$result) { + // Note: 0 means no rows changed, which can happen if the only + // thing we changed was the icon, since it's not altered until + // the next step. + if ($result === false) { common_log_db_error($this->app, 'UPDATE', __FILE__); $this->serverError(_('Could not update application.')); } -- 2.39.5