? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
28 Feb 2016

Pull Request for follow up Issue #9230.

Summary of Changes

Now we use a function that also support Updating from 2.5.x to 3.5

Testing Instructions

Expected result

Update works (without any error message) but you got logged out

Notes

Please note that the only change included in my package is the one from this PR and #9230 . So you might encounter other issues which were already fixed since the release of beta2.

@wilsonge @tecpromotion @Harmageddon can you check if that works for you too? Thanks.

avatar zero-24 zero-24 - open - 28 Feb 2016
avatar zero-24 zero-24 - change - 28 Feb 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Feb 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - test_item - 28 Feb 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Feb 2016

I have tested this item :white_check_mark: successfully on 0abbc48

Tested a 2.5.28 to your custom 3.5.0-beta2 upgrade with success.
Yesterday got a fatal error, today has ok, so success.

Used Extensions -> Install -> Upload method.

Note to other testers: PHP and webserver upload max size must be equal or more that 12MB (mine was at 8MB - had to change it to 12MB).


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

avatar andrepereiradasilva andrepereiradasilva - test_item - 28 Feb 2016 - Tested successfully
avatar brianteeman brianteeman - change - 28 Feb 2016
Category Updating
avatar Harmageddon Harmageddon - test_item - 28 Feb 2016 - Tested successfully
avatar Harmageddon
Harmageddon - comment - 28 Feb 2016

I have tested this item :white_check_mark: successfully on 0abbc48

Sorry for not testing 2.5 yesterday! Thank you @zero-24! The update now works fine from 2.5, too.


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

avatar richard67 richard67 - test_item - 28 Feb 2016 - Tested successfully
avatar richard67
richard67 - comment - 28 Feb 2016

I have tested this item :white_check_mark: successfully on 0abbc48

I have tested this with success.

For the test I used my own update zip packages based on latest staging plus my PR #9221

  • 1st container without this patch here

  • 2nd container with this patch here

Result:

Test 1: Update the 2.5.28 with container 1 using upload and install package method, ended with lots of PHP error messages "Error on updating manifest cache ...". Then could login and saw success message for package installation and the statistics plugin question box, i.e. all was normal then, and I could do the successful database fix as made possible with my PR I mentioned.

Test 2: Same with container 2. Result: No PHP errors :+1: , cleanly logged out, and after login again all normal, i.e. success message, statistics plugin message displayed, database fix worked.


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

avatar richard67
richard67 - comment - 28 Feb 2016

P.S.: I also tested with same container 2 from my previous test if updating a 3.4.8 still works and can confirm this PR here braks nothing, i.e. if this PR here and mine are merged, all problems related to updates from 2.5.28 on to 3.5 are solved, and so Beta 3 can be shipped :smile:


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

avatar wilsonge wilsonge - change - 28 Feb 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-02-28 13:00:00
Closed_By wilsonge
avatar wilsonge wilsonge - close - 28 Feb 2016
avatar wilsonge wilsonge - reference | 5a91daa - 28 Feb 16
avatar wilsonge wilsonge - merge - 28 Feb 2016
avatar wilsonge wilsonge - close - 28 Feb 2016
avatar wilsonge wilsonge - change - 28 Feb 2016
Milestone Added:
avatar zero-24 zero-24 - head_ref_deleted - 28 Feb 2016
avatar zero-24
zero-24 - comment - 28 Feb 2016

Thanks for the testers! And @wilsonge for the check and merge :+1:

Add a Comment

Login with GitHub to post a comment