[asterisk-dev] [Code Review] 3375: [res_config_odbc/res_odbc] Fixed handling of non-text columns updates with empty values

zvision reviewboard at asterisk.org
Thu Mar 27 23:24:12 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3375/
-----------------------------------------------------------

(Updated March 27, 2014, 11:24 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 411399


Bugs: ASTERISK-23459
    https://issues.asterisk.org/jira/browse/ASTERISK-23459


Repository: Asterisk


Description
-------

This review request is a patch for the issue reviewed in https://reviewboard.asterisk.org/r/3335 but for non-trunk versions.

Changes:
- correct check for keyfield existence,
- new res_odbc.conf variable allow_empty_string_in_nontext,
- proper empty string => NULL conversion with allow_empty_string_in_nontext option disabled
  for non-text columns.

This patch is based on branch 11 svn, but code for 1.8 and 10 is the same, so it should apply with no problems.


Diffs
-----

  http://svn.asterisk.org/svn/asterisk/branches/11/res/res_odbc.exports.in 411017 
  http://svn.asterisk.org/svn/asterisk/branches/11/res/res_odbc.c 411017 
  http://svn.asterisk.org/svn/asterisk/branches/11/res/res_config_odbc.c 411017 
  http://svn.asterisk.org/svn/asterisk/branches/11/include/asterisk/res_odbc.h 411017 
  http://svn.asterisk.org/svn/asterisk/branches/11/configs/res_odbc.conf.sample 411017 

Diff: https://reviewboard.asterisk.org/r/3375/diff/


Testing
-------

Tested with Asterisk 11 + ODBC + SIP realtime + PostgreSQL with both allow_empty_string_in_nontext settings
to ensure no regression is introduced.


Thanks,

zvision

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140328/9edd81e3/attachment.html>


More information about the asterisk-dev mailing list