Error: No idea.....

NZ_Kiwis

I'm trying to send data to my PHP script. It works fine until I send Body information and then I get this error

Warning: Missing argument 1 for AddNews() in /home/mysite/public_html/addnews.php on line 20

Line 20 is the last line in the below code and $Body is in the HTML tags below. Whats wrong and how can I fix it?

<?php
// Pull in the NuSOAP code
require_once('lib/nusoap.php');
// Create the server instance
$server = new soap_server();
// Initialize WSDL support
$server->configureWSDL('AddNewswsdl', 'urn:hellowsdl');
// Register the method to expose
$server->register('AddNews',                // method name
    array('Header' => 'xsd:string', 'Author' => 'xsd:string', 'Source' => 'xsd:string', 'IMG' => 'xsd:string', 'Body' => 'xsd;string'),         // input parameters
    array('return' => 'xsd:string'),      // output parameters
    'urn:AddNewswsdl',                      // namespace
    'urn:AddNewswsdl#AddNews',                // soapaction
    'rpc',                                // style
    'encoded',                            // use
    'Says hello to the caller'            // documentation
);
// Define the method as a PHP function
function AddNews($Header, $Author, $Source, $IMG, $Body) {


Seven Australian based players were today included in the New Zealand under 18 side to play two tests against the Australian Schoolboys over the next fortnight.

Their selection followed a high-scoring trial at Mt Smart Stadium in which a team of New Zealand based players faced a combination of New Zealand eligible players from Australia. 

Five of them are from Sydney clubs while the other two are living in Brisbane. 

Also named in the team are several players contracted to the Vodafone Warriors including 17-year-old prop Russell Packer and utility Constantine Mika, also 17, who both trained fulltime with the clubs NRL squad throughout 2007. Along with another of the club’s development players - prop Leeson Ah Mau - they were regulars for the Auckland Lions in the New South Wales Premier League this year. 

All three were also in the national under 18 side last year - Packer was captain – while Meli Koliavu, Mose Masoe and Ben Matulino are other survivors from that side. 

The team, coached by Ivan Harrison (Central Districts), is in camp at Aucklands Millennium Institute to prepare for two internationals against the Australian Schoolboys. The first will kick off at 1.00pm at Mt Smart Stadium this Saturday (September 29) with the second test starting at 2.00pm at the Rotorua International Stadium on October 6.


NEW ZEALAND UNDER 18 TEAM


1. ZORAM WATENE Waicoa Bay
2 ALVIN MATAVAO Sydney
3 MARK TALANOA Sydney
4 MELI KOLIAVU Wellington
5 CHIPPIE KORUSTCHUK Canterbury
6 A J TAUMATA Sydney
7 DARREN TONIHI Canterbury
8 RUSSELL PACKER Manawatu
9 MANAIA OSBOURNE Wellington
10 LEESON AH MAU Auckland
11 BEN MATULINO Wellington
12 JASON SHIRNACK Sydney
13 ROMAN LEOLAHI Sydney
14 MOSE MASOE Wellington
15 CONSTANTINE MIKA Auckland
16 KENNY EDWARDS Brisbane
17 RICARDO PARATA Brisbane

18 DANIEL PALAVI Auckland

Standby players. Kurt Kara (Waicoa Bay), Alehana Mara (Wellington), Rusty Bristow (Auckland), Siosia Vave (Sydney)

dagon

show the full AddNews function and your call to it

NZ_Kiwis

My full script is...

<?php
// Pull in the NuSOAP code
require_once('lib/nusoap.php');
// Create the server instance
$server = new soap_server();
// Initialize WSDL support
$server->configureWSDL('AddNewswsdl', 'urn:hellowsdl');
// Register the method to expose
$server->register('AddNews',                // method name
    array('Header' => 'xsd:string', 'Author' => 'xsd:string', 'Source' => 'xsd:string', 'IMG' => 'xsd:string', 'Body' => 'xsd:string'
    ),         // input parameters
    array('return' => 'xsd:string'),      // output parameters
    'urn:AddNewswsdl',                      // namespace
    'urn:AddNewswsdl#AddNews',                // soapaction
    'rpc',                                // style
    'encoded',                            // use
    'Says hello to the caller'            // documentation
);
// Define the method as a PHP function
function AddNews($Header, $Author, $Source, $IMG, $Body) {

mysql_connect (localhost, **, **);
mysql_select_db (**);

$LASTUPDATE = time();
$Body = str_replace ("'", "", $Body); 
$Body = mysql_escape_string($Body);
$result = mysql_query ("INSERT INTO news (`id`,`header`,`author`,`source`,`body`,`LASTUPDATED`) Values('','11$Header','$Author','$Source','$Body','$LASTUPDATE')") or die("INSERT error: ".mysql_error()); 
$id = mysql_insert_id();      

return $id;   

}

// Use the request to (try to) invoke the service
$HTTP_RAW_POST_DATA = isset($HTTP_RAW_POST_DATA) ? $HTTP_RAW_POST_DATA : '';
$server->service($HTTP_RAW_POST_DATA);

?>

And is called by a VB.NET script.

dagon

Missing argument 1 for AddNews() .. i assume the call is incorrect

NZ_Kiwis

Well this part is in VB.NET

Dim Header As String = Add_Header.Text
        Dim Source As String = Add_Source.Text
        Dim Author As String = Add_Author.Text
        Dim Body As String = Add_Body.Text
        Dim User As String = "ZZZTEST" 
        Dim IMG As String = "ZZZTest"

    Dim ReturnID As String
    Try
        ReturnID = ws.AddNews(Header, Author, Source, IMG, Body)
    Catch ex As Exception
        MessageBox.Show(ex.ToString)
        Call Enable()
        Exit Sub
    End Try

And when I debug it Body does have this text in it. Take body out and it works...

NZ_Kiwis

Anybody able to help?/

halojoy

$result = mysql_query ("INSERT INTO news (id,header,author,source,body,LASTUPDATED) Values('','11$Header',

11$Header
????

NZ_Kiwis

Yeah that's there for a reason. Has nothing to do with the problem as I've removed the SQL line as well.

igorlopez

Let me get this straight:
You have a VB script somewhere that is trying to use a php function that somehow is registered (exposed) to the VB script via SOAP...
I have no idea on how that is supposed to work but maybe i can point out some troubleshooting tips.

First you say that everything worked fine.... or rather

It works fine until I send Body information ...

What do you mean by that?
Is it that in an earlier version where you did not send the $Body information everything worked as expected. If so my focus would be to the differences between now and then.

Then when I look at where you are explaining how you are making the call,

Dim Header As String = Add_Header.Text
        Dim Source As String = Add_Source.Text
        Dim Author As String = Add_Author.Text
        Dim Body As String = Add_Body.Text
        Dim User As String = "ZZZTEST" 
        Dim IMG As String = "ZZZTest"

    Dim ReturnID As String
    Try
        ReturnID = ws.AddNews(Header, Author, Source, IMG, Body)
    Catch ex As Exception
        MessageBox.Show(ex.ToString)
        Call Enable()
        Exit Sub
    End Try

I am starting to think that when you say if you remove Body in the above

ws.AddNews(Header, Author, Source, IMG, Body)

that it works, hm hm. Do you mean that without Body in the ws.AddNews call it works with the old php file where you only have 4 arguments to

function AddNews($Header, $Author, $Source, $IMG) {

And when you are executing ws.AddNews with the Body parameter and the modified php file with 5 arguments to AddNews you get the error on missing argumnet for AddNews...

Well this turned only out to be questions rather than answers, sorry but I am still interested and will monitor this thread for the solution.

Good Luck.