From 9719eff91e5fc7382d3ae42dd50580c638f63b93 Mon Sep 17 00:00:00 2001 From: shankar Date: Mon, 5 Apr 2010 14:02:51 +0000 Subject: Fixing invalid error loggings git-svn-id: http://svn.apache.org/repos/asf/axis/axis2/c/core/trunk@930860 13f79535-47bb-0310-9956-ffa450edef68 --- axiom/src/om/om_children_qname_iterator.c | 2 -- axiom/src/soap/soap_body.c | 29 +++++++++++++++++------------ axiom/src/soap/soap_builder.c | 1 + samples/server/notify/notify.c | 2 +- 4 files changed, 19 insertions(+), 15 deletions(-) diff --git a/axiom/src/om/om_children_qname_iterator.c b/axiom/src/om/om_children_qname_iterator.c index d508a9e..b1b8152 100644 --- a/axiom/src/om/om_children_qname_iterator.c +++ b/axiom/src/om/om_children_qname_iterator.c @@ -38,8 +38,6 @@ axiom_children_qname_iterator_create( { axiom_children_qname_iterator_t *iterator = NULL; - AXIS2_ENV_CHECK(env, NULL); - AXIS2_PARAM_CHECK(env->error, current_child, NULL); iterator = (axiom_children_qname_iterator_t *)AXIS2_MALLOC(env->allocator, sizeof(axiom_children_qname_iterator_t)); diff --git a/axiom/src/soap/soap_body.c b/axiom/src/soap/soap_body.c index 8fb7d8f..69ee7ae 100644 --- a/axiom/src/soap/soap_body.c +++ b/axiom/src/soap/soap_body.c @@ -144,21 +144,26 @@ axiom_soap_body_has_fault( } else { - while(!axiom_node_is_complete(soap_body->om_ele_node, env)) + /* This soap body could have been built programatically. Do the following only if soap + * body is created from soap_builder */ + if(soap_body->soap_builder) { - if(axiom_soap_builder_next(soap_body->soap_builder, env) != AXIS2_SUCCESS) + while(!axiom_node_is_complete(soap_body->om_ele_node, env)) { - /* problem in building the SOAP body. Note that has_fault is about soap fault, - * not about problem in building the node. So, even though there is a problem - * building the body, has_fault should be AXIS2_FALSE - */ - break; - } + if(axiom_soap_builder_next(soap_body->soap_builder, env) != AXIS2_SUCCESS) + { + /* problem in building the SOAP body. Note that has_fault is about soap fault, + * not about problem in building the node. So, even though there is a problem + * building the body, has_fault should be AXIS2_FALSE + */ + break; + } - if(soap_body->soap_fault) - { - soap_body->has_fault = AXIS2_TRUE; - break; + if(soap_body->soap_fault) + { + soap_body->has_fault = AXIS2_TRUE; + break; + } } } } diff --git a/axiom/src/soap/soap_builder.c b/axiom/src/soap/soap_builder.c index 0b5829e..f8236ad 100644 --- a/axiom/src/soap/soap_builder.c +++ b/axiom/src/soap/soap_builder.c @@ -151,6 +151,7 @@ axiom_soap_builder_create( soap_builder->soap_version = AXIOM_SOAP12; soap_builder->last_node_status = -1; soap_builder->om_builder = stax_builder; + soap_builder->done = AXIS2_FALSE; status = axiom_soap_builder_identify_soap_version(soap_builder, env, soap_version); if(status != AXIS2_SUCCESS) diff --git a/samples/server/notify/notify.c b/samples/server/notify/notify.c index 80dc5bd..4264854 100644 --- a/samples/server/notify/notify.c +++ b/samples/server/notify/notify.c @@ -59,7 +59,7 @@ axis2_notify_notify( { axis2_char_t *text_str = (axis2_char_t *) axiom_text_get_value(text, env); - AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, "Notification received : %s", text_str); + AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, "Notification received : %s", text_str); } } else -- cgit v1.1-32-gdbae