Skip to content

Fix duplicate and inconsistent update messages in Upload & Update view #46338#47550

Closed
pratibha-builds wants to merge 811 commits intojoomla:5.4-devfrom
pratibha-builds:bug/46338-fix-update-message-duplication
Closed

Fix duplicate and inconsistent update messages in Upload & Update view #46338#47550
pratibha-builds wants to merge 811 commits intojoomla:5.4-devfrom
pratibha-builds:bug/46338-fix-update-message-duplication

Conversation

@pratibha-builds
Copy link
Copy Markdown

Pull Request resolves #46338

  • I read the Generative AI policy and my contribution is compatible with the policy and GNU/GPL 2 or later.

Before_Update.mp4

Summary of Changes

This PR refactors the rendering of update messages in the Upload & Update view.

Previously:

  • The generic update instruction and direct update package message were rendered in a way that caused duplication and inconsistent UI.
  • Messages could appear multiple times or in separate alert blocks, leading to confusion.

Changes made:

  • Consolidated both messages into a single alert container.
  • Always display the generic update instruction message.
  • Conditionally display the direct download message only when update data is available.
  • Removed redundant and duplicate rendering paths.

This improves UI clarity, avoids repetition, and maintains both functional use cases.


Testing Instructions

  1. Navigate to:
    Administrator → System → Joomla Update → Upload & Update

  2. Test two scenarios:

    Case 1: Update available

    • Ensure both messages are displayed inside a single alert box:
      • Generic instruction message
      • Direct update package message

    Case 2: No update available (simulated)

    • Temporarily set:
      $hasUpdate = false;
    • Verify that only the generic instruction message is displayed.
  3. Ensure:

    • No duplicate messages appear
    • Only one alert container is rendered
    • Layout remains clean and consistent

Actual result BEFORE applying this Pull Request

  • Multiple alert boxes were displayed
  • Duplicate or repeated messages appeared
  • UI was cluttered and confusing

Expected result AFTER applying this Pull Request

  • A single alert box is displayed
  • Messages are clearly structured and non-duplicated
  • Direct download message appears only when update data is available
  • Improved readability and user experience

Screenshots / Testing Evidence

Before Fix (Duplicate / inconsistent messages):
https://github.com/user-attachments/assets/9bf8500b-b59c-4ddf-bcd4-59802e0df3e7

After Fix (Single alert, clean UI, no duplication):

Screenshot 2026-04-06 122003

Bodge-IT and others added 30 commits January 8, 2026 07:58
* Update create-translation-pull-request-v6.yml
* Language update
* Use PAT for translation bot
Merge pull request #46640 from Bodge-IT/upmerges/2026-01-08
Broken articles table in backend fix
…ion for Category List (#46539)

Co-authored-by: Brian Teeman <brian@teeman.net>
Co-authored-by: Brian Teeman <brian@teeman.net>
Co-authored-by: Quy <quy@nomonkeybiz.com>
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
Co-authored-by: Quy <quy@nomonkeybiz.com>
Co-authored-by: Hannes Papenberg <info@joomlager.de>
Co-authored-by: Denitz <denitz@site.com>
Co-authored-by: Phil Walton <phil@softforge.co.uk>
Co-authored-by: Harald Leithner <leithner@itronic.at>
…iously made with PR 46646 (#46661)

* Composer update paragonie/sodium_compat to v1.24.0

* Composer update altcha-org/altcha to v1.3.1
* Update create-translation-pull-request-v6.yml
* Language updates
* Use PAT for translation bot
* fix build error
https://github.com/joomla-translation-bot/joomla-cms/actions/runs/20963499191/job/60247089852



---------

Co-authored-by: Gary Barclay <1881642+Bodge-IT@users.noreply.github.com>
Co-authored-by: Stefan Wendhausen <stefan.wendhausen@tec-promotion.de>
Co-authored-by: Gary Barclay <1881642+Bodge-IT@users.noreply.github.com>
…ies 2026-01-10 (#46663)

* NPM audit fix 2026-01-08

* NPM audit fix 2026-01-10
* Improve language caching error handling
* Code simplify
* Catch Log::add RuntimeException
@muhme
Copy link
Copy Markdown
Contributor

muhme commented Apr 9, 2026

Thank you @pratibha-builds for your contribution.

Please note:

  • The 5.4-dev branch is for bug fixes only; with refactoring the rendering of update messages this PR needs to be rebased to 6.2-dev
  • Removing the copyright notice is not allowed
  • Refactoring or bug fixing should only modify the relevant lines

Please rebase and update this PR, or it will be closed in two weeks.

@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@muhme muhme added the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Apr 10, 2026
@brianteeman
Copy link
Copy Markdown
Contributor

you have changed your base branch but you are still trying to submit a pr to 5.4

image

@pratibha-builds pratibha-builds force-pushed the bug/46338-fix-update-message-duplication branch from c12152c to c469db5 Compare April 10, 2026 09:57
@richard67
Copy link
Copy Markdown
Member

@pratibha-builds I just see you have opened a new PR for 6.2-dev which replaces this one here. Could you refer to this PR here in the description of the other PR, e.g. with Replaces #47550 , and close this PR here? Thanks in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Conflicting Files PR-5.4-dev Updates Requested Indicates that this pull request needs an update from the author and should not be tested.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Joomla upload and update messages