Общая функция для вызова dot
Issue #78
closed
Originally reported on Google Code with ID 78 ``` Функцию вызова dot размещаем в классе типа вопроса Функции две две, в них входит все содержимое draw_nfa кроме собственно генерации файла.
Первая - проверка и создание подкаталога в темп - введите параметр с его именем $componentname, возвращает путь к гарантированно существующему каталогу.
Вторая - собственно вызов dot - его надо будет совершенствовать, т.к. есть специальное расширение PHP для работы с dot, если оно установлено то надо работать через него - но это позже, сначала надо централизовать то, что есть.
Добавляем проверку наличия пути к dot и самого dot; значение по умолчанию для пути к dot - пустая строка.
jpgfilename я бы поставил значение по умолчанию null и брал бы тогда имя от исходного файла, меняя расширение
Глобалки должны объявляться в начале функции, никогда не в середине...
Я думаю вам несложно будет поправить вызовы в обоих матчерах... ```
Reported by `oasychev` on 2011-12-13 10:46:40
Comments (11)
-
-
reporter
``` Я вот тут подумал - не лучше ли этой функции быть в preg_regex_handler? Я пока не могу придумать кто кроме обработчиков регексов может пользоваться dot - а такое размещение существенно упрощает вызов, т.к. функции нормально всеми наследуются... ```
Reported by `oasychev` on 2011-12-19 12:48:37
-
reporter
Reported by `oasychev` on 2011-12-23 10:44:10 - Status changed: `InProgress`
-
Reported by `vostreltsov` on 2011-12-23 13:30:07 - Status changed: `Fixed`
-
reporter
``` $regexhandler = new preg_regex_handler(); $dir = $regexhandler->get_temp_dir('dfa');
В dfa - функция draw член класса матчера? Если да, то зачем отдельный объект?! Достаточно $this-> ```
Reported by `oasychev` on 2011-12-23 13:50:59
-
``` Ага, исправил. ```
Reported by `vostreltsov` on 2011-12-23 18:45:40
-
reporter
``` К сожалению, поскольку рефакторинг - issue 80 - перемещал файлы, слить ваши изменения проблематично.
Придется попросить вас произвести их еще раз в новом клоне (можно попробовать сгенерировать патч и подменить имена файлов, после чего применить - или просто глянуть на diff и перенести код). ```
Reported by `oasychev` on 2011-12-26 06:57:11 - Status changed: `InProgress`
-
reporter
Reported by `oasychev` on 2011-12-27 10:11:51 - Labels added: Milestone-Release2.2 - Labels removed: Milestone-Release2.1
-
Reported by `vostreltsov` on 2012-01-08 22:30:47 - Status changed: `Fixed`
-
reporter
``` Спасибо. :) ```
Reported by `oasychev` on 2012-01-09 18:53:44
-
reporter
Reported by `oasychev` on 2012-03-20 11:40:03 - Status changed: `Done`
- Log in to comment
Reported by `vostreltsov` on 2011-12-16 18:19:21 - Status changed: `Fixed`