Skip to content

Commit 30aa95b

Browse files
committed
PHPC-152: Throw MongoDB\Driver\InvalidArgumentException on argument errors
1 parent 5ce5b70 commit 30aa95b

File tree

4 files changed

+85
-1
lines changed

4 files changed

+85
-1
lines changed

config.m4

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ if test "$PHONGO" != "no"; then
190190
PHONGO_MONGODB_EXCEPTIONS="\
191191
src/MongoDB/Exception.c \
192192
src/MongoDB/RuntimeException.c \
193+
src/MongoDB/InvalidArgumentException.c \
193194
src/MongoDB/ConnectionException.c \
194195
src/MongoDB/AuthenticationException.c \
195196
src/MongoDB/SSLConnectionException.c \

php_phongo.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ zend_class_entry* phongo_exception_from_phongo_domain(php_phongo_error_domain_t
7474
{
7575
switch (domain) {
7676
case PHONGO_ERROR_INVALID_ARGUMENT:
77-
return spl_ce_InvalidArgumentException;
77+
return php_phongo_invalidargumentexception_ce;
7878
case PHONGO_ERROR_RUNTIME:
7979
return php_phongo_runtimeexception_ce;
8080
case PHONGO_ERROR_MONGOC_FAILED:
@@ -1587,6 +1587,7 @@ PHP_MINIT_FUNCTION(phongo)
15871587

15881588
PHP_MINIT(Exception)(INIT_FUNC_ARGS_PASSTHRU);
15891589
PHP_MINIT(RuntimeException)(INIT_FUNC_ARGS_PASSTHRU);
1590+
PHP_MINIT(InvalidArgumentException)(INIT_FUNC_ARGS_PASSTHRU);
15901591
PHP_MINIT(ConnectionException)(INIT_FUNC_ARGS_PASSTHRU);
15911592
PHP_MINIT(AuthenticationException)(INIT_FUNC_ARGS_PASSTHRU);
15921593
PHP_MINIT(SSLConnectionException)(INIT_FUNC_ARGS_PASSTHRU);

php_phongo_classes.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,7 @@ extern PHONGO_API zend_class_entry *php_phongo_writeresult_ce;
209209

210210
extern PHONGO_API zend_class_entry *php_phongo_exception_ce;
211211
extern PHONGO_API zend_class_entry *php_phongo_runtimeexception_ce;
212+
extern PHONGO_API zend_class_entry *php_phongo_invalidargumentexception_ce;
212213
extern PHONGO_API zend_class_entry *php_phongo_connectionexception_ce;
213214
extern PHONGO_API zend_class_entry *php_phongo_authenticationexception_ce;
214215
extern PHONGO_API zend_class_entry *php_phongo_sslconnectionexception_ce;
@@ -248,6 +249,7 @@ PHP_MINIT_FUNCTION(WriteResult);
248249

249250
PHP_MINIT_FUNCTION(Exception);
250251
PHP_MINIT_FUNCTION(RuntimeException);
252+
PHP_MINIT_FUNCTION(InvalidArgumentException);
251253
PHP_MINIT_FUNCTION(ConnectionException);
252254
PHP_MINIT_FUNCTION(AuthenticationException);
253255
PHP_MINIT_FUNCTION(SSLConnectionException);
Lines changed: 80 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,80 @@
1+
/*
2+
+---------------------------------------------------------------------------+
3+
| PHP Driver for MongoDB |
4+
+---------------------------------------------------------------------------+
5+
| Copyright 2013-2014 MongoDB, Inc. |
6+
| |
7+
| Licensed under the Apache License, Version 2.0 (the "License"); |
8+
| you may not use this file except in compliance with the License. |
9+
| You may obtain a copy of the License at |
10+
| |
11+
| http://www.apache.org/licenses/LICENSE-2.0 |
12+
| |
13+
| Unless required by applicable law or agreed to in writing, software |
14+
| distributed under the License is distributed on an "AS IS" BASIS, |
15+
| WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
16+
| See the License for the specific language governing permissions and |
17+
| limitations under the License. |
18+
+---------------------------------------------------------------------------+
19+
| Copyright (c) 2014, MongoDB, Inc. |
20+
+---------------------------------------------------------------------------+
21+
*/
22+
23+
#ifdef HAVE_CONFIG_H
24+
# include "config.h"
25+
#endif
26+
27+
/* External libs */
28+
#include <bson.h>
29+
#include <mongoc.h>
30+
31+
/* PHP Core stuff */
32+
#include <php.h>
33+
#include <php_ini.h>
34+
#include <ext/standard/info.h>
35+
#include <Zend/zend_interfaces.h>
36+
#include <ext/spl/spl_iterators.h>
37+
/* Our Compatability header */
38+
#include "php_compat_53.h"
39+
40+
/* Our stuffz */
41+
#include "php_phongo.h"
42+
#include "php_bson.h"
43+
#include <ext/spl/spl_exceptions.h>
44+
45+
46+
PHONGO_API zend_class_entry *php_phongo_invalidargumentexception_ce;
47+
48+
/* {{{ MongoDB\Driver\InvalidArgumentException */
49+
50+
static zend_function_entry php_phongo_invalidargumentexception_me[] = {
51+
PHP_FE_END
52+
};
53+
54+
/* }}} */
55+
56+
57+
/* {{{ PHP_MINIT_FUNCTION */
58+
PHP_MINIT_FUNCTION(InvalidArgumentException)
59+
{
60+
(void)type; (void)module_number;
61+
zend_class_entry ce;
62+
63+
INIT_NS_CLASS_ENTRY(ce, "MongoDB\\Driver", "InvalidArgumentException", php_phongo_invalidargumentexception_me);
64+
php_phongo_invalidargumentexception_ce = zend_register_internal_class_ex(&ce, spl_ce_InvalidArgumentException, NULL TSRMLS_CC);
65+
zend_class_implements(php_phongo_invalidargumentexception_ce TSRMLS_CC, 1, php_phongo_exception_ce);
66+
67+
return SUCCESS;
68+
}
69+
/* }}} */
70+
71+
72+
73+
/*
74+
* Local variables:
75+
* tab-width: 4
76+
* c-basic-offset: 4
77+
* End:
78+
* vim600: noet sw=4 ts=4 fdm=marker
79+
* vim<600: noet sw=4 ts=4
80+
*/

0 commit comments

Comments
 (0)