Вопросы специалистов

Сайты на ShopModxBox

Attempt to save lazy object при повторном сохранении объекта в процессоре

удаление товара из заказа в корзине

А каким образом вы добавили колонки вашему объекту OrderProduct?

А каким образом вы добавили колонки вашему объекту

действовал по этой инструкции: modxclub.ru/blog/dokumentatsiya-dlya-spetsialistov/258.html

действовал по этой инструкции: modxclub.ru/blog/do

Выложите мап-файл вашего класса OrderProduct куда-нибудь.

Выложите мап-файл вашего класса OrderProduct куда-

orderproduct.map.inc.php https://bitbucket.org/snippets/tzx1z/X84E

orderproduct.map.inc.php https://bitbucket.org/sni

На вид все ОК. Шлите мне на почту адрес сайта, логин и пароль в админку, гляну что не так.

На вид все ОК. Шлите мне на почту адрес сайта, лог

public function beforeSave(){
        print_r($this->object->_lazy);
        
        return "Debug";
        return parent::beforeSave();
}

Array
(
    [7] => testut
    [10] => duration
)

Зачем вам частичное описание этой и этой колонок?

public function beforeSave(){
        print_r($this->object->_lazy);
        
        return "Debug";
        return parent::beforeSave();
} Что видим? Array
(
    [7] => testut
    [10] => duration
) Зачем вам частичное описание этой и этой колонок?

public function beforeSave(){
        print_r($thi

Ага, раньше было нужно, потом функционал изменился, а описание колонок осталось. 
 Большое спасибо!
 Хотя до конца не ясно, почему описание этих колонок вызывало такую ошибку.


Ага, раньше было нужно, потом функционал изменился

в поле duration заносится ориентировочное время на трансфер в core/components/billing/processors/mgr/orders/products/create.class.php
 а testut — уже ненужен
 Еще раз большое спасибо!
 

в поле duration заносится ориентировочное время на трансфер в core/components/billing/processors/mgr/orders/products/create.class.php
 а testut — уже ненужен
 Еще раз большое спасибо! ?

в поле duration заносится ориентировочное время на

Хотя до конца не ясно, почему описание этих колонок вызывало такую ошибку.

Потому что есть описание колонок, но нет самих колонок в перечислении. То есть данные для сохранения как бы есть, но их как бы некуда сохранять. Поэтому xPDO и матерится. По этой причине не совсем ясен смысл отделения колонок от описания колонок. ИМХО надо было описания колонок фигачить в массив колонок, а не плодить два отдельных массива. Но сейчас уже наверняка это никто менять не будет.

Хотя до конца не ясно, почему описание этих колонок вызывало такую ошибку. Потому что есть описание колонок, но нет самих колонок в перечислении. То есть данные для сохранения как бы есть, но их как бы некуда сохранять. Поэтому xPDO и матерится. По этой причине не совсем ясен смысл отделения колонок от описания колонок. ИМХО надо было описания колонок фигачить в массив колонок, а не плодить два отдельных массива. Но сейчас уже наверняка это никто менять не будет.

Хотя до конца не ясно, почему описание этих колоно

Пожалуйста!

надо было описания колонок фигачить в массив колонок

А где находится массив колонок?


надо было описания колонок фигачить в массив колонок А где находится массив колонок?

надо было описания колонок фигачить в массив колон

Имеются ввиду элементы fields и fieldMeta. Не ясно зачем их разбили по отдельности. Их можно было объединить и подобных проблем бы не возникало.

Имеются ввиду элементы fields и fieldMeta. Не ясно

Ясно, Я увидел что поле size описано в fields и fieldMeta, так и делал.
 gist.github.com/Fi1osof/f0372195175ade7a72fb/revisions

Ясно, Я увидел что поле size описано в fields и fi

Я не говорю, что вам не надо было так делать. Я говорю странно, что Джейсон Ковард в xPDO разбил эти данные на две отдельных сущности. Практического применения этому я так и не нашел.

Я не говорю, что вам не надо было так делать. Я го

ааа, Sorry, Я Вас неправильно понял)))

