НКА - вбить значения по умолчанию для настроек в код.

[Oleg Sychev]

Originally reported on Google Code with ID 52 ``` Это недопустимо!

Функция определена в базовом классе и просто возвращает заполненное значение из $this->left. Переопределять ее нельзя (это не С++, здесь нет перегрузки по типам параметров). Это просто нарушает работу самого вопроса (надо бы не только юнит-тесты иногда запускать....)

Если вам нужна другая функция, поменяйте имя. Например determine_characters_left или что-нибудь в этом роде.

Исправить срочно! ```

Reported by `oasychev` on 2011-11-19 15:15:10

[Oleg Sychev]

Reported by `oasychev` on 2011-11-19 15:19:51 - Labels added: Milestone-Release2.1

[Valeriy Streltsov]

``` Исправлено. ```

Reported by `vostreltsov` on 2011-11-19 15:22:24

[Oleg Sychev]

``` Когда работа над issue закончена, привыкайте менять статус на Fixed вместе с сообщением. Сделайте это сейчас... ```

Reported by `oasychev` on 2011-11-19 15:34:58

[Valeriy Streltsov]

Reported by `vostreltsov` on 2011-11-19 15:38:38 - Status changed: `Fixed`

[Oleg Sychev]

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

Сделайте в коде условия, что если настройки размера автомата не установлены или равны 0, null и т.д. - то чтобы они брались по 250...

```

Reported by `oasychev` on 2011-11-19 15:55:12 - Status changed: `Accepted`

[Valeriy Streltsov]

``` Если я правильно понял, то такая проверка уже реализована, и вы это вытянули в основной репозиторий. А в вашем клоне под 2.1 я этого не вижу.

http://code.google.com/r/vostreltsov-nfa-preg/source/browse/question/type/preg/nfa_preg_matcher.php

Строки 355, 360 как раз устанавливают лимиты вручную, но только равными единице - чтобы принудить админа сделать подходящие для него настройки... ```

Reported by `vostreltsov` on 2011-11-19 16:19:26

[Oleg Sychev]

``` Ну дело в том, что админы (а тем более конкретные учителя) не все так умны, чтобы догадаться о причинах. И если не очень понятная ошибка случается сразу - имеют тенденцию считать что программа глючная и вообще бросать ею пользоваться.... "Не напрягай юзера без нужды" (с) бритва Оккама для программеров :)

Ошибка критична для тех, кто помогает нам в тестировании - того же Joseph Rezeau, потому что при нормальном процессе апгрейда ее не будет... Но вот для работающих с девелоперским кодом это неприятно... В принципе должно обойтись, я потому и не выталкивал пока смерженный код 2.1, чтобы присоединить настройки к апгрейду, но лучше бы поменять от греха подальше.

Лучше сделать на работоспособные настройки, а не на 1. ```

Reported by `oasychev` on 2011-11-19 16:27:12

[Valeriy Streltsov]

Reported by `vostreltsov` on 2011-11-19 16:34:29 - Status changed: `Fixed`

[Oleg Sychev]

``` Интегрировано в основной код. Спасибо. ```

Reported by `oasychev` on 2011-11-19 16:45:36 - Status changed: `Done`