
Timur
13.01.2017
19:03:24
Я ещё @Symfony добавляю, но тут у всех разные фреймворки.
У Symfony там свои особенности есть.

Виталий
13.01.2017
19:05:55
Расскажи плиз как именно ты его используешь?

Google

Timur
13.01.2017
19:09:32
https://github.com/FriendsOfPHP/PHP-CS-Fixer
Этот конфиг кладёшь в корень проекта под именем ".php_cs.dist". Потом ставишь Composer'ом и пускаешь так: $ ./vendor/bin/php-cs-fixer fix
Глобально ставить его не люблю. У себя ставлю локально.
В мануалах там описано всё.
В .gitignore добавляешь .php_cs и .php_cs.cache файлы.
Можешь глобально поставить. Тут каждый сам решает.

F01134H
13.01.2017
19:10:42
получается он и на проде крутится чоле?

Timur
13.01.2017
19:10:58
Зачем? Нет. Локально запустил, он всё и пофиксил.
В require-dev его надо.

Виталий
13.01.2017
19:11:45
так а для чего есть в проект подключать?
Немного не понимаю. Это ведь больше инструмент разработчика и его к шторму надо подключать.

Timur
13.01.2017
19:12:05
У каждого проекта свой стиль кодирования.
У меня так по-крайней мере в команде.
У Symfony 3.* проекта свой, у Yii 2 проекта свой, у древнего Yii 1.1 проекта свой.
Поэтому у нас per project.

Виталий
13.01.2017
19:13:15
угу, понял. И у вас есть пхп файлик в корне проекта запуская который вы все фиксите, так?

Google

Timur
13.01.2017
19:15:49
Ну, да, так. PHP файл этот просто настройки для PHP CS Fixer хранит.
https://github.com/symfony/symfony/blob/master/.php_cs.dist
https://github.com/laravel/framework/blob/5.3/.php_cs
В Laravel набор правил вообще свой.

Виталий
13.01.2017
19:17:55
да, крутяк. Спасибо! Очень помог:)
чет пропустил я эти фишки с phpcs
Коллеги, забыл как в PHP Storm хоткей для интерактивного переимнования переменной везде где она юзается.
Ставишь курсор на переменную, жмакаешь хоткей и она выделяется рамочкой иможно печатать и когда меняешь название она автоматом меняется везде по коду, интрактивно.
Перерыл все доки, не могу найти.

F01134H
13.01.2017
20:25:40
вроде бы через ctrl+f решается

dypa
13.01.2017
20:25:48
code -> refactor

F01134H
13.01.2017
20:25:59
ну лан :D

dypa
13.01.2017
20:26:07
у всех хоткеи разные

Виталий
13.01.2017
20:26:30
ну вот как так!
Только написал вам и сразу налось решение. Два вечера искал хоткей.
Спасибо!

Максим
13.01.2017
20:26:46
ctrl+shift+r переиновать в папке

Виталий
13.01.2017
20:27:42
shift + f6
а я f7 жмакал

dypa
13.01.2017
20:27:51
pdf'ку с хоткеями можно сделать через help -> keymap reference

F01134H
13.01.2017
20:28:07

Google

dypa
13.01.2017
20:28:08

Максим
13.01.2017
20:28:22

Виталий
13.01.2017
20:28:35
пальцы привыкли. и в голове комбинация не запомнилась, а потом дого не кодил и пальзы забыли и кароче трабла ппц (
mac os

dypa
13.01.2017
20:28:57

Виталий
13.01.2017
20:29:00
я через ПКМ на переменную > Refactor
а я блин наугад два вечера подбирал https://yadi.sk/d/7BxSSBt838rCJR
разве по пср так правильно?
<?php foreach ($authClientCollection as $client) : ?>
а так нет?
<?php foreach ($authClientCollection as $client): ?>

Fayozjon [CybernatiC]
13.01.2017
20:42:36

Виталий
13.01.2017
20:42:48
в пробеле между ")" и двоеточнием

Fayozjon [CybernatiC]
13.01.2017
20:42:55
?

Виталий
13.01.2017
20:43:31
ну вот мне phpcs подсвечивает, грит мол ожидается 1 пробел после скобки

Timur
13.01.2017
20:53:17
PHP CS Fixer ещё можно встроить в Continious Integration процесс.
При сборке и тестировании пускай выявляет косяки в стиле кодирования.
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/472#issuecomment-53514112
Вот тут они обсуждали это.
И ниже.
Короче просто забили. Там походу все юзают обычный синтаксис или Twig.
У себя в Yii2 проекте я просто исключил шаблоны из анализа и правки. ?

Google

Виталий
13.01.2017
21:01:05
Коллеги, а как использовать файл .php_cs, не могу нагуглить примеров.
Кому его скармливать? или он просто запускается как php .php_cs
или даже он просто должен лежать в папке проекта, а потом php-cs-fixer сам его подхватит

F01134H
13.01.2017
21:56:41
Народ, а вы юзаете public при создании метода? Это в PSR описано где-нибудь? Ну т.е. метод по умолчанию public, но можно жеж дописать при желании. Интересует - нужно ли оно

da horsie
13.01.2017
21:58:37

F01134H
13.01.2017
21:58:52
а по русски

Admin
ERROR: S client not available

F01134H
13.01.2017
21:58:56
не все ж в США живут

da horsie
13.01.2017
21:59:10
по-русски это к 1С-никам

F01134H
13.01.2017
21:59:17
((

da horsie
13.01.2017
21:59:28
область видимости должна быть прописана
у методов и у свойств
обязательно

F01134H
13.01.2017
21:59:43
оо
спасибо

da horsie
13.01.2017
22:00:30
и именно в таком порядке (abstract|final) (public|protected|private) (static)

Роман
13.01.2017
22:01:55
/yazaban
?
Прочитал ЭТО...
Я тоже пишу скобочку где надо (на той же строке). Будем вместе гореть.

Google

da horsie
13.01.2017
22:03:01
/yazaban
че это такое

F01134H
13.01.2017
22:06:40
ого себе

Роман
13.01.2017
22:08:19
Команда должна была быть может. Но нету. ?
http://pastebin.ru/gwjpSY1J
С самого начала писал так, все остальные стили не понимаю. ? Особенно когда скобка на следующей строке.
А ещё отвратительный стиль когда между арифметическими знаками не ставят пробелы. Вот за это я бы наверное в яму на сутки сажал. ?
Вокруг точнее, а не между.

Dmitriy
13.01.2017
22:13:44

Роман
13.01.2017
22:13:59
?

Dmitriy
13.01.2017
22:14:21
CI проверяет на соотвествие стандарту

Роман
13.01.2017
22:14:23
Да я сам по себе. Мелкие заказы на фрилансе. В команде не работал никогда.

Виталий
13.01.2017
22:14:35
зачем называть файл .php_cs.dist ?

Роман
13.01.2017
22:15:01

Dmitriy
13.01.2017
22:15:32

Виталий
13.01.2017
22:15:43
dist = distribution (распространяемый)