При сканировании устройств получаем (DS18B20 3 штуки)

Fatal error: Call to undefined function bccomp() in /var/www/modules/onewire/ownet.php on line 516

После в списке устройств есть только одно обнаруженное устройство.

28.6164123C64EE (28.6164123C64EE, Online)
Next check: 2018-01-20 22:56:30 

Остальных два в списке нет.

28.6164123C64EE | 28.6164123C64EE | 
28.616412266DFD | 28.616412266DFD | 
28.6164123FF5E5 | 28.6164123FF5E5 | 

I wanted to add new method and discovered that code has different styles inside single class. Some methods use 1 space identation, other methods 2 spaces. Would be nice to have some definition of php coding standards. All contributors will be able to configure favorite IDE and provide new well-formatted code. Legacy code will be formated during various refactorings in future. DoD:

• [ ] Add php coding standards on wiki or link to readme
• [ ] Prepare continuous integration server somewhere (for example travis-ci.org)
• [ ] Configure PHP_CodeSniffer according to coding standards
• [ ] Set trigger to fail build if number of violations are increased

https://github.com/sergejey/majordomo/blob/5271bad4d73fb20afd1097b325b5e46a22350564/lib/mysqli.class.php#L268

Что передача только изменившихся полей, что всей записи обрабатывается mysql одинаково, причем If you set a column to the value it currently has, MySQL notices this and does not update it. https://dev.mysql.com/doc/refman/8.0/en/update.html В чём выигрыш? того же самого можно добиться, передавая только нужные поля+id $jobs['ID'] = $jobs[$i]['ID']; $jobs['PROCESSED'] = 1; $jobs['STARTED'] = date('Y-m-d H:i:s'); SQLUpdate('jobs', $jobs); и, по-прежнему, можно свой индекс выбрать, а не зашитый хардкодом теперь ID

При использовании кеша - все значения берутся из таблицы кеша , если же там они отсутствую то из пвалуе таблицы. Уменьшает нагрузку на накопители что очень полезно для систем НЕ держащих всю базу данных в памяти... Включение-выключение кеширования свойств обьектов производится в общих настройках - система.

Хочется подготовить плацдарм, для того, чтобы можно было использовать какую нибудь другую базу данных (Кроме MySQL). Поэтому нужно вынести всю работу с базой данных в отдельные места. Я попробую потихонечку заняться этим.

В корневой папке проекта лежит небезопасный файл php.php. Он сообщает всю информацию о сервере любому, у кого есть доступ к сайту Majordomo, т.е. фактически систему выставлять в интернет опасно.

Удалена SERVER_URL Вместо нее есть BASE_URL И она в конфиге описана Вот оно if (isset($_ENV["S2G_BASE_URL"]) && $_ENV["S2G_BASE_URL"]) { Define('BASE_URL', $_ENV["S2G_BASE_URL"]); } else { // Your base URL:port (!!!) Define('BASE_URL', 'http://127.0.0.1:80');
}

Исправления, связанные с работой из подкаталога.

И, Сергей, такой вопрос, константа BASE_URL в файле config.php подразумевает только адрес и порт, или ещё и подкаталог, в котором располагается Мажордомо? Просто, я столкнулся с тем, что в некоторых местах системы к BASE_URL добавляется ROOTHTML, а в некоторых нет. Какова изначальная идея переменной? В какую сторону идеологически правильно будет править код?

Некорректный урл вебсокета т.к. порт и часть адреса прописаны не в настройках, а в коде. Выявилось при попытке открыть список городов в приложении OpenWeather. Список так и не открылся.

MajorDoMo установлен из исходников gitgub'a. Система: Linux 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt25-2+deb8u3 (2016-07-02) x86_64 GNU/Linux

