?
avatar PhilETaylor
PhilETaylor
22 May 2021

Steps to reproduce the issue

Has something just broke this... I swear I have now deleted and reinstalled, and recompiled the JS and CSS many times in the last 10 mins and both Safari on mac and Google Chrome on mac are showing this wrong now. iPhone 12 pointed at the same installation looks perfect though!!

Screenshot 2021-05-22 at 17 27 05

avatar PhilETaylor PhilETaylor - open - 22 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 22 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 22 May 2021
avatar PhilETaylor PhilETaylor - change - 22 May 2021
Title
[4] dashboard box colours?
RELEASE BLOCKER [4] dashboard box colours?
avatar PhilETaylor PhilETaylor - edited - 22 May 2021
avatar PhilETaylor PhilETaylor - change - 22 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-22 16:36:44
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 22 May 2021
avatar PhilETaylor PhilETaylor - change - 22 May 2021
Title
RELEASE BLOCKER [4] dashboard box colours?
[4] dashboard box colours?
avatar PhilETaylor PhilETaylor - edited - 22 May 2021
avatar PhilETaylor
PhilETaylor - comment - 22 May 2021

Confused.

avatar PhilETaylor PhilETaylor - change - 22 May 2021
Status Closed New
Closed_Date 2021-05-22 16:36:44
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - reopen - 22 May 2021
avatar PhilETaylor PhilETaylor - change - 22 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-22 16:45:39
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 22 May 2021
avatar PhilETaylor
PhilETaylor - comment - 22 May 2021

Hmmmmm.... seems a complete reinstall from scratch fixed it... but maybe that means we have an upgrade issue... I'll close for now.

avatar PhilETaylor
PhilETaylor - comment - 23 May 2021

@richard67 when you are next testing your upgrade paths, please see if you accidentally replicate this.

I did npm builds loads and it never fixed it and it was not a cache issue, which leads me to believe it was a db issue as after reinstalling from /installation/ it was fine again.

avatar richard67
richard67 - comment - 23 May 2021

@PhilETaylor I think it comes from the backend template repaint for which I had to add later update SQL for the Atum template style parameters: #33375 . Have you run the SQL update scrips, too, after having pulled changes and run npm? The database fix will not run that because it's DML (data manipulation language) and not DDL (data definition language).

avatar PhilETaylor
PhilETaylor - comment - 23 May 2021

I cannot remember the last time I reinstalled the db from scratch, and I always forget to run update sql's...

I did not have this problem until yesterday. which is strange.

I think the check for db schema updates should be on the home dashboard and "in your face"

avatar brianteeman
brianteeman - comment - 23 May 2021

I think the check for db schema updates should be on the home dashboard and "in your face"

No as its really only relevant to those few of us who are testing and not going through an install

avatar richard67
richard67 - comment - 23 May 2021

@PhilETaylor The check for db schema update will not help here because it checks only DDL statements which change structure, and the schema version. So in this case it will only show that the schema version is not matching, and when you use the fix button, it will fix the schema version in db.

But it will not sun completely those SQL scripts which according to the schema version haven't run yet, like it would do e.g. a tool like https://gist.github.com/mbabker/d7bfb4e1e2fbc6b7815a733607f89281 .

I think about implementing that since long time, something like The database checker has detected a possibly incomplete update. If this is the case, use the "Complete update" button to fix that..

But it has certain risks, e.g. if a script has aborted somewhere in the middle so some update statements (DML) did not run and because we haven't reliable criteria in every case to check in db if we have to run them again or not.

We would need a database checker not only for the schema (structure) but also for data, and we would have to link every insert or update or delete statement to a condition which allows to verify if that statement needs to be run or not (not in all cases the WHERE clause of the particular statement is sufficient for that).

avatar PhilETaylor
PhilETaylor - comment - 23 May 2021

Thanks for the explanation. If you are all happy Im happy :-)

I guess I was just shocked and worried so close to RC release to see what I saw, which persisted after NPM updates... but I now understand - thanks.

avatar richard67
richard67 - comment - 23 May 2021

That was why I explained so much, because I wanted you to understand and not just to believe.

Add a Comment

Login with GitHub to post a comment