Ничего страшного.

Чаще всего такое происходит, когда в мап-файле не для всех колонок есть описания. Смотри ветку здесь.
 

На счет твоего сохранения двух объектов: как я понимаю, это на самом деле повторное сохранение одного и того же объекта, просто в первом случае он создается новый, а во втором он дергается из базы. В целом твой процессор логически не правильный. Исключение — это только если ты допускаешь, что созданный $secondOp может оставаться сохраненным в базе данных, даже если первичный объект не удалось сохранить. Надо же понимать, что в beforeSet() create-процессора $this->object еще не сохранен, и у него нет ID-шника. При попытке выполнения этого процессора у тебя еще в beforeSet() сохраняется вторичный объект, хотя в итоге этот метод может вернуть ошибку (а если не он, то еще есть beforeSave(), который так же может вернуть ошибку), и первичный объект не сохранится, а вторичный объект уже будет в БД. Все-таки гораздо правильней в подобных случаях через связанные объекты делать. То есть задай связи этим двум классам и в beforeSet() пропиши типа $this->object->secondOp = $this->modx->newObject('Operation'); И при сохранении первичного объекта у тебя автоматически и вторичный сохранится. Конечно и здесь есть логические ошибки в xPDO, ибо при сохранении первичного объекта сначала сохраняются вторичные объекты, и только потом первичный (а потом и опять вторичные), но здесь уже вероятность ошибок будет гораздо меньше. Плюс к этому ты на любом этапе до сохранения первичного объекта можешь устанавливать любые значения вторичному объекту, и он будет сохранен только при попытке сохранить первичный объект. Если до первичного объекта дело не дойдет, то и вторичный не будет записан в БД.

Илья, привет. Чаще всего такое происходит, когда в мап-файле не для всех колонок есть описания. Смотри ветку здесь. На счет твоего сохранения двух объектов: как я понимаю, это на самом деле повторное сохранение одного и того же объекта, просто в первом случае он создается новый, а во втором он дергается из базы. В целом твой процессор логически не правильный. Исключение — это только если ты допускаешь, что созданный $secondOp может оставаться сохраненным в базе данных, даже если первичный объект не удалось сохранить. Надо же понимать, что в beforeSet() create-процессора $this->object еще не сохранен, и у него нет ID-шника. При попытке выполнения этого процессора у тебя еще в beforeSet() сохраняется вторичный объект, хотя в итоге этот метод может вернуть ошибку (а если не он, то еще есть beforeSave(), который так же может вернуть ошибку), и первичный объект не сохранится, а вторичный объект уже будет в БД. Все-таки гораздо правильней в подобных случаях через связанные объекты делать. То есть задай связи этим двум классам и в beforeSet() пропиши типа $this->object->secondOp = $this->modx->newObject('Operation'); И при сохранении первичного объекта у тебя автоматически и вторичный сохранится. Конечно и здесь есть логические ошибки в xPDO, ибо при сохранении первичного объекта сначала сохраняются вторичные объекты, и только потом первичный (а потом и опять вторичные), но здесь уже вероятность ошибок будет гораздо меньше. Плюс к этому ты на любом этапе до сохранения первичного объекта можешь устанавливать любые значения вторичному объекту, и он будет сохранен только при попытке сохранить первичный объект. Если до первичного объекта дело не дойдет, то и вторичный не будет записан в БД.

Илья, привет. Чаще всего такое происходит, когда в

О, а точно, спасибо, переделаю через связанные объекты. Посмотрим, что будет))

О, а точно, спасибо, переделаю через связанные объ

Но это не решит твоей проблемы с лейзи. Проверяй мапу. Ссылку я дал.

Но это не решит твоей проблемы с лейзи. Проверяй м

Клево. Переделал код так:
 

<?php
class OperationCreateProcessor extends modObjectCreateProcessor {
    
    /* 
     * Процессор создает "Операции" для управления финансами
     * Операция - это расход, приход или перевод со счета на счет
     */
     
    public $classKey = 'Operation';
    public $objectType = 'object';
    
    public function beforeSet() {
        $amount = $this->getProperty('amount');
        switch ($this->getProperty('type')) {
            case 'charge':
                $this->setProperty('amount', -1 * $amount);
                break;
            case 'income': 
                $this->setProperty('amount', 1 * $amount);
                break;
            case 'transfer':
                $this->object->Related = $this->modx->newObject('Operation');
                $this->object->Related->fromArray($this->getProperties());
                $this->object->Related->set('amount', 1 * $amount);
                $this->object->Related->set('account', $this->getProperty('to'));
                $this->object->Related->set('relation_type', $this->getProperty('type'));

                $this->setProperty('amount', -1 * $amount);
                $this->setProperty('account', $this->getProperty('from'));
                $this->setProperty('related', $secondOp->id);
                $this->setProperty('relation_type', $this->getProperty('type'));
                break;
            default:
                $this->modx->error->addField('type', 'Unknown type');
                return false;
        }
        return true;
    }
    
    public function afterSave() {
        if ($this->getProperty('type') == 'transfer') {
            if ($this->object->Related) {
                $this->object->Related->set('related', $this->object->id);
                $this->object->Related->save();
            }
        }
        
        return true;
    }

}

return 'OperationCreateProcessor';

И вот здесь опять вопрос. Вот есть строка, где связанные объекты сохраняются второй раз и в них передаются ключи основного объекта для сохранения связи. Однако у меня без блока afterSave у связанного объекта поле related оказывается незаполненным.
 

// ...
    'Related' => 
    array (
      'class' => 'Operation',
      'local' => 'related',
      'foreign' => 'id',
      'cardinality' => 'one',
      'owner' => 'foreign',
    ),
// ...

По идее должно работать. Но почему-то не хочет…

Клево. Переделал код так: <?php
class OperationCreateProcessor extends modObjectCreateProcessor {
    
    /* 
     * Процессор создает "Операции" для управления финансами
     * Операция - это расход, приход или перевод со счета на счет
     */
     
    public $classKey = 'Operation';
    public $objectType = 'object';
    
    public function beforeSet() {
        $amount = $this->getProperty('amount');
        switch ($this->getProperty('type')) {
            case 'charge':
                $this->setProperty('amount', -1 * $amount);
                break;
            case 'income': 
                $this->setProperty('amount', 1 * $amount);
                break;
            case 'transfer':
                $this->object->Related = $this->modx->newObject('Operation');
                $this->object->Related->fromArray($this->getProperties());
                $this->object->Related->set('amount', 1 * $amount);
                $this->object->Related->set('account', $this->getProperty('to'));
                $this->object->Related->set('relation_type', $this->getProperty('type'));

                $this->setProperty('amount', -1 * $amount);
                $this->setProperty('account', $this->getProperty('from'));
                $this->setProperty('related', $secondOp->id);
                $this->setProperty('relation_type', $this->getProperty('type'));
                break;
            default:
                $this->modx->error->addField('type', 'Unknown type');
                return false;
        }
        return true;
    }
    
    public function afterSave() {
        if ($this->getProperty('type') == 'transfer') {
            if ($this->object->Related) {
                $this->object->Related->set('related', $this->object->id);
                $this->object->Related->save();
            }
        }
        
        return true;
    }

}

return 'OperationCreateProcessor'; И вот здесь опять вопрос. Вот есть строка, где связанные объекты сохраняются второй раз и в них передаются ключи основного объекта для сохранения связи. Однако у меня без блока afterSave у связанного объекта поле related оказывается незаполненным. Вот описание связи: // ...
    'Related' => 
    array (
      'class' => 'Operation',
      'local' => 'related',
      'foreign' => 'id',
      'cardinality' => 'one',
      'owner' => 'foreign',
    ),
// ... По идее должно работать. Но почему-то не хочет…

