Fix ContactEndpoint & tests

This commit is contained in:
Philipp Holzer 2020-09-12 10:01:13 +02:00
parent 1008f408c0
commit 61014feec3
No known key found for this signature in database
GPG Key ID: 9A28B7D4FF5667BD
3 changed files with 10 additions and 10 deletions

View File

@ -73,7 +73,7 @@ abstract class ContactEndpoint extends BaseApi
throw new HTTPException\NotFoundException(DI::l10n()->t('User not found')); throw new HTTPException\NotFoundException(DI::l10n()->t('User not found'));
} }
$uid = $user['uid']; $uid = (int)$user['uid'];
} }
return $uid; return $uid;
@ -111,7 +111,7 @@ abstract class ContactEndpoint extends BaseApi
'next_cursor_str' => $return['next_cursor_str'], 'next_cursor_str' => $return['next_cursor_str'],
'previous_cursor' => $return['previous_cursor'], 'previous_cursor' => $return['previous_cursor'],
'previous_cursor_str' => $return['previous_cursor_str'], 'previous_cursor_str' => $return['previous_cursor_str'],
'total_count' => $return['total_count'], 'total_count' => (int)$return['total_count'],
]; ];
return $return; return $return;
@ -153,7 +153,7 @@ abstract class ContactEndpoint extends BaseApi
'pending' => false 'pending' => false
]; ];
$total_count = DBA::count('contact', $condition); $total_count = (int)DBA::count('contact', $condition);
if ($cursor !== -1) { if ($cursor !== -1) {
if ($cursor > 0) { if ($cursor > 0) {
@ -171,7 +171,7 @@ abstract class ContactEndpoint extends BaseApi
// Cursor is on the user-specific contact id since it's the sort field // Cursor is on the user-specific contact id since it's the sort field
if (count($ids)) { if (count($ids)) {
$previous_cursor = -$ids[0]; $previous_cursor = -$ids[0];
$next_cursor = $ids[count($ids) -1]; $next_cursor = (int)$ids[count($ids) -1];
} }
// No next page // No next page

View File

@ -89,7 +89,7 @@ class User extends BaseEntity
*/ */
public function __construct(array $publicContact, array $apcontact = [], array $userContact = [], $skip_status = false, $include_user_entities = true) public function __construct(array $publicContact, array $apcontact = [], array $userContact = [], $skip_status = false, $include_user_entities = true)
{ {
$this->id = $publicContact['id']; $this->id = (int)$publicContact['id'];
$this->id_str = (string) $publicContact['id']; $this->id_str = (string) $publicContact['id'];
$this->name = $publicContact['name']; $this->name = $publicContact['name'];
$this->screen_name = $publicContact['nick'] ?: $publicContact['name']; $this->screen_name = $publicContact['nick'] ?: $publicContact['name'];
@ -143,10 +143,10 @@ class User extends BaseEntity
$this->notifications = false; $this->notifications = false;
// Friendica-specific // Friendica-specific
$this->uid = $userContact['uid'] ?? 0; $this->uid = (int)$userContact['uid'] ?? 0;
$this->cid = $userContact['id'] ?? 0; $this->cid = (int)$userContact['id'] ?? 0;
$this->pid = $publicContact['id']; $this->pid = (int)$publicContact['id'];
$this->self = $userContact['self'] ?? false; $this->self = (boolean)$userContact['self'] ?? false;
$this->network = $publicContact['network']; $this->network = $publicContact['network'];
$this->statusnet_profile_url = $publicContact['url']; $this->statusnet_profile_url = $publicContact['url'];
} }

View File

@ -234,7 +234,7 @@ class ContactEndpointTest extends FixtureTest
'uid' => 42, 'uid' => 42,
'cid' => 44, 'cid' => 44,
'pid' => 45, 'pid' => 45,
'self' => 0, 'self' => false,
'network' => 'dfrn', 'network' => 'dfrn',
'statusnet_profile_url' => 'http://localhost/profile/friendcontact', 'statusnet_profile_url' => 'http://localhost/profile/friendcontact',
]; ];