Skip to content

Conversation

@Vlczech
Copy link

@Vlczech Vlczech commented Oct 5, 2020

  • bug fix / new feature? Partially both
  • BC break? no
  • doc PR: nette

If data contains special char, table is named with quotation marks also. Thus regclass returns name with quotes and for example operation ->table() delimites such delimited regclass name obtained from SELECT for the second time. So for example $tbl='someschema."some-table"' is in $db->table() delimited to "someschema"."""some-table""".

Note: Tested for PostgreSQL only.

dg and others added 13 commits July 10, 2020 18:29
If data contains special char, table is named with quotation marks also. Thus regclass returns name with quotes and for example operation ->table() delimites such delimited regclass name obtained from SELECT for the second time. So for example $tbl='someschema."some-table"' is in $db->table() delimited to "someschema"."""some-table""".
Note: Tested for PostgreSQL only.
@dg
Copy link
Member

dg commented Oct 6, 2020

Does this feature apply only to Postgresql, or any other databases?

@dg dg force-pushed the master branch 2 times, most recently from a9e5763 to c3cc31a Compare October 6, 2020 17:24
@Vlczech
Copy link
Author

Vlczech commented Oct 7, 2020

@dg: I tried to code it to be universal as well for other databases for reverse operation to delimite(), but don't know if any other database has some feature like regclass in postgresql. I don't like to have this feature in ResultSet if it's usable for something specific only (in this case PostgreSQL), so i decided to put it at the driver level :-)

@dg dg added this to the v3.1 milestone Oct 8, 2020
@dg dg force-pushed the master branch 2 times, most recently from e91b234 to 953502b Compare October 15, 2020 22:53
@dg dg force-pushed the master branch 6 times, most recently from 52d72e1 to fd0d57c Compare October 30, 2020 13:01
@dg dg force-pushed the master branch 2 times, most recently from f06c30e to 1b974d3 Compare November 25, 2020 22:28
@dg dg force-pushed the master branch 21 times, most recently from cdb8832 to a1798c0 Compare September 6, 2024 15:11
@dg dg force-pushed the master branch 2 times, most recently from 1769a67 to b02f117 Compare January 9, 2025 22:12
@dg dg force-pushed the master branch 2 times, most recently from 2d78899 to 2f04c73 Compare June 5, 2025 23:26
@dg dg force-pushed the master branch 2 times, most recently from e8459f8 to 23617a8 Compare June 19, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants