?
avatar mbabker
mbabker
22 May 2018

This is moreso a reminder for myself than anything.

Today I found a directory that shipped in 3.0 and was moved in 3.1 was never added to the deleted files array in the update script so those files were never removed from those sites.

Validate the array is in expected state and fix it if not (use the script in build directory for this).

avatar mbabker mbabker - open - 22 May 2018
avatar joomla-cms-bot joomla-cms-bot - change - 22 May 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 22 May 2018
avatar brianteeman
brianteeman - comment - 22 May 2018

Pretty sure that @AlexRed reviewed this not too long ago but only starting at about 3.4

avatar mbabker
mbabker - comment - 22 May 2018

Well, clearly that wasn't enough then 😉

With https://github.com/joomla/joomla-cms/blob/staging/build/deleted_file_check.php it's actually much simpler to do now than when he did it (I only did that script fairly recently to make sure all the 3.8 stuff was right with the namespacing and moving 300 files).

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 May 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 22 May 2018
Category com_joomlaupdate
avatar brianteeman
brianteeman - comment - 22 May 2018

so are we ok now?

avatar mbabker
mbabker - comment - 22 May 2018

At least for that batch of files I added just before release today, yes. It'd still be good to run that script in batches and make sure what it spits out (which is the result of git diff and tracking renames and deletes) matches what we have in the array for each branch (for the most part if someone other than me runs it you should be able to paste the output directly into the array and things just work, you just need to make sure deleted folders are added to the second list in the end), just to make sure we don't run into anymore potential "this doesn't work because I'm using the files from 3.0.3 while I'm on 3.8.7" problems.

avatar mbabker mbabker - change - 24 May 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-05-24 01:13:50
Closed_By mbabker
avatar mbabker
mbabker - comment - 24 May 2018

PR #20564

avatar mbabker mbabker - close - 24 May 2018

Add a Comment

Login with GitHub to post a comment