Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Let's fix the Update from 2.5 to 3.5 with a method both supports #9240

Merged
merged 1 commit into from Feb 28, 2016

Conversation

zero-24
Copy link
Member

@zero-24 zero-24 commented Feb 28, 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.

@andrepereiradasilva
Copy link
Contributor

I have tested this item ✅ 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.

@Harmageddon
Copy link
Contributor

I have tested this item ✅ 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.

@richard67
Copy link
Member

I have tested this item ✅ 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 👍 , 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.

@richard67
Copy link
Member

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 😄


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

wilsonge added a commit that referenced this pull request Feb 28, 2016
Let's fix the Update from 2.5 to 3.5 with a method both supports
@wilsonge wilsonge merged commit 5a91daa into joomla:staging Feb 28, 2016
@wilsonge wilsonge added this to the Joomla! 3.5.0 milestone Feb 28, 2016
@zero-24 zero-24 deleted the fixSessionUpdateCheckFrom25 branch February 28, 2016 18:33
@zero-24
Copy link
Member Author

zero-24 commented Feb 28, 2016

Thanks for the testers! And @wilsonge for the check and merge 👍

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.

None yet

6 participants