Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Configuring PostgreSQL charset #71

Closed
wants to merge 3 commits into from
Closed

Configuring PostgreSQL charset #71

wants to merge 3 commits into from

Conversation

wandersonwhcr
Copy link
Contributor

I just enabled the charset configuration to PostgreSQL, like I said in issue #70.

In the second commit, I just configure the postgres password to enable authentication, because I received an PDOException to an empty password.

PHP Fatal error:  Uncaught exception 'PDOException' with message 'SQLSTATE[08006] [7] fe_sendauth: no password supplied' in /vagrant/test/IntegrationTestListener.php:63

@wandersonwhcr
Copy link
Contributor Author

gotcha!

@wandersonwhcr
Copy link
Contributor Author

@Maks3w Must I use the master branch?

@mwillbanks
Copy link
Contributor

Can you please rebase against develop, or if you were planning to do master, please cherry-pick your commits to a new branch and resubmit the PR. Right now you're pulling in additional items.

@wandersonwhcr
Copy link
Contributor Author

ACK

@wandersonwhcr
Copy link
Contributor Author

Done 😄

$this->markTestSkipped('pgsql extension not loaded');
}

$this->setExpectedException('Zend\Db\Adapter\Exception\RuntimeException');
Copy link
Contributor

Choose a reason for hiding this comment

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

setExpectedException() is deprecated, use expectException() instead.

@weierophinney weierophinney added this to the 2.6.3 milestone Apr 12, 2016
@weierophinney weierophinney self-assigned this Apr 12, 2016
@weierophinney
Copy link
Member

Really nice patch, @wandersonwhcr ! Thanks!

@weierophinney weierophinney modified the milestones: 2.7.1, 2.6.3 Apr 12, 2016
weierophinney added a commit that referenced this pull request Apr 12, 2016
Configuring PostgreSQL charset
weierophinney added a commit that referenced this pull request Apr 12, 2016
weierophinney added a commit that referenced this pull request Apr 12, 2016
@wandersonwhcr wandersonwhcr deleted the issue-70 branch April 12, 2016 13:58
@wandersonwhcr
Copy link
Contributor Author

@weierophinney ty 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants