[#82] Added FROM clause to update builder #168

sivagollapalli · 2018-12-05T10:15:22Z

sivagollapalli · 2018-12-05T16:29:20Z

@lann Could you please let me know the comments?

lann · 2018-12-05T20:03:12Z

update_test.go

+func TestUpdateBuilderFromClause(t *testing.T) {
+	sql, _, err := Update("employees").Set("sales_count", 100).From("accounts").Where("accounts.name = ?", "ACME").ToSql()
+	assert.NoError(t, err)
+	assert.Equal(t, "UPDATE employees SET sales_count = ? WHERE accounts.name = ?", sql)


This doesn't look right.

teresa-recursion · 2020-05-21T16:54:22Z

Is there another approach to get a FROM clause on an update? This still isn't working for me even though the referenced issue above is closed.

[Masterminds#82] Added FROM clause to update builder

e472f58

lann reviewed Dec 5, 2018

View reviewed changes

mlaflamm mentioned this pull request Jul 28, 2020

Support FROM clause in UpdateBuilder #82

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#82] Added FROM clause to update builder #168

[#82] Added FROM clause to update builder #168

sivagollapalli commented Dec 5, 2018 •

edited

Loading

sivagollapalli commented Dec 5, 2018

lann Dec 5, 2018

teresa-recursion commented May 21, 2020

[#82] Added FROM clause to update builder #168

Are you sure you want to change the base?

[#82] Added FROM clause to update builder #168

Conversation

sivagollapalli commented Dec 5, 2018 • edited Loading

sivagollapalli commented Dec 5, 2018

lann Dec 5, 2018

Choose a reason for hiding this comment

teresa-recursion commented May 21, 2020

sivagollapalli commented Dec 5, 2018 •

edited

Loading