[HELP] Change value of XML file

[[M]ister]

Hello I am creating a system for teams and plan to use xml to save data from gangs. But I'm having some difficulty using it and I would do the following: Well I'm in the team 'Gang1' and type in the command ex: '/ gang tag ' and with this he change the value 'tag' of the xml file .

gangs.xml

<teams> 
    <team name="Gang1" tag="TAG1" color="#FF9e00"></team> 
    <team name="Gang2" tag="TAG2" color="#00FF00"></team> 
    <team name="Gang3" tag="TAG3" color="#777777"></team> 
</teams> 

script.lua

addCommandHandler("gang", function(player,cmd,subcmd,newtag) 
    if subcmd == "tag" then 
    -- help me with code ? 
    end 
end) 

Could anyone help me with this?

[Bonsai]

I think the wiki has some nice examples how to use xml functions.

Overall, its nothing to hard.

[Et-win]

xmlNodeGetChildren 
xmlNodeGetAttribute 

[[M]ister]

It changes the value of the 'tag' in XML, but prints the message 'Error' twice

addCommandHandler("gang", function(player,cmd,subcmd,newtag) 
    if subcmd == "tag" then 
        local xml = xmlLoadFile("gangs.xml") 
        local children = xmlNodeGetChildren(xml) 
        teams = {} 
        for i,nodes in pairs(children) do 
            teams[i] = xmlNodeGetAttribute(nodes,"name") 
            if teams[i] == getTeamName(getPlayerTeam(player)) then 
                xmlNodeSetAttribute(nodes,"tag",newtag) 
            else 
                outputChatBox("Error") 
            end 
        end 
        xmlSaveFile(xml) 
        xmlUnloadFile(xml) 
    end 
end) 

[Et-win]

Remove after teams.

[[M]ister]

the 'Error' still continues to appear twice

[Et-win]

Never mind what I said about , I looked wrong. But why putting it into a table and not just a variable?

[[M]ister]

I had tried just with variables, but as the error persisted I decided to try several things '-'

[Et-win]

Well, it's this line atleast:

if teams[i] == getTeamName(getPlayerTeam(player)) then 

Put 'getTeamName(getPlayerTeam(player))' into a variable and output it, also for 'teams' and check or they are both the same. 1 of these must be wrong.

[[M]ister]

addCommandHandler("gang", function(player,cmd,subcmd,newtag) 
    if subcmd == "tag" then 
        local xml = xmlLoadFile("gangs.xml") 
        local children = xmlNodeGetChildren(xml) 
        teams = {} 
        for i,nodes in pairs(children) do 
            teams[i] = xmlNodeGetAttribute(nodes,"name") 
            teamName = getTeamName(getPlayerTeam(player)) 
            outputChatBox("Team name in xml: "..teams[i].."  Team name of player: "..teamName) 
            if teams[i] == teamName then 
                xmlNodeSetAttribute(nodes,"tag",newtag) 
                --outputChatBox("You changed the tag") 
            else 
                --outputChatBox("Error") 
            end 
        end 
        xmlSaveFile(xml) 
        xmlUnloadFile(xml) 
    end 
end) 

[Bonsai]

Why do you store the team anyway? Couldn't you just go through all nodes, get the name, and compare with the team name?

  
if xmlNodeGetAttribute(nodes,"name") == teamName then 
   xmlNodeSetAttribute(nodes,"tag",newtag) 
   return 
end 
  

[Et-win]

Why do you store the team anyway? Couldn't you just go through all nodes, get the name, and compare with the team name?

  
if xmlNodeGetAttribute(nodes,"name") == teamName then 
   xmlNodeSetAttribute(nodes,"tag",newtag) 
   return 
end 
  

Because it can return false. And why would you check something that failed? Check or it's false and if so output it, always handy for typos in XML file.