Replace remaining references to default banner image by api.mastodon_banner configuration value

- Ensure leading slash is present at every place the configuration value is used
This commit is contained in:
Hypolite Petovan 2023-08-17 07:35:19 -04:00
parent 5d7985dcff
commit dad3404ac5
5 changed files with 61 additions and 6 deletions

47
src/Contact/Header.php Normal file
View file

@ -0,0 +1,47 @@
<?php
/**
* @copyright Copyright (C) 2010-2023, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*
*/
namespace Friendica\Contact;
use Friendica\Core\Config\Capability\IManageConfigValues;
class Header
{
/** @var IManageConfigValues */
private $config;
public function __construct(IManageConfigValues $config)
{
$this->config = $config;
}
/**
* Returns the Mastodon banner path relative to the Friendica folder.
*
* Ensures the existence of a leading slash.
*
* @return string
*/
public function getMastodonBannerPath(): string
{
return '/' . ltrim($this->config->get('api', 'mastodon_banner'), '/');
}
}

View file

@ -22,6 +22,7 @@
namespace Friendica\Model; namespace Friendica\Model;
use Friendica\Contact\Avatar; use Friendica\Contact\Avatar;
use Friendica\Contact\Header;
use Friendica\Contact\Introduction\Exception\IntroductionNotFoundException; use Friendica\Contact\Introduction\Exception\IntroductionNotFoundException;
use Friendica\Contact\LocalRelationship\Entity\LocalRelationship; use Friendica\Contact\LocalRelationship\Entity\LocalRelationship;
use Friendica\Content\Conversation as ConversationContent; use Friendica\Content\Conversation as ConversationContent;
@ -1899,7 +1900,7 @@ class Contact
switch ($platform) { switch ($platform) {
case 'friendica': case 'friendica':
case 'friendika': case 'friendika':
$header = DI::baseUrl() . '/images/friendica-banner.jpg'; $header = DI::baseUrl() . (new Header(DI::config()))->getMastodonBannerPath();
break; break;
case 'diaspora': case 'diaspora':
/** /**

View file

@ -23,6 +23,7 @@ namespace Friendica\Module\Api\Mastodon;
use Exception; use Exception;
use Friendica\App; use Friendica\App;
use Friendica\Contact\Header;
use Friendica\Core\Config\Capability\IManageConfigValues; use Friendica\Core\Config\Capability\IManageConfigValues;
use Friendica\Core\L10n; use Friendica\Core\L10n;
use Friendica\Core\System; use Friendica\Core\System;
@ -49,6 +50,9 @@ class InstanceV2 extends BaseApi
/** @var IManageConfigValues */ /** @var IManageConfigValues */
private $config; private $config;
/** @var Header */
private $contactHeader;
public function __construct( public function __construct(
App $app, App $app,
L10n $l10n, L10n $l10n,
@ -64,8 +68,9 @@ class InstanceV2 extends BaseApi
) { ) {
parent::__construct($app, $l10n, $baseUrl, $args, $logger, $profiler, $response, $server, $parameters); parent::__construct($app, $l10n, $baseUrl, $args, $logger, $profiler, $response, $server, $parameters);
$this->database = $database; $this->database = $database;
$this->config = $config; $this->config = $config;
$this->contactHeader = new Header($config);
} }
/** /**
@ -82,7 +87,7 @@ class InstanceV2 extends BaseApi
$version = '2.8.0 (compatible; Friendica ' . App::VERSION . ')'; $version = '2.8.0 (compatible; Friendica ' . App::VERSION . ')';
$description = $this->config->get('config', 'info'); $description = $this->config->get('config', 'info');
$usage = $this->buildUsageInfo(); $usage = $this->buildUsageInfo();
$thumbnail = new InstanceEntity\Thumbnail($this->baseUrl->withPath('images/friendica-banner.jpg')); $thumbnail = new InstanceEntity\Thumbnail($this->baseUrl . $this->contactHeader->getMastodonBannerPath());
$languages = [$this->config->get('system', 'language')]; $languages = [$this->config->get('system', 'language')];
$configuration = $this->buildConfigurationInfo(); $configuration = $this->buildConfigurationInfo();
$registration = $this->buildRegistrationsInfo(); $registration = $this->buildRegistrationsInfo();

View file

@ -22,6 +22,7 @@
namespace Friendica\Module; namespace Friendica\Module;
use Friendica\BaseModule; use Friendica\BaseModule;
use Friendica\Contact\Header;
use Friendica\Core\Logger; use Friendica\Core\Logger;
use Friendica\Core\Protocol; use Friendica\Core\Protocol;
use Friendica\Database\DBA; use Friendica\Database\DBA;
@ -482,6 +483,6 @@ class Photo extends BaseApi
if (!empty($photo)) { if (!empty($photo)) {
return $photo; return $photo;
} }
return MPhoto::createPhotoForImageData(file_get_contents(DI::basePath() . '/images/friendica-banner.jpg')); return MPhoto::createPhotoForImageData(file_get_contents(DI::basePath() . (new Header(DI::config()))->getMastodonBannerPath()));
} }
} }

View file

@ -24,6 +24,7 @@ namespace Friendica\Object\Api\Mastodon;
use Friendica\App; use Friendica\App;
use Friendica\App\BaseURL; use Friendica\App\BaseURL;
use Friendica\BaseDataTransferObject; use Friendica\BaseDataTransferObject;
use Friendica\Contact\Header;
use Friendica\Core\Config\Capability\IManageConfigValues; use Friendica\Core\Config\Capability\IManageConfigValues;
use Friendica\Database\Database; use Friendica\Database\Database;
use Friendica\DI; use Friendica\DI;
@ -91,7 +92,7 @@ class Instance extends BaseDataTransferObject
$this->version = '2.8.0 (compatible; Friendica ' . App::VERSION . ')'; $this->version = '2.8.0 (compatible; Friendica ' . App::VERSION . ')';
$this->urls = ['streaming_api' => '']; // Not supported $this->urls = ['streaming_api' => '']; // Not supported
$this->stats = new Stats($config, $database); $this->stats = new Stats($config, $database);
$this->thumbnail = $baseUrl . $config->get('api', 'mastodon_banner'); $this->thumbnail = $baseUrl . (new Header($config))->getMastodonBannerPath();
$this->languages = [$config->get('system', 'language')]; $this->languages = [$config->get('system', 'language')];
$this->max_toot_chars = (int)$config->get('config', 'api_import_size', $config->get('config', 'max_import_size')); $this->max_toot_chars = (int)$config->get('config', 'api_import_size', $config->get('config', 'max_import_size'));
$this->registrations = ($register_policy != Register::CLOSED); $this->registrations = ($register_policy != Register::CLOSED);