Клево. Переделал код так: <?php
class OperationCre

Вот теперь код более чистый. Правда рудимент видимо остался — $this->setProperty('related', $secondOp->id); Объекта $secondOp нет у тебя в том методе.
 

По идее должно работать. Но почему-то не хочет… 

У тебя связь неправильно прописана (если я не ошибаюсь, что эта связь прописана для мапы $this->object). У тебя в ней сказано, что owner — foreign, то есть будет использоваться значение из внешнего объекта (из его колонки id (ключ foreign)). То есть это описание для изменения локальной колонки $this->object->related, которая примет значение от $this->object->Related->id, а не наоборот. Если ты хочешь менять колонку внешнего объекта, то мапа должна быть такой:
 

// ...
    'Related' =>
    array (
      'class' => 'Operation',
      'local' => 'id',
      'foreign' => 'related',
      'cardinality' => 'one',
      'owner' => 'local',
    ),
// ...

Вот теперь код более чистый. Правда рудимент видимо остался — $this->setProperty('related', $secondOp->id); Объекта $secondOp нет у тебя в том методе. По идее должно работать. Но почему-то не хочет… У тебя связь неправильно прописана (если я не ошибаюсь, что эта связь прописана для мапы $this->object). У тебя в ней сказано, что owner — foreign, то есть будет использоваться значение из внешнего объекта (из его колонки id (ключ foreign)). То есть это описание для изменения локальной колонки $this->object->related, которая примет значение от $this->object->Related->id, а не наоборот. Если ты хочешь менять колонку внешнего объекта, то мапа должна быть такой: // ...
    'Related' =>
    array (
      'class' => 'Operation',
      'local' => 'id',
      'foreign' => 'related',
      'cardinality' => 'one',
      'owner' => 'local',
    ),
// ...

Вот теперь код более чистый. Правда рудимент видим

$this->setProperty('related', $secondOp->id);

лишняя (related проставит сам MODX, когда будет сохранять основной объект, на основе связи).
 

И понял, что без afterSave не обойтись. У меня оба объекта равнозначны и должны быть связаны между собой. То есть
 

$first = $this->object;
$second = $this->object->Related;
$second->addOne($this->object);

в итоге это не одна связь, а две. Поэтому еще одно сохранение делать придется в любом случае)

Да, строчка $this->setProperty('related', $secondOp->id); лишняя (related проставит сам MODX, когда будет сохранять основной объект, на основе связи). И понял, что без afterSave не обойтись. У меня оба объекта равнозначны и должны быть связаны между собой. То есть $first = $this->object;
$second = $this->object->Related;
$second->addOne($this->object); в итоге это не одна связь, а две. Поэтому еще одно сохранение делать придется в любом случае)

