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