Script is working, and shows it is. [aka SOLVED]

[Gamesnert]

Hi again all,

Another problem... Now about an "emergency saving system" for mabako's map editor. (since it won't save now and then, I made this just in case )

The problem is: I let it look where the script has gone wrong, and print that in the server screen. For some reason, the script below gave the last possible error:

"Error: Not all necessary variables could be saved! Save aborted!"

Now if you watch closely, I already wanted to check what was wrong, and put xmlSaveFile just in front of the error. I look in the file, and I see... No errors..? What is quite strange, because it DOES return false, and DOES give the error in the server window.

Can anyone see what I did do wrong? (probably something stupid again... )

function saveVehicleLocation(thePlayer) 
    if (isPlayerInVehicle(thePlayer)) then 
        local vehicle=getPlayerOccupiedVehicle(thePlayer) 
        local model=getVehicleID(vehicle) 
        local x,y,z=getElementPosition(vehicle) 
        local rx,ry,rz=getVehicleRotation(vehicle) 
        rz=(360-rz) 
         
        local xmlFile=xmlLoadFile("vehicles.xml") 
        if xmlFile then 
            local newVehicleLine=xmlCreateSubNode(xmlFile,"vehicle") 
            if newVehicleLine then 
                local sucMod=xmlNodeSetAttribute(newVehicleLine,"model",model) 
                local sucX=xmlNodeSetAttribute(newVehicleLine,"posX",x) 
                local sucY=xmlNodeSetAttribute(newVehicleLine,"posY",y) 
                local sucZ=xmlNodeSetAttribute(newVehicleLine,"posZ",z) 
                local sucRX=xmlNodeSetAttribute(newVehicleLine,"rotX",rx) 
                local sucRY=xmlNodeSetAttribute(newVehicleLine,"rotY",ry) 
                local sucRZ=xmlNodeSetAttribute(newVehicleLine,"rotZ",rz) 
                if (sucMod and sucX and sucY and sucZ and secRX and sucRY and sucRZ) then 
                    xmlSaveFile(xmlFile) 
                    return true 
                else 
                    xmlSaveFile(xmlFile) 
                    outputDebugString("Error: Not all necessary variables could be saved! Save aborted!") 
                    return false 
                end 
            else 
                outputDebugString("Error: New vehicle line was not created!") 
            end 
        else 
            outputDebugString("Error: File could not be opened!") 
            return false 
        end 
    else 
        return nil 
    end 
end  

A final note: This code is being called by another resource, my commands part. Although I don't think there should be any problem in there, I post it just to be sure:

call(getResourceFromName("total-war-XMLdb"),"saveVehicleLocation",thePlayer) 

And the reason for the strange resource name? That's just so not the ENTIRE gamemode has to start over by 1 change, but only the section XMLdb for example.

Edited July 13, 2008 by Guest

[DutchCaffeine]

Do you have in your meta.xml the next line:

<export function="function_name" /> 

[Gamesnert]

Well if I didn't, I wouldn't see this in the server window eh?

"Error: Not all necessary variables could be saved! Save aborted!"

[Phat Looser]

I think, after working through it, that I have the solution of the problem.

When you call a script in LUA, it has the bug that LUA thinks this function is somewhat loaded INSIDE the script that calls it, which means you would have to put your XML file in the resource which calls the function.

Since this is a theory of mine, which I developed after a few tests, I would like to hear your result.

This disabled my anticheat, by the way. It made it impossible to work correctly with multiple resources.

[Gamesnert]

Although it did look like that was the solution, it wasn't...

There HAS to be something with "if (sucMod and sucX and sucY and sucZ and secRX and sucRY and sucRZ) then"(or the places where they're defined) I think... The rest works perfectly! It saves the file correctly and everything... Only wrong message... (and it only saved because I added that to test which var was wrong ... NONE... )

[Phat Looser]

"and secRX and"

trivial, after I looked through it without the intention to save the world.

You even copied and pasted this mistake twice without seeing it.

[Gamesnert]

I knew it would be something stupid... Thanks...