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]: index.php/settings/apps incorrectly shows "All apps are up-to-date." while there is 1 not up-to-date #31915

Closed
6 of 8 tasks
march42 opened this issue Apr 9, 2022 · 8 comments · Fixed by #32114
Closed
6 of 8 tasks
Labels
2. developing Work in progress bug

Comments

@march42
Copy link
Contributor

march42 commented Apr 9, 2022

⚠️ 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.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

The displayed status for available updates is wrong under some conditions.

  • Under "Your apps" and "Updates" shows "All apps are up-to-date." when there is one update available.
  • Under "Active apps" shows "All apps are up-to-date." when one and two updates (maybe more) are available.

Steps to reproduce

  1. Nextcloud v23.0.3 and Nextcloud 24.0.0 beta 3
  2. 1 or more updates for apps available
  3. check index.php/settings/apps etc. for the status line above the apps list

(Your apps, 1 update, 24.0.0 beta 3)
2022-04-08 (1)
(Your apps, 1 update, 23.0.3)
2022-04-08 (2)
(Active apps, 2 updates)
2022-04-09 (1)

Expected behavior

Displaying the actual count of available updates.
2022-04-09

Installation method

Manual installation

Operating system

Debian/Ubuntu

PHP engine version

PHP 7.4

Web server

Apache (supported)

Database engine version

MariaDB

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

{
    "system": {
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
            "localhost",
            "frija.kleines-miststueck.de",
            "nc.march42.net",
            "march42.net",
            "kleines-miststueck.de",
            "enyo.kleines-miststueck.de"
        ],
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "mysql",
        "version": "23.0.3.2",
        "overwrite.cli.url": "http:\/\/localhost",
        "dbname": "***REMOVED SENSITIVE VALUE***",
        "dbhost": "***REMOVED SENSITIVE VALUE***",
        "dbport": "",
        "dbtableprefix": "oc_",
        "mysql.utf8mb4": true,
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "installed": true,
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpmode": "smtp",
        "mail_sendmailmode": "pipe",
        "memcache.local": "\\OC\\Memcache\\APCu",
        "default_language": "de",
        "default_locale": "de_DE",
        "default_phone_region": "DE",
        "logtimezone": "Europe\/Berlin",
        "mail_domain": "***REMOVED SENSITIVE VALUE***",
        "mail_smtphost": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpport": "587",
        "mail_smtpsecure": "tls",
        "mail_smtpauth": "true",
        "mail_smtpauthtype": "LOGIN",
        "mail_smtpname": "***REMOVED SENSITIVE VALUE***",
        "mail_smtppassword": "***REMOVED SENSITIVE VALUE***",
        "ldapProviderFactory": "OCA\\User_LDAP\\LDAPProviderFactory",
        "maintenance": false,
        "redis": {
            "host": "***REMOVED SENSITIVE VALUE***",
            "timeout": 2,
            "dbindex": 2
        },
        "memcache.distributed": "\\OC\\Memcache\\Redis",
        "memcache.locking": "\\OC\\Memcache\\Redis",
        "preview_max_x": 600,
        "preview_max_y": 600,
        "enabledPreviewProviders": [
            "OC\\Preview\\Image",
            "OC\\Preview\\TXT",
            "OC\\Preview\\Markdown",
            "OC\\Preview\\SVG",
            "OC\\Preview\\TIFF",
            "OC\\Preview\\Photoshop",
            "OC\\Preview\\PDF"
        ],
        "enable_previews": true,
        "filelocking.enabled": true
    }
}

List of activated Apps

