Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@masiuchi
Copy link
Contributor

$sth cannot be used in $dbd->fetch_id.
https://github.com/sixapart/data-objectdriver/blob/0.11/lib/Data/ObjectDriver/Driver/DBI.pm#L406

$sth is closed before calling $dbd->fetch_id.
https://github.com/sixapart/data-objectdriver/blob/0.11/lib/Data/ObjectDriver/Driver/DBI.pm#L398

I want to change the closing timing of $sth.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.52% when pulling ca0300c on movabletype:modify_sth_close_timing into 4cfa74c on sixapart:master.

ziguzagu added a commit that referenced this pull request May 14, 2015
Change the closing timing of $sth when inserting/creating records.
@ziguzagu ziguzagu merged commit 2818c3e into sixapart:master May 14, 2015
@ziguzagu
Copy link
Contributor

thanks

ziguzagu added a commit that referenced this pull request May 18, 2015
Changelog diff is:

diff --git a/Changes b/Changes
index 17ae8ba..0cb62d5 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,9 @@
 {{$NEXT}}

+0.12 2015-05-18T05:33:33Z
+
+    - Fix broken $sth at inserting new records (masiuchi) #19
+
 0.11 2015-03-28T10:04:24Z

     - Fix test (ambs) #16
ziguzagu added a commit that referenced this pull request May 20, 2015
Fix the order of processes changed in pullreq #19.
ziguzagu added a commit that referenced this pull request May 20, 2015
Changelog diff is:

diff --git a/Changes b/Changes
index 0cb62d5..53c63c4 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,9 @@
 {{$NEXT}}

+0.13 2015-05-20T02:26:46Z
+
+    - Fix 0.12 changes with braking inserting new records (masiuchi) #20
+
 0.12 2015-05-18T05:33:33Z

     - Fix broken $sth at inserting new records (masiuchi) #19
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