Skip to content

Commit 2197a45

Browse files
ZNeumannlavarou
ZNeumann
authored and
lavarou
committed
refactor(agent): remove oapi 'clean' callback
1 parent f30642f commit 2197a45

24 files changed

+262
-269
lines changed

‎agent/fw_cakephp.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -331,8 +331,8 @@ void nr_cakephp_enable_2(TSRMLS_D) {
331331
nr_cakephp_name_the_wt_2 TSRMLS_CC);
332332
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
333333
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
334-
nr_php_wrap_user_function_before_after_clean(
335-
NR_PSTR("CakeException::__construct"), nr_cakephp_problem_2, NULL, NULL);
334+
nr_php_wrap_user_function_before_after(
335+
NR_PSTR("CakeException::__construct"), nr_cakephp_problem_2, NULL);
336336
#else
337337
nr_php_wrap_user_function(NR_PSTR("CakeException::__construct"),
338338
nr_cakephp_problem_2 TSRMLS_CC);

‎agent/fw_drupal.c

Lines changed: 10 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -305,10 +305,10 @@ NR_PHP_WRAPPER(nr_drupal_http_request_after) {
305305
= nr_drupal_http_request_get_method(NR_EXECUTE_ORIG_ARGS);
306306

307307
external_params.encoded_response_header
308-
= nr_drupal_http_request_get_response_header(&func_return_value);
308+
= nr_drupal_http_request_get_response_header(NR_GET_RETURN_VALUE_PTR);
309309

310310
external_params.status
311-
= nr_drupal_http_request_get_response_code(&func_return_value);
311+
= nr_drupal_http_request_get_response_code(NR_GET_RETURN_VALUE_PTR);
312312
if (NRPRG(txn) && NRTXN(special_flags.debug_cat)) {
313313
nrl_verbosedebug(
314314
NRL_CAT, "CAT: outbound response: transport='Drupal 6-7' %s=" NRP_FMT,
@@ -331,16 +331,6 @@ NR_PHP_WRAPPER(nr_drupal_http_request_after) {
331331
}
332332
NR_PHP_WRAPPER_END
333333

334-
NR_PHP_WRAPPER(nr_drupal_http_request_clean) {
335-
NR_UNUSED_SPECIALFN;
336-
NR_UNUSED_FUNC_RETURN_VALUE;
337-
(void)wraprec;
338-
339-
NR_PHP_WRAPPER_REQUIRE_FRAMEWORK(NR_FW_DRUPAL);
340-
341-
NRPRG(drupal_http_request_depth) -= 1;
342-
}
343-
NR_PHP_WRAPPER_END
344334
#else
345335

346336
/*
@@ -778,14 +768,6 @@ NR_PHP_WRAPPER(nr_drupal_wrap_module_invoke_all_after) {
778768
}
779769
NR_PHP_WRAPPER_END
780770

781-
NR_PHP_WRAPPER(nr_drupal_wrap_module_invoke_all_clean) {
782-
NR_UNUSED_SPECIALFN;
783-
NR_UNUSED_FUNC_RETURN_VALUE;
784-
(void)wraprec;
785-
nr_drupal_invoke_all_hook_stacks_pop();
786-
}
787-
NR_PHP_WRAPPER_END
788-
789771
#else
790772
NR_PHP_WRAPPER(nr_drupal_wrap_module_invoke_all) {
791773
zval* hook = NULL;
@@ -834,14 +816,14 @@ void nr_drupal_enable(TSRMLS_D) {
834816
nr_drupal_cron_run TSRMLS_CC);
835817
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
836818
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
837-
nr_php_wrap_user_function_before_after_clean(
838-
NR_PSTR("QFormBase::Run"), nr_drupal_qdrupal_name_the_wt, NULL, NULL);
839-
nr_php_wrap_user_function_before_after_clean(
819+
nr_php_wrap_user_function_before_after(
820+
NR_PSTR("QFormBase::Run"), nr_drupal_qdrupal_name_the_wt, NULL);
821+
nr_php_wrap_user_function_before_after(
840822
NR_PSTR("drupal_page_cache_header"), nr_drupal_name_wt_as_cached_page,
841-
NULL, NULL);
842-
nr_php_wrap_user_function_before_after_clean(
823+
NULL);
824+
nr_php_wrap_user_function_before_after(
843825
NR_PSTR("drupal_http_request"), nr_drupal_http_request_before,
844-
nr_drupal_http_request_after, nr_drupal_http_request_clean);
826+
nr_drupal_http_request_after);
845827
#else
846828
nr_php_wrap_user_function(NR_PSTR("QFormBase::Run"),
847829
nr_drupal_qdrupal_name_the_wt TSRMLS_CC);
@@ -860,10 +842,9 @@ void nr_drupal_enable(TSRMLS_D) {
860842
nr_drupal_wrap_module_invoke TSRMLS_CC);
861843
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
862844
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
863-
nr_php_wrap_user_function_before_after_clean(
845+
nr_php_wrap_user_function_before_after(
864846
NR_PSTR("module_invoke_all"), nr_drupal_wrap_module_invoke_all_before,
865-
nr_drupal_wrap_module_invoke_all_after,
866-
nr_drupal_wrap_module_invoke_all_clean);
847+
nr_drupal_wrap_module_invoke_all_after);
867848
#else
868849
nr_php_wrap_user_function(NR_PSTR("module_invoke_all"),
869850
nr_drupal_wrap_module_invoke_all TSRMLS_CC);

‎agent/fw_drupal8.c

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -76,13 +76,12 @@ static void nr_drupal8_add_method_callback(const zend_class_entry* ce,
7676

7777
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
7878
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
79-
static void nr_drupal8_add_method_callback_before_after_clean(
79+
static void nr_drupal8_add_method_callback_before_after(
8080
const zend_class_entry* ce,
8181
const char* method,
8282
size_t method_len,
8383
nrspecialfn_t before_callback,
84-
nrspecialfn_t after_callback,
85-
nrspecialfn_t clean_callback) {
84+
nrspecialfn_t after_callback) {
8685
zend_function* function = NULL;
8786

8887
if (NULL == ce) {
@@ -106,9 +105,9 @@ static void nr_drupal8_add_method_callback_before_after_clean(
106105
"%.*s::%.*s", NRSAFELEN(nr_php_class_entry_name_length(ce)),
107106
nr_php_class_entry_name(ce), NRSAFELEN(method_len), method);
108107

109-
nr_php_wrap_user_function_before_after_clean(
108+
nr_php_wrap_user_function_before_after(
110109
class_method, nr_strlen(class_method),
111-
before_callback, after_callback, clean_callback);
110+
before_callback, after_callback);
112111

113112
nr_free(class_method);
114113
}
@@ -542,12 +541,6 @@ NR_PHP_WRAPPER(nr_drupal94_invoke_all_with_after) {
542541
nr_drupal_invoke_all_hook_stacks_pop();
543542
}
544543
NR_PHP_WRAPPER_END
545-
546-
NR_PHP_WRAPPER(nr_drupal94_invoke_all_with_clean) {
547-
(void)wraprec;
548-
nr_drupal_invoke_all_hook_stacks_pop();
549-
}
550-
NR_PHP_WRAPPER_END
551544
#endif // OAPI
552545

553546
/*
@@ -584,11 +577,10 @@ NR_PHP_WRAPPER(nr_drupal8_module_handler) {
584577
/* Drupal 9.4 introduced a replacement method for getImplentations */
585578
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
586579
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
587-
nr_drupal8_add_method_callback_before_after_clean(
580+
nr_drupal8_add_method_callback_before_after(
588581
ce, NR_PSTR("invokeallwith"),
589582
nr_drupal94_invoke_all_with,
590-
nr_drupal94_invoke_all_with_after,
591-
nr_drupal94_invoke_all_with_clean);
583+
nr_drupal94_invoke_all_with_after);
592584
#else
593585
nr_drupal8_add_method_callback(ce, NR_PSTR("invokeallwith"),
594586
nr_drupal94_invoke_all_with TSRMLS_CC);
@@ -709,10 +701,10 @@ void nr_drupal8_enable(TSRMLS_D) {
709701
*/
710702
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
711703
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
712-
nr_php_wrap_user_function_before_after_clean(
704+
nr_php_wrap_user_function_before_after(
713705
NR_PSTR("Symfony\\Component\\HttpKernel\\EventListe"
714706
"ner\\RouterListener::onKernelRequest"),
715-
nr_drupal8_name_the_wt_via_symfony, NULL, NULL);
707+
nr_drupal8_name_the_wt_via_symfony, NULL);
716708
#else
717709
nr_php_wrap_user_function(NR_PSTR("Symfony\\Component\\HttpKernel\\EventListe"
718710
"ner\\RouterListener::onKernelRequest"),

‎agent/fw_laravel.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -772,8 +772,8 @@ static void nr_laravel5_wrap_middleware(zval* app TSRMLS_DC) {
772772
Z_STRVAL_P(classname));
773773
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
774774
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
775-
nr_php_wrap_user_function_before_after_clean(
776-
name, nr_strlen(name), nr_laravel5_middleware_handle, NULL, NULL);
775+
nr_php_wrap_user_function_before_after(
776+
name, nr_strlen(name), nr_laravel5_middleware_handle, NULL);
777777
#else
778778
nr_php_wrap_user_function(name, nr_strlen(name),
779779
nr_laravel5_middleware_handle TSRMLS_CC);
@@ -833,8 +833,8 @@ static void nr_laravel_add_callback_method(const zend_class_entry* ce,
833833

834834
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
835835
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
836-
nr_php_wrap_user_function_before_after_clean(
837-
class_method, nr_strlen(class_method), callback, NULL, NULL);
836+
nr_php_wrap_user_function_before_after(
837+
class_method, nr_strlen(class_method), callback, NULL);
838838
#else
839839
nr_php_wrap_user_function(class_method, nr_strlen(class_method),
840840
callback TSRMLS_CC);
@@ -1231,9 +1231,9 @@ void nr_laravel_enable(TSRMLS_D) {
12311231
*/
12321232
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
12331233
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
1234-
nr_php_wrap_user_function_before_after_clean(
1234+
nr_php_wrap_user_function_before_after(
12351235
NR_PSTR("Illuminate\\Console\\Application::doRun"),
1236-
nr_laravel_console_application_dorun, NULL, NULL);
1236+
nr_laravel_console_application_dorun, NULL);
12371237
#else
12381238
nr_php_wrap_user_function(NR_PSTR("Illuminate\\Console\\Application::doRun"),
12391239
nr_laravel_console_application_dorun TSRMLS_CC);

‎agent/fw_laravel_queue.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -864,18 +864,18 @@ void nr_laravel_queue_enable(TSRMLS_D) {
864864
* took.
865865
*/
866866

867-
nr_php_wrap_user_function_before_after_clean(
867+
nr_php_wrap_user_function_before_after(
868868
NR_PSTR("Illuminate\\Queue\\Worker::raiseBeforeJobEvent"), NULL,
869-
nr_laravel_queue_worker_raiseBeforeJobEvent_after, NULL);
870-
nr_php_wrap_user_function_before_after_clean(
869+
nr_laravel_queue_worker_raiseBeforeJobEvent_after);
870+
nr_php_wrap_user_function_before_after(
871871
NR_PSTR("Illuminate\\Queue\\Worker::raiseAfterJobEvent"),
872-
nr_laravel_queue_worker_raiseAfterJobEvent_before, NULL, NULL);
873-
nr_php_wrap_user_function_before_after_clean(
872+
nr_laravel_queue_worker_raiseAfterJobEvent_before, NULL);
873+
nr_php_wrap_user_function_before_after(
874874
NR_PSTR("Illuminate\\Queue\\SyncQueue::raiseBeforeJobEvent"),
875-
nr_laravel_queue_syncqueue_raiseBeforeJobEvent_before, NULL, NULL);
876-
nr_php_wrap_user_function_before_after_clean(
875+
nr_laravel_queue_syncqueue_raiseBeforeJobEvent_before, NULL);
876+
nr_php_wrap_user_function_before_after(
877877
NR_PSTR("Illuminate\\Queue\\SyncQueue::raiseAfterJobEvent"),
878-
nr_laravel_queue_worker_raiseAfterJobEvent_before, NULL, NULL);
878+
nr_laravel_queue_worker_raiseAfterJobEvent_before, NULL);
879879

880880
#else
881881

‎agent/fw_lumen.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -225,9 +225,9 @@ void nr_lumen_enable(TSRMLS_D) {
225225
nr_lumen_handle_found_route TSRMLS_CC);
226226
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
227227
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
228-
nr_php_wrap_user_function_before_after_clean(
228+
nr_php_wrap_user_function_before_after(
229229
NR_PSTR("Laravel\\Lumen\\Application::sendExceptionToHandler"),
230-
nr_lumen_exception, NULL, NULL);
230+
nr_lumen_exception, NULL);
231231
#else
232232
nr_php_wrap_user_function(
233233
NR_PSTR("Laravel\\Lumen\\Application::sendExceptionToHandler"),

‎agent/fw_magento2.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -440,9 +440,9 @@ void nr_magento2_enable(TSRMLS_D) {
440440
*/
441441
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
442442
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
443-
nr_php_wrap_user_function_before_after_clean(
443+
nr_php_wrap_user_function_before_after(
444444
NR_PSTR("Magento\\Framework\\App\\Action\\Action::dispatch"),
445-
nr_magento2_action_dispatch, NULL, NULL);
445+
nr_magento2_action_dispatch, NULL);
446446
#else
447447
nr_php_wrap_user_function(
448448
NR_PSTR("Magento\\Framework\\App\\Action\\Action::dispatch"),
@@ -472,10 +472,10 @@ void nr_magento2_enable(TSRMLS_D) {
472472
*/
473473
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
474474
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
475-
nr_php_wrap_user_function_before_after_clean(
475+
nr_php_wrap_user_function_before_after(
476476
NR_PSTR(
477477
"Magento\\Webapi\\Controller\\Rest\\InputParamsResolver::resolve"),
478-
nr_magento2_inputparamsresolver_resolve, NULL, NULL);
478+
nr_magento2_inputparamsresolver_resolve, NULL);
479479
#else
480480
nr_php_wrap_user_function(
481481
NR_PSTR(
@@ -497,14 +497,14 @@ void nr_magento2_enable(TSRMLS_D) {
497497
nr_magento2_soap_iswsdllistrequest TSRMLS_CC);
498498
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
499499
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
500-
nr_php_wrap_user_function_before_after_clean(
500+
nr_php_wrap_user_function_before_after(
501501
NR_PSTR("Magento\\Webapi\\Controller\\Soap\\Request\\Handler::_"
502502
"prepareRequestData"),
503-
nr_magento2_soap_handler_preparerequestdata, NULL, NULL);
504-
nr_php_wrap_user_function_before_after_clean(
503+
nr_magento2_soap_handler_preparerequestdata, NULL);
504+
nr_php_wrap_user_function_before_after(
505505
NR_PSTR("Magento\\Webapi\\Controller\\Soap\\Request\\Handler::"
506506
"prepareOperationInput"),
507-
nr_magento2_soap_handler_prepareoperationinput, NULL, NULL);
507+
nr_magento2_soap_handler_prepareoperationinput, NULL);
508508

509509
#else
510510
nr_php_wrap_user_function(

‎agent/fw_slim.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -181,17 +181,17 @@ void nr_slim_enable(TSRMLS_D) {
181181
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
182182

183183
/* Slim 3 */
184-
nr_php_wrap_user_function_before_after_clean(
185-
NR_PSTR("Slim\\Route::run"), nr_slim3_4_route_run, NULL, NULL);
184+
nr_php_wrap_user_function_before_after(
185+
NR_PSTR("Slim\\Route::run"), nr_slim3_4_route_run, NULL);
186186

187187
/* Slim 4 */
188-
nr_php_wrap_user_function_before_after_clean(
189-
NR_PSTR("Slim\\Routing\\Route::run"), nr_slim3_4_route_run, NULL, NULL);
188+
nr_php_wrap_user_function_before_after(
189+
NR_PSTR("Slim\\Routing\\Route::run"), nr_slim3_4_route_run, NULL);
190190

191191
/* Slim 4 */
192-
nr_php_wrap_user_function_before_after_clean(
192+
nr_php_wrap_user_function_before_after(
193193
NR_PSTR("Slim\\Routing\\Dispatcher::dispatch"), nr_slim4_route_dispatch,
194-
NULL, NULL);
194+
NULL);
195195
#else
196196
/* Slim 4*/
197197
nr_php_wrap_user_function(NR_PSTR("Slim\\Routing\\Route::run"),

‎agent/fw_symfony4.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -269,9 +269,9 @@ void nr_symfony4_enable(TSRMLS_D) {
269269
*/
270270
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
271271
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
272-
nr_php_wrap_user_function_before_after_clean(
272+
nr_php_wrap_user_function_before_after(
273273
NR_PSTR("Symfony\\Component\\Console\\Command\\Command::run"),
274-
nr_symfony4_console_application_run, NULL, NULL);
274+
nr_symfony4_console_application_run, NULL);
275275
#else
276276
nr_php_wrap_user_function(
277277
NR_PSTR("Symfony\\Component\\Console\\Command\\Command::run"),

‎agent/fw_wordpress.c

Lines changed: 10 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -682,16 +682,6 @@ NR_PHP_WRAPPER(nr_wordpress_handle_tag_stack_after) {
682682
}
683683
NR_PHP_WRAPPER_END
684684

685-
NR_PHP_WRAPPER(nr_wordpress_handle_tag_stack_clean) {
686-
NR_UNUSED_SPECIALFN;
687-
NR_UNUSED_FUNC_RETURN_VALUE;
688-
(void)wraprec;
689-
if (0 != NRINI(wordpress_hooks)) {
690-
clean_wordpress_tag_stack(auto_segment);
691-
}
692-
}
693-
NR_PHP_WRAPPER_END
694-
695685
NR_PHP_WRAPPER(nr_wordpress_apply_filters_after) {
696686
/* using nr_php_get_user_func_arg() so that we don't perform another copy
697687
* when all we want to do is check the string length */
@@ -763,8 +753,8 @@ NR_PHP_WRAPPER(nr_wordpress_add_filter) {
763753
if (NULL != wordpress_plugin_theme || NRPRG(wordpress_core)) {
764754
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
765755
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
766-
callback_wraprec = nr_php_wrap_callable_before_after_clean(
767-
zf, NULL, nr_wordpress_wrap_hook, nr_wordpress_wrap_hook);
756+
callback_wraprec = nr_php_wrap_callable_before_after(
757+
zf, NULL, nr_wordpress_wrap_hook);
768758
#else
769759
callback_wraprec = nr_php_wrap_callable(zf, nr_wordpress_wrap_hook);
770760
#endif
@@ -823,22 +813,22 @@ void nr_wordpress_version() {
823813
void nr_wordpress_enable(TSRMLS_D) {
824814
#if ZEND_MODULE_API_NO >= ZEND_8_0_X_API_NO \
825815
&& !defined OVERWRITE_ZEND_EXECUTE_DATA
826-
nr_php_wrap_user_function_before_after_clean(
816+
nr_php_wrap_user_function_before_after(
827817
NR_PSTR("apply_filters"), nr_wordpress_apply_filters,
828-
nr_wordpress_apply_filters_after, nr_wordpress_handle_tag_stack_clean);
818+
nr_wordpress_apply_filters_after);
829819

830820
if (0 != NRINI(wordpress_hooks)) {
831-
nr_php_wrap_user_function_before_after_clean(
821+
nr_php_wrap_user_function_before_after(
832822
NR_PSTR("apply_filters_ref_array"), nr_wordpress_exec_handle_tag,
833-
nr_wordpress_handle_tag_stack_after, nr_wordpress_handle_tag_stack_clean);
823+
nr_wordpress_handle_tag_stack_after);
834824

835-
nr_php_wrap_user_function_before_after_clean(
825+
nr_php_wrap_user_function_before_after(
836826
NR_PSTR("do_action"), nr_wordpress_exec_handle_tag,
837-
nr_wordpress_handle_tag_stack_after, nr_wordpress_handle_tag_stack_clean);
827+
nr_wordpress_handle_tag_stack_after);
838828

839-
nr_php_wrap_user_function_before_after_clean(
829+
nr_php_wrap_user_function_before_after(
840830
NR_PSTR("do_action_ref_array"), nr_wordpress_exec_handle_tag,
841-
nr_wordpress_handle_tag_stack_after, nr_wordpress_handle_tag_stack_clean);
831+
nr_wordpress_handle_tag_stack_after);
842832
if (0 != NRPRG(wordpress_plugins)) {
843833
nr_php_wrap_user_function(NR_PSTR("add_filter"), nr_wordpress_add_filter);
844834
}

0 commit comments

Comments
 (0)