\templates\websockets.html

 function startWebSockets() {
            var loc = window.location, new_uri;
            var serverUrl='';
            if (loc.protocol === "https:") {
                    serverUrl = "wss:";
            } else {
                    serverUrl = "ws:";
            }
            serverUrl += "//" + loc.host + ':<#WEBSOCKETS_PORT#>/majordomo';
...

\load_settings.php

if (!defined('WEBSOCKETS_PORT'))
   Define('WEBSOCKETS_PORT', 8001);

Как результат получаем это:

Вариант решения: Вынести в config.php порт web-сокета и его адрес.

При нажатии выключателя в меню в связанный метод не передаются парметр VALUE в xray видится так

.switch {" VALUE" :" " ," OLD_VALUE" :" 0" ," ORIGINAL_OBJECT_TITLE" :" lamp1" }

в отладчике в браузере запрос выглядит так http://192.168.1.208/ajax/commands.html?op=value_changed&item_id=118&new_value=

Храниить текущее время в базе ради удобства доступа - расточительно) Поэтому просто отдаем его сразу по запросу. Возможно лучше его вообще убрать, мало где используется (но тем неменее создает хоть и небольшую, но бессмысленную нагрузку на базу)

https://github.com/sergejey/majordomo/blob/fe0ac29059437457d6ca0272db5ab8c72bead689/lib/messages.class.php#L149

пришлось добавить ещё : и слеш if (preg_match('/image:([\w\d\:\\\_\/\-\.]+)/is',$ph,$m)) {

https://github.com/sergejey/majordomo/blob/c772b18d1ddb0579820031a0b8f6d6161a333683/modules/devices/devices.class.php#L1475

<#LANG_DEVICES_MODULE_TITLE#> https://github.com/sergejey/majordomo/blob/c772b18d1ddb0579820031a0b8f6d6161a333683/modules/devices/devices.class.php#L23

https://github.com/sergejey/majordomo/blob/c772b18d1ddb0579820031a0b8f6d6161a333683/lib/objects.class.php#L389

Опираясь на постулат, что имя объекта в системе уникально, можно упростить весьма часто используемую функцию

function getObject($name)
{
    if (trim($name)=='') return 0;
    if (preg_match('/^(.+?)\.(.+?)$/', $name, $m)) { //уберём класс спереди
        $name = $m[2];
    }
    $rec = SQLSelectOne("SELECT ID FROM objects WHERE TITLE = '".DBSafe($name)."'");
    if ($rec['ID']) {
        include_once(DIR_MODULES . 'objects/objects.class.php');
        $obj = new objects();
        $obj->id = $rec['ID'];
        $obj->loadObject($rec['ID']);
        return $obj;
    }

    return 0;
}

https://github.com/sergejey/majordomo/blob/c772b18d1ddb0579820031a0b8f6d6161a333683/modules/objects/objects.class.php#L322 Основываясь на предположении, что объект ДОЛЖЕН принадлежать к какому-нибудь классу избавляемся от лишнего запроса

 
function loadObject($id)
    {
        $rec = SQLSelectOne("SELECT  objects.*,classes.TITLE CL_TITLE  FROM objects join classes ON objects.CLASS_ID = classes.ID WHERE objects.ID=" . (int)$id);
        if (IsSet($rec['ID'])) {
            $this->id = $rec['ID'];
            $this->object_title = $rec['TITLE'];
            $this->class_id = $rec['CLASS_ID'];
            $this->class_title = $rec['CL_TITLE'];
            $this->description = $rec['DESCRIPTION'];
            $this->location_id = $rec['LOCATION_ID'];
            if (preg_match('/^sdevice(.+?)/', $rec['SYSTEM'], $m)) {
                $this->device_id = $m[1];
            }
        } else {
            return false;
        }
    }

https://github.com/sergejey/majordomo/blob/e453668e32c17bf4c3d751ea2efcd8494bc3b11a/lib/common.class.php#L314

При ожидании Ок от getURL возможна задержка до 45 с. При долгой задаче выдается ошибка, которой нет + задерживается выполнение следующей задачи, если ее время запуска то же( Предлагается не ждать ответа и использовать getURLBackground. Проверено на себе)

        if ($jobs[$i]['COMMANDS'] != '') {
            $url = BASE_URL . '/objects/?system_call=1&job=' . $jobs[$i]['ID'];
            getURLBackground($url);
            /*
            $result = trim(getURL($url, 0));
            $result = preg_replace('/<!--.+-->/is', '', $result);
            if (!preg_match('/OK$/', $result)) {
                DebMes(sprintf('Error executing job %s (%s): %s', $jobs[$i]['TITLE'], $jobs[$i]['ID'], $result) . ' (' . __FILE__ . ')');
            }
            */
        }