Domoticz

mcmikev wrote:Just discovered that It is not the switch name but device name of the iphone

Changed that and now I get more
But get error from line 53

2016-09-14 12:16:19.411 Error: EventSystem: in /home/pi/domoticz/scripts/lua/script_time_checkphones.lua: /home/pi/domoticz/scripts/lua/script_time_checkphones.lua:53: bad argument #1 to 'sub' (string expected, got nil)

prev_dist = tonumber(string.sub(string.match(otherdevices['Position ' .. user], '%(.*%)'), 2,-5))
if prev_distance == nil then prev_distance = 1000 end

prev_distance_str = string.match(otherdevices['Position ' .. user], '%(.*%)') or '(1000 km)'
prev_dist = tonumber(string.sub(prev_distance_str, 2,-5))

mcmikev wrote:Tried this, but same results however.

mvzut wrote:

mcmikev wrote:Just discovered that It is not the switch name but device name of the iphone

Changed that and now I get more
But get error from line 53

2016-09-14 12:16:19.411 Error: EventSystem: in /home/pi/domoticz/scripts/lua/script_time_checkphones.lua: /home/pi/domoticz/scripts/lua/script_time_checkphones.lua:53: bad argument #1 to 'sub' (string expected, got nil)

Ah, that's another thing that apparently needs some polishing. I wanted to only update the text devices when their location has changed significantly (I use the "radius" variable for that again). I get the "old" distance from the text device itself. However, new text devices automatically contain the text "Hello world!", so you won't find any distance in there, and the script stops because LUA doesn't want to work with empty strings. I tried to solve it with the 'if prev_distance == nil then prev_distance = 1000 end" line, to give it an initial value that is large enough so that the text device gets updated, but it doesn't even get to that point apparently. You could change the contents of the text field into something like "(1000 km)" by hand, but I don't find that a very elegant solution. I think I found a better solution now:

Please change these lines

Code: Select all

prev_dist = tonumber(string.sub(string.match(otherdevices['Position ' .. user], '%(.*%)'), 2,-5))
if prev_distance == nil then prev_distance = 1000 end

into this

Code: Select all

prev_distance_str = string.match(a, '%(.*%)')
if prev_distance_str == nil then prev_distance_str = '(1000 km)' end
prev_dist = tonumber(string.sub(prev_distance_str, 2,-5))

mcmikev wrote:Same here.
Error :
2016-09-14 13:38:05.117 Error: EventSystem: in /home/pi/domoticz/scripts/lua/script_time_checkphones.lua: /home/pi/domoticz/scripts/lua/script_time_checkphones.lua:53: bad argument #1 to 'match' (string expected, got nil)

Line:
prev_distance_str = string.match(a, '%(.*%)')

mcmikev wrote:That did it. Working now !!

cattoo wrote:Is the original script updated? So i could let go for the moment and fix everything after work?

cattoo wrote:Okey, i took the script now from the first post, and i still got errors..

Code: Select all

 Error: EventSystem: in Check_iphone: [string "-- Script to check the location of multiple i..."]:53: bad argument #1 to 'match' (string expected, got nil)
2016-09-14 14:35:30.046 H

prev_distance_str = string.match(otherdevices['Position ' .. user], '%(.*%)') or '(1000 km)'

prev_distance_str = string.match(otherdevices_svalues['Position ' .. user], '%(.*%)') or '(1000 km)'

Egregius wrote:Yes, 5 minutes later

deejc wrote:hi,
i got this to work 1 time, but it wont fire any more.
restarted domoticz etc, resaved an event to reset the events system but it does not fire, nothing in the log etc.. very strange.. any ideas if i can force it to run to see if it logs anything ?

mvzut wrote: If your position doesn't change no device will update and you won't see anything in the log. Are you really sure the script is not running? You can check by putting a "print('test')" command in the top, directly after "commandArray = {}". If you don't see this in your log, and you named the script correctly (i.e. script_time_something.lua) then I'm afraid I'm out of advice to give...