Skip to content

keep star on history#24731

Merged
SniperSister merged 9 commits intojoomla:stagingfrom
alikon:patch-92
May 8, 2019
Merged

keep star on history#24731
SniperSister merged 9 commits intojoomla:stagingfrom
alikon:patch-92

Conversation

@alikon
Copy link
Copy Markdown
Contributor

@alikon alikon commented Apr 26, 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

@wilsonge
Copy link
Copy Markdown
Contributor

Looks about right to me

@ghost
Copy link
Copy Markdown

ghost commented Apr 28, 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.

@alikon
Copy link
Copy Markdown
Contributor Author

alikon commented Apr 28, 2019

did you fix the database ?

@ghost
Copy link
Copy Markdown

ghost commented Apr 28, 2019

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

@alikon
Copy link
Copy Markdown
Contributor Author

alikon commented Apr 28, 2019

yes, sorry for uncomplete test info

@ghost
Copy link
Copy Markdown

ghost commented Apr 28, 2019

Updated Instructions.

@ghost
Copy link
Copy Markdown

ghost commented Apr 28, 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.

@HLeithner HLeithner added this to the Joomla 3.9.6 milestone Apr 29, 2019
@ChristineWk
Copy link
Copy Markdown

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.

@ghost
Copy link
Copy Markdown

ghost commented Apr 30, 2019

@ChristineWk please mark your test as successfully > https://docs.joomla.org/Testing_Joomla!_patches#Recording_test_results

@ChristineWk
Copy link
Copy Markdown

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

@chmst
Copy link
Copy Markdown
Contributor

chmst commented Apr 30, 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.

@ghost
Copy link
Copy Markdown

ghost commented Apr 30, 2019

@chmst please mark your test as unsuccessfully > https://docs.joomla.org/Testing_Joomla!_patches#Recording_test_results

@chmst
Copy link
Copy Markdown
Contributor

chmst commented Apr 30, 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.

@ReLater
Copy link
Copy Markdown
Contributor

ReLater commented Apr 30, 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.

@ChristineWk
Copy link
Copy Markdown

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

@ChristineWk
Copy link
Copy Markdown

ChristineWk commented Apr 30, 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

@alikon
Copy link
Copy Markdown
Contributor Author

alikon commented Apr 30, 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

@ghost
Copy link
Copy Markdown

ghost commented Apr 30, 2019

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

@HLeithner HLeithner modified the milestones: Joomla 3.9.6, Joomla 3.9.7 Apr 30, 2019
@ReLater
Copy link
Copy Markdown
Contributor

ReLater commented Apr 30, 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.

@alikon
Copy link
Copy Markdown
Contributor Author

alikon commented May 1, 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

@ReLater
Copy link
Copy Markdown
Contributor

ReLater commented May 1, 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.

@ReLater
Copy link
Copy Markdown
Contributor

ReLater commented May 1, 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.

@ChristineWk
Copy link
Copy Markdown

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.

@alikon
Copy link
Copy Markdown
Contributor Author

alikon commented May 3, 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

@joomla-cms-bot joomla-cms-bot removed this from the Joomla 3.9.7 milestone May 3, 2019
@alikon
Copy link
Copy Markdown
Contributor Author

alikon commented May 3, 2019

thanks for testing folks

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 3, 2019
@ReLater
Copy link
Copy Markdown
Contributor

ReLater commented May 3, 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.

@SniperSister SniperSister merged commit 14edbc9 into joomla:staging May 8, 2019
@SniperSister
Copy link
Copy Markdown
Contributor

Merged, thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 8, 2019
@SniperSister SniperSister added this to the Joomla 3.9.7 milestone May 8, 2019
@alikon alikon deleted the patch-92 branch May 8, 2019 11:59
tecpromotion pushed a commit to tecpromotion/joomla-cms that referenced this pull request May 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants