updating to 28.0.10 Repair warning: Failed to read dimensions from logo

[partofac]

update with occ in upgrade phase
getting error

Repair warning: Failed to read dimensions from logo

i am using my own design

this happens since 28.0.8 i think.

[j-ed]

@partofac

i am using my own design

Wouldn’t it make sense to provide DETAILED information about your own design, about the logo you are using, which format it has and which dimension etc., etc.?

[partofac]

I use my own theme which I adopted from the example theme of nextcloud.
My logo is type of svg and has a size of 7kB and 1024/488px.
As I mentioned, this warning came up first with nc vers. 28.0.8

[partofac]

to correct: the logo size in server.css is w/h 256/128px

[j-ed]

I’m personally not using the Theming app, but as I searched for the reported message I found a match in:

<Nextcloud-webroot-folder>/lib/private/Repair/RepairLogoDimension.php

Here you find the following code snippets:

    public function run(IOutput $output): void {
        $logoDimensions = $this->config->getAppValue('theming', 'logoDimensions');
        if (preg_match('/^\d+x\d+$/', $logoDimensions)) {
            $output->info('Logo dimensions are already known');
            return;

...

        if (!$dimensions) {
            $output->warning('Failed to read dimensions from logo');
            $this->config->deleteAppValue('theming', 'logoDimensions');
            return;
        }

For me it looks like the app is looking for a configuration parameter which cannot be read. You should check the app settings using the occ command, like e.g. occ config:app:get theming. Can you find a parameter called “logoDimensions”?

https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/theming.html#theming

[partofac]

when you set in config.php ‘theme’ => ‘mytheme’, the theming app is not used.

there was no param ‘logoDimensions’ in the database.
I inserted it manualy with the value of ‘256 x 128’.
I have not changed anything in the theming in the /themes/mytheme directory. it works since 2016.

lets see, if this helps?
with which version of nextcloud has the var $logoDimensions been introduced?