Комментарии 7
Простите, а что полезного для читателей в этой статье, если вы не написали ничего конкретного, даже в чем бага? Была бага, пофиксел, написал автору в гитхаб, конец. Это такой миниатюрный курс о пулл реквестах гитхаба?
а в код сложно посмотреть?
я ссылку привел на pullrequest и кратко описал в чем именно была проблема.
Давайте для вас более подробно распишу:
там в коде идет посимвольный парсинг строчек файла.
и в этом парсинге есть следующая проверка:
Если встретился символ ';' и это не внутри комментария и т.п. — то проверяем — есть ли что в текущей строке слева от точки с запятой.
Если есть — добавляем в буфер с запросом, добавляем туда же перевод строки и точку с запятой.
и вот тут баг. вместо того, чтобы закрыть if автор там дальше, в том же if вызывает pg_query
вот и получается, что если в строчке слева от точки с запятой ничего нет (символ ';' первый в строке) — то запрос на выполнение не уходит.
ошибка то элементарная, интерес вызывает количество лет, которое она находится в коде достаточно популярного приложения, и достаточная экзотичность условий, которые приводят к ее появлению
я ссылку привел на pullrequest и кратко описал в чем именно была проблема.
Давайте для вас более подробно распишу:
там в коде идет посимвольный парсинг строчек файла.
и в этом парсинге есть следующая проверка:
Если встретился символ ';' и это не внутри комментария и т.п. — то проверяем — есть ли что в текущей строке слева от точки с запятой.
Если есть — добавляем в буфер с запросом, добавляем туда же перевод строки и точку с запятой.
и вот тут баг. вместо того, чтобы закрыть if автор там дальше, в том же if вызывает pg_query
вот и получается, что если в строчке слева от точки с запятой ничего нет (символ ';' первый в строке) — то запрос на выполнение не уходит.
ошибка то элементарная, интерес вызывает количество лет, которое она находится в коде достаточно популярного приложения, и достаточная экзотичность условий, которые приводят к ее появлению
О, это та самая рубрика странная со странными работами над ошибками… Прошлая была на -13 рейтинга…
Прочитал описание пуллреквеста — эм, английский бы очень не плохо подтянуть… Что такое «right patch for fix error»? :)
А еще там есть разметка, которой можно код выделить…
Прочитал описание пуллреквеста — эм, английский бы очень не плохо подтянуть… Что такое «right patch for fix error»? :)
А еще там есть разметка, которой можно код выделить…
Это что, и себе статьи начать писать о каждом pull request?
Ну исправили, спасибо вам от многих пользователей, но на статью для хабра это не тянет.
Ну исправили, спасибо вам от многих пользователей, но на статью для хабра это не тянет.
Мне интересно, где же вы таких клиентов нашли которые сами дамп заливают, да еще через phpPgAdmin?
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Работа над ошибками — phppgadmin