? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
5 Jul 2020

Summary of Changes

Show a message when restoring history version fails.

Testing Instructions

Create an article.
Make some edits to create multiple versions.
Click Versions button.
In the modal select a version and click Restore.

Actual result BEFORE applying this Pull Request

Version is not restore but success message is shown.

Expected result AFTER applying this Pull Request

Error message is shown:

Error
Column 'ordering' cannot be null

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 5 Jul 2020
avatar SharkyKZ SharkyKZ - change - 5 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jul 2020
Category Libraries
avatar ceford
ceford - comment - 6 Jul 2020

This is confusing for me. I created an article with one line, saved, added a line, saved, added a line saved. So I have three Versions.
Question: why does it let me choose more than one version and put up an alert to tell me to select One version?
It is the first version that says it has saved but hasn't - so the patch works. But why can't I restore the first version? And what on earth do I do about Error: Column 'ordering' cannot be null. It seems to me there ought to be a better solution.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30011.
avatar ChristineWk ChristineWk - test_item - 6 Jul 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 6 Jul 2020

I have tested this item successfully on e00f6d6


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

avatar ChristineWk
ChristineWk - comment - 6 Jul 2020

This is confusing for me. I created an article with one line, saved, added a line, saved, added a line saved. So I have three Versions.
Question: why does it let me choose more than one version and put up an alert to tell me to select One version?
It is the first version that says it has saved but hasn't - so the patch works. But why can't I restore the first version? And what on earth do I do about Error: Column 'ordering' cannot be null. It seems to me there ought to be a better solution.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30011.

@ceford see: https://issues.joomla.org/tracker/joomla-cms/30006

avatar SharkyKZ
SharkyKZ - comment - 6 Jul 2020

@ceford This PR does what it says it does. It shows a message to let users know that an error has occurred. It does not solve the error.

avatar toivo toivo - test_item - 6 Jul 2020 - Tested successfully
avatar toivo
toivo - comment - 6 Jul 2020

I have tested this item successfully on e00f6d6

Tested successfully in Beta3-dev of 6 July. After the column 'ordering' was set to Null, the results were as described.


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

avatar toivo
toivo - comment - 6 Jul 2020

I have tested this item successfully on e00f6d6

Tested successfully in Beta3-dev of 6 July. After the column 'ordering' was set to Null, the results were as described.


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

avatar richard67 richard67 - change - 6 Jul 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 6 Jul 2020

RTC


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

avatar richard67 richard67 - change - 6 Jul 2020
Labels Added: ? ?
avatar wilsonge wilsonge - change - 6 Jul 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-06 13:23:48
Closed_By wilsonge
Labels
avatar wilsonge wilsonge - close - 6 Jul 2020
avatar wilsonge wilsonge - merge - 6 Jul 2020
avatar wilsonge
wilsonge - comment - 6 Jul 2020

Thanks!

Add a Comment

Login with GitHub to post a comment