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

[4.0] Fix errors during upgrade (PostgreSQL) #25477

Merged
merged 8 commits into from
Jul 9, 2019
Merged

Conversation

twister65
Copy link
Contributor

@twister65 twister65 commented Jul 8, 2019

Pull Request for Issue #25467 .

Summary of Changes

Fix errors of sql files in administrator/components/com_admin/sql/updates/postgresql.
See the commit comments.

Testing Instructions

  1. Install a copy of Joomla 3.x (I was using 3.9.9 RC) on postgres
  2. Apply this PR to your joomla source directory.
  3. Build an update package to your joomla-4.0-dev source directory:
  • git tag 4.0.0-alpha11-dev
  • php build/build.php
  • Update Joomla 3.9.x with the new update package.
  1. Check if Joomla upgrades successfully

Expected result

No Warning or Error.

Actual result

Error
The template for this display is not available.

is going away.

database_check

Documentation Changes Required

Copy link
Member

@richard67 richard67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think @wilsonge could merge that by review. Code review looks good, and when this is merged we can use the nightly build after that merge for further testing updates.

@richard67
Copy link
Member

richard67 commented Jul 8, 2019

For testers: You can find a modified nightly build from today 2019-07-08 morning with this PR applied here:
https://test5.richard-fath.de/Joomla_4.0.0-alpha11-dev-Development-Update_Package_2019-07-08_pr-25477.zip

@richard67
Copy link
Member

richard67 commented Jul 8, 2019

I have tested this item ✅ successfully on b60b991

Tested with the regular and my modified update package from today's nightly build, which I linked in my previous comment.

Regular update package: SQL errors, and the template not found warnings mentioned in issue #25467 don't go away.

Modified update package with this PR: No SQL errors, behavior like descibed in issue #25467, i.e. there are template not found warnings shown, but after update has finished, they go away and not appear again when navigating in the backend.

So this PR makes the update complete without crash. Without this PR update crashes because of SQL errors and so is not complete.

Other errors remain for updating 3.9.x to 4.0-dev, e.g. the menu entry for com_search is still there, but these are not subject of this PR. They shall (and I will do so) be reported in issue #25467 .


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

@alikon
Copy link
Contributor

alikon commented Jul 9, 2019

I have tested this item ✅ successfully on b60b991


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

@alikon
Copy link
Contributor

alikon commented Jul 9, 2019

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 9, 2019
@wilsonge wilsonge merged commit 865bdbf into joomla:4.0-dev Jul 9, 2019
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 9, 2019
@wilsonge
Copy link
Contributor

wilsonge commented Jul 9, 2019

Thanks guys! Much appreciated :)

@wilsonge wilsonge added this to the Joomla 4.0 milestone Jul 9, 2019
@wilsonge
Copy link
Contributor

wilsonge commented Jul 9, 2019

I've retriggered a nightly build of this so if any of you want to test using the nightly you can

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.

5 participants