Skip to content

Commit

Permalink
Make protected method protected.
Browse files Browse the repository at this point in the history
PHPUnit loses its lunch when trait methods are renamed. This change also
simplifies the classes using PDODriverTrait.
  • Loading branch information
markstory committed Mar 24, 2013
1 parent 40389af commit 4263940
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 11 deletions.
6 changes: 3 additions & 3 deletions lib/Cake/Model/Datasource/Database/Driver/Mysql.php
Expand Up @@ -17,13 +17,13 @@
*/ */
namespace Cake\Model\Datasource\Database\Driver; namespace Cake\Model\Datasource\Database\Driver;


use Cake\Model\Datasource\Database\Dialect\MysqlDialectTrait;
use PDO; use PDO;


class Mysql extends \Cake\Model\Datasource\Database\Driver { class Mysql extends \Cake\Model\Datasource\Database\Driver {


use PDODriverTrait { use PDODriverTrait;
connect as protected _connect; use MysqlDialectTrait;
}


/** /**
* Base configuration settings for MySQL driver * Base configuration settings for MySQL driver
Expand Down
4 changes: 2 additions & 2 deletions lib/Cake/Model/Datasource/Database/Driver/PDODriverTrait.php
Expand Up @@ -28,7 +28,7 @@ trait PDODriverTrait {
* @param array $config configuration to be used for creating connection * @param array $config configuration to be used for creating connection
* @return boolean true on success * @return boolean true on success
*/ */
public function connect(array $config) { protected function _connect(array $config) {
$connection = new PDO( $connection = new PDO(
$config['dsn'], $config['dsn'],
$config['login'], $config['login'],
Expand All @@ -46,7 +46,7 @@ public function connect(array $config) {
* *
* @return mixed connection object used internally * @return mixed connection object used internally
*/ */
public function connection($connection = null) { public function connection($connection = null) {
if ($connection !== null) { if ($connection !== null) {
$this->_connection = $connection; $this->_connection = $connection;
} }
Expand Down
4 changes: 1 addition & 3 deletions lib/Cake/Model/Datasource/Database/Driver/Postgres.php
Expand Up @@ -23,9 +23,7 @@


class Postgres extends \Cake\Model\Datasource\Database\Driver { class Postgres extends \Cake\Model\Datasource\Database\Driver {


use PDODriverTrait { use PDODriverTrait;
connect as protected _connect;
}
use PostgresDialectTrait; use PostgresDialectTrait;


/** /**
Expand Down
4 changes: 1 addition & 3 deletions lib/Cake/Model/Datasource/Database/Driver/Sqlite.php
Expand Up @@ -23,9 +23,7 @@


class Sqlite extends \Cake\Model\Datasource\Database\Driver { class Sqlite extends \Cake\Model\Datasource\Database\Driver {


use PDODriverTrait { use PDODriverTrait;
connect as protected _connect;
}
use SqliteDialectTrait; use SqliteDialectTrait;


/** /**
Expand Down

0 comments on commit 4263940

Please sign in to comment.