with a php version LESS than 8.1
install and configure joomla 4.4
set the update url to https://update.joomla.org/core/nightlies/next_major_list.xml
check for updates
something happens. Probably an error message about not having the required php version. OR more helpfully the preinstall check shows exactly what the problem is. Otherwise it does make the very first check redundant as you can only see that if you are already compliant
Generic we can not find an update url message. 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.
Labels |
Added:
No Code Attached Yet
|
Title |
|
There was an attempt to fix it but it was given up: #38799 .
The issue was reported in past with #38694 , which was closed when the PR mentioned above was created, but it was not reopened when that PR was closed.
As far as I can see, that PR would not have solved the issue when the current installation meets the requirements but the updated site would not, and it checked the requirements of the database driver, which are lower than those of the CMS.
Also DB version check would be good,
I was able to run update on MySQL 5.6 that should not be possible :)
(it is half failed but the site still works)
@Fedik Problem with DB version check is that currently we have the information about the DB requirement of the CMS only in the installation folder in the DatabaseHelper and in the XML files of the update server, but nowhere in a regular installation locally. The check with the minimum version required by the framework's database driver results in a lower requirement.
We probably can hardcode it somewhere, in update script/model/view or controller,
Not sure currently how it should be displayed and processed
For the PHP version it is hardcoded in the index.php files in the root, in administrator and in installation.
$this->checkMinRequirement();
uses the database driver's requirement, as far as I remember, and that's not the same as the CMS requirement.
Ah no, I mean a new method $this->blablaCheckMinRequirement()
, nothing from existing.
Where we can hardcode everything ?
Hm, but we already have it https://update.joomla.org/core/test/extension_test.xml
<supported_databases mysql="8.0.13" mariadb="10.4.0" postgresql="12.0"/>
It just not processed/validated, only check for type
Okay, so it seems just need one more method and option for the overview list :)
something happens. Probably an error message about not having the required php version.
@brianteeman I not sure now, where the issue is, when I example modify the requirement to php 9, then it shows me failed "check".
That what I would expect, maybe I missing something?
Hm, but we already have it https://update.joomla.org/core/test/extension_test.xml
<supported_databases mysql="8.0.13" mariadb="10.4.0" postgresql="12.0"/>
It just not processed/validated, only check for type
This helps only if the update site is reachable and an update would be found.
@richard67 here is for DB version check:
Ah, I understood now, the issue that it does not show anything usefull, and user may think that there is no update.
Labels |
Added:
bug
|
The issue that Joomla\CMS\Updater\Update
joomla-cms/libraries/src/Updater/Update.php
Lines 345 to 400 in 1e7527b
Ah, I understood now, the issue that it does not show anything usefull, and user may think that there is no update.
exactly
I have done what I can to raise this important issue and request that it is a release blocker. I can only do so much.
As far as I remember there was once in 4.x made an optimization for the Joomla Updates Quick Icon. Maybe that's the reason that it shows an available update when the Joomla Update component doesn't find one? The PHP and database requirements are in the details XML file of the update site. If the quickicon doesn't check the details xml but only the list XML (where we also have the targetplatformversion
but not the PHP and DB requirements), it will explain that behaviour.
There are 2 possible ways out without having to rewrite the complete updater library for that case:
So or so it is an old problem not introduced with 4.4 or 5.0 so I wouldn't say it's a release blocker.
We let the quick icon also check the details XML so it doesn't show the update when the PHP and DB requirements are not met.
That just moves the problem - user hears about a new version but their site makes no mention of the update and they dont know why
So or so it is an old problem not introduced with 4.4 or 5.0 so I wouldn't say it's a release blocker.
Of course its a new problem. We didnt have a higher version of php or joomla as a requirement before.
But it seems everyone thinks it is acceptable to hide the availability of an update without explaining why.
I think Quickicon should do the same as Component do. At current point: "show nothing" when compatible update did not found.
Then we should look how to improve it.
Maybe need some new mechanism (or changes to existing) to load update information, separately.
What Joomla\CMS\Updater\Update
is doing is extarcting download url for compatible update. (technicaly we can have diferent packges: only for Mysql, or only for PHP 5 etc)
It is not very usable for retrieve Update information.
Currently I have no idea how it can be done.
Of course its a new problem.
If I right remember, it was the same for 3 to 4 update.
If I right remember, it was the same for 3 to 4 update.
I remember the same.
I dont' I am sure it came in when the emptystate code was added in j4
but whichever is the case it doesnt resolve the problem
Is this an acceptable quick fix ??
I would think yes, but to be sure I'd like to see the code.
No code - just a string edit
- COM_JOOMLAUPDATE_NODOWNLOAD_EMPTYSTATE_TITLE="We can't find a download URL"
+ COM_JOOMLAUPDATE_NODOWNLOAD_EMPTYSTATE_TITLE="This site can not be updated to Joomla %1$s as this web server does not meet the minimum requirements."
Looks not bad.
We can't find a download URL
We can't find a download URL for your server environment
People not always read the description ;)
No code - just a string edit
I see. So it's shown always when no update found.
Not realy, it will be shown when Update available, but download URL is empty:
Ah.
Maybe the message could refer to the documentation? We could e.g. add to the end:
<br>Check <a class=\"alert-link\" href=\"https://downloads.joomla.org/technical-requirements\" target=\"_blank\" rel=\"noopener noreferrer\">the technical requirements</a> and switch to a suitable server, or check <a class=\"alert-link\" href=\"https://downloads.joomla.org/latest\" target=\"_blank\" rel=\"noopener noreferrer\">the official Joomla download page</a> for a Joomla <em><strong>Upgrade Package</strong></em> which works with your server.
agreed - it was just as poc
of course it needs to be a link to a page with actual content ;)
I will wait for a release lead approval before making a pr
of course it needs to be a link to a page with actual content ;)
@brianteeman Currently the page for which I suggested the link is empty, but it says it will be updated soon and it links to the developer docs with the right info, so I think we still can use that and not the developer docs link which we later might have to change back.
Can this issue be closed as we have a PR? OR is something remaining to be done?
i just hadnt pressed the button yet - you were too fast
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-10-09 18:13:53 |
Closed_By | ⇒ | brianteeman |
@brianteeman In section „Actual result“ you wrote:
I assume you meant „we can’t find”.