? ? ? Pending

User tests: Successful: 2 ChristineWk, alikon Unsuccessful: 0

avatar brianteeman
brianteeman
3 Oct 2019

Currently the button is called Fix and based on the number of times I see people thinking they need to do this all the time and it will magically fix things this PR changes the name to Update Structure

Hopefully it will reduce the number of people who think its a solution to every possible bug

image

avatar brianteeman brianteeman - open - 3 Oct 2019
avatar brianteeman brianteeman - change - 3 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Oct 2019
Category Administration Language & Strings
avatar brianteeman brianteeman - change - 3 Oct 2019
Labels Added: ? ?
avatar ChristineWk ChristineWk - test_item - 3 Oct 2019 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 3 Oct 2019

I have tested this item successfully on 6690b24


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

avatar alikon alikon - test_item - 3 Oct 2019 - Tested successfully
avatar alikon
alikon - comment - 3 Oct 2019

I have tested this item successfully on 6690b24


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

avatar alikon alikon - change - 3 Oct 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 3 Oct 2019

RTC


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

avatar ChristineWk
ChristineWk - comment - 3 Oct 2019

@brianteeman
..."Hopefully it will reduce the number of people who think its a solution to every possible bug"

of course :-) OT: That's why I'm checking before:
database
sorry, I didn't find the image button.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26458.
avatar wilsonge
wilsonge - comment - 3 Oct 2019

So the fun part is it does more than db structure :D it also deals with deleting unused files (also normally managed by joomla update and even a piece of 2.5 code migration - that can probably be removed than J4 https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_installer/Model/DatabaseModel.php#L268-L270)

Although in principal this should be separate of course. Some of this stuff is why we need a status page rather than this magic button

avatar brianteeman
brianteeman - comment - 3 Oct 2019

Stage 1 is to rename the button - this PR
Stage 2 is separate the non-database code
Stage 3 create the status page

avatar wilsonge
wilsonge - comment - 3 Oct 2019

Please make sure at least part 2 comes in the not too distant. else we'll need to revert it before going stable if it's still doing non database things

avatar wilsonge wilsonge - change - 3 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-03 15:10:56
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 3 Oct 2019
avatar wilsonge wilsonge - merge - 3 Oct 2019
avatar wilsonge
wilsonge - comment - 3 Oct 2019

Thanks!

Add a Comment

Login with GitHub to post a comment