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: trunk Block user comment: N Private report: N New Comment: OK. There will be full-featured OO API without adding new functions into legacy API (e.g. no session support). Though old functions will be provided with two features (in sake of code cimplicity): * multi-OID * strict output typing Previous Comments: ------------------------------------------------------------------------ [2010-12-29 18:22:55] paj...@php.net No need to deprecate it, but tell anyone asking for a feature for the legacy API to migrate to the new shiny one :) ------------------------------------------------------------------------ [2010-12-29 18:21:57] paj...@php.net It is much easier and cleaner to simply go straight to OO then, without procedural API, and keep the old for legacy apps. ------------------------------------------------------------------------ [2010-12-29 18:17:11] lytboris at gmail dot com You have guessed my next target - OO interface. :-) If you apply new patch you'll see there is very small piece of code making SNMPv1 functions be polymorphic - dozen of lines, not more. My thoughts are that old API should be deprecated sometime, so there will be session-only support: both OO and non-OO. ------------------------------------------------------------------------ [2010-12-29 18:02:23] paj...@php.net What do you think about keeping the old APIs as it is, froze it and add a much nicer and flexible OO one instead? For any new improvements? That's what I did for zip and brings much more rooms for new stuff while reducing the maintenance work load. ------------------------------------------------------------------------ [2010-12-29 17:56:28] lytboris at gmail dot com Reuploaded patches (to be allied against trunk) + session-like workflow: $session = snmp_session_open(...); $result = snmpget($oid); ... snmp_session_close($session); + support for SNMPv3 contextName & contextEngineID properties in new syntax style + tests for new features snmp(real)?{get,walk,set} functions are now polymorphic - they accepts both old (SNMPv1) arguments and new session-like. All version protocols are handled by the since protocol version is now set using snmp_session_open. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/bug.php?id=53594 -- Edit this bug report at http://bugs.php.net/bug.php?id=53594&edit=1