[svn-commits] mjordan: trunk r411513 - /trunk/

SVN commits to the Digium repositories svn-commits at lists.digium.com
Fri Mar 28 11:49:15 CDT 2014


Author: mjordan
Date: Fri Mar 28 11:49:09 2014
New Revision: 411513

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=411513
Log:
Blocked revisions 411512

........
res_config_odbc/res_odbc: Fix handling of non-text columns updates with empty values.

This patch fixes setting nullable integer columns to NULL instead of an empty
string, which fails for PostgreSQL, for example. The current code is supposed
to do so, but the check is broken. The patch also allows the first column in
the list to be a nullable integer.

This patch also adds a compatibility setting in res_odbc.conf,
allow_empty_string_in_nontext. It is enabled by default. It should be disabled
for database backends (such as PostgreSQL) that require NULL instead of an
empty string for Integer columns.

Review: https://reviewboard.asterisk.org/r/3375

(issue ASTERISK-23459)
Reported by: zvision
patches:
  res_config_odbc.diff uploaded by zvision (License 5755)
........

Merged revisions 411399 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 411408 from http://svn.asterisk.org/svn/asterisk/branches/11

Modified:
    trunk/   (props changed)

Propchange: trunk/
------------------------------------------------------------------------------
--- branch-12-blocked (original)
+++ branch-12-blocked Fri Mar 28 11:49:09 2014
@@ -1,1 +1,1 @@
-/branches/12:398559,398578,399306,401391,403291,403779-403780,405350,405587,409762,410006
+/branches/12:398559,398578,399306,401391,403291,403779-403780,405350,405587,409762,410006,411512




More information about the svn-commits mailing list