Pull to refresh
3
0
Виталий Воскобович @voskobovich

PHP Developer | 8+ years exp.

Send message
Архитектура этого не запрещает)
Поведение просто сохраняет массив ID к нужной связи, а как его формировать — уже дело каждого)
Отличный пример. Только как в этом может помочь поведение?
Передайте ID первой категории GET параметром и потом перед отображением формы, в контроллере, запишите его
$productModel->categories_list[] = $_GET['cat_id']

Это позволит сразу выделить первую категорию в мульти-select`e, а сохраниться она в базу уже после сохранения товара)
Не совсем понял, второй пункт «Также удобно было бы, чтобы была возможность для новых моделей иницализировать список связей вручную (например, я перешел по ссылке «создать продукт в этой категории»).» — можно еще пример?

Про удаление связей вообще — хорошая идея. Учтем.
Сейчас я работаю над поддержкой остальных типов связи: has many и has one.
Залью код в новую ветку.

Переехал на GitHub: ManyToManyBehavior. Буду ждать PullRequest`ов.
Не подумал, что мне могут помочь с разработкой. На Bitbucket у меня все приватные репо и по традиции…
Но, я перееду на github, раз такое дело)

И с версиями я тоже намудрил, делаю тэгами, а надо ветками.
Все исправлю!
Мой PhpStorm пишет вот так:
if($a == 100)
	$a = 300;
$b = 5;

если писать сперва первую строку, потом вторую строку и третью)

Дискуссия хороша, но не имеет смысла. Против общепризнанных стандартов я идти не собираюсь)
Я тоже бываю не прав и сейчас как раз такая ситуация.
Раз я пишу публичное расширение для проекта то должен соблюдать кодстайл принятый в проекте!
Я лишь говорю что, если фигурные скобки расположены на одной вертикали, то проще найти их пару. Согласен?
Если взять во внимание, что условия это примерно 65% всего кода, то простота нахождения пары условий должна быть максимальной)

Про забывания поставить скобки и баги… не замечал. IDE же форматирует код так, что не заметить такую оплошность нельзя (PhpStorm).

Ты можешь писать как угодно и каким хочешь почерком, главное чтобы потом было удобно читать)
Я студент и мне знакома ситуация, когда на паре писал и было все понятно, а при подготовке к экзамену материшься на свой же почерк)
А можно услышать эти самые причины по которым они перешли на скобки?

Я считаю, что если уж и ставить скобки, то только так, чтобы они были на одной вертикали.
Потому что искать открывающую скобку по диагонали при теле if`a в 20 и больше задача для глаз не из легких.
С вертикальным размещением все на порядок проще + IDE подсвечивает. Веди вверх взгляд и ищи светящуюся скобку)
Что скажешь?
Мне не нравится современные кодстайлы. Читать такой код не совсем удобно.
Я считаю, что если PHP позволяет опустить скобки — значит этим нужно пользоваться.
Чем меньше символов в файле, тем быстрее и легче он анализируется глазами, разве не так?
Точно так же и пространство между строками кода. Чем просторнее и логичнее разделен на блоки код, тем меньше напрягается зрение для анализа и понимания алгоритма.

Когда я писал поведение, я не думал его публиковать вот и оформил не по стандартам.
Со следующим обновлением сделаю форматирование кода по стандарту)
К сожалению (или к счастью) Yii2 сильно отличается от первой Yii1. Другая реализация связей и перенимать от туда что-либо сложно)
Давайте я выкачу новую версию, а аудитория уже выскажет свое мнение)
Ты правильно понял и я с тобой полностью согласен.
Через пару дней устраню это неудобство и сделаю поведение более гибкое)
Какая именно была ошибка? Можно попробовать разобраться.

Information

Rating
Does not participate
Location
Одесса, Одесская обл., Украина
Registered
Activity