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

Fix undefined function xml_convert at Database\BaseUtils #1209

Merged
merged 4 commits into from
Oct 17, 2018

Conversation

samsonasik
Copy link
Member

by add xml helper ported from CI3 source

@jim-parry
Copy link
Contributor

It looks like this one was missed. Did you want to add the docs page for it, and add it to the helpers/index.rst in the user guide?

@jim-parry jim-parry added this to the 4.0.0 milestone Sep 27, 2018
@samsonasik
Copy link
Member Author

@jim-parry I've added documentation for it, and add MySQLi util class for class driver for it when using MySQL. It was only for PostgreSQL.

@lonnieezell
Copy link
Member

@jim-parry Thoughts on this one?

@jim-parry
Copy link
Contributor

I am fine with this. It originally arose because the xml_helper wasn't ported to CI4, and it looks like the also missing MySQLi/Utils has been addressed too.

@lonnieezell lonnieezell merged commit 648a213 into codeigniter4:develop Oct 17, 2018
@samsonasik samsonasik deleted the xml-convert branch October 17, 2018 05:02
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