Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Wrong type: $message is array, not string #89

Closed
bitbucket-import opened this Issue · 0 comments

2 participants

@bitbucket-import

In DB_Driver.php, the following code near line 1163 is setting $message to a strong:

{{{
#!php

    if ($native == TRUE)
    {
        $message = $error;
    }

}}}

It should be

{{{
#!php
$message[] = $error;
}}}

because later in the code $message is expected to be an array, but $error is a string.

@derekjones derekjones referenced this issue from a commit
@philsturgeon philsturgeon Fixed a bug (Reactor #89) where MySQL export would fail if the table …
…had hyphens or other non alphanumeric/underscore characters.
b62cf1e
@toopay toopay referenced this issue from a commit
@toopay toopay Merge branch 'develop' into debugger
* develop: (65 commits)
  Fixing the documentation link in the Unit_test library so that it points to the correct page
  Fixing the documentation link in the Typography library so that it's valid
  Fixing the Encryption link in the Sha1 library so that it's valid
  Fixing the documentation url given in the Table library
  Update the ChangeLog
  Suppress warnings generated by get_magic_quotes_gpc() (issue #467)
  Skip is_array() check
  Fix a variable type mismatch (issue #89) in system/database/DB_driver.php
  Escape WHERE clause field names in the DB update_string() method
  fixed issue #192
  Add brackets to the for() loop
  Add ->db->insert_batch() support to the OCI8 (Oracle) driver
  Documented third $after_field parameter of dbforge->add_column()
  Added a condition so that the previous link respects use_page_numbers configuration.
  Fix issue #182 in system/database/drivers/oci8_result.php by caching the num_rows property after statement execution
  Updating changelog
  Fixed style guide suggestion from philsturgeon on code review
  Migrations: Added a config that allows the system to migration to the latest migration when you load the library. This way you do not have to call migrations anywhere else in your code and can always be at the latest migration
  Added support to select the name of the database table you are going to use in Migrations
  Updated ChangeLog.
  ...
b3c00ce
@sviande sviande referenced this issue from a commit in sviande/CodeIgniter
@philsturgeon philsturgeon Fixed a bug (Reactor #89) where MySQL export would fail if the table …
…had hyphens or other non alphanumeric/underscore characters.
1eac784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.