yii2 mac-address validator

[vakorovin]

Приветствую!

Ребята, работаю в провайдере, несколько лет занимался разработкой собственного движка, однако недавно убедил руководство переходить на yii2, т.к. использую его в других проектах.

В-общем начал потихоньку переносить свои наработки в yii2. Вчера набросал валидатор для mac-адресов:

https://github.com/vakorovin/yii2-macaddress-validator

Там всё расписано, что и как, тем не менее остановлюсь на форматах:

AA:BB:CC:DD:EE:FF
AA-BB-CC-DD-EE-FF
AA BB CC DD EE FF

AABB.CCDD.EEFF
AABB CCDD EEFF

AABBCC DDEEFF
AABBCC-DDEEFF
AABBCC:DDEEFF

AABBCCDDEEFF

В моём конкретном случае маки хранятся в PostgreSQL, с явным указанием типа macaddr. Все эти форматы для PostgreSQL являются валидными, отсюда и такой их список. Хотя возможно я учёл не все форматы, если вы сталкивались с другими - пожалуйста напишите, я добавлю поддержку.

Да и вообще, конструктивная критика приветствуется! С написанием composer.json раньше не сталкивался, но судя по документации оформил более или менее правильно, в любом случае хотелось бы услышать комментарии по оформлению этого файла.

[zelenin]

считаю, что валидатор должен в этом случае выступать и нормализатором, то есть приводить формат мака к общепринятому. Неважно, какой формат готова принять база - важно, в каком формате его ожидает клиент. Думаю, стандартный вариант AA:BB:CC:DD:EE:FF.

https://github.com/vakorovin/yii2-macad ... poser.json
неотформатировано, осталось упоминание оригинального репо, откуда вы пример взяли, видимо.
stability я бы убрал. бранч какой-то внизу. вы это используете?

[vakorovin]

считаю, что в валидатор должен в этом случае выступать и нормлаизатором, то есть приводить формат мака к общепринятому. Неважно, какой формат готова принять база - важно, в каком формате его ожидает клиент. Думаю, стандартный вариант AA:BB:CC:DD:EE:FF.

https://github.com/vakorovin/yii2-macad ... poser.json
неотформатировано, осталось упоминание оригинального репо, откуда вы пример взяли, видимо.
stability я бы убрал. бранч какой-то внизу. вы это используте?

Ага, спасибо. По поводу нормализатора, да, хорошая мысль, поправлю (для нас не особо важно, т.к. обработками занимаются триггеры в самой БД).

Да вот у меня вообще каша с этим composer'ом, прочитал на офф.сайте, видимо не всё понял, пересмотрел кучу готовых репозиториев, кстати заметил, что многие не устанавливались с той же ошибкой, что и мой (пока не додумался в самом проекте изменить stability). Буду очень благодарен, если дадите советы как его оформить - не от лени прошу, а от непонимания.

[zelenin]

Да вот у меня вообще каша с этим composer'ом, прочитал на офф.сайте, видимо не всё понял, пересмотрел кучу готовых репозиториев, кстати заметил, что многие не устанавливались с той же ошибкой, что и мой (пока не додумался в самом проекте изменить stability). Буду очень благодарен, если дадите советы как его оформить - не от лени прошу, а от непонимания.

советы дал. в остальном все ок.

[zelenin]

бранч удалите.
https://github.com/vakorovin/yii2-macad ... r/releases добавьте релиз, который указан в composer.json
если хук привязан к пакаджисту, то релиз автоматически должен появиться здесь https://packagist.org/packages/vakorovi ... -validator

[vakorovin]

бранч удалите.
https://github.com/vakorovin/yii2-macad ... r/releases добавьте релиз, который указан в composer.json
если хук привязан к пакаджисту, то релиз автоматически должен появиться здесь https://packagist.org/packages/vakorovi ... -validator

Сделал. Бранч удалил, с релизами разобрался (и картинка сложилась после этого), хук привязал. Спасибо за помощь.

[zelenin]

версия-то не появилась на пакаджисте. что-то не так

[vakorovin]

версия-то не появилась на пакаджисте. что-то не так

вот так правильно? ))

[zelenin]

вроде ок