? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
29 Apr 2020

Pull Request for Issue # .

Summary of Changes

Adds missing import.

Testing Instructions

Code review.

Or go to com_installer configuration.
Set Updates Caching (in hours) to 0.
Go to Postinstall Messages.
In The Joomla! Update Notification will not run in this configuration message click Set it back to the default setting (6 Hours) button.

Expected result

No errors.

Actual result

Class 'Table' not found

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 29 Apr 2020
avatar SharkyKZ SharkyKZ - change - 29 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Apr 2020
Category Front End Plugins
avatar ChristineWk
ChristineWk - comment - 29 Apr 2020

Confirm Actual result

postinstallmessage_catch

avatar ChristineWk
ChristineWk - comment - 29 Apr 2020

I have tested this item successfully on 2465c15

Code review


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

avatar ChristineWk ChristineWk - test_item - 29 Apr 2020 - Tested successfully
avatar richard67
richard67 - comment - 29 Apr 2020

I have tested this item successfully on 2465c15

Code review: Function "updatecachetime_postinstall_action" uses the "Table" class, but there is no import for it. This is corrected by this PR.


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

avatar richard67 richard67 - test_item - 29 Apr 2020 - Tested successfully
avatar richard67 richard67 - change - 29 Apr 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 29 Apr 2020

RTC


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

avatar richard67 richard67 - change - 29 Apr 2020
Labels Added: ? ?
avatar Quy Quy - change - 29 Apr 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-04-29 13:47:45
Closed_By Quy
Labels
avatar Quy Quy - close - 29 Apr 2020
avatar Quy Quy - merge - 29 Apr 2020

Add a Comment

Login with GitHub to post a comment