Add the primary font color to the capabilities as well
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
parent
95b7fa306e
commit
b7b4cfd6a1
|
@ -37,6 +37,9 @@ class Capabilities implements IPublicCapability {
|
|||
/** @var ThemingDefaults */
|
||||
protected $theming;
|
||||
|
||||
/** @var Util */
|
||||
protected $util;
|
||||
|
||||
/** @var IURLGenerator */
|
||||
protected $url;
|
||||
|
||||
|
@ -45,11 +48,13 @@ class Capabilities implements IPublicCapability {
|
|||
|
||||
/**
|
||||
* @param ThemingDefaults $theming
|
||||
* @param Util $util
|
||||
* @param IURLGenerator $url
|
||||
* @param IConfig $config
|
||||
*/
|
||||
public function __construct(ThemingDefaults $theming, IURLGenerator $url, IConfig $config) {
|
||||
public function __construct(ThemingDefaults $theming, Util $util, IURLGenerator $url, IConfig $config) {
|
||||
$this->theming = $theming;
|
||||
$this->util = $util;
|
||||
$this->url = $url;
|
||||
$this->config = $config;
|
||||
}
|
||||
|
@ -68,6 +73,7 @@ class Capabilities implements IPublicCapability {
|
|||
'url' => $this->theming->getBaseUrl(),
|
||||
'slogan' => $this->theming->getSlogan(),
|
||||
'color' => $this->theming->getColorPrimary(),
|
||||
'color-text' => $this->util->invertTextColor($this->theming->getColorPrimary()) ? '#000000' : '#FFFFFF',
|
||||
'logo' => $this->url->getAbsoluteURL($this->theming->getLogo()),
|
||||
'background' => $backgroundLogo === 'backgroundColor' ?
|
||||
$this->theming->getColorPrimary() :
|
||||
|
|
|
@ -23,6 +23,9 @@ namespace OCA\Theming\Tests;
|
|||
|
||||
use OCA\Theming\Capabilities;
|
||||
use OCA\Theming\ThemingDefaults;
|
||||
use OCA\Theming\Util;
|
||||
use OCP\App\IAppManager;
|
||||
use OCP\Files\IAppData;
|
||||
use OCP\IConfig;
|
||||
use OCP\IURLGenerator;
|
||||
use Test\TestCase;
|
||||
|
@ -52,34 +55,38 @@ class CapabilitiesTest extends TestCase {
|
|||
$this->theming = $this->createMock(ThemingDefaults::class);
|
||||
$this->url = $this->getMockBuilder(IURLGenerator::class)->getMock();
|
||||
$this->config = $this->createMock(IConfig::class);
|
||||
$this->capabilities = new Capabilities($this->theming, $this->url, $this->config);
|
||||
$util = new Util($this->config, $this->createMock(IAppManager::class), $this->createMock(IAppData::class));
|
||||
$this->capabilities = new Capabilities($this->theming, $util, $this->url, $this->config);
|
||||
}
|
||||
|
||||
public function dataGetCapabilities() {
|
||||
return [
|
||||
['name', 'url', 'slogan', 'color', 'logo', 'background', 'http://absolute/', [
|
||||
['name', 'url', 'slogan', '#FFFFFF', 'logo', 'background', 'http://absolute/', [
|
||||
'name' => 'name',
|
||||
'url' => 'url',
|
||||
'slogan' => 'slogan',
|
||||
'color' => 'color',
|
||||
'color' => '#FFFFFF',
|
||||
'color-text' => '#000000',
|
||||
'logo' => 'http://absolute/logo',
|
||||
'background' => 'http://absolute/background',
|
||||
]],
|
||||
['name1', 'url2', 'slogan3', 'color4', 'logo5', 'background6', 'http://localhost/', [
|
||||
['name1', 'url2', 'slogan3', '#01e4a0', 'logo5', 'background6', 'http://localhost/', [
|
||||
'name' => 'name1',
|
||||
'url' => 'url2',
|
||||
'slogan' => 'slogan3',
|
||||
'color' => 'color4',
|
||||
'color' => '#01e4a0',
|
||||
'color-text' => '#FFFFFF',
|
||||
'logo' => 'http://localhost/logo5',
|
||||
'background' => 'http://localhost/background6',
|
||||
]],
|
||||
['name1', 'url2', 'slogan3', 'color4', 'logo5', 'backgroundColor', 'http://localhost/', [
|
||||
['name1', 'url2', 'slogan3', '#000000', 'logo5', 'backgroundColor', 'http://localhost/', [
|
||||
'name' => 'name1',
|
||||
'url' => 'url2',
|
||||
'slogan' => 'slogan3',
|
||||
'color' => 'color4',
|
||||
'color' => '#000000',
|
||||
'color-text' => '#FFFFFF',
|
||||
'logo' => 'http://localhost/logo5',
|
||||
'background' => 'color4',
|
||||
'background' => '#000000',
|
||||
]],
|
||||
];
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue