Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Проблемы с обновлением сделки #10

Open
drillcoder opened this issue Jul 1, 2016 · 5 comments
Open

Проблемы с обновлением сделки #10

drillcoder opened this issue Jul 1, 2016 · 5 comments

Comments

@drillcoder
Copy link

При обновлении полей в сделки, если не заданы поля "имя сделки" и "статус" то значения из низ пропадают((
В коде так и не нашёл причины подобного поведения, если в ручную делать запрос без обёртки всё ок, обновляются только те поля которые были указаны, остальные остаются неизменными.

@stepkagr
Copy link

stepkagr commented Oct 1, 2016

request.php

перед
$this->params = $params;
добавить

foreach ($params as $key => $value) {
            if (empty($value)) {
                unset($params->$key);
            }
        }

@s-belichenko
Copy link
Owner

А в каком месте именно добавлять? Там есть такая строка в конструкторе и есть в private function createPostRequest.

@stepkagr
Copy link

В конструкторе

@s-belichenko
Copy link
Owner

Проверил - в итоге при создании сделки там же в конструкторе пустой id пока что, и в итоге этот код его удаляет, в итоге в createPostRequest в строке $id = $this->params[0]->id; не находится id и происходит ошибка.

@dmitry4m
Copy link

предложенный выше код идеологически верный, но требует два уточнения:

  1. не на empty() проверять, а на ==NULL
  2. проверять, что вызов идёт на обновление, а не добавление.

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

No branches or pull requests

4 participants