Solved Problem with movements scripts [TFS 1.0 10.41]

[xardas33]

Hello guys. I have problem with my scripts...
Look at them first:
Movements.xml

<!-- Tiles with Req -->
   <movement type="StepIn" actionid="61000" script="tiles/PremiumBridge.lua"/>
   <movement type="StepIn" actionid="61001" script="tiles/EntranceBridges.lua"/>

tiles/PremiumBridge.lua

function onStepIn(cid, item, position, fromPosition)

   if item.actionid == 61000 then
     if isPlayer(cid) then
       if isPremium(cid) == true then
         doTeleportThing(cid, fromPosition)
         doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, "You need premium account to pass this bridge.")
       end
     end
   end
return true
end

tiles/EntranceBridges.lua

function onStepIn(cid, item, position, fromPosition)

   if item.actionid == 61001 then
     if isPlayer(cid) then
       if getPlayerLevel(cid) < 4 then
         doTeleportThing(cid, fromPosition)
         doPlayerSendTextMessage(cid, MESSAGE_INFO_DESCR, "You need 4 level to pass this bridge.")
       end
     end
   end
return true
end

So, the problem is when I step on scripted tile, it's pushing me back and send text "The tile seems to be protected against unwanted intruders."
How to fix that?

On TFS 0.3.6 V8.2 all scripts worked good.

 

[xardas33]

bump

 

[Junpa]

Change:
<movement type="StepIn" actionid="61000" script="tiles/PremiumBridge.lua"/>
<movement type="StepIn" actionid="61001" script="tiles/EntranceBridges.lua"/>

For:
<movevent event="StepIn" actionid="61000" script="tiles/PremiumBridge.lua"/>
<movevent event="StepIn" actionid="61001" script="tiles/EntranceBridges.lua"/>

Also the Premium bridge script is not working as you want. You want to allow players with premium account pass, so this line should be:
"if isPremium(cid) == false then"

 

[Printer]

You need also convert them to metatables, we have dropped the support for compat.lua

If you are using the ORTS datapack, also it always good to convert your old scripts to meta:

function onStepIn(cid, item, position, fromPosition)
    local player = Player(cid)
    if not player then
        return false
    end

    if player:getPremiumDays() == 0 then
        player:sendTextMessage(MESSAGE_INFO_DESCR, 'You need premium account to pass this bridge.')
        player:teleportTo(fromPosition, true)
        return false
    end
    return true   
end

function onStepIn(cid, item, position, fromPosition)
    local player = Player(cid)
    if not player then
        return false
    end

    if player:getLevel() < 4 then
        player:sendTextMessage(MESSAGE_INFO_DESCR, 'You need atleast level 4 to pass this bridge.')
        player:teleportTo(fromPosition, true)
        return false
    end
    return true
end

 

[Summ]

There are tiles which are already registered to tiles.lua, which might be executed instead of your script therefore.

<movevent event="StepIn" itemid="416" script="tiles.lua"/>
<movevent event="StepOut" itemid="417" script="tiles.lua"/>
<movevent event="StepIn" itemid="426" script="tiles.lua"/>
<movevent event="StepOut" itemid="425" script="tiles.lua"/>
<movevent event="StepIn" itemid="446" script="tiles.lua"/>
<movevent event="StepOut" itemid="447" script="tiles.lua"/>
<movevent event="StepIn" itemid="3216" script="tiles.lua"/>
<movevent event="StepOut" itemid="3217" script="tiles.lua"/>
<movevent event="StepIn" itemid="11062" script="tiles.lua"/>
<movevent event="StepOut" itemid="11063" script="tiles.lua"/>

 

[xardas33]

Thank you all, i changed all, converted scripts and it doesn't work.... I tried to remove:

<movevent event="StepIn" itemid="446" script="tiles.lua"/>
<movevent event="StepOut" itemid="447" script="tiles.lua"/>

But it still doesn't work ;/ Same error..
I don't use ORTS.

 

[xardas33]

bump

 

[xardas33]

Oh I forgot about change movements to movevents.... Now it's working... Thanks Simple & Perfect