? Pending

User tests: Successful: Unsuccessful:

avatar roland-d
roland-d
12 Sep 2019

Pull Request for Issue #25885

Summary of Changes

This change makes sure a database will be created during the installation of Joomla.

Testing Instructions

  1. Get a clean copy of Joomla 4
  2. Start the installation process
  3. Go through the process until you get to the database part
  4. Select here the PDO driver
  5. Fill-in a database name that doesn't exist yet
  6. Complete the installation
  7. Installation doesn't complete because the database name does not exist.
  8. Clean the session cookies
  9. Apply the patch
  10. Restart the installation process
  11. Go through the process until you get to the database part
  12. Select here the PDO driver
  13. Fill-in a database name that doesn't exist yet
  14. Complete the installation
  15. Installation completes successfully

Expected result

The database will be created by the installer

Actual result

The database is not created and throws an error

Documentation Changes Required

None

avatar roland-d roland-d - open - 12 Sep 2019
avatar roland-d roland-d - change - 12 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Sep 2019
Category Installation
avatar roland-d roland-d - change - 12 Sep 2019
Title
Handle PDO driver database creation
[4.0] Handle PDO driver database creation
avatar roland-d roland-d - edited - 12 Sep 2019
avatar brianteeman
brianteeman - comment - 12 Sep 2019

Many people will actually consider it a problem that joomla can create a new database in the installation and/or you should not be using a database user who has creation rights. But I guess consistency across all databases is good

avatar roland-d roland-d - change - 12 Sep 2019
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 13 Sep 2019

Works fine on MySQL PDO but getting authentication error on PostgreSQL:

Could not connect to PDO: SQLSTATE[08006] [7] FATAL: password authentication failed for user

For what it's worth, creating DB on PgSQL doesn't work in 3.x either. So maybe limit this change to MySQL PDO only to restore 3.x behavior?

avatar SharkyKZ
SharkyKZ - comment - 13 Sep 2019

@brianteeman I have no opinion regarding this. Are you suggesting to remove the ability to create database during installation?

avatar brianteeman
brianteeman - comment - 13 Sep 2019

basically yes. although its useful when building a site locally its something that is pretty unique to joomla iirc

avatar roland-d
roland-d - comment - 13 Sep 2019

It is not only useful for building sites locally. If you create sites on
the fly, it is also a very useful feature. Like I do for the pbf testing
sites for example. I really don't want to create a 100 databases by hand.

Why even remove a useful feature?

On September 13, 2019 17:00:36 Brian Teeman notifications@github.com wrote:

basically yes. although its useful when building a site locally its
something that is pretty unique to joomla iirc

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

avatar Wolf-Rost Wolf-Rost - test_item - 19 Oct 2019 - Tested successfully
avatar Wolf-Rost
Wolf-Rost - comment - 19 Oct 2019

I have tested this item successfully on 51fab6a


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

avatar richard67
richard67 - comment - 15 Mar 2020

@roland-d Oh I just see this PR here. I somehow missed it and made an own one, independently from yours, and that has been merged meanwhile: #28338 . So sorry I made your PR here obsolete.

avatar richard67
richard67 - comment - 15 Mar 2020

Closing as meanwhile solved with #28338 ... again sorry Roland.

avatar richard67 richard67 - change - 15 Mar 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-03-15 17:37:56
Closed_By richard67
avatar richard67 richard67 - close - 15 Mar 2020

Add a Comment

Login with GitHub to post a comment