[asterisk-dev] [Code Review] 3346: [res_config_pgsql] Correct handling of nullable int fields in update_realtime

[zvision]

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

(Updated March 17, 2014, 9:57 a.m.)


Review request for Asterisk Developers.


Changes
-------

Double count++ fixed. Set all columns to NULL except for text ones, if a parameter value is empty.


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


Repository: Asterisk


Description
-------

This patch fixes handling of nullable int columns in update_realtime function. It checks if a value is empty and sets the column to NULL instead of '', which raises an error.
Additionally, it checks for existence of the keyfield column instead of the first parameter column.


Diffs (updated)
-----

  http://svn.asterisk.org/svn/asterisk/branches/1.8/res/res_config_pgsql.c 410663 

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


Testing
-------

Only tested for successful compilation. Someone needs to confirm that the patch works fine.


Thanks,

zvision

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140317/7002ceaa/attachment.html>