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: Introducing full-featured OO API. It covers bug #46065 also. Old API *_set_* functions actually sets `global' preferences, they are used when creating new object instance. $object->... properties are used object(session)-wise. Previous Comments: ------------------------------------------------------------------------ [2010-12-31 19:52:07] lytboris at gmail dot com 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 ------------------------------------------------------------------------ [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. ------------------------------------------------------------------------ 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