Enabled:
  - activity: 2.15.0
  - backup: 1.0.6
  - calendar: 3.2.2
  - circles: 23.1.0
  - cloud_federation_api: 1.6.0
  - contacts: 4.1.0
  - contactsinteraction: 1.4.0
  - dav: 1.21.0
  - deck: 1.6.1
  - duplicatefinder: 0.0.13
  - event_update_notification: 1.5.0
  - external: 3.10.2
  - federatedfilesharing: 1.13.0
  - federation: 1.13.0
  - files: 1.18.0
  - files_external: 1.15.0
  - files_pdfviewer: 2.4.0
  - files_sharing: 1.15.0
  - files_trashbin: 1.13.0
  - files_versions: 1.16.0
  - files_videoplayer: 1.12.0
  - fileslibreofficeedit: 1.0.4
  - groupfolders: 11.1.2
  - logreader: 2.8.0
  - lookup_server_connector: 1.11.0
  - mail: 1.11.7
  - metadata: 0.15.0
  - notes: 4.3.1
  - notifications: 2.11.1
  - oauth2: 1.11.0
  - password_policy: 1.13.0
  - photos: 1.5.0
  - previewgenerator: 4.0.0
  - privacy: 1.7.0
  - provisioning_api: 1.13.0
  - serverinfo: 1.13.0
  - settings: 1.5.0
  - sharebymail: 1.13.0
  - spreed: 13.0.5
  - systemtags: 1.13.0
  - text: 3.4.1
  - theming: 1.14.0
  - transfer: 0.2.1
  - twofactor_backupcodes: 1.12.0
  - updatenotification: 1.13.0
  - user_ldap: 1.13.1
  - user_status: 1.3.1
  - viewer: 1.7.0
  - workflowengine: 2.5.0
Disabled:
  - accessibility: 1.9.0
  - admin_audit
  - comments: 1.13.0
  - dashboard: 7.3.0
  - encryption
  - files_antivirus: 3.2.2
  - files_rightclick: 1.2.0
  - firstrunwizard: 2.12.0
  - gpgmailer: 0.0.4
  - nextcloud_announcements: 1.12.0
  - recommendations: 1.2.0
  - richdocuments: 5.0.3
  - richdocumentscode: 21.11.204
  - support: 1.6.0
  - survey_client: 1.11.0
  - weather_status: 1.3.0

Nextcloud Signing status

No errors have been found.

Nextcloud Logs

{"reqId":"MQVHJoaOfUlBNJuttNuN","level":3,"time":"2022-04-08T23:22:15+02:00","remoteAddr":"","user":"--","app":"core","method":"","url":"--","message":"OC_Image::fixOrientation(): No image loaded","userAgent":"--","version":"23.0.3.2"}
{"reqId":"MQVHJoaOfUlBNJuttNuN","level":3,"time":"2022-04-08T23:22:18+02:00","remoteAddr":"","user":"--","app":"core","method":"","url":"--","message":"OC_Image::fixOrientation(): No image loaded","userAgent":"--","version":"23.0.3.2"}

Additional info

No response

@march42 march42 added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Apr 9, 2022
@JordanDIG
Copy link

I also have the same problem on the 2 instances in version 23.0.3 and nextcloud 24 in beta 2

@tflidd
Copy link
Contributor

tflidd commented Apr 25, 2022

I can confirm the issue for NC24RC1.

On NC 22.2.6, I don't have the problem on a similar setup.

@szaimen
Copy link
Contributor

szaimen commented Apr 25, 2022

When this is happening since 23.0.3 too it must be something that was backported to 23...
#31504
#31591
#31645

@szaimen
Copy link
Contributor

szaimen commented Apr 25, 2022

Question: does it show the update all button when 2 or 3 apps have updates available?

@szaimen
Copy link
Contributor

szaimen commented Apr 25, 2022

We might be able to fix it by changing this

return this.apps.filter(app => app.update).length > 1

to

return this.apps.filter(app => app.update).length > 0

@szaimen
Copy link
Contributor

szaimen commented Apr 25, 2022

cc @skjnldsv would the above be a good fix or is it not expected that the update all button shows up if only one app update is available?

@szaimen
Copy link
Contributor

szaimen commented Apr 25, 2022

we should probably also add enabled here:

return this.hasPendingUpdate && ['installed', 'updates'].includes(this.category)

@szaimen
Copy link
Contributor

szaimen commented Apr 25, 2022

fix in #32114

@szaimen szaimen added 2. developing Work in progress and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants