#75 Add Uuid support #77
Conversation
| public function __construct($name) | ||
| { | ||
| parent::__construct($name); | ||
| $this->setAllowedPattern(self::UUID_PATTERN); |
There was a problem hiding this comment.
А почему бы в этом классе вместо конструктора сразу не настроить свойство allowedPattern? А вот метод setAllowedPattern перекрыть и бросать Exception что бы не меняли эту настройку.
|
Убрал метод obtainUuid из PgSQL и вынес его в метод UuidUtils :-) |
|
а с этим чего? |
|
Как-то оно заглохло и забылось. Мне так и не нравится хак PgSql::obtainSequence - то что там статический вызов не радует (может я хочу Uuid по другим правилам генерить) и то что UuidUtils класс требует установленного extension'а - фиг кто прогонит у себя тесты с Uuid. Тот же MySql мало кто и редко когда гоняют :) |
можно добавить проверку на личие extension'а, если нет - генерить своей функцией
а в чем проблема? можно же объект сразу вместе с id собрать |
тогда можно вообще не вставлять этого кода в PgSql раз можно сразу вместе с id собирать
лучше что б оно было не через статический вызов а сделать возможность сетить в PgSql нужный Uuid генератор. Если ничего не засетили, то тогда уж что б создавал для себя какой-то генератор по умолчанию. |
Собственно спасибо Алексею ))) за мини туториал по тому как делать правильно ;)
Обсуждаем коллеги :-)