Tryton - Issues

Using sao, when tryting to add a Report to an email template I get the following error:

Traceback (most recent call last):
  File "/trytond/wsgi.py", line 109, in dispatch_request
    return endpoint(request, **request.view_args)
  File "/trytond/protocols/dispatcher.py", line 47, in rpc
    return methods.get(request.rpc_method, _dispatch)(
  File "/trytond/wsgi.py", line 77, in auth_required
    return wrapped(*args, **kwargs)
  File "/trytond/protocols/wrappers.py", line 131, in wrapper
    return func(request, pool, *args, **kwargs)
  File "/trytond/protocols/dispatcher.py", line 181, in _dispatch
    result = rpc.result(meth(*c_args, **c_kwargs))
  File "/trytond/model/modelsql.py", line 1246, in search
    tables, expression = cls.search_domain(domain)
  File "/trytond/model/modelsql.py", line 1432, in search_domain
    expression = convert(domain)
  File "/trytond/model/modelsql.py", line 1429, in convert
    return And((convert(d) for d in (
  File "/trytond/model/modelsql.py", line 1429, in <genexpr>
    return And((convert(d) for d in (
  File "/trytond/model/modelsql.py", line 1420, in convert
    expression = field.convert_domain(domain, tables, cls)
  File "/trytond/model/fields/field.py", line 528, in convert_domain
    return super(FieldTranslate, self).convert_domain(
  File "/trytond/model/fields/field.py", line 391, in convert_domain
    assert name == self.name
AssertionError

Ready for testing.

On 2020-07-01 23:35, Sergi Almacellas Abellana wrote:
> - Do you plan to allow the user to see all the sent emails for a record? As we have them stored on the server I think it makes sense to allow other users to see all the emails sent for a record (probably with a similar design like attachments and notes).

Yes that's why I created a general relate on all models.

> - Why not sotoring also emails sent by the notification_email module? This will make sense for me to allow the user to see that some notification has sent for the record. Probably we can merge the notification.email.log with the ir.email models and store everything together.

This is something that should be discussed and balanced. But for me, it
is out of the scope of this feature as it would require migration for
notification_email module and this feature is already big enough.

I have two doubts here: 

- Do you plan to allow the user to see all the sent emails for a record? As we have them stored on the server I think it makes sense to allow other users to see all the emails sent for a record (probably with a similar design like attachments and notes).

- Why not sotoring also emails sent by the notification_email module? This will make sense for me to allow the user to see that some notification has sent for the record. Probably we can merge the notification.email.log with the ir.email models and store everything together.

Following https://discuss.tryton.org/t/set-up-emails-and-easy-email-delivery/2116/5