Is this possible?

[Margolin]

Hello OTLand. Im wondering if this is possible:

Player can only buy a house if he is a citizen of the city where he pretends to buy.
Example: !buyhouse CANCEL-> You need to be a citizen of Thais to buy this house.


When you look into a player it says which city he's from and if owns a house.
Example: He is a citizen of Thais.

NPC can sell a citizenship for cash or items.

 

[Sajgon]

Yes this is possible, go to talkactions > scripts > buyhouse.lua and add this code

if player:getTown() ~= house:getTown() then
    player:sendCancelMessage("You must be a citizen of this town.")
    return false
end

I suppose you use TFS 1+

 

[Margolin]

This symbol is giving me an error "~"

Yes this is possible, go to talkactions > scripts > buyhouse.lua and add this code

if player:getTown ~= house:getTown() then
    player:sendCancelMessage("You must be a citizen of this town.")
    return false
end

I suppose you use TFS 1+

 

[slavi]

getTown() instead of just getTown

bssh, symbol(s) ~= means "not equal"

 

[Margolin]

Still not working

getTown() instead of just getTown

bssh, symbol(s) ~= means "not equal"

 

[Lay]

tfs 1.2+:

function onSay(player, words, param)
    local housePrice = configManager.getNumber(configKeys.HOUSE_PRICE)
    if housePrice == -1 then
        return true
    end

    if not player:isPremium() then
        player:sendCancelMessage("You need a premium account.")
        return false
    end
   
    local position = player:getPosition()
    position:getNextPosition(player:getDirection())

    local tile = Tile(position)
    if not tile then
        player:sendCancelMessage("You have to be looking at the door of the house you would like to buy.")
        return false
    end

    local tile = Tile(position)
    local house = tile and tile:getHouse()
    if not house then
        player:sendCancelMessage("You have to be looking at the door of the house you would like to buy.")
        return false
    end
   
    if player:getTown() ~= house:getTown() then
        player:sendCancelMessage("You must be a citizen of this town.")
        return false
    end
   
    if house:getOwnerGuid() > 0 then
        player:sendCancelMessage("This house already has an owner.")
        return false
    end

    if player:getHouse() then
        player:sendCancelMessage("You are already the owner of a house.")
        return false
    end

    local price = house:getTileCount() * housePrice
    if not player:removeMoney(price) then
        player:sendCancelMessage("You do not have enough money.")
        return false
    end

    house:setOwnerGuid(player:getGuid())
    player:sendTextMessage(MESSAGE_EVENT_ADVANCE, "You have successfully bought this house, be sure to have the money for the rent in the bank.")
    return false
end

 

[Margolin]

Not working for me

tfs 1.2+:

function onSay(player, words, param)
    local housePrice = configManager.getNumber(configKeys.HOUSE_PRICE)
    if housePrice == -1 then
        return true
    end

    if not player:isPremium() then
        player:sendCancelMessage("You need a premium account.")
        return false
    end
  
    local position = player:getPosition()
    position:getNextPosition(player:getDirection())

    local tile = Tile(position)
    if not tile then
        player:sendCancelMessage("You have to be looking at the door of the house you would like to buy.")
        return false
    end

    local tile = Tile(position)
    local house = tile and tile:getHouse()
    if not house then
        player:sendCancelMessage("You have to be looking at the door of the house you would like to buy.")
        return false
    end
  
    if player:getTown() ~= house:getTown() then
        player:sendCancelMessage("You must be a citizen of this town.")
        return false
    end
  
    if house:getOwnerGuid() > 0 then
        player:sendCancelMessage("This house already has an owner.")
        return false
    end

    if player:getHouse() then
        player:sendCancelMessage("You are already the owner of a house.")
        return false
    end

    local price = house:getTileCount() * housePrice
    if not player:removeMoney(price) then
        player:sendCancelMessage("You do not have enough money.")
        return false
    end

    house:setOwnerGuid(player:getGuid())
    player:sendTextMessage(MESSAGE_EVENT_ADVANCE, "You have successfully bought this house, be sure to have the money for the rent in the bank.")
    return false
end

 

[Lay]

Any error? paste here urs script.

 

[Margolin]

Nevermind, It's working fine. I do not know what happened.
Now, is there a way to look at a player and display which city he belongs to and if he owns a house?

 

[Il Knight]

data/events
onLook.
player:getTown()
player:getGuid()

function getHouseByPlayerGUID(playerGUID)
for _, house in ipairs(Game.getHouses()) do
if house:getOwnerGuid() == playerGUID then
return house:getId()
end
end
return nil
end

good luck

 

[Margolin]

Can you tell me exactly how to put this code?

data/events
onLook.
player:getTown()
player:getGuid()

function getHouseByPlayerGUID(playerGUID)
for _, house in ipairs(Game.getHouses()) do
if house:getOwnerGuid() == playerGUID then
return house:getId()
end
end
return nil
end

good luck

 

[Andréew]

@Il Knight i would like to know aswell, ... data > events > scripts > player. but where on OnLook should it go?

 

[Sajgon]

@Margolin

@Il Knight i would like to know aswell, ... data > events > scripts > player. but where on OnLook should it go?

events.xml

<event class="Player" method="onLook" enabled="1" />

scripts > player.lua

function Player:onLook(thing, position, distance)
local description = "You see " .. thing:getDescription(distance)

--// view players house if he got one onLook
if thing:getId() and isPlayer(thing:getId()) and thing:getHouse() then
   description = description .. " Owner of the house " ..  player:getHouse():getName() .. "."
end

self:sendTextMessage(MESSAGE_INFO_DESCR, description)
end

This should work.

 

[Margolin]

@Sajgon after adding this: <event class="Player" method="onLook" enabled="1" /> I can not longer look to anything

 

[StreamSide]

lmao, edit the onlook function, you havent read those post anyways, you only want a fully working code and no opinions about it. That is pretty funny.

 

[Margolin]

@StreamSide Seen

 

[Sajgon]

@Sajgon after adding this: <event class="Player" method="onLook" enabled="1" /> I can not longer look to anything

I updated my post, please try again ^_^

 

[Margolin]

I updated my post, please try again ^_^

Quesiton, where exactly is most convenient to paste that line of code?

 

[Margolin]

Quesiton, where exactly is most convenient to paste that line of code?

Im getting error in function "isPlayer"

 

[Sajgon]

Im getting error in function "isPlayer"

Try the code from my earlier post again, sorry for errors but I am not sure why that wouldn't work. Which TFS do you use?