Active Record Call Stored Procedure #2047

Closed
wants to merge 4 commits into
from

Projects

None yet

3 participants

ilk33r commented Dec 2, 2012

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

ilk33r added some commits Dec 2, 2012
@ilk33r ilk33r Active Record Call Stored Procedure
calling mysql stored procedure and fix Commands out of sync problem.
This feature only support mysqli driver.
b42d156
@ilk33r ilk33r Revert "Active Record Call Stored Procedure"
This reverts commit b42d156.
b781188
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 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.

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")');
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