Project

General

Profile

Bug #81310

.store() sur un formdef en lecture seule

Added by Sentry Io about 1 year ago. Updated about 1 year ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
18 September 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/113439/

AssertionError: 
(...)
  File "wcs/admin/forms.py", line 681, in _q_index
    'include_management_link': bool(user.is_admin or self.formdef.is_of_concern_for_user(user)),
  File "wcs/formdef.py", line 1712, in is_of_concern_for_user
    data_class = self.data_class()
  File "wcs/formdef.py", line 294, in data_class
    table_name = sql.get_formdef_table_name(self)
  File "wcs/sql.py", line 290, in get_formdef_table_name
    formdef.store(object_only=True)
  File "wcs/formdef.py", line 399, in store
    assert not self.is_readonly()

Très curieux d'avoir table_name absent lors d'une visite d'un snapshot https://.../backoffice/forms/206/history/10111/view/

mais on gagne à juste ignorer.

Associated revisions

Revision a4a3f9cb (diff)
Added by Frédéric Péters about 1 year ago

sql: do not store table_name to read-only formdef (#81310)

History

#1

Updated by Frédéric Péters about 1 year ago

  • Project changed from Suivi des traces to w.c.s.
  • Assignee set to Frédéric Péters
#2

Updated by Robot Gitea about 1 year ago

  • Status changed from Nouveau to En cours

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#3

Updated by Robot Gitea about 1 year ago

  • Status changed from En cours to Solution proposée
#4

Updated by Robot Gitea about 1 year ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea about 1 year ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#6

Updated by Transition automatique about 1 year ago

  • Status changed from Résolu (à déployer) to Solution déployée
#7

Updated by Transition automatique 11 months ago

Automatic expiration

Also available in: Atom PDF