? ? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
30 Aug 2019

Pull Request for Issue #26078

Summary of Changes

Make sure mariadb versions are correctly identified by the updater

Testing Instructions

Expected result

pre update checker and install of the update is proposed

Actual result

image

Documentation Changes Required

In order to make this work this here implements a new version detection for mariadb and also requires mariadb to be specified in the XML. This new behavior needs to be documented.

<supported_databases mysql="5.6" mariadb="10.4" postgresql="11.0"/>
avatar zero-24 zero-24 - open - 30 Aug 2019
avatar zero-24 zero-24 - change - 30 Aug 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Aug 2019
Category Libraries
avatar richard67
richard67 - comment - 30 Aug 2019

@zero-24

<supported_databases mysql="5.6" mariadb="10.4" postgresql="11.0"/>

Shall it define the supported versions, like the name of the item suggests, e.g. with a regex mariadb="10.[456]", or shall it be the minimum supported version for each DB? If the latter, then the name "supported_databases" might be a bit misleading.

P.S.: Is just a question for my understanding, not that I require any change. I can't test it anyway because I don't have MariaDB and for installing one it needs to kill my MySQL Db first, which I would like to avoid.

avatar richard67
richard67 - comment - 30 Aug 2019

@zero-24 Forget my question in my previous comment. I've found it in the docs.

avatar zero-24 zero-24 - change - 3 Sep 2019
Labels Added: ? ?
avatar richard67
richard67 - comment - 4 Sep 2019

@zero-24 My comment here #26104 (comment) applies also to this PR here. Shall I provide fixes for both PRs with making PRs to your repo?

avatar zero-24
zero-24 - comment - 4 Sep 2019

Merged thanks @richard67 ?

avatar richard67
richard67 - comment - 4 Sep 2019

Unfortunately I don't use MariaDB, sounds too catholic to me, am protestant ;-) So I can't help with a test.

avatar fancyFranci
fancyFranci - comment - 5 Sep 2019

I tested this with 10.3.16-MariaDB and the updater works again, after adding the patch.
But my MariaDB version is not one of the supported databases. Is this intended?

Btw System Information displays before and after the patch:
Database Type: mysql
Database Version: 5.5.5-10.3.16-MariaDB


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

avatar zero-24
zero-24 - comment - 5 Sep 2019

I tested this with 10.3.16-MariaDB and the updater works again, after adding the patch.
But my MariaDB version is not one of the supported databases. Is this intended?

It is more an minimum_databases tag ;) As the updates points to 10.1 you are good to go with 10.3.16

Btw System Information displays before and after the patch:

Yes this is intended as 5.5.5.XX.XX.XX is the actual version that the database returns.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Sep 2019
avatar fancyFranci fancyFranci - test_item - 5 Sep 2019 - Tested successfully
avatar fancyFranci
fancyFranci - comment - 5 Sep 2019

I have tested this item successfully on 193cc4f

Catholic enough


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

avatar fancyFranci
fancyFranci - comment - 5 Sep 2019

I tested this with 10.3.16-MariaDB and the updater works again, after adding the patch.
But my MariaDB version is not one of the supported databases. Is this intended?

It is more an minimum_databases tag ;) As the updates points to 10.1 you are good to go with 10.3.16

This is the point that confuses me, as 10.3. is lower than the minimum 10.4., isn't it? :)

avatar roland-d roland-d - test_item - 5 Sep 2019 - Tested successfully
avatar roland-d
roland-d - comment - 5 Sep 2019

I have tested this item successfully on 193cc4f

Before applying the patch Joomla said that the update was not available, after applying the patch the update of Joomla 4 shows up.


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

avatar zero-24
zero-24 - comment - 5 Sep 2019

This is the point that confuses me, as 10.3. is lower than the minimum 10.4., isn't it? :)

The minimum configured on the update server right now is:

<supported_databases mysql="5.6" mariadb="10.0" postgresql="11.0"/>

So anything above that should pass using the testing update server provided above. This is not an official statement on what would be the actual minimum for sure. ;)

avatar zero-24
zero-24 - comment - 5 Sep 2019

Ah ok i now got where the confusion is comming from @fancyFranci it is my fault :D

The update server itself points to mariadb 10.0 in my description here i used 10.4 bis this is not the version set in the xml and should have only represent an example how the xml could look like.

So in conclusion everything works as expected :)

avatar roland-d
roland-d - comment - 5 Sep 2019

The PHP version seems incorrect:
image

This should say PHP Version >= 7.2.0

avatar zero-24
zero-24 - comment - 5 Sep 2019

Yes and no :D You are correct this should be the case for 4.0 stable but as of now the update server in this example still points to

<php_minimum>7.0.0</php_minimum>

https://www.jah-tz.de/downloads/core/nightlies/next_major_extension.xml

I'm not at my desk so i can not update it yet but it is totally working than expected for now :)

avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Sep 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Sep 2019

Status "Ready To Commit".

avatar HLeithner HLeithner - change - 5 Sep 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-09-05 12:23:47
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 5 Sep 2019
avatar HLeithner HLeithner - merge - 5 Sep 2019
avatar HLeithner
HLeithner - comment - 5 Sep 2019

Thanks for making J3 a bit more mariaDB aware.

avatar zero-24
zero-24 - comment - 13 Sep 2019

Create an PR against the updater repo to ship the mariadb tag in the xmls

Based on: joomla/update.joomla.org#145 (comment) mariadb minimum is going to be 10,1 for now. Here is the update server PR to do the required change: joomla/update.joomla.org#146 cc @HLeithner @wilsonge

Update docs https://docs.joomla.org/Deploying_an_Update_Server

Added a note about the case implemented here.

Add a Comment

Login with GitHub to post a comment