Edit report at http://bugs.php.net/bug.php?id=53594&edit=1
ID: 53594 Comment by: lytboris at gmail dot com Reported by: lytboris at gmail dot com Summary: php-snmp rewrite Status: Feedback Type: Feature/Change Request Package: SNMP related Operating System: irrelevant PHP Version: 5.2.16 Block user comment: N Private report: N New Comment: This patch covers bugs #44193 #45893 #51336 Previous Comments: ------------------------------------------------------------------------ [2010-12-23 07:20:18] lytboris at gmail dot com I know about version lifecycle. But Cacti 087 (Larry mentioned it) runs on 5.2 branch. Here you are patch for trunk. There is no difference between 5.3 and trunk for now so this patch may be appied to 5.3 branch too. ------------------------------------------------------------------------ [2010-12-22 22:07:14] paj...@php.net Thanks for this great patch :) However 5.3 is in maintenance mode (and 5.2 is dead btw), please provide a patch against trunk instead. ------------------------------------------------------------------------ [2010-12-22 20:05:04] lytboris at gmail dot com Patch was created for 5.3 branch and then adopted to be used in 5.2 branch. ------------------------------------------------------------------------ [2010-12-22 19:49:37] lytboris at gmail dot com Description: ------------ The main new feature is multi OID get/getnext/set commands. Another one - strong and simple return value logic: if command fails, return nothing but FALSE. No empty strings, no SNMP error messages as values, etc. Just FALSE. Another effort was to cover source code with unit tests. Results: 100% functions (-zm_info_snmp, but it is not actually snmp function), 94% source code lines. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/bug.php?id=53594&edit=1
