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.

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

Jam::update() does not support joins #19

Open
hkdobrev opened this issue Aug 6, 2013 · 4 comments
Open

Jam::update() does not support joins #19

hkdobrev opened this issue Aug 6, 2013 · 4 comments

Comments

@hkdobrev
Copy link
Contributor

hkdobrev commented Aug 6, 2013

Jam::update() should support the ->join() syntax similar to other builder methods in Jam.

Right now the only alternative is using the plain old Query Builder or plain text queries.

@ghost ghost assigned ivank Aug 6, 2013
@ivank
Copy link
Contributor

ivank commented Aug 6, 2013

Yeah I know, that's a limitation of Kohana Database Module - to fix that properly we'll have to go deeper and modify Database Module. I even think that the old Query builder did not work either it just presented that interface.

For now you'll have to do this as a custom query.

@hkdobrev
Copy link
Contributor Author

hkdobrev commented Aug 6, 2013

@ivank Actually when dug deeper the query builder of Kohana does not allow it as well.

@ivank
Copy link
Contributor

ivank commented Aug 6, 2013

@hkdobrev Yeah that's why I hope we're going to fix it when we extract the database module in a composer package.

@hkdobrev
Copy link
Contributor Author

One more "+1" for this one. I am constantly using

UPDATE `updated_table_name`
JOIN `joined_table_name`
ON ...
SET

in migrations.

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

No branches or pull requests

2 participants