#6508: Suggestion: Remove 'connect' from $_baseConfig in DboPostgres
--------------------------+-------------------------------------------------
Reporter: cuppett | Type: Optimization
Status: new | Priority: Very Low
Milestone: 1.2.x.x | Component: PostgreSQL
Version: 1.2 Final | Severity: Trivial
Keywords: | Php_version: PHP 5
Cake_version: 1.2.3 |
--------------------------+-------------------------------------------------
The 'connect' parameter of $_baseConfig is never used in DboPostgres.
It's a little misleading as it suggests changing it from pg_pconnect to
pg_connect might get you a non-persistent database handle; however, only
the 'persistent' parameter does that:
if (!$config['persistent']) {
$this->connection = pg_connect($conn,
PGSQL_CONNECT_FORCE_NEW);
} else {
$this->connection = pg_pconnect($conn);
}
Can easily remove 'connect' entirely to have a 1 entry shorter array to
deal with or somehow tie the value supplied there into how the connection
is processed and the function that is actually called.
--
Ticket URL: <https://trac.cakephp.org/ticket/6508>
CakePHP : The Rapid Development Framework for PHP <https://trac.cakephp.org/>
Cake is a rapid development framework for PHP which uses commonly known design
patterns like ActiveRecord, Association Data Mapping, Front Controller and MVC.
Our primary goal is to provide a structured framework that enables PHP users at
all levels to rapidly develop robust web applications, without any loss to
flexibility.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"tickets cakephp" group.
To post to this group, send email to tickets-cakephp@googlegroups.com
To unsubscribe from this group, send email to
tickets-cakephp+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/tickets-cakephp?hl=en
-~----------~----~----~----~------~----~------~--~---
