? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
23 Nov 2019

Summary of Changes

remove obsolete 3.x sql updates

Testing Instructions

code review

avatar alikon alikon - open - 23 Nov 2019
avatar alikon alikon - change - 23 Nov 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Nov 2019
Category SQL Administration com_admin
avatar brianteeman
brianteeman - comment - 23 Nov 2019

Surely if I upgrade from 3.7 directly to 4.0 they are needed

avatar alikon
alikon - comment - 23 Nov 2019

maybe i can be wrong but you cannot upgrade to 4 from anything else than 3.10

avatar infograf768
infograf768 - comment - 23 Nov 2019

I thought one had to first update to 3.10

avatar brianteeman
brianteeman - comment - 23 Nov 2019

But the schedule is to release both on the same day?

avatar infograf768
infograf768 - comment - 23 Nov 2019

It's imho just a matter of forcing joomalupdate to first propose update to 3.10 for any site and then propose 4.0 for updating.

avatar HLeithner
HLeithner - comment - 23 Nov 2019

@brianteeman you can only update from Joomla 3.10 to 4.0

avatar richard67
richard67 - comment - 23 Nov 2019

@HLeithner That's not really right: Currently you can update also from 3.9. As long as the 3.10-dev branch was not up to date with latest changes in staging, we always tested the update path with 3.9.x.

As far as I know it is like @infograf768 said: The update to 4.0 will just not be proposed for 3.9 or earlier.

But up to now I can see no code keeping someone from updating a 3.9 or earlier to a 4.0 alpha or nightly using the Upload&Update tab of the Joomla Update component.

avatar richard67
richard67 - comment - 23 Nov 2019

@wilsonge Regarding updating a pre-3.10 to 4.0 using the Upload&Update tab of the Joomla Update component: Is it planned to add some code in future to block this? If so, this PR here is absolutely ok. If not, it could be a problem because we have no control on which version will be updated. People could try to do that with a 3.5 maybe. Up to now, when we tested the update path we were using latest 3.9. For this scenario, this PR here would also be ok. Only for updating a pre-3.9 it would be a problem.

avatar HLeithner
HLeithner - comment - 23 Nov 2019

Joomla 3.10 should include an upgrade check for Joomla 4 (PHP version checking, mysql version checking) and so on. This also allows us some cleanups like removing the sql files from 10 years.

avatar richard67
richard67 - comment - 23 Nov 2019

@HLeithner If J 3.10 includes such check, it does not keep people from updating a pre-3.10 to a 4.0 using the Upload&Update tab of the Joomla Update component. It will need a check in 4.0 in the update component that the version to be updated is a 3.10 and nothing older.

avatar brianteeman
brianteeman - comment - 23 Nov 2019

Exactly

avatar mbabker
mbabker - comment - 23 Nov 2019

You can’t put code in the 4.0 update component either because that won’t run until the entire filesystem is updated. Not to mention there isn’t a good “what version am I upgrading from” check in that code.

Any limitation is artificial at best.

avatar infograf768
infograf768 - comment - 24 Nov 2019

Any limitation is artificial at best.

Then what do you suggest?

avatar mbabker
mbabker - comment - 24 Nov 2019

Nothing more than is already done. Limit when updates are shown through the update server configuration. If you’re trying to code against the “but what if someone on 3.6.3 tries to upload the 4.0 package and direct upgrade” scenario, it’s too late for that. There isn’t a pre-upgrade abort hook and any code in the new version runs after the filesystem is updated so the site is already for all intents and purposes at the new version you don’t want them upgrading to in the way they did.

Maybe someone with some foresight can rework things so that a pre-upgrade abort is possible but it is not usable for any version already released.

avatar richard67
richard67 - comment - 24 Nov 2019

So this PR here should be ok, right?

avatar infograf768
infograf768 - comment - 24 Nov 2019

Or not.

avatar wilsonge
wilsonge - comment - 25 Nov 2019

@zero-24 before this is merged I'd like to remove 3.9.x as an available update path for 4.x

We've always said that we're only going to allow updates from the latest 3.x (originally 3.9 and then after the privacy changes 3.10) to 4.x. I don't see this as strictly different to how we've managed this in the past. We've always said for example from 1.7 you have to go to 2.5 to go to 3.x otherwise bad things can/will happen and of course that doesn't stop people from doing differently. We will definitely not allow this on the update site side (4.x will only show to sites on 4.x).

Yes people can shortcut this - but so can they by overriding files of Joomla 3.1 - and from there you can currently only get to 3.5 through the normal autoupdate process and no one screamed at that.

TL/DR Yes this PR is fine

avatar zero-24
zero-24 - comment - 25 Nov 2019
avatar wilsonge wilsonge - change - 25 Nov 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-11-25 23:02:40
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 25 Nov 2019
avatar wilsonge wilsonge - merge - 25 Nov 2019
avatar wilsonge
wilsonge - comment - 25 Nov 2019

Thanks!

Add a Comment

Login with GitHub to post a comment