237 serializer test #308
237 serializer test #308
Conversation
-and edit NetworkTimingsParser.php
-added a test to check the operation of products and transactions between themselves
- the test works with the help of services
-add improvements to the method DEAL_PRODUCT_ROW so that it can use the currency
…l.get,crm.deal.productrows.get)
… это путем замены области видимости с private на protected в DealProductRowItemResult.php.(Так можно делать ?)
-delete var_dump
- edit method in DealProductRows.php(work)
- edit test (delete testBatch)
| $var = $this->data[$offset] * 100; | ||
| return new Money((string)$var,new Currency($this->currency->getCode())); | ||
| } | ||
| return null; |
There was a problem hiding this comment.
Добавь остальные поля
* @property-read Money $PRICE_EXCLUSIVE
* @property-read Money $PRICE_NETTO
* @property-read Money $PRICE_BRUTTO
| /** | ||
| * @return DealProductRowItemResult[] | ||
| * @throws BaseException | ||
| */ | ||
| public function getProductRows(): array | ||
| { | ||
| $res = []; | ||
| foreach ($this->getCoreResponse()->getResponseData()->getResult() as $productRow) { | ||
| $res[] = new DealProductRowItemResult($productRow); | ||
| foreach ($this->getCoreResponse()->getResponseData()->getResult()['result']['rows'] as $productRow) { |
There was a problem hiding this comment.
А почему ты тут добавил?
['result']['rows']
There was a problem hiding this comment.
Вообщем у меня с этим сейчас возникли проблемы , но я вроде бы пофиксил это добавив дополнительное условие. Проблема была в том что наш метод очень умный getSuperSuperSmart(get). И если мы принимаем только айдишник сделки, то у нас выполняется batch запрос в котором мы возвращаем саму сделку и строку товара, это все у нас записывается в переменную $res. Потом мы ее добавляем в наш объект DealProductRowItemsResult. После чего нам приходится дополнительно указывать [result][rows] , так как есть еще выбор выбрать [result][deal]. А вот если мы указываем в нашем методе и айдишник и валюту , то тогда в DealProductRowItemsResult у нас приходит лишь один запрос с одним массивом и нам не нужно указывать [result][rows].
| */ | ||
| public function get(int $dealId): DealProductRowItemsResult | ||
| public function getStupid(int $dealId, Currency $currency): DealProductRowItemsResult |
There was a problem hiding this comment.
С этим понятно всё, как отладим, можно удалять
| * @throws \Bitrix24\SDK\Core\Exceptions\BaseException | ||
| * @throws \Bitrix24\SDK\Core\Exceptions\TransportException | ||
| */ | ||
| public function getSuperSmart(int $dealId): DealProductRowItemsResult |
There was a problem hiding this comment.
с этим тоже понятно, как отладим, можно удалять
| * @throws \Bitrix24\SDK\Core\Exceptions\BaseException | ||
| * @throws \Bitrix24\SDK\Core\Exceptions\TransportException | ||
| */ | ||
| public function getSuperSuperSmart(int $dealId, Currency $currency = null): DealProductRowItemsResult |
| use Money\Currencies\ISOCurrencies; | ||
| use Money\Currency; | ||
| use Money\Formatter\DecimalMoneyFormatter; | ||
| use Money\Money; |
There was a problem hiding this comment.
Причеши тесты, оставь тест на get( с двумя вариантами:
- мы не знаем валюту
- мы знаем валюту
There was a problem hiding this comment.
ну тесты я делаю не очень , надеюсь готово)
…(Теперь в тестах можем и указывать валюту или не указывать). - убрал лишние методы - поправил тесты - добавил в AbstractCrmItem.php остальные поля связанные с деньгами.
mesilov
added
hacktoberfest-accepted
Занимаюсь добавлением поддержки phpmoney как базового типа для денег