207 похожих чатов

Смотрю код одного модуля, там: payment_ids = self.env['account.payment'] payment_ids |=

order.mapped('purchase_payment_ids')
Оно чем-то лучше простого
payment_ids = self.purchase_payment_ids
?

11 ответов

31 просмотр

ну как минимум это красиво :D :D :D

Может в предыдущих версиях не было поля purchase_payment_ids

Andro- Автор вопроса

Та не, тут же итерация по self: for order in self: payment_ids = self.env['account.payment'] payment_ids |= order.mapped('purchase_payment_ids')

Andro
Та не, тут же итерация по self: for order ...

если было вот так: payment_ids = self.env['account.payment'] for order in self: payment_ids |= order.mapped('purchase_payment_ids') тогда ясно что почем и зачем но в любом случае тогда неясно, почему не сделать self.mapped( короче, или была логика, которую почикали, или рефакторили много и не до конца

ничем не лучше. Кода больше(= саппортить дороже), работает медленнее, а смысл сего усложнения кода - не просматривается,...

Sergej R
если было вот так: payment_ids = self.env['account...

имейте ввиду еще, что когда мы пишем .mapped - мы получаем set, и если надо список повторяющихся значений (для _ids - не актуально), то в общем виде self.mapped('field') != self.field

Anton Goroshkin
имейте ввиду еще, что когда мы пишем .mapped - мы ...

да, это известная шутка. но в конструкции for order in self: payment_ids = self.env['account.payment'] payment_ids |= order.mapped('purchase_payment_ids') вообще как-то выглядит... потеряно, что ли :D

Andro- Автор вопроса
Sergej R
да, это известная шутка. но в конструкции for or...

сама конструкция - дикая. Цикл по селфу, чтобы потом из каждого выбрать purchase_payment_ids как говориться: вот а хєлл из гоуин он?

Andro
screenshot list ведь или о другом речь?

лист для нерекордов, рекордсет для рекородов

Похожие вопросы

Обсуждают сегодня

У меня это всегда вопрос вызывало.. Нафига писать код так, чтобы потом ошибки вылавливать?
Nik
44
Добрый день! Пробую отловить исключение EConvertError: function _TryTextToDouble(var DoubleVar: Extended; Text: String): Boolean; begin try DoubleVar := StrToFloat...
Kirill Filippenok
19
Привет, коллеги! Возникла задача ограничить максимальный размер вложений для определённых расширений, например, чтобы для изображений лимит был 10 МБ, а для видео — 100 МБ. Ог...
Andro
1
Сделал краткий обзор магазина из статьи и заодно там же записал инструкцию по развёртыванию. https://www.youtube.com/watch?v=8-sE1vNk95Q (чё думаете по звуку? кое-что поменял...
Andrey K
17
Кароче инфа для тех кто собирает пакеты и в пакетах есть платформозависимые модули. Т.е. такие которые должны подключаться по определенным условиям. У меня в CPU-View как раз ...
Александр (Rouse_) Багель
5
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
+ ну такое. не для новичков. Если нет консоли на вирт хостинге? + нет отсылки ни в видео ни в описании на процесс создания магазина. Сеошники и юзеры не одобрят :) settings.ph...
Дмитрий
12
Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Всем здравствуйте, Я использую LibUsbDotNet, пытаясь написать библиотека (под linux/Mac OS) для взаимодействия с usb-устройством, который отправляет текст на дисплей покупат...
Асылбек Анарбай
2
Что значит "cs в биосе"?
Крокант
7
Карта сайта