Protoqol / Prequel

Prequel for Laravel. Clear and concise database management. Unfortunately, not actively maintained due to time constraints.
https://packagist.org/packages/protoqol/prequel
MIT License
1.54k stars 96 forks source link

[BUG] Custom Database Connection Name #139

Open fauzie opened 3 years ago

fauzie commented 3 years ago

Describe the bug Not support with custom connection name that difference from driver name. e.g: in config database.php i am using connections with name defaultdb and driver using mysql

To Reproduce Assume you are using mysql database. Steps to reproduce the behavior:

  1. Edit 'config/database.php' edit in connection array key with driver mysql to other than mysql. eg:
    'connections' => [
        'mydb' => [
              'driver' => 'mysql'
              .....
  2. Leave the config/prequel.php as is
  3. Open prequel on browser
  4. Error with Exception Selected invalid or unsupported database driver

Next try i change my config path prequel.database.connection to mydb, same like on database config Open prequel on browser, and is still error with message:

Your database connection might not be supported yet, currently supported: 'mysql', 'pgsql'. (Yours is: 'mydb').

Expected behavior Prequel database config must follow the configuration from the database config, and make the connection details such as host, port, database, username, password is optional in the prequel config, just store the connection name on prequel config, and follow the rest from database config.

Screenshots https://ibb.co/9WMmwfz https://ibb.co/8M02W7d

PHP Version

Database type

Desktop (please complete the following information):

QuintenJustus commented 2 years ago

Hey @fauzie, thanks for submitting this bug! Very sorry about the very late answer. Prequel is going back in to active development and a v2.0 is being developed right now (see #156). I will fix this in that release.

Hope you haven't been discouraged from using Prequel because of this! I'll keep this issue open until the fix is in the aforementioned PR.