? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
26 Apr 2019

Pull Request for Issue #9287 .

Summary of Changes

added field ordering to ignoreChanges on #__content_types table

Testing Instructions

run the query via phpmyadmin
Create an article & save (art1)
Do some changes in that article (art1)
Create another article (art2)

Expected result

In the article Item Version History the last version is marked with a star

Actual result

after creating art2 the star is missed on art1 article Item Version History

credits

@wilsonge

avatar alikon alikon - open - 26 Apr 2019
avatar alikon alikon - change - 26 Apr 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Apr 2019
Category SQL Administration com_admin Postgresql MS SQL Installation
avatar wilsonge
wilsonge - comment - 27 Apr 2019

Looks about right to me

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Apr 2019

I have tested this item 🔴 unsuccessfully on 90a67ec

after creating art2 the star is missed on art1 article Item Version History


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 28 Apr 2019 - Tested unsuccessfully
avatar alikon
alikon - comment - 28 Apr 2019

did you fix the database ?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Apr 2019

No, will retest (Fix Database after "apply Patch"?)

avatar alikon
alikon - comment - 28 Apr 2019

yes, sorry for uncomplete test info

avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Apr 2019
The description was changed
avatar franz-wohlkoenig franz-wohlkoenig - edited - 28 Apr 2019
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Apr 2019

Updated Instructions.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Apr 2019

I have tested this item ✅ successfully on 90a67ec


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 28 Apr 2019 - Tested successfully
avatar alikon alikon - change - 29 Apr 2019
Labels Added: ?
avatar ChristineWk
ChristineWk - comment - 29 Apr 2019

Before applying the Patch, I made following tests:
Created an article & saved (art1)
Some changes in that article (art1)
Created another article (art2) etc.

Actual result

after creating of art2 the star was not missed on art1 article Item Version History.
You can keep it as current version. Or to activate (select) required version for restoring (star).


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24731.
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Apr 2019
avatar ChristineWk
ChristineWk - comment - 30 Apr 2019

@franz-wohlkoenig: I didn't use the Patch. My "Actual result" was without applying the Patch.

avatar chmst
chmst - comment - 30 Apr 2019

I have tested (patch applied via patchtester on a 3.9.5 installation).
Before patch, the issue is as described.
After applying the patch and fixing the database - same issue as before, no star is displayed in versions of article1.

So it was not successful for me.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Apr 2019
avatar chmst
chmst - comment - 30 Apr 2019

I have tested this item 🔴 unsuccessfully on c655a92


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

avatar chmst
chmst - comment - 30 Apr 2019

I have tested this item 🔴 unsuccessfully on c655a92


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

avatar chmst chmst - test_item - 30 Apr 2019 - Tested unsuccessfully
avatar ReLater
ReLater - comment - 30 Apr 2019

I have tested this item 🔴 unsuccessfully on c655a92

