В случае с отправки почты чуть сложнее

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

Чего мелочиться с названиями методов, почему так коротко getDatabase?:)

потому что так приятно 🙂 согласно всяким PSR )

Почему бы сразу не воспользоваться опытом того же философа и не назвать getDatabaseFromMyDatabasesServer

я сомневаюсь, что филосов каждый раз это руками долбил

Что именно принято? Делать названия длинными?:)

делать названия понятными (:

Тут пару дней назад человек поле psword назвал неправильно и полдня промучался

а если бы он назвал его p, то не мучился бы?

мучился бы другой потом значит

если код писать в админке, то да, плохо

а в шторме пофиг

Ну ладно надо добавить get и сделать camelCase, но зачем при этом удлиннять то?:)

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

Ну ладно надо добавить get и сделать camelCase, но зачем при этом удлиннять то?:)

переходим на неймспейсы)

С таким подходом скоро вас тут скоро останется человек пять и все гуру)))

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

особенно в эпоху IDE

С таким подходом скоро вас тут скоро останется человек пять и все гуру)))

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

PSR ведь не просто так пишутся

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

Не видел нигде правила заменять короткие всем известные названия на длинные

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

Не видел нигде правила заменять короткие всем известные названия на длинные

там не в замене названий смысл пиара

Не видел нигде правила заменять короткие всем известные названия на длинные

выше была не замена переменной, а нормальный рефаткоринг кода

Такое чувство, что сейчас все не для удобства пользования эво делается, а для удобства нескольких человек в рамках эво отказаться от эво))

или вы не чувствуете разницы между $this->property и $this->method() ?

бля, это делается, чтобы система была гибче

там не в замене названий смысл пиара

Ну так тем более в чем смысл такого переименования

SOLID принципы хотя бы читаните, чтобы понимать зачем это делается

бля, это делается, чтобы система была гибче

Разблякался он, бля

Че ты лезешь, если ты это не используешь?

потому что проходили это уже в рево 🙂

ничего ж не меняется

все хотят 3-5-10 версии, но чтобы ничего не менять

Я же написал выше, хочешь по пср назови getDb вместо db, причем тут этот Database

а смысл экономить на буквах?

зато хоть читать по человечески можно

и понимать, что там кто до тебя понаписал

Иди рево свое читай, пока оно тормозит как раз время есть

Иди рево свое читай, пока оно тормозит как раз время есть

Разницы что писать читать нету что ево что рево проблемы одни и теже

А так большущее спасибо Жене за проделанную работу

Не нравятся длинные название надо было сделать короткие в чем проблема то?

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

Причем тут эти идиотские фразы "не хотят переучиваться", как будто смысл этой учебы в том, чтобы вместо коротких названий длинные выучить

Не нравится как сделано сделайте лучше ну в чем проблема?

Весь код на гитхаб делай нехочу

Но что то что в ево что в рево не больше 10 человек кто что то делает.

Я считаю что принимать решение должен тот кто что то делает. Если вам не нравиться то банально не в чате пишите а напишите комментарий к пул реквесту.

Может Женя скажет что да действительно db так же понятней но короче и примет ваше замечание;)

Как будто от того, что весь код класса парсера воткнули в \EvolutinCMS\Core ну прям суперсовременно стало

А если в чате писать нельзя про это, так и хрен с этим чатом тогда, варитесь тут сами:)

В чате писать можно чего ж нельзя;)

Только пользы от этого??

Как будто от того, что весь код класса парсера воткнули в \EvolutinCMS\Core ну прям суперсовременно стало

Стало лучше чем было и ближе к современным реалиям Это уже лучше чем ничего.

Как будто от того, что весь код класса парсера воткнули в \EvolutinCMS\Core ну прям суперсовременно стало

Пока весь, со временем разнесется по разным классам

Пока весь, со временем разнесется по разным классам

Не ну это не дело 🙂 надо так что б завтра все было по новому и с полной обратной совместимостью :)))))

Похоже, я много пропустил...

Ага не успел поучавствовать :))

Ну впадло было человеку писать Database он хотел полегче и чтобы ничего не переделывать) Если бы никто ничего в мире не переделывал, то до сих пор езди ли бы на лошадях.

Ага и на вопрос что вы хотите ответ: Более быструю лошадь

В доках тоже методы новые надо будет?

угумс

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

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

Но так для версии 2.0 нужно будет от db избавлятся

что там нынче в тренде ) на GO перепишем все:)

Йопть, любой северокорейский чат позавидует единодушию мнений и рвению подпевал этого милого сообщества:)

Ну блин, вот что плохого в том что сделали код читабельным?

Или это прям сложно написать не 2, а 11 символов?

угумс

А откуда инфу брать?

А откуда инфу брать?

когда допилим будет инфа

И напоминаю про синхронизацию доков

Йопть, любой северокорейский чат позавидует единодушию мнений и рвению подпевал этого милого сообщества:)

Я бы поддержал тебя если бы хотя бы выкладывал новости по своим дополнениям. А то судя по гиту что то пилишь но хоть бы пару релизов сделал. Но нет оно не тебе не надо. Зачем же писать что то и так хорошо. Поэтому сори но сейчас ты не прав.

Мнение большенства в том что они получают за спасибо новый функционал и развитие системы. Я тоже обеими руками за то что б кто то сделал все за меня:)

Или это прям сложно написать не 2, а 11 символов?

В твоих визитках не сложно:)

В твоих визитках не сложно:)

Давайте не будем обсуждать свой говнокод 🙂 говнокодить все умеем 🙂

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

В твоих визитках не сложно:)

вот зашел бы сюда: https://github.com/evolution-cms/evolution/pull/725 и написал бы по части того что getDb было и делов

Я тут скорее к тому, что это требует времени. А это время маловероятно будут оплачивать заказчики.

Если клиент не хочет это оплачивать то не делай