[Bug]: Disabling User Theming through OCC is Broken

[i2h3]

⚠️ This issue respects the following points: ⚠️

• This is a bug, not a question or a configuration/webserver/proxy issue.
• This issue is not already reported on Github OR Nextcloud Community Forum (I've searched it).
• Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
• I agree to follow Nextcloud's Code of Conduct.

Bug description

The special OCC theming command to disable user theming does not accept any value due to a type conflict.

Steps to reproduce

Run this in your shell as in the server documentation:

./occ theming:config disable-user-theming no

See this error message:

In AppConfig.php line 482:
                                          
  conflict with value type from database  
                                          

theming:config [-r|--reset] [--] [<key> [<value>]]

This also fails with other attempted values like false or 0.

Expected behavior

disable-user-theming in oc_appconfig is set to 0.

Nextcloud Server version

31

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

Fresh Nextcloud Server install

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

• Default user-backend (database)
• LDAP/ Active Directory
• SSO - SAML
• Other

Configuration report

Irrelevant. Just take the Nextcloud Docker image (31.0.9 or latest for 32).

List of activated Apps

Irrelevant. Just take the Nextcloud Docker image (31.0.9 or latest for 32).

Nextcloud Signing status

Irrelevant. Just take the Nextcloud Docker image (31.0.9 or latest for 32).

Nextcloud Logs

Irrelevant. Just take the Nextcloud Docker image (31.0.9 or latest for 32).

Additional info

For anyone reading along, coming by or being stuck on an older version, this more general command works fine:

./occ config:app:set theming disable-user-theming --value 0

[joshtrichards]

#56016 / #56039

Though I'm not sure if the CLI fully supports "no" at the moment even after the IAppConfig swap:

server/apps/theming/lib/ThemingDefaults.php

Lines 438 to 441 in 76d878e

	case ConfigLexicon::USER_THEMING_DISABLED:
	$value = $value === 'true' || $value === 'yes' || $value === '1';
	$this->appConfig->setAppValueBool(ConfigLexicon::USER_THEMING_DISABLED, $value);
	break;

[i2h3]

@joshtrichards It does not work regardless of the argument value.

[joshtrichards]

Sorry I should have been more clear; it was just ported to IAppConfig in #56039 so only the current master branch and, now after a recent backport, the v32.0.2 maintenance release has the fix.

Just take the Nextcloud Docker image (31.0.9 or latest for 32).

The latest tag for the image yesterday would still have been v32.0.1.

[icewind1991]

This is still an issue on 32.0.2

#56626 has the proper fix