Skip to content

Update jcomments.ajax.php#29

Open
volandku wants to merge 1 commit intoexstreme:Developfrom
volandku:patch-3
Open

Update jcomments.ajax.php#29
volandku wants to merge 1 commit intoexstreme:Developfrom
volandku:patch-3

Conversation

@volandku
Copy link
Copy Markdown
Contributor

Многочисленные ошибки, в частности при отправке жалобы на коммент если идут warning, а они идут, если конечно не выставлено отображение ошибок в Нет
ИМХО для современных версий php, даже начиная с 5,6 актуально.
ЗЫ. Это касается только аякс-ответов, где напрямую варнинги не видны

Многочисленные ошибки, в частности при отправке жалобы на коммент если идут warning, а они идут, если конечно не выставлено отображение ошибок в Нет
ИМХО для современных версий php, даже начиная с 5,6 актуально.
ЗЫ. Это касается только аякс-ответов, где напрямую варнинги не видны
@exstreme
Copy link
Copy Markdown
Owner

Есть пример актуальных варнингов ? А не которые в официальной версии?
Такой хардкод будет потом мешать в поиске проблемы, когда окажется, что логика скрыта

@volandku
Copy link
Copy Markdown
Contributor Author

Я уж закрыл тот проект.
На 5.6 в аякс-ответе сыпались при жалобе на коммент (без указания причины) от гостя
В итоге в ответе вместо json получалась фигня

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants