Структура директорий

[stev]

~ не большие изменения в структуре директорий

[dovg]

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

[dovg]

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

[AlexeyDsov]

И правда, почему бы не вынести View в отдельную папку.
и +1 к @dovg - по поводу мусора в диффе.

[stev]

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

[AlexeyDsov]

когда-то ранее, обсуждалось что в onPHP пустые строки не удаляются а "tab'ируются" до уровня начала строки, то есть:

<?php
    class MyClass
    {
        //вот тут пустая строка
        public function run()
        {
            //вот тут пустая строка
        }
    }

И если посмотреть файлы они так в большинстве и написаны и объекты тоже в таком стиле генеряться.

[dovg]

@stev https://raw.github.com/onPHP/onphp-framework/master/doc/CodingStyle пустые строки часть нашего coding style

[stev]

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

Вернуть пробелы?

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

[AlexeyDsov]

Если вдруг решиться убрать пробелы - то это явно стоит делать другим коммитом по всему коду, а в этом тикете что б не было ничего лишнего в diff'е, то, думаю, стоит вернуть пробелы.

[dovg]

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

Это coding style. Тут нет "правильно" и "неправильно". Есть то, чему следуют все. )

[stev]

Если вдруг решиться убрать пробелы - то это явно стоит делать другим коммитом по всему коду, а в этом тикете > что б не было ничего лишнего в diff'е, то, думаю, стоит вернуть пробелы.

Ok

[stev]

@dovg я приветствую консерватизм, в определенных соотношениях.

Это coding style. Тут нет "правильно" и "неправильно". Есть то, чему следуют все. )

-если есть смысл следовать - то Да.
это то же не библя :) чтоб принимать все как есть.

В общем, я сторонник изменений, С преемственностью конечно. Пустые пробелы - не относится к сильным сторонам onPHP и не ухудшают их.

зы. onPHP - может быть лучше :) поэтому иногда стоить вносить изменения в соглашения.

[dovg]

@stev сделай лучше что-нибудь хорошее.
Файлы перекладывать и пробелы убирать - это пустое, ИМХО.

[stev]

Файлы перекладывать и пробелы убирать - это пустое, ИМХО.

это подготовка почвы )