Skip to content

Conversation

createit-ru
Copy link
Contributor

Что оно делает?

Методы add, change, remove класса msCartHandler возвращают на фронт много информации, такой как текущая корзина, общая стоимость товаров и их вес и другое.
Но нет информации о том, сколько товара было в корзине ДО совершения действия.
Например: в корзине было 2 единицы товара, и мы добавили еще один. В ответе от сервере будет count = 3. Но для различных ситуаций может потребоваться информация о том, сколько товаров было ДО совершения действия (2 товара). Самый простой пример - это передача данных в системы электронной коммерции.

Это изменение добавляет новый ключ changes в ответ сервера:
image

change содержит в себе:

  • count - текущее кол-во товара в корзине
  • old_count - прошлое кол-во товара
  • delta - разница между count и old_count
  • method - метод, который был вызван (add, change, remove)

Зачем это нужно?

На фронте будет доступно больше данных об изменениях после выполнения действий (add, change, remove) с товарами в корзине.

Связанные проблема(ы)/PR(ы)

#908

@createit-ru createit-ru marked this pull request as ready for review September 29, 2025 19:34
Copy link
Member

@biz87 biz87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@createit-ru ты уж если начал строгую типизацию вносить в класс, то и параметры тоже типизируй, а не только ответ

@createit-ru
Copy link
Contributor Author

@biz87 хорошо, доберусь, но уже через неделю. Что думаешь про идею в целом и наименование ключа changes в частности?

@createit-ru
Copy link
Contributor Author

@createit-ru ты уж если начал строгую типизацию вносить в класс, то и параметры тоже типизируй, а не только ответ

Добавил, и в базовый интерфейс msCartInterface тоже. Как думаешь, не затронет это никого?

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.

3 participants