Thanks to visit codestin.com
Credit goes to github.com

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

Get the connected dsn string that is now stored when the pdo connection is made#5211

Closed
Paul-PSDigital wants to merge 1 commit into
zendframework:masterfrom
Paul-PSDigital:zf5094
Closed

Get the connected dsn string that is now stored when the pdo connection is made#5211
Paul-PSDigital wants to merge 1 commit into
zendframework:masterfrom
Paul-PSDigital:zf5094

Conversation

@Paul-PSDigital
Copy link
Copy Markdown
Contributor

Issue:
#5094

This stores the dsn that is generated when creating a pdo connection which can they be retrieved.

If connection has not yet been made it will attempt to connect first

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

double space after @param for consistency

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@samsonasik By the moment I think that we should stop asking about alignments. Today or Tomorrow some change will break the alignment,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Maks3w ok, should I close #4998 ?

ralphschindler pushed a commit that referenced this pull request Dec 3, 2013
Merge branch 'zf5094' of git://github.com/thepauleh/zf2 into thepauleh-zf5094

* 'zf5094' of git://github.com/thepauleh/zf2:
  Get the connected dsn string that is now stored when the pdo connection is made
ralphschindler pushed a commit that referenced this pull request Dec 3, 2013
Merge branch 'thepauleh-zf5094' into develop

* thepauleh-zf5094:
  Simplification of DSN retrieval from PDO Connection class
  Get the connected dsn string that is now stored when the pdo connection is made
@ralphschindler
Copy link
Copy Markdown
Member

Merged a variation of this into develop

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants