Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Update to Nextcloud 25.0.3 doesn't migrate theming images #36206

Closed
6 of 9 tasks
DATA-HORIZON opened this issue Jan 18, 2023 · 11 comments
Closed
6 of 9 tasks

[Bug]: Update to Nextcloud 25.0.3 doesn't migrate theming images #36206

DATA-HORIZON opened this issue Jan 18, 2023 · 11 comments
Assignees
Labels
1. to develop Accepted and waiting to be taken care of 25-feedback bug

Comments

@DATA-HORIZON
Copy link

DATA-HORIZON commented Jan 18, 2023

⚠️ 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 (I've searched it).
  • Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
  • Nextcloud Server is running on 64bit capable CPU, PHP and OS.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

After updating several Nextcloud instances from Nextcloud 24.0.8 to 25.0.2 and 24.0.9 to 25.0.3 we are facing always the same problem: all images under theming are removed from the file system (but not from the database). As a result the design is broken and 404's are thrown for the images lost.

Steps to reproduce

  1. add images to the theming app in your Nextcloud 24.0.9 ("Logo" / "Background and login image" / "Header logo" / "Favicon")
  2. update your Nextcloud to version 25.0.3
  3. enjoy the broken design

Expected behavior

Images of the theming app should not get "lost" when upgrading to Nextcloud 25.0.3.

After investigating the issue we could determine the following work-around:

  • the problem is the new directory structure of the theming app:
  • from a backup before the upgrade copy the images of the folder...
    "data/appdata_**************/theming/*"
  • ...to the new subdirectory "global/images" that is created on the update process "data/appdata_**************/theming/global/images"
  • from a backup before the upgrade copy the icon/favicon directory of the folder...
    "data/appdata_**************/theming/[digits]"
  • ...to the new subdirectory "global" that is created on the update process
    "data/appdata_**************/theming/global/[digits]"

Installation method

Community Manual installation with Archive

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.0

Web server

Apache (supported)

Database engine version

MariaDB

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

Updated to a major version (ex. 22.2.3 to 23.0.1)

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

No response

List of activated Apps

Enabled:
  - activity: 2.17.0
  - audioplayer: 3.3.1
  - circles: 25.0.0
  - cloud_federation_api: 1.8.0
  - contactsinteraction: 1.6.0
  - dashboard: 7.5.0
  - dav: 1.24.0
  - federatedfilesharing: 1.15.0
  - files: 1.20.1
  - files_pdfviewer: 2.6.0
  - files_rightclick: 1.4.0
  - files_sharing: 1.17.0
  - files_trashbin: 1.15.0
  - files_versions: 1.18.0
  - firstrunwizard: 2.14.0
  - logreader: 2.10.0
  - lookup_server_connector: 1.13.0
  - nextcloud_announcements: 1.14.0
  - notes: 4.6.0
  - notifications: 2.13.1
  - oauth2: 1.13.0
  - onlyoffice: 7.6.8
  - password_policy: 1.15.0
  - photos: 2.0.1
  - previewgenerator: 5.1.1
  - provisioning_api: 1.15.0
  - quota_warning: 1.15.0
  - related_resources: 1.0.4
  - serverinfo: 1.15.0
  - settings: 1.7.0
  - sharebymail: 1.15.0
  - text: 3.6.0
  - theming: 2.0.1
  - twofactor_backupcodes: 1.14.0
  - user_status: 1.5.0
  - viewer: 1.9.0
  - weather_status: 1.5.0
  - workflowengine: 2.7.0
Disabled:
  - admin_audit
  - bruteforcesettings: 2.4.0
  - camerarawpreviews: 0.8.0
  - comments: 1.8.0
  - encryption
  - federation: 1.8.0
  - files_external: 1.9.0
  - privacy: 1.0.0
  - recommendations: 0.6.0
  - support: 1.1.0
  - survey_client: 1.6.0
  - suspicious_login
  - systemtags: 1.8.0
  - twofactor_totp
  - updatenotification: 1.11.0
  - user_ldap
  - video_converter: 1.0.5

Nextcloud Signing status

No errors have been found.

Nextcloud Logs

No response

Additional info

No response

@DATA-HORIZON DATA-HORIZON added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Jan 18, 2023
@szaimen
Copy link
Contributor

szaimen commented Jan 18, 2023

From which version did you upgrade?

From 24.0.x to 25.0.3?

@szaimen

This comment was marked as resolved.

@OlliZabal

This comment was marked as resolved.

@andy-smith-work

This comment was marked as resolved.

@thomas-svw

This comment was marked as off-topic.

@jsalatiel

This comment was marked as off-topic.

1 similar comment
@Bouli911

This comment was marked as off-topic.

@DATA-HORIZON
Copy link
Author

DATA-HORIZON commented Jan 19, 2023

This bug report is not about the "Header Logo is ignored". Maybe you should file another issue for that.
Here the problem is the major upgrade from Nextcloud 24.0.8 to 25.0.2 or 24.0.9 to 25.0.3 that leads to the loss of all theming images.

@skjnldsv
Copy link
Member

I had the same logo & image issue after upgrading from v24.0.9 to 25.0.3. I re-uploaded my images in Settings > Theming. The Logo, Background & Login Image, and Favicon are good after re-uploading the files, but the Header Logo is ignored (the Logo is used instead).

This has been fixed already in upcoming 25.0.4.
Sorry for the troubles 😖

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 19, 2023
@PVince81
Copy link
Member

Ref: this is the PR that added the scope: #34599

there were some additional fixes but they were related to migrating user backgrounds from the dashboard app to the user based theme

@szaimen
Copy link
Contributor

szaimen commented Jan 20, 2023

Will be fixed with 25.0.4 when migrating from 24 to 25. For everyone that already ran into this and didnt revert to 24: you need to manually re-add the assets in the admin theming settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of 25-feedback bug
Projects
None yet
Development

No branches or pull requests

9 participants