# Infinite loop due to fields.depends adding the same field in a function field

**URL:** https://discuss.tryton.org/t/infinite-loop-due-to-fields-depends-adding-the-same-field-in-a-function-field/2694
**Category:** Developer
**Created:** [May 9, 2020, 10:30am UTC](https://discuss.tryton.org/t/infinite-loop-due-to-fields-depends-adding-the-same-field-in-a-function-field/2694 "2020-05-09T10:30:19Z")
**Posts on this page:** 3
**Page:** 1

<div class="post-metadata">

### Author: ![albert](https://discuss-cdn.tryton.org/user_avatar/discuss.tryton.org/albert/32/21_2.png) [@albert](https://discuss.tryton.org/u/albert)
#### Post date: [May 9, 2020, 10:30am UTC](https://discuss.tryton.org/t/infinite-loop-due-to-fields-depends-adding-the-same-field-in-a-function-field/2694/1 "2020-05-09T10:30:19Z")

</div>

We hit a problem which it seems to me that it could be considered a bug, but I’m not 100% sure.

The problem was with our account\_invoice\_line\_payment module.

We have a “company” field as a fields.Function that uses this on\_change\_with\_company method:

> <https://github.com/NaN-tic/trytond-account_invoice_line_payment/blob/main/payment.py#L378>

As you can see we had the “company” field itself as a “depends” field. It is important to note that in this case we removed this dependency because we really don’t need it and that solved the problem. But having it there caused an infinite loop.

The client did this RPC call:

`ERROR trytond.protocols.dispatcher <class 'trytond.pool.account.invoice.line.payment'>.read(*([30972], ['company', 'difference', 'difference_move', 'company.rec_name', 'difference_move.rec_name', 'rec_name', '_timestamp'], ....`

But the request failed with an infinte loop where the looping part looks like this:

```
Traceback (most recent call last):
File "/trytond/trytond/model/modelstorage.py", line 1557, in __getattr__
read_data = self.read(list(ids), list(ffields.keys()))
File "/trytond/trytond/model/modelsql.py", line 836, in read
getter_results = field.get(ids, cls, field_list, values=result)
File "/trytond/trytond/model/fields/function.py", line 106, in get
return dict((name, call(name)) for name in names)
File "/trytond/trytond/model/fields/function.py", line 106, in <genexpr>
return dict((name, call(name)) for name in names)
File "/trytond/trytond/model/fields/function.py", line 101, in call
return dict(([r.id](http://r.id), method(r, name)) for r in records)
File "/trytond/trytond/model/fields/function.py", line 101, in <genexpr>
return dict(([r.id](http://r.id), method(r, name)) for r in records)
File "/trytond/trytond/model/fields/field.py", line 116, in wrapper
_set_value(self, field)
File "/trytond/trytond/model/fields/field.py", line 91, in _set_value
if not hasattr(record, field):
File "/trytond/trytond/model/fields/field.py", line 334, in __get__
return inst. __getattr__ ([self.name])

```

The problem is that in field.py:116 Tryton tries to set (and thus compute) the value for the ‘company’ field before calling on\_change\_with\_company() because ‘company’ itself is in the depends. So there’s an infinite loop.

As I already stated, there’s no problem for us in this case but there’s a couple of observations I wanted to make:

- I was surprised to see that @fields.depends() is trying to run `_set_value()` here. Unless I miss something, it looks like a waste of CPU cycles when on\_change\_with is being called from the ORM. Of course, it is necessary when it’s being called from RPC. I don’t know if there’s something we could do to prevent it.
- I can imagine some (maybe corner) case in which it would could be desireable to have the field itself in the list of depends fields. For example, if the field was a Function field with setter we may be interested in its value in on\_change\_with.

For example, the following would be a valid scenario that would fail:

```
my_field = fields.Function(fields.Char('My Field'), 'on_change_with_my_field',
        setter='set_my_field')

@fields.depends('other_field', 'my_field')
def on_change_with_my_field(self, name=None):
    if self.my_field:
        return self.my_field
    return self.other_field

```

What do you think? Am I missing something? Is this actually a bug?

---

<div class="post-metadata">

### Author: ![ced](https://discuss-cdn.tryton.org/user_avatar/discuss.tryton.org/ced/32/1237_2.png) [@ced](https://discuss.tryton.org/u/ced)
#### Post date: [May 9, 2020, 11:35am UTC](https://discuss.tryton.org/t/infinite-loop-due-to-fields-depends-adding-the-same-field-in-a-function-field/2694/2 "2020-05-09T11:35:10Z")

</div>

> [@albert](#):
>
> the following would be a valid scenario that would fail

It does not look like a valid scenario. Something can not depend on its own value to define its own value.

---

<div class="post-metadata">

### Author: ![albert](https://discuss-cdn.tryton.org/user_avatar/discuss.tryton.org/albert/32/21_2.png) [@albert](https://discuss.tryton.org/u/albert)
#### Post date: [May 9, 2020, 2:24pm UTC](https://discuss.tryton.org/t/infinite-loop-due-to-fields-depends-adding-the-same-field-in-a-function-field/2694/3 "2020-05-09T14:24:34Z")

</div>

> [@ced](#):
>
> It does not look like a valid scenario. Something can not depend on its own value to define its own value.

You’re right. It makes sense only as an on\_change\_with but not as a getter method.