I followed exactly the instructions here and there (#9287) and also fixed the database.

Result: Always 2 version entries for "Article 1" but no star before and after patch.


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

avatar ReLater ReLater - test_item - 30 Apr 2019 - Tested unsuccessfully
avatar ChristineWk
ChristineWk - comment - 30 Apr 2019

@chmst & @ReLater: Thanks for testing. Sorry, maybe I made some steps too much before patch.

avatar ChristineWk
ChristineWk - comment - 30 Apr 2019

I have tested this item 🔴 unsuccessfully on c655a92

Create an article & save (art1) > here was my failure (in previous test): I saved & closed & reopened
Do some changes in that article (art1)
Create another article (art2)

Tested again now - Before Patch: Result: no star in versions of article 1.

After applying the patch and fixing the database - same as above: Test unsuccessfully.


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

Edit: Sorry, I don't know, why I have h2 above instead of p

avatar ChristineWk
ChristineWk - comment - 30 Apr 2019

I have tested this item 🔴 unsuccessfully on c655a92

Create an article & save (art1) > here was my failure (in previous test): I saved & closed & reopened
Do some changes in that article (art1)
Create another article (art2)

Tested again now - Before Patch: Result: no star in versions of article 1.

After applying the patch and fixing the database - same as above: Test unsuccessfully.


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

avatar ChristineWk ChristineWk - test_item - 30 Apr 2019 - Tested unsuccessfully
avatar alikon
alikon - comment - 30 Apr 2019

that's weird if I run the same query in this patch via phpmyadmin administrator/components/com_admin/sql/updates/postgresql/3.9.7-2019-04-26.sql i got the correct result

Screenshot from 2019-04-30 18-56-48
i.e ignoreChanges":["modified_by", "modified", "checked_out", "checked_out_time", "version", "hits", "ordering"]
viceversa

if I run via database fix got wrong result as you reported

i.e ignoreChanges":["modified_by", "modified", "checked_out", "checked_out_time", "version", "hits"]
without "ordering"

need some more investigation... sorry

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Apr 2019

@alikon As i tested first Time got unsuccessfully Result, 2nd time got successfully Test so i thought i had Mistakes in 1st run.

avatar ReLater
ReLater - comment - 30 Apr 2019

that's weird if I run the same query in this patch via phpmyadmin administrator/components/com_admin/sql/updates/postgresql/3.9.7-2019-04-26.sql i got the correct result

Confirmed. The same with https://github.com/alikon/joomla-cms/blob/patch-92/administrator/components/com_admin/sql/updates/mysql/3.9.7-2019-04-26.sql . It works via phpMyAdmin.

avatar alikon
alikon - comment - 1 May 2019

Ok so that is the situation:
The Database Fix (com_installer) only work for DDL and not with DML (ie don't run UPDATE)
instead
The Joomla update (com_joomlaupdate) run both DDL and DML

so the easy way to test is to run the query via phpmyadmin

avatar alikon alikon - change - 1 May 2019
The description was changed
avatar alikon alikon - edited - 1 May 2019
avatar ReLater
ReLater - comment - 1 May 2019

I have tested this item ✅ successfully on c655a92

1. test via phpMyAdmin
2. test via installation of https://github.com/alikon/joomla-cms/archive/patch-92.zip


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

avatar ReLater ReLater - test_item - 1 May 2019 - Tested successfully
avatar ReLater
ReLater - comment - 1 May 2019

@alikon

The Database Fix (com_installer) only work for DDL and not with DML (ie don't run UPDATE)

Isn't that a new issue in Joomla 3? Because the database fix runs without any messages but didn't do its job and says "database Ok" afterwards.

avatar ChristineWk
ChristineWk - comment - 3 May 2019

I have tested this item ✅ successfully on c655a92

Test via phpMyAdmin


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

avatar ChristineWk ChristineWk - test_item - 3 May 2019 - Tested successfully
avatar alikon
alikon - comment - 3 May 2019

@ReLater some kind of messages is displayed after all

1 database changes did not alter table structure and were skipped.

so maybe something should be addressed in 4.x

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

thanks for testing folks

RTC


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

avatar alikon
alikon - comment - 3 May 2019

thanks for testing folks

RTC


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

avatar ReLater
ReLater - comment - 3 May 2019

1 database changes did not alter table structure and were skipped.

Ah, "structure" is the magic difference. Nice to know. I've overlooked that since ever. I've always messages like that when I fix a database and always thought "everything's OK now".
Thus when one updates a crashed website via FTP and uses the fix button which is (maybe) not sufficient one should recommend a final Joomla Update > Upload & Update.

avatar SniperSister SniperSister - change - 8 May 2019
Labels Added: ?
avatar SniperSister SniperSister - change - 8 May 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-08 10:07:32
Closed_By SniperSister
avatar SniperSister SniperSister - close - 8 May 2019
avatar SniperSister SniperSister - merge - 8 May 2019
avatar SniperSister
SniperSister - comment - 8 May 2019

Merged, thanks!

Add a Comment

Login with GitHub to post a comment