не, давай уже серьезно конкретно твоя работа

Разработка функционала на OroCRM)

а - я работаю в продуктовой компании, вот црм это как я работаю директором, в фирме ) большой )

ыы

ни о чем не говорит, кроме попыток показать какой ты крутой )

ни о чем не говорит, кроме попыток показать какой ты крутой )

ни разу не старался) Это ты все додумал

Разработка функционала на OroCRM)

ОУ!!! Что это значит? Я уже подумал, что ты написал ОроЦРМ и ГитХаб. И решил уже заткнуться.

Разработка функционала на OroCRM)

А магазин на ней быстро поднимается?

Вы спросили я ответил)

ни разу не старался) Это ты все додумал

а я не додумываю, а предлагаю показать

покажи конкретно что ты можешь )

А магазин на ней быстро поднимается?

НУ если бы ты посмотрел то это CRM правда уже зареоизился OroCommerce но там b2b

А магазин на ней быстро поднимается?

crm, там b2b, он работает там, что не понятного то? ?

Итак, приходит ко мне клиент. Хочу, говорит, магазин. А я ему - на ороцрм. Там магазина нет, зато все тестами покрыто

НУ если бы ты посмотрел то это CRM правда уже зареоизился OroCommerce но там b2b

а может ты там и не работаешь? пруф в студию )

а может ты там и не работаешь? пруф в студию )

Ок не работаю Можешь в линкедыне меня найти)

пруф что ты там работаешь, пруф вклада ) если слово "примонтировать" вызывает трудности, а клиента протокола от клиента маунта не отличаешь у меня большие вообще вопросы в твоей компетенции )

Итак, приходит ко мне клиент. Хочу, говорит, магазин. А я ему - на ороцрм. Там магазина нет, зато все тестами покрыто

Лолчто? Я не предлагал делать магащин на оро)

Ок не работаю Можешь в линкедыне меня найти)

а.. я поискать должен )

ага, понял)

огоцрм поищи.. я хз где оно, может и нет такой меня поищи )

Битрикс популярен прежде всего тем, что магазин намнем очень быстро поднимается

Итак, приходит ко мне клиент. Хочу, говорит, магазин. А я ему - на ороцрм. Там магазина нет, зато все тестами покрыто

какими тестами, у него глобал вызывает трудности уже, на входе..

В противовес приводилась магента. Ну ок, давай сравним проекты. И тут нам говорят, что oroCRM. Мы же про магазины говорили. Если сравнивать CRM, то это другой разговор

а если приватный класс тестировать, это же ваще ппц, не справится)

пруф что ты там работаешь, пруф вклада ) если слово "примонтировать" вызывает трудности, а клиента протокола от клиента маунта не отличаешь у меня большие вообще вопросы в твоей компетенции )

NDA)) Ну если сильно захочешь можешь поискать на гитхабе)

оу, волшебное слово нда, ну ок )

оу, волшебное слово нда, ну ок )

ДА не вопрос, не веришь можешь сам узнать)

видимо урл огоцрм тоже нда не дает )

Ок все ребя твы меня пабидили)

да никто тебя пабиждать не собирался )

видимо урл огоцрм тоже нда не дает )

https://github.com/orocrm/crm

остынь, над тобой просто посмеялись и не больше ) ладно, давай ... продолжение надеюсь знаешь)

остынь, над тобой просто посмеялись и не больше ) ладно, давай ... продолжение надеюсь знаешь)

дык у меня как у некоторых не пригорает)

А прибыть oroCRM приносит? Большую?

пфф

И кому?

https://github.com/orocrm/crm

среди топа апруверов тебя не видно)

шо мы на человека накинулись?

среди топа апруверов тебя не видно)

ок не вопрос)

а не, нашел, самый заминусованный же )) 8,235 ++ / 20,618 —

ыыы

бугага

работает себе, делает вклад в срм

шо мы на человека накинулись?

никто не накинулся, как сказал бы тот самый человек "это ты додумал" :)

ночь, битрикс меня замучал модуль рассылок

На самом деле тут два подхода к продукту

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

а тут такое)

Каждый на битрикс по-разному смотрит

он всего лишь более за качество битрикса

и за уменьшение кол-ва багов

За качество кода

но по сути да

а не, нашел, самый заминусованный же )) 8,235 ++ / 20,618 —

