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

Fix for MODX updates widget #16694

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Conversation

smg6511
Copy link
Collaborator

@smg6511 smg6511 commented Jan 24, 2025

What does it do?

Fixes error in array selection when only one update is available.

Why is it needed?

MODX update status is not showing properly (both when there is an update and when MODX is up to date) when the software update API only returns one option.

How to test

  1. Verify software status displays "Up to Date" on this dev branch
  2. Manually apply this change on the 3.0.x branch to verify the 3.1.0 update shows and is downloadable via the widget

Related issue(s)/PR(s)

None

Fix updates display when only a single modx update is available
@smg6511 smg6511 added the pr/review-needed Pull request requires review and testing. label Jan 24, 2025
@opengeek opengeek merged commit 42fe47a into modxcms:3.x Jan 24, 2025
7 checks passed
opengeek pushed a commit that referenced this pull request Jan 24, 2025
### What does it do?
Fixes error in array selection when only one update is available.

### Why is it needed?
MODX update status is not showing properly (both when there is an update
and when MODX _is_ up to date) when the software update API only returns
one option.

### How to test

1. Verify software status displays "Up to Date" on this dev branch
2. Manually apply this change on the 3.0.x branch to verify the 3.1.0
update shows and is downloadable via the widget

### Related issue(s)/PR(s)
None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/review-needed Pull request requires review and testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants