Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Table prefix replacement also happens on the values #2562

Closed
nitriques opened this issue Feb 10, 2016 · 5 comments
Closed

Table prefix replacement also happens on the values #2562

nitriques opened this issue Feb 10, 2016 · 5 comments
Assignees
@nitriques
Labels
1 - Minor Fix found Fixed in 3.0.x
Milestone
3.0.0

Comments

@nitriques
Copy link
Member

I do not know how to fix it, but any values containing tbl_something will get transformed....
@nitriques nitriques added this to the 2.7.0 milestone Feb 10, 2016
@michael-e
Copy link
Member

Cool find!

@nitriques
Copy link
Member Author

Indeed. And that's the 2nd today haha

@brendo
Copy link
Member

brendo commented Feb 13, 2016

The magic all happens on this line, so fixing this may be possible by adjusting the regular expression :)

@nitriques
Copy link
Member Author

so fixing this may be possible by adjusting the regular expression :)

I know... but that's going to be pretty hard !

@nitriques
Copy link
Member Author

Thinking about this, PDO will resolve the problem since values and queries are separated. Postponing to 3.0.0

@nitriques nitriques modified the milestones: 3.0.0, 2.7.0 Feb 19, 2016
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Sep 27, 2017
@nitriques
Introduce a new database API, based on PDO
69439c6 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re symphonycms#2604
Fixes symphonycms#2562
Re symphonycms#2472
Re symphonycms#2448
Fixes #2425
Re symphonycms#2015
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Feb 1, 2018
@nitriques
Introduce a new database API, based on PDO
01cfd4e 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re symphonycms#2604
Fixes symphonycms#2562
Re symphonycms#2472
Re symphonycms#2448
Fixes #2425
Re symphonycms#2015

Picked from 69439c6
@nitriques nitriques self-assigned this Feb 1, 2018
nitriques pushed a commit to DeuxHuitHuit/symphonycms that referenced this issue Feb 14, 2018
@nils-werner @brendo @nitriques
PDO and Query Builer implementation
8e7861b 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Feb 15, 2018
@nitriques
Introduce a new database API, based on PDO
33102ad 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re symphonycms#2604
Fixes symphonycms#2562
Re symphonycms#2472
Re symphonycms#2448
Fixes #2425
Re symphonycms#2015

Picked from 69439c6
nitriques pushed a commit to DeuxHuitHuit/symphonycms that referenced this issue Feb 15, 2018
@nils-werner @brendo @nitriques
PDO and Query Builer implementation
2b1774a 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
nitriques pushed a commit to DeuxHuitHuit/symphonycms that referenced this issue Feb 15, 2018
@nils-werner @brendo @nitriques
PDO and Query Builer implementation
e6c7673 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Apr 20, 2018
@nitriques
Introduce a new database API, based on PDO
a939719 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re symphonycms#2604
Fixes symphonycms#2562
Re symphonycms#2472
Re symphonycms#2448
Fixes #2425
Re symphonycms#2015

Picked from 69439c6
Picked from 33102ad
nitriques pushed a commit to DeuxHuitHuit/symphonycms that referenced this issue Apr 20, 2018
@nils-werner @nitriques
PDO and Query Builer implementation
f873618 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>

Picked from e6c7673
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Apr 20, 2018
@nitriques
Introduce a new database API, based on PDO
5355f60 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re symphonycms#2604
Fixes symphonycms#2562
Re symphonycms#2472
Re symphonycms#2448
Fixes #2425
Re symphonycms#2015

Picked from 69439c6
Picked from 33102ad
nitriques pushed a commit to DeuxHuitHuit/symphonycms that referenced this issue Apr 20, 2018
@nils-werner @nitriques
PDO and Query Builer implementation
77192a0 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>

Picked from e6c7673
nitriques pushed a commit to DeuxHuitHuit/symphonycms that referenced this issue Apr 20, 2018
@nils-werner @nitriques
PDO and Query Builer implementation
fee6f90 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
Co-authored-by: Nicolas Brassard <nitriques@users.noreply.github.com>

Picked from e6c7673
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Mar 29, 2019
@nitriques
Introduce a new database API, based on PDO
fb6bcec 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re symphonycms#2604
Fixes symphonycms#2562
Re symphonycms#2472
Re symphonycms#2448
Fixes #2425
Re symphonycms#2015

Picked from 69439c6
Picked from 33102ad
Picked from 5355f60
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Mar 29, 2019
@nitriques
PDO and Query Builer implementation
a35ce85 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
Co-authored-by: Nicolas Brassard <nitriques@users.noreply.github.com>

Picked from e6c7673
Picked from fee6f90
nitriques added a commit that referenced this issue Apr 8, 2019
@nitriques
Introduce a new database API, based on PDO
e3f83c7 
This commit brings a new API for the datbase layer.
Instead of creating our dynamic SQL queries with concat and sprintf

The way it works is that you first start with on of the factory methods
available in the Database class. You then use the created
DatabaseStatement and its chainable API to add `part` to the clause.

This is very much a WIP. The API will change at some point.

Re #2604
Fixes #2562
Re #2472
Re #2448
Fixes #2425
Re #2015

Picked from 69439c6
Picked from 33102ad
Picked from 5355f60
Picked from fb6bcec
nitriques added a commit that referenced this issue Apr 8, 2019
@nitriques
PDO and Query Builer implementation
acec98f 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re #2015
Fixes #2425
Fixes #2448
Fixes #2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
Co-authored-by: Nicolas Brassard <nitriques@users.noreply.github.com>

Picked from e6c7673
Picked from fee6f90
Picked from a35ce85
nitriques added a commit to DeuxHuitHuit/symphonycms that referenced this issue Apr 8, 2019
@nitriques
PDO and Query Builer implementation
e27e913 
This commit is based on the original PDO implementation from the
integration branch. It replaces the original PDO changes with the new
Query Builder API.

Picked from 989299b
Picked from 5ba3664
Re symphonycms#2015
Fixes #2425
Fixes symphonycms#2448
Fixes symphonycms#2562

Co-authored-by: Nils Werner <nils.werner@gmail.com>
Co-authored-by: Brendan Abbott <brendan@bloodbone.ws>
Co-authored-by: Nicolas Brassard <nitriques@users.noreply.github.com>

Picked from e6c7673
Picked from fee6f90
Picked from a35ce85
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nitriques nitriques

Labels
1 - Minor Fix found Fixed in 3.0.x
Projects
None yet
Milestone
3.0.0
Development

No branches or pull requests
3 participants
@michael-e @brendo @nitriques