Рассудите, виноват я или прав?

Question

[Михаил Шатилов]

Я - разработчик. Началась история с того, как мне месяц назад пришел проект "на доработку" (сайт-магазин). Нововведения списка задач пересекаются с изменением уже существующего кода, как вы знаете, часто проекты состоят из жуткого говно-кода, и данный - не исключение. Ладно, думаю, разберусь и сделаю. Уже заканчиваю последние задачи и дошел до бага, который сложно исправить, ввиду его непостоянности. Проблема в том, что иногда если не ввести email, то выдается ошибка, иногда - нет (наличие 2х форм, одна из которых скрыта является причиной непостоянности..).
Как следствие, последнее время стали часто приписывать ЛЮБЫЕ (ночью могут позвонить) проблемы с данной формой (багом), хотя я вообще мог не работать с ней, глобальный костыль предыдущих программистов дает о себе знать.

Я считаю, что если мне дали задание поправить вывод ошибок с формы, то это не значит, что я должен выкинуть весь код форм и написать свой без костылей.
Вышестоящий считает (начальник и разработчик), что за сайт в ответе я и все нужно сделать красиво.

P.S. все это на битриксе.

Comments

[Михаил Шатилов]

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

Просто очень не приятно, когда все косяки валят на меня (сам на проекте), даже если код не мой/верстальщик где-то запорол.

Answer 1

[murich]

Должен делать то за что платят. Хотят новую форму - поясни сроки и перепиши заново если согласятся.

Comments

[Михаил Шатилов]

Я сказал, что перепись займет недель (с прозапасом и тестированием), конечно, не согласились.

[Михаил Шатилов]

Я сказал, что перепись займет недель (с прозапасом и тестированием), конечно, не согласились.

[Пума Тайланд]

Ну не согласились и не согласились, че звонить то ? Так и скажи им

[Михаил Шатилов]

@opium ничего не докажешь)

[Пума Тайланд]

А чего тут доказывать то ? Прямым текстом скажите чтобы по этим вопросам не звонили так как ОНИ не согласны, а не вы.

Answer 2

[Yakov Akulov]

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

Я так понял вы баг этот по контракту должны исправить.

Магии в программировании не бывает (почти) - прогуляйтесь пару часиков на свежем воздухе, а потом с головой окунитесь в проблему на несколько часов, вооружившись всеми возможными отладочными инструментами. Я уверен, проблема решится, а у Вас будет +1 к самооценке и опыту)

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

Comments

[Михаил Шатилов]

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

[Михаил Шатилов]

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

Answer 3

[Алексей]

Виноват битрикс и вы, потому что связались с ним.
Ну а если серьезно, вам нужно доделать\исправить.

Comments

[Crash]

+1. Где можно оправданно использовать битрикс? — В анекдотах (с)

[Михаил Шатилов]

Я бы с ним не связывался, но не я решаю что мне делать :)
Т.к. намечались 2 новых проекта по битриксу, то уволился. Банально не хочу мучатся и копаться в нем.