diff --git a/src/Model/Post/DeliveryData.php b/src/Model/Post/DeliveryData.php index 327e6f9121..c1aad730c7 100644 --- a/src/Model/Post/DeliveryData.php +++ b/src/Model/Post/DeliveryData.php @@ -86,19 +86,19 @@ class DeliveryData switch ($protocol) { case self::ACTIVITYPUB: - $increments[] = ["`activitypub` = `activitypub` + 1"]; + $increments[] = "`activitypub` = `activitypub` + 1"; break; case self::DFRN: - $increments[] = ["`dfrn` = `dfrn` + 1"]; + $increments[] = "`dfrn` = `dfrn` + 1"; break; case self::LEGACY_DFRN: - $increments[] = ["`legacy_dfrn` = `legacy_dfrn` + 1"]; + $increments[] = "`legacy_dfrn` = `legacy_dfrn` + 1"; break; case self::DIASPORA: - $increments[] = ["`diaspora` = `diaspora` + 1"]; + $increments[] = "`diaspora` = `diaspora` + 1"; break; case self::OSTATUS: - $increments[] = ["`ostatus` = `ostatus` + 1"]; + $increments[] = "`ostatus` = `ostatus` + 1"; break; } diff --git a/tests/src/Database/DatabaseTest.php b/tests/src/Database/DatabaseTest.php index 47d4032209..e906834614 100644 --- a/tests/src/Database/DatabaseTest.php +++ b/tests/src/Database/DatabaseTest.php @@ -85,4 +85,19 @@ class DatabaseTest extends FixtureTest ])); self::assertEquals(1, $db->selectFirst('gserver', ['registered-users'], ['nurl' => 'http://friendica.local'])['registered-users']); } + + public function testUpdateWithArray() + { + $db = $this->getDbInstance(); + + self::assertTrue($db->update('gserver', ['active-week-users' => 0, 'registered-users' => 0], ['nurl' => 'http://friendica.local'])); + + $fields = ["`registered-users` = `registered-users` + 1"]; + $fields[] = "`active-week-users` = `active-week-users` + 2"; + + self::assertTrue($db->update('gserver', $fields, ['nurl' => 'http://friendica.local'])); + + self::assertEquals(2, $db->selectFirst('gserver', ['active-week-users'], ['nurl' => 'http://friendica.local'])['active-week-users']); + self::assertEquals(1, $db->selectFirst('gserver', ['registered-users'], ['nurl' => 'http://friendica.local'])['registered-users']); + } }