Да, строчка $this->setProperty('related', $secondO

У тебя связь один-к-одному. Зачем тебе две равнозначные связи на два объекта? Ты через один всегда сможешь понять есть у тебя второй объект или нет. 98% у тебя не оптимальная структура.

У тебя связь один-к-одному. Зачем тебе две равнозн

lazy

ShopModxBox

корзина

процессоры

# Николай Ланец aka Fi1osof
**Lead Fullstack Developer / Ментор / Евангелист современного JS‑стека**

---

## О себе

Я — не просто Senior Developer с 2007 года. Я — тот, кто строит продукты будущего на базе современных технологий и помогает другим подниматься на новую ступень айти-карьеры. Создатель и идейный вдохновитель freecode.academy — платформы, где обучение перестает быть скучной обязаловкой и становится настоящим технологическим приключением.

---

## Мой стек и философия разработки

Основу моего рабочего арсенала составляют:

- [Javascript](/technologies/ck1faudv928l40a89c4u9zxlm), [TypeScript](/technologies/ckcvwwx010kov08009u7y2rav)
- [React](/technologies/ck1fawbdw28v20a891k4wlh27), [Next.js](/technologies/ckg2jd4ffr9260730aog66dri)
- [Node.js](/technologies/ckt55wm59272831wo9gg9r8ynx), [GraphQL](/technologies/ck1fawhqk28vu0a8995lqvlsj), [Apollo Client](/technologies/ck2zdcg0vpvc10a89xjmhj1e0), [Nexus](/technologies/ckl2pbh0fxew60730ifl3zozp), [Prisma](/technologies/ck2zdbmi7pv8r0a89ntc2r0f1)
- DevOps: [Docker](/technologies/ck2y1x9lho9cy0a8953xzfjr8), [docker-compose](/technologies/ck2y1xqino9g80a89tsftxin2), [Caddy](/technologies/ck2zd8ovqpuyl0a8909748sbu), [Ubuntu](/technologies/cktu8giks3593911vmle30ztqxt)
- UI-инструменты: [Styled-Components](/technologies/ckk8myysio7lw0730csszvrdw), [Material-UI](/technologies/ckljv3jwd7bgf0730huxlko9n), [react-hook-form](/technologies/ckmx8cn9fmwrv073032e67722)
- Всё актуальное для разработки, CI/CD и автоматизации 
- Эксперименты с [WebGL](/technologies/ckan9anntc12f08003saqttrq), [A‑Frame](/technologies/ck2tgoiz8j37y0a89c7gxsrxc), [Three.js](/technologies/ck2tgns1nj32z0a89vmw7xw3f), [Unreal Engine 4](/technologies/ck7puswy6g4yb07004cdp6lbr), [Elixir](/technologies/ckn0olavrnnzi0730xs8rctn8), [C++](/technologies/cknin1y8lu2fr07307mskn2fn)

---

## Разработка и менторство

- Более 15 лет **hands-on** разработки и проджект‑менеджмента
- Главный разработчик и преподаватель платформы [freecode.academy](https://freecode.academy)
- **Ведущий менти-пул на платформе** — лично выращиваю новых звезд отрасли

### Мои менти и их результаты

| Менти (ФИО \| username)                               | Уровень | Рейтинг |
|-------------------------------------------------------|---------|---------|
| [Попов Дмитрий \| linklib](/users/cjoe890c706ji0d962oismzoe)           | 3       | 315     |
| [Сэт \| defk1lla](/users/ckzh1ita312978121imr9hpqfc64)                 | 3       | 670     |
| [Ксения Матвеева \| Ксения Матвеева](/users/ckz4awwlh10627771imr4wvpb7cj) | 2       | 0       |
| [Михаил \| Balamut](/users/cl1npq92z10258261vmbq0w6vfh2)               | 2       | 0       |
| [Ольга Ершова \| angel](/users/cjoe87zhk00250d969arluevk)              | 1       | 0       |
| [Егор \| rogedkone](/users/cksrdp47d3308261ijuq7wfrxdl)                | 3       | 0       |
| [Roman \| Roman43](/users/ckur2qtt64721431xoboyjcte52)                 | 2       | 0       |
| [Бут Игорь \| santiment](/users/ckttd9x20893951vmltb8t7sha)            | 1       | 0       |

**Доношу простую мысль: успех — это не магия, это осознанная системная работа и кайф от роста.**
Мои менти выводят решения в продакшн, пробиваются в лидеры рейтингов — кому не хочется быть в такой команде?

---

## Почему я?

- **Прокачаю проект** — от TDD архитектуры до культурного CI/CD
- **Вдохновлю команду** — и научу мыслить по‑новому, где ИИ — напарник, а не угроза
- **Выращу мидлов и сеньоров** — доказано платформой и рейтингами моих учеников

**Сила — в сообществе, а техника — это просто смазка для ваших побед. Присоединяйтесь!**

---

## Контакты

- 📧 [n.lanets@freecode.academy](mailto:n.lanets@freecode.academy)
- Телеграм: [@freecode_academy](https://t.me/freecode_academy)

---

**P.S.**: Мой опыт — ваш драйвер роста. Хотите в команду? Ищете менторство? Пишите. Растём вместе!