Страница 124 из 139

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 9:07 am
tarasfrompir
что возвращать то ? как по мне условия вполне правильные даже если случайно и включить ночной режим и не знать об этом - все равно будет говорить при условии что уровень сообщения не выше системного... так по моему прописано в функции say для воспроизведения сообщений(если за полгода ничего не изменилось).
Я ведь честно говоря ничего не фантазировал - а брал из имеющихся обкатанных вариантов...
ПС
По поводу вариантов кода - некоторым писателям модулей вообще отдельное спасибо - красиво внятно, ну и передрать можна.... очень просто

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 9:34 am
skysilver
tarasfrompir1 писал(а): Ср окт 31, 2018 9:07 am как по мне условия вполне правильные
Ну у меня вообще может не быть такого режима и этих свойств. Что тогда?

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 9:46 am
tarasfrompir
skysilver писал(а): Ср окт 31, 2018 9:34 am
tarasfrompir1 писал(а): Ср окт 31, 2018 9:07 am как по мне условия вполне правильные
Ну у меня вообще может не быть такого режима и этих свойств. Что тогда?
ну и нету его и что помоему эта часть отлично справится - or getGlobal(DarknessMode.active)==0 ..
Или может что не так я понимаю... Тыкните носом в пример чтобы раздуплился , ато если упрусь то сам не вижу чего так а не этак...

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 10:16 am
skysilver
tarasfrompir1 писал(а): Ср окт 31, 2018 9:46 am ну и нету его и что помоему эта часть отлично справится - or getGlobal(DarknessMode.active)==0 ..
А условие $levelmes>=$level тогда будет проигнорировано. А оно мне нужно.

Второе - почему режим "темное время суток"? Как он вообще соотносится с голосом? Ночной режим, режим тишины еще куда ни шло, но темное время суток прям совсем не вижу логики.

Третье - добавляются накладные расходы в виде двух запросов к БД на каждый вызов processSubscription()

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 10:33 am
tarasfrompir
skysilver писал(а): Ср окт 31, 2018 10:16 am
tarasfrompir1 писал(а): Ср окт 31, 2018 9:46 am ну и нету его и что помоему эта часть отлично справится - or getGlobal(DarknessMode.active)==0 ..
А условие $levelmes>=$level тогда будет проигнорировано. А оно мне нужно.

Второе - почему режим "темное время суток"? Как он вообще соотносится с голосом? Ночной режим, режим тишины еще куда ни шло, но темное время суток прям совсем не вижу логики.

Третье - добавляются накладные расходы в виде двух запросов к БД на каждый вызов processSubscription()
Логика проста - для дневного режима - не страшно воспроизводить сообщения с любым уровнем - потому что тут не используется функция say. Это же адресное сообщение... даже если мы прописываем его в свое логике - мы ведь хотим его услышать...
А для ночного - дабы избавить нас от собственной забывчивости....
Но я пересмотрю функцию say - и сделаю реквест так как там сделано ... и все...
И на этом я думаю наши умоизвращения и закончатся...
ПС
Как в анекдоте - "со - не дысыс, а как дысал, как дысал...."

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 10:41 am
skysilver
tarasfrompir1 писал(а): Ср окт 31, 2018 10:33 am Логика проста - для дневного режима - не страшно воспроизводить сообщения с любым уровнем - потому что тут не используется функция say. Это же адресное сообщение... даже если мы прописываем его в свое логике - мы ведь хотим его услышать...
А для ночного - дабы избавить нас от собственной забывчивости....
День/ночь - да, логику можно подобрать. Но темно/светло - это уже не та степь.
У меня вот еще тихий час актуален среди дня, тоже нужно режим тишины соблюдать. Поэтому всю эту логику нужно отдавать на откуп юзерам, а не хардкодить в ядре.

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 10:52 am
tarasfrompir
skysilver писал(а): Ср окт 31, 2018 10:41 am
tarasfrompir1 писал(а): Ср окт 31, 2018 10:33 am Логика проста - для дневного режима - не страшно воспроизводить сообщения с любым уровнем - потому что тут не используется функция say. Это же адресное сообщение... даже если мы прописываем его в свое логике - мы ведь хотим его услышать...
А для ночного - дабы избавить нас от собственной забывчивости....
День/ночь - да, логику можно подобрать. Но темно/светло - это уже не та степь.
У меня вот еще тихий час актуален среди дня, тоже нужно режим тишины соблюдать. Поэтому всю эту логику нужно отдавать на откуп юзерам, а не хардкодить в ядре.
Ну ведь день/ночь из стандратной коробки убрали...

Код: Выделить всё

  if ($terminal_rec['LINKED_OBJECT'] && $terminal_rec['LEVEL_LINKED_PROPERTY']) {
   $min_level=(int)getGlobal($terminal_rec['LINKED_OBJECT'].'.'.$terminal_rec['LEVEL_LINKED_PROPERTY']);
  } else {
   $min_level=(int)getGlobal('minMsgLevel');
  }
  if ($level < $min_level) {
   return 0;
  }
сделаю так как написано в sayTo... Правильно?

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 11:12 am
skysilver
tarasfrompir1 писал(а): Ср окт 31, 2018 10:52 am сделаю так как написано в sayTo... Правильно?
На первый взгляд да. )

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 11:16 am
tarasfrompir
skysilver писал(а): Ср окт 31, 2018 11:12 am
tarasfrompir1 писал(а): Ср окт 31, 2018 10:52 am сделаю так как написано в sayTo... Правильно?
На первый взгляд да. )
решил не гемороить - а сделать как было до этого..

Re: Ещё немного обновлений

Добавлено: Ср окт 31, 2018 11:21 am
xor
tarasfrompir1 писал(а): Ср окт 31, 2018 10:52 am
skysilver писал(а): Ср окт 31, 2018 10:41 am
tarasfrompir1 писал(а): Ср окт 31, 2018 10:33 am Логика проста - для дневного режима - не страшно воспроизводить сообщения с любым уровнем - потому что тут не используется функция say. Это же адресное сообщение... даже если мы прописываем его в свое логике - мы ведь хотим его услышать...
А для ночного - дабы избавить нас от собственной забывчивости....
День/ночь - да, логику можно подобрать. Но темно/светло - это уже не та степь.
У меня вот еще тихий час актуален среди дня, тоже нужно режим тишины соблюдать. Поэтому всю эту логику нужно отдавать на откуп юзерам, а не хардкодить в ядре.
Ну ведь день/ночь из стандратной коробки убрали...

Код: Выделить всё

  if ($terminal_rec['LINKED_OBJECT'] && $terminal_rec['LEVEL_LINKED_PROPERTY']) {
   $min_level=(int)getGlobal($terminal_rec['LINKED_OBJECT'].'.'.$terminal_rec['LEVEL_LINKED_PROPERTY']);
  } else {
   $min_level=(int)getGlobal('minMsgLevel');
  }
  if ($level < $min_level) {
   return 0;
  }
сделаю так как написано в sayTo... Правильно?
Еще тут напишу про апдейты в terminals.class.php:
1. А для Windows TTS создаются файлы вида '/cms/cached/voice/sapi_' . md5($message) . '.mp3'. Это не нужно добавить?
2.

Код: Выделить всё

registerError('ssdp_finder', get_class($e) . ', ' . $e->getMessage());  
это что за сообщение, нужно такой модуль ставить?