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

Active Record Call Stored Procedure #2047

Closed
wants to merge 4 commits into from
Closed

Active Record Call Stored Procedure #2047

wants to merge 4 commits into from

Conversation

ilk33r
Copy link

@ilk33r ilk33r commented Dec 2, 2012

calling mysql stored procedure and fix Commands out of sync problem.
This feature only support mysqli driver.

calling mysql stored procedure and fix Commands out of sync problem.
This feature only support mysqli driver.
@narfbg
Copy link
Contributor

narfbg commented Dec 2, 2012

No one-driver-only features should be implemented - inconsistency is bad and such features are hard to maintain and usually not implemented the right way to be made portable for other drivers. For example - oci8 has a stored_procedure() method, which doesn't even handle Oracle's stored procedues well, because it expects them to be inside a package, which is not always the case.

Plus, #436 seems to be a lot simpler solution (though I'm not sure how appropriate).

@ilk33r
Copy link
Author

ilk33r commented Dec 2, 2012

Other way change mysqli driver execute function but if procedure do not return any data, mysql will give a error.
in this case stored procedure working other function but only a special case of mysql. should not affect other database drivers.
I think it looks like the only way to add a new function active record.

@WDC
Copy link
Contributor

WDC commented Jan 15, 2013

Is it that hard to do something like this?

$query = $this->db->query('CALL stored_procedure_name ("param1", 2, "param3")');

@narfbg
Copy link
Contributor

narfbg commented Jul 17, 2015

Closing this now, will be resolved via #3981.

@narfbg narfbg closed this Jul 17, 2015
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