|
отлично.
==================
хотя вместо "select InventItemLocation where InventItemLocation"
лучше было сделать InventItemLocation::find
тогда код можно было бы упростить.
================
кроме того, если уж вы попытались сделать комбинации...
код учитыват только некоторые стандартные складские аналитики.
Вы не учитываете, что:
1. в системе есть инвентаризация партии. Т.е. вполне возможно стоит делать записи и по партиям
2. в системе можно добавить свои складские аналитики
3. в 3.0, для каждой номенклатуры, есть список допустимых комбинаций. И по идее надо бы заполнять не все комбинации, а только допустимые
==============
А почему вы не на query делали?
Я, конечно, могу ошибаться, но мне кажется, это упростило бы код и добавило возможность указывать диапазоны.
Кто нибудь хочет еще написать более оптимальный код?
|