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

V3.2 #1324

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

V3.2 #1324

wants to merge 31 commits into from

Conversation

matthiaz
Copy link
Contributor

No description provided.

spekary and others added 30 commits March 19, 2017 09:07
Makes a Blob type in PHP more akin to a Binary type. Text and VarChars are now both Text types in PHP, since PHP really doesn't care how the text is saved in the database.
…This is an important step to being able to move the database adapters to their own repository. Also adds a basic unit test for querying cursors.
…ency

bugfix-RemoveQueryBuilderDependency
InsertOrUpdate is a great method that is currently only used by the dbsessionhandler. It allows you to remove a query to determine if you want to insert or update because mysql will do this for you. 
Unfortunately, when you use this method, you didn't get the LAST_INSERT_ID on insert. I added this, plus when there is a duplicate, qcubed will return the id found in the update. 

To simplify the INSERT statement I also used the alternative INSERT syntax. INSERT INTO tablename SET col=val in stead of INSERT INTO tablename (col) values(val)

I'm running dbbackedsessionhandler in production.and it sees no difference between the old and new function. So there shouldn't be an impact on the rest of qcubed.
InsertOrUpdate now returns the autoincremented ID
…e the devtools on deploy, but lets make sure that this thing cannot run without EnableProfiling turned on to make sure.
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.

None yet

2 participants