а ты уверен что это рейтинг?)))

protected function changeOnDeleteToCascade(Schema $schema, array $data) + { + foreach ($data as $tableName => $columns) { + $table = $schema->getTable($tableName); + foreach ($columns as $column) { + $foreignKeys = $table->getForeignKeys(); + foreach ($foreignKeys as $foreignKey) { + $foreignKeyColumns = $foreignKey->getUnquotedLocalColumns(); + if ($foreignKeyColumns === [$column]) { + if ($foreignKey->getOption('onDelete') !== 'CASCADE') { + $table->removeForeignKey($foreignKey->getName()); + $table->addForeignKeyConstraint( + $foreignKey->getUnqualifiedForeignTableName(), + $foreignKeyColumns, + $foreignKey->getUnquotedForeignColumns(), + ['onDelete' => 'CASCADE', 'onUpdate' => $foreignKey->getOption('onUpdate')] + ); + } + + break; + } + } + } + } + } +} и этот человек говорит о качестве кода? может еще 10 уровней вложенности докрутить? почемуб нет)

давно так не ржал, спасибо)

protected function changeOnDeleteToCascade(Schema $schema, array $data) + { + foreach ($data as $tableName => $columns) { + $table = $schema->getTable($tableName); + foreach ($columns as $column) { + $foreignKeys = $table->getForeignKeys(); + foreach ($foreignKeys as $foreignKey) { + $foreignKeyColumns = $foreignKey->getUnquotedLocalColumns(); + if ($foreignKeyColumns === [$column]) { + if ($foreignKey->getOption('onDelete') !== 'CASCADE') { + $table->removeForeignKey($foreignKey->getName()); + $table->addForeignKeyConstraint( + $foreignKey->getUnqualifiedForeignTableName(), + $foreignKeyColumns, + $foreignKey->getUnquotedForeignColumns(), + ['onDelete' => 'CASCADE', 'onUpdate' => $foreignKey->getOption('onUpdate')] + ); + } + + break; + } + } + } + } + } +} и этот человек говорит о качестве кода? может еще 10 уровней вложенности докрутить? почемуб нет)

дык покажи класс, отрефакторь)

без целевой системы будет сложно.. но сейчас попробую )

это даже забавно)

это даже забавно)

В общем закинешь PR комьюнити тебе спс скажет

да ну) это же тестить надо) я тебе сейчас скину, протестишь) будет явно читабельнее)

10 минут) тут ничего сложного)

да ну) это же тестить надо) я тебе сейчас скину, протестишь) будет явно читабельнее)

В пакете уже есть все необходимые тесты. Делай PR

сколько платишь?

да ну) это же тестить надо) я тебе сейчас скину, протестишь) будет явно читабельнее)

Ну ты же сам предлагаешь

я показал на косячный код)

засомневался в адекватности, вижу подтверждение

ERROR: S client not available

ты пытаешься поддеть, могу показать как правильно, мне ен сложно)

я показал на косячный код)

Ну это еще нужно доказать что он косячный

антипаттерн - пять уровней вложенности

второе - они не обязательны)

для начала $foreignKeys = $table->getForeignKeys(); можно получить вначале два foreach прогонять ради того чтобы найти if ($foreignKeyColumns === [$column]) { (остальное пропускается) - антипаттерн

думаю есть еще логическая ошибка - в foreach идет запрос, тут уже надо смотреть целевую систему, есть вероятность что делает обращение к внешнему источнику, но даже без него.. получается забавно

до трех уровней можно сократить)

если хочешь, могу пару частных уроков дать ;) бугага но не больше)

если хочешь, могу пару частных уроков дать ;) бугага но не больше)

Дык сделай. Глядишь на работу пригласят, как хорошего спеца)

ахаха

Для того код и лежит на гитхабе что бы его комьюнити улучшало)

жги еще ?

меня на работу пригласят, радость то какая ???

жги еще ?

Т.е. ты не настолько хорошиый спец?)))

а я тебя не возьму, уж прости)

бля, чувак, я техдир) со своим офисом и очень интересными большими проектами напуркуа мне какая то твоя црм?

ахахах

блин, спасибо, ты мне сделал вечер ?

бля, чувак, я техдир) со своим офисом и очень интересными большими проектами напуркуа мне какая то твоя црм?

