Edit report at https://bugs.php.net/bug.php?id=40816&edit=1

 ID:                 40816
 Comment by:         c...@php.net
 Reported by:        ch at westend dot com
 Summary:            Add "snmptranslate" function
 Status:             Feedback
 Type:               Feature/Change Request
 Package:            SNMP related
 Operating System:   linux
 PHP Version:        5.2.1
 Assigned To:        lytboris
 Block user comment: N
 Private report:     N

 New Comment:

Well, throwing exceptions everywhere is exactly what I'd with for but to 
discuss this I opened #55542. This one can be closed as we now have a way to 
check for the presence of a MIB (more or less convenient).


Previous Comments:
------------------------------------------------------------------------
[2011-08-30 15:18:14] lytbo...@php.net

I have read a bunch of threads on php-dev about extensions throwing exceptions 
and the last thing I remember that extension should not throw any exception 
except object creation if there is no way to disable them.

Anyway, if to throw exceptions, library will throw an exception for each error, 
not only parsing error.

------------------------------------------------------------------------
[2011-08-30 15:10:48] ch at westend dot com

Now the following works..

  @$snmp->get(array($oid1, $oid2));
  if ($snmp->getErrno() == SNMP::ERRNO_OID_PARSING_ERROR) {
    throw new Exception($snmp->getError());
  }

... but why is there only a warning? Calling get() with a OID that does not 
even parses should throw an Exception? Especially as when using OOP, one does 
not expect having to call getErrno() after every call as we did in the last 
millenium or even worse don't catch this error.

Can you wrapt this into a nice SnmpOidParsingException?

------------------------------------------------------------------------
[2011-08-30 14:30:34] lytbo...@php.net

Indeed. Commited sources into PHP_5_4 branch.

------------------------------------------------------------------------
[2011-08-30 09:52:01] ch at westend dot com

As of SVN revision 315762 from now, the constant only appears in the tests, not 
in the class source. Running the tests yield to:

Fatal error: Undefined class constant 'ERRNO_OID_PARSING_ERROR' in 
/home/chammers/workspace/php-src-5.4/ext/snmp/tests/snmp-object-errno-errstr.php
 on line 45

Maybe it's not yet committet?

------------------------------------------------------------------------
[2011-08-30 09:42:08] lytbo...@php.net

Please check that you have ERRNO_OID_PARSING_ERROR in snmp extension sources. 
This constant appeared on 27th of august.

------------------------------------------------------------------------


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

    https://bugs.php.net/bug.php?id=40816


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=40816&edit=1

Reply via email to