Skip to content

Comments

[4.4] Noupdate available to joomla 5#42103

Merged
MacJoom merged 5 commits intojoomla:4.4-devfrom
brianteeman:noupdate
Oct 13, 2023
Merged

[4.4] Noupdate available to joomla 5#42103
MacJoom merged 5 commits intojoomla:4.4-devfrom
brianteeman:noupdate

Conversation

@brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Oct 9, 2023

Pull Request for Issue #42034 .

Summary of Changes

If your joomla 4.4 web site is on a server with a php or db version below the minimum requirements for Joomla 5 then we do NOT offer an update.

Testing Instructions

Actual result BEFORE applying this Pull Request

No mention at all that its because the site doesnt match the requirements for 5.0. Especially confusing because the dashboard says a new version is available.
image

Expected result AFTER applying this Pull Request

Clear explanation that you cannot update because of server requirements. Advice to contact your host and link to the requirments

image

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.4-dev labels Oct 9, 2023
@brianteeman brianteeman changed the title Noupdate [4.4] Noupdate available to joomla 5 Oct 9, 2023
@richard67 richard67 added the bug label Oct 9, 2023
@Quy
Copy link
Contributor

Quy commented Oct 9, 2023

Warning: Undefined property: Joomla\Component\Joomlaupdate\Administrator\View\Joomlaupdate\HtmlView::$reasonNoDownload in \administrator\components\com_joomlaupdate\tmpl\joomlaupdate\noupdate.php on line 23

@toivo
Copy link
Contributor

toivo commented Oct 10, 2023

I have tested this item ✅ successfully on 815e885

Tested successfully in Joomla 4.4.0-rc2-dev of 9 October and Wampserver 3.3.1 with PHP 8.0.26 and 8.2.11.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42103.

@richard67
Copy link
Member

I have tested this item ✅ successfully on 815e885

I've tested with an own update site on which I've adjusted the requirements.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42103.

@richard67
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42103.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 10, 2023
@brianteeman
Copy link
Contributor Author

What is preventing this from being merged? Without it the upgrade experience is attrocious

@richard67
Copy link
Member

What is preventing this from being merged? Without it the upgrade experience is attrocious

@brianteeman I don't know. It's a decision by the release managers.

@MacJoom
Copy link
Contributor

MacJoom commented Oct 12, 2023

Thanks - we are looking into it

Copy link
Contributor

@tecpromotion tecpromotion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks all

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants