Building UPDATE JOIN statement by SQL Builder #903

Closed
temp-impl opened this Issue Jan 24, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@temp-impl

MySQL has a statement like below.

UPDATE table1 a
INNER JOIN table2 b USING (ID)
SET a.value = b.value 

I think SQL Builder Class cannot build statement like this.

new SQL()
.UPDATE("table1 a")
.INNER_JOIN("table2 b USING (ID)")
.SET("a.value = b.value");

// this drops INNER JOIN statement

So, I wish the feature of building UPDATE JOIN statement.

@h3adache h3adache self-assigned this Feb 25, 2017

@h3adache h3adache added this to the 3.4.3 milestone Feb 25, 2017

@h3adache h3adache referenced this issue Feb 25, 2017

Merged

Fix for #903 #926

h3adache added a commit that referenced this issue Feb 25, 2017

@h3adache h3adache closed this Feb 25, 2017

@kazuki43zoo

This comment has been minimized.

Show comment
Hide comment
@kazuki43zoo

kazuki43zoo Mar 11, 2017

Member

@temp-impl @h3adache

Is not need support on DELETE ?
MySQL is support JOIN phrase on DELETE.

See as following links:

Member

kazuki43zoo commented Mar 11, 2017

@temp-impl @h3adache

Is not need support on DELETE ?
MySQL is support JOIN phrase on DELETE.

See as following links:

@h3adache

This comment has been minimized.

Show comment
Hide comment
@h3adache

h3adache Mar 14, 2017

Member

Sure we can add that @kazuki43zoo
I didn't go through the sql grammar to see what else was missing at the time.

Member

h3adache commented Mar 14, 2017

Sure we can add that @kazuki43zoo
I didn't go through the sql grammar to see what else was missing at the time.

@kazuki43zoo

This comment has been minimized.

Show comment
Hide comment
@kazuki43zoo

kazuki43zoo Mar 14, 2017

Member

@h3adache Thanks for reply!!
I think adding is not required. However If there is request for adding, we will be better that consider support it on DELETE phrase.

Member

kazuki43zoo commented Mar 14, 2017

@h3adache Thanks for reply!!
I think adding is not required. However If there is request for adding, we will be better that consider support it on DELETE phrase.

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