Сергей Протько

Клиент понимает только цифры, ему плевать на качество кода ровно до той поры, пока поддержка кода не станет ему в копеечку лишнюю. Приведите ему реальные доводы ЗА рефакторинг выражающиеся профитом в денежном эквиваленте и вуаля. Ну а если этих доводов нет - только личное мироощущение, то нужен ли рефакторинг?

Ну и коль уж мы про цифры, клиентов и технический долг, а как вы убедили клиента оплачивать вам время на написание тестов? Вы включаете это во время разработки при оценке стоимости? Что мешает заложить и рефакторинг критических мест. Вы не пишите тесты? Тогда о каком рефакторинге имеет смысл вести разговор? Тогда доводы должны быть просто железобетонные, что бы не тратить время на фул-тэст и поиск регрессий.

вы плохо читали советы гугла. Он обычно рекомендует вставить в конец страницы скрипт, который добавляет link в head. Тип так:

link=document.createElement('link');
link.href='href';
link.rel='rel';

document.getElementsByTagName('head')[0].appendChild(link);

В этом случае стили будут загружаться тогда, когда будет загружен весь DOM. А для того что бы страница не выглядела стремно, тот же гугл рекомендует инлайнить above-the-fold css в тело страницы.

Поймите, GIT - распределенная система. Вы и ваш коллега будете работать в абсолютно разных репозиториях, у каждого свой. Удаленный репозиторий будет служить точкой синхронизации. Так что если вы сделаете локальный брэкн, он будет только у вас. То есть что бы этот брэнч был доступен другому разработчику, вы должны его запушить в общий удаленный репозиторий. И тут веселье, так как ваш коллега работает с форком проекта, у вас два удаленных репозитория.

В целом меньше боли будет если вы оба будете синхронизировать изменения между собой только через master. Читать про git-flow.

скорее всего сервер редеректнул вас на server-error, и запрос по редиректу шел уже по GET

Какую архитектуру(mvc, hmvc...) выбрать для интернет магазина?

CQRS + Event Sourcing.

REST или SOAP, склоняюсь больше к SOAP. т.к платежные системы начали переходить на SOAP

только REST (ну или RPC), SOAP это боль и унижение (если вы конечно не пишите и клиент и сервер на .NET)

php 7? можно попробовать на

А смысл? Потом проапдейтитесь.

Нагрузка предполагаемая около 100 000 посещений в день, максимум.

Ну это тип... мелочь. Пока забейте на производительность. Лучше потом этим заняться, вооружившись профайлером и нагрузочными тестами. И оптимизация эта будет включать введение дополнительных слоев кеширования и т.д.

Обмен данными асинхронный (ajax или websokets не знаю что лучше выбрать..)

Не знаю как правильно пишется, Июнь или Июль? Ирак или Иран?. websockets помогут если у вас очень интенсивный обмен данными между клиентом и сервером. В остальном long-poling попроще. Хотя я думаю вам push-уведомления вообще не нужны.

redis(для кэша, push, всяких оповещений)

Только для кэша и можно еще как шину данных и очередь использовать.

Клиент написан на js.

А можно на чем-то другом? (языки транслирующиеся в js не в счет).

Да везде.

Допустим несколько сот тысяч долларов. А теперь вычитаем из этой суммы то, сколько заказчику потребуется последующую поддержку и рефакторинг говнокода, что останется после вас, и расстраиваемся. А сколько разработчикам после вас придется пить бета-блокаторов...

что бы велосипед не изобретался заново

реализовать небольшой Object-Oriented JavaScript Framework

Мне кажется в этих двух цитатах есть противоречие.

Вот вам и идея проекта. Web-сайт агрегатор идей и поиск единомышленников.

Почему при создании класса переменная не обнуляется?

А кто вам сказал что класс создается каждый раз?

Вообще возьмите себе в привычку, что менять конструктор репозитория это не ок. И делать любые сервисы statefull тоже не ок, но иногда нужно.

Если вам нужно какое-то базовое условие запихнуть - можно вынести это в свой метод-фабрику, которая будет создавать вам QB.

class ItemRepository extends EntityRepository
{

    public function findOneByUrl($url)
    {
        return $this->filterIsActive()
            ->andWhere('i.url = :url')
            ->setParameter('url', $url)
            ->getQuery()->getSingleResult();

    }

    public function findTop()
    {
        return $this->filterIsActive()
            ->getQuery()->getResult();

    }

    public function findAll()
    {
        return $this->filterIsActive()
            ->getQuery()->getResult();

    }


    public function findByCategory($category)
    {
        return $this->filterIsActive()
            ->andWhere('i.category = :category')
            ->setParameter('category', $category)
            ->getQuery()->getResult();
    }

    private function filterIsActive() 
    {
           return $this->createQueryBuilder('i')
                ->where('i.isActive = true');
    }


}

Достаточно иметь доступ к DNS записям.