Skip to content
This repository

ODBC error #287

Closed
wants to merge 6 commits into from

1 participant

M. Vugteveen
M. Vugteveen

This is a fix for issue #24. The current ODBC driver calls the parent CI_DB, this should be CI_DB_driver.

Michiel Vugt... added some commits August 23, 2011
M. Vugteveen

mmm there's something wrong... The correct commits are:

I'm sorry, I am new to Git and Github...

oci8 quote fix 931ff69
M. Vugteveen it-can closed this August 24, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 6 unique commits by 1 author.

Aug 23, 2011
readme file 709569b
Aug 24, 2011
readme weg 1111ad0
Merge remote-tracking branch 'upstream/develop' into develop 6ea67c6
incorrect parent call 50e0505
changelog update e98d4f9
oci8 quote fix 931ff69
This page is out of date. Refresh to see the latest.
1  system/database/drivers/oci8/oci8_driver.php
@@ -404,6 +404,7 @@ function escape_str($str, $like = FALSE)
404 404
 		}
405 405
 
406 406
 		$str = remove_invisible_characters($str);
  407
+		$str = str_replace("'", "''", $str);
407 408
 
408 409
 		// escape LIKE condition wildcards
409 410
 		if ($like === TRUE)
2  system/database/drivers/odbc/odbc_driver.php
@@ -50,7 +50,7 @@ class CI_DB_odbc_driver extends CI_DB {
50 50
 
51 51
 	function CI_DB_odbc_driver($params)
52 52
 	{
53  
-		parent::CI_DB($params);
  53
+		parent::CI_DB_driver($params);
54 54
 
55 55
 		$this->_random_keyword = ' RND('.time().')'; // database specific random keyword
56 56
 	}
2  user_guide/changelog.html
@@ -106,6 +106,8 @@
106 106
 	<li>Fixed a bug (#160) - Removed unneeded array copy in the file cache driver.</li>
107 107
 	<li>Fixed a bug (#150) - <samp>field_data()</samp> now correctly returns column length.</li>
108 108
 	<li>Fixed a bug (#8) - <samp>load_class()</samp> now looks for core classes in <samp>APPPATH</samp> first, allowing them to be replaced.</li>
  109
+	<li>Fixed a bug (#24) - ODBC database driver called incorrect parent.</li>
  110
+	<li>Fixed a bug (#85) - OCI8 (Oracle) database driver escaping quotes did not work correctly.</li>
109 111
 </ul>
110 112
 
111 113
 <h2>Version 2.0.3</h2>
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.