ок пабидил) Правда техдир боиться PR сделать ну ок

блин, спасибо, ты мне сделал вечер ?

И тебе спс, я получил что хотел

ок пабидил) Правда техдир боиться PR сделать ну ок

какой пиар)

Pull Request.

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

foreach ($columns as $column) { foreach ($foreignKeys as $foreignKey) { $foreignKeyColumns = $foreignKey->getUnquotedLocalColumns(); if ($foreignKeyColumns === [$column]) { поиск таблицы с одной записью чтоли?

всмысле массива

а, хотя нет, сравнение двух массивов.. блин.. вот логика..

Дык сделай. Глядишь на работу пригласят, как хорошего спеца)

protected function changeOnDeleteToCascade(Schema $schema, array $data) { foreach ($data as $tableName => $columns) { $table = $schema->getTable($tableName); $foreignKeys = $table->getForeignKeys(); $arrForeignKeyColumns = []; foreach ($foreignKeys as $key=>$foreignKey) { $foreignKeyColumns = $foreignKey->getUnquotedLocalColumns(); $arrForeignKeyColumns[$key] = $foreignKeyColumns; } foreach ($columns as $key=>$column) { $columns[$key] = $column; } $arDiff = array_intersect($arrForeignKeyColumns, $columns); $arKeys = array_keys($arDiff); foreach($arKeys as $key) { $foreignKey = $foreignKeys[$key]; if ($foreignKey->getOption('onDelete') !== 'CASCADE') { $table->removeForeignKey($foreignKey->getName()); $table->addForeignKeyConstraint( $foreignKey->getUnqualifiedForeignTableName(), $foreignKeyColumns, $foreignKey->getUnquotedForeignColumns(), ['onDelete' => 'CASCADE', 'onUpdate' => $foreignKey->getOption('onUpdate')] ); } } } } } наверняка опечатался, потому что делал без возможности перепроверки, но дикую вложенность убил, да и прирост будет наверняка на поиске, надо лишь проверить, и возможно устранить досадные опечатки можешь не благодарить

?

protected function changeOnDeleteToCascade(Schema $schema, array $data) { foreach ($data as $tableName => $columns) { $table = $schema->getTable($tableName); $foreignKeys = $table->getForeignKeys(); $arrForeignKeyColumns = []; foreach ($foreignKeys as $key=>$foreignKey) { $foreignKeyColumns = $foreignKey->getUnquotedLocalColumns(); $arrForeignKeyColumns[$key] = $foreignKeyColumns; } foreach ($columns as $key=>$column) { $columns[$key] = $column; } $arDiff = array_intersect($arrForeignKeyColumns, $columns); $arKeys = array_keys($arDiff); foreach($arKeys as $key) { $foreignKey = $foreignKeys[$key]; if ($foreignKey->getOption('onDelete') !== 'CASCADE') { $table->removeForeignKey($foreignKey->getName()); $table->addForeignKeyConstraint( $foreignKey->getUnqualifiedForeignTableName(), $foreignKeyColumns, $foreignKey->getUnquotedForeignColumns(), ['onDelete' => 'CASCADE', 'onUpdate' => $foreignKey->getOption('onUpdate')] ); } } } } } наверняка опечатался, потому что делал без возможности перепроверки, но дикую вложенность убил, да и прирост будет наверняка на поиске, надо лишь проверить, и возможно устранить досадные опечатки можешь не благодарить

$columns[$key] = [$column]; конечно же..

ну и если все эти foreach не являются скрытыми yeild обращениями к бд на самом деле, т.е. не вызовут проблем с нагрузкой, то лучше рефактор вот еще такой провести foreach ($columns as $key=>$column) { $columns[$key] = [$column]; } заменить на array_map(function($column){ return [$column] }, $columns); т.к. нагляднее и заменить foreach ($foreignKeys as $key=>$foreignKey) { $foreignKeyColumns = $foreignKey->getUnquotedLocalColumns(); $arrForeignKeyColumns[$key] = $foreignKeyColumns; } на array_map(function($v){ return $foreignKey->getUnquotedLocalColumns(); }, $foreignKeys); т.к. опять же и нагляднее и не вынуждает заводить временную переменную, которую завел чтобы не терять контекст

Можно в следующий раз это куда-нибудь на pastebin размещать?