Don't send the header

This commit is contained in:
Michael 2022-12-12 03:13:24 +00:00
parent a2f386cd2f
commit 5312b23f03
4 changed files with 23 additions and 12 deletions

View File

@ -110,10 +110,10 @@ class Lists extends ContactEndpoint
$ids = array_reverse($ids); $ids = array_reverse($ids);
} }
self::setLinkHeader();
$return = self::list($ids, $total_count, $uid, $cursor, $count, $skip_status, $include_user_entities); $return = self::list($ids, $total_count, $uid, $cursor, $count, $skip_status, $include_user_entities);
$this->response->setHeader(self::getLinkHeader());
$this->response->exit('lists', ['lists' => $return]); $this->response->exit('lists', ['lists' => $return]);
} }
} }

View File

@ -77,7 +77,7 @@ class Lists extends ContactEndpoint
} }
} else { } else {
$params = ['order' => ['cid' => true], 'limit' => $count]; $params = ['order' => ['cid' => true], 'limit' => $count];
$condition = ['relation-cid' => $cid, 'follows' => true]; $condition = ['relation-cid' => $cid, 'follows' => true];
$total_count = (int)DBA::count('contact-relation', $condition); $total_count = (int)DBA::count('contact-relation', $condition);
@ -110,10 +110,10 @@ class Lists extends ContactEndpoint
$ids = array_reverse($ids); $ids = array_reverse($ids);
} }
self::setLinkHeader();
$return = self::list($ids, $total_count, $uid, $cursor, $count, $skip_status, $include_user_entities); $return = self::list($ids, $total_count, $uid, $cursor, $count, $skip_status, $include_user_entities);
$this->response->setHeader(self::getLinkHeader());
$this->response->exit('lists', ['lists' => $return]); $this->response->exit('lists', ['lists' => $return]);
} }
} }

View File

@ -21,7 +21,6 @@
namespace Friendica\Module\Api\Twitter\Friendships; namespace Friendica\Module\Api\Twitter\Friendships;
use Friendica\Core\System;
use Friendica\Database\DBA; use Friendica\Database\DBA;
use Friendica\Module\Api\Twitter\ContactEndpoint; use Friendica\Module\Api\Twitter\ContactEndpoint;
use Friendica\Module\BaseApi; use Friendica\Module\BaseApi;
@ -81,7 +80,7 @@ class Incoming extends ContactEndpoint
$return = self::ids($ids, $total_count, $cursor, $count, $stringify_ids); $return = self::ids($ids, $total_count, $cursor, $count, $stringify_ids);
self::setLinkHeader(); $this->response->setHeader(self::getLinkHeader());
$this->response->exit('incoming', ['incoming' => $return]); $this->response->exit('incoming', ['incoming' => $return]);
} }

View File

@ -139,13 +139,13 @@ class BaseApi extends BaseModule
} }
/** /**
* Set the "link" header with "next" and "prev" links * Get the "link" header with "next" and "prev" links
* @return void * @return string
*/ */
protected static function setLinkHeader() protected static function getLinkHeader(): string
{ {
if (empty(self::$boundaries)) { if (empty(self::$boundaries)) {
return; return '';
} }
$request = self::$request; $request = self::$request;
@ -164,7 +164,19 @@ class BaseApi extends BaseModule
$prev = $command . '?' . http_build_query($prev_request); $prev = $command . '?' . http_build_query($prev_request);
$next = $command . '?' . http_build_query($next_request); $next = $command . '?' . http_build_query($next_request);
header('Link: <' . $next . '>; rel="next", <' . $prev . '>; rel="prev"'); return 'Link: <' . $next . '>; rel="next", <' . $prev . '>; rel="prev"';
}
/**
* Set the "link" header with "next" and "prev" links
* @return void
*/
protected static function setLinkHeader()
{
$header = self::getLinkHeader();
if (!empty($header)) {
header($header);
}
} }
/** /**