zeckson / 125753-yeticave-1

Евгений Щепотьев

125753-yeticave-1's Issues

Fix time to auction end

Remove docker-compose.yml

Remove and add docker-compose.yml

Get rid of `empty()`

Empty is bad practice due to cast everything to everything
http://php.net/manual/en/function.empty.php

Handle Swift Transport exception

Есть ещё ошибки при отправке почты в редких случаях, но это не совсем про базовый PHP, так что не надо делать: https://yadi.sk/i/vYPWo0rnt_fQ2w

Fix category title

На странице категории заголовок остается "заглавная". Не критично.

Fix XSS

на странице лота не делаем точно, https://yadi.sk/i/N2gWqXKzhnxb9Q
возможно, где-то ещё...

Fix ago date format

Fix DDL

вместо DROP DATABASE yeticave; следовало написать DROP DATABASE IF EXISTS yeticave;

При логине с неверным паролем выскакивает ошибка

Fatal error: Uncaught TypeError: Return value of login() must be of the type array, null returned in /Users/ilia.kondrashov/Downloads/project-40819/src/user_queries.php:65 Stack trace: #0 /Users/ilia.kondrashov/Downloads/project-40819/login.php(30): login(Object(mysqli), '[email protected]...', 'qqq') #1 {main} thrown in /Users/ilia.kondrashov/Downloads/project-40819/src/user_queries.php on line 65

На странице лота для гостя и автора лота не показывается ни цена, ни история ставок

В ТЗ есть условия на вывода блока добавления ставок, но не на цену и историю ставок.

Add finishing indicator

There is finishing indicator on time timer--finishing that is not used currently

Add pagination

Add days to countdown

• функция работы с датой считает только часы и минуты? Дни, хотя бы было бы круто добавить... https://yadi.sk/i/egtyJgHbTJlVDA

Fix errors on lot.php

https://yadi.sk/i/LZkuW8DlVquCXA нет проверки на то, что число в значении нашего параметра?

константы не подключаем, например, в lot.php, от этого ошибка: https://yadi.sk/i/XdcrdBx1MRqn4Q

Fix category empty page

• 404 не отдаём https://yadi.sk/i/3ePlR2EKmaQrOw (но тут получилось подобие шаблона что надо)) Осталось только подписать "нет товаров в категории", если категория есть, но нет товаров и "нет такой категории" в другом случае..

Fix expired lot timer

Add declare(strict_types=1); everywhere

Не во всех файлах я вижу в начале declare(strict_types=1);, а без этого жесткая типизация во-многом теряет смысл.

Fix category links

в таблицу категорий можно было бы добавить поле alias или url, тогда бы могли генерить ссылки /lot_list.php?category=boots вместо /lot_list.php?category=2, а тут и до ЧПУ не долго)

Handle closed lots properly

• add closed state for FSI lot
• do not add bids for close lots

Ошибка на странице поиска

http://0.0.0.0:3000/search.php?query=scoop

Warning: mysqli_stmt_bind_param() expects parameter 1 to be mysqli_stmt, boolean given in /Users/ilia.kondrashov/Downloads/project-40819/src/utils/db.php on line 35

Warning: mysqli_stmt_execute() expects parameter 1 to be mysqli_stmt, boolean given in /Users/ilia.kondrashov/Downloads/project-40819/src/utils/db.php on line 38

Fatal error: Failed SQL-query: "SELECT lot.id, lot.name, start_price, description, bid_step, lot.created_at, lot.closed_at, lot.author_id, IFNULL(MAX(bid.amount), start_price) AS price, image_url AS image, category.name AS category, count(bid.id) AS bids_count FROM lots lot LEFT JOIN categories category ON lot.category_id = category.id LEFT JOIN bids bid ON lot.id = bid.lot_id WHERE TIMESTAMP('2018-10-15 17:42:38') < lot.closed_at AND MATCH(lot.name, lot.description) AGAINST (?) GROUP BY lot.id ORDER BY lot.created_at DESC" with error: Can't find FULLTEXT index matching the column list in /Users/ilia.kondrashov/Downloads/project-40819/src/utils/db.php on line 42

Fix return types

For all funs return array
For get single funs return ?array

Add template for 404 page

• для страницы 404 можно было бы сделать шаблон, вообще на 5 мин работы

На главной странице подпись "Стартовая цена", а выводится текущая.

Предлагаю убрать слово "Стартовая"

На форме логина пароль при вводе показывается в открытом виде