backoffice: expand form_attachments_* in inspect (#16507) #684

Merged
fpeters merged 1 commits from wip/16507-form-attachments-in-inspect into main 2023-09-18 10:49:48 +02:00
Owner
No description provided.
fpeters force-pushed wip/16507-form-attachments-in-inspect from 94120666a5 to b5f8f18ca1 2023-09-17 08:43:22 +02:00 Compare
fpeters force-pushed wip/16507-form-attachments-in-inspect from b5f8f18ca1 to 3e698fe7e2 2023-09-17 09:32:39 +02:00 Compare
fpeters changed title from WIP: backoffice: expand form_attachments_* in inspect (#16507) to backoffice: expand form_attachments_* in inspect (#16507) 2023-09-17 09:50:27 +02:00
fpeters reviewed 2023-09-17 09:56:50 +02:00
@ -651,3 +651,3 @@
assert resp.request.environ['PATH_INFO'].endswith(file1.filename)
file2 = attachments.created_doc[1]
file2 = attachments.created_doc[0]
Author
Owner

Il y avait un reverse() qui est retiré, pour avoir les fichiers attachés indexés chronologiquement, il y a juste ce petit bout de test qui demande adaptation.

Il y avait un reverse() qui est retiré, pour avoir les fichiers attachés indexés chronologiquement, il y a juste ce petit bout de test qui demande adaptation.
@ -3886,3 +3886,3 @@
'display_name': v['form_display_name'],
}
elif hasattr(v, 'inspect_keys'):
elif hasattr(v, 'inspect_keys') and not getattr(v, 'include_in_inspect', False):
Author
Owner

Introduction de cet attribut "include_in_inspect" sans quoi le form_attachments_xxx n'apparait pas (parce qu'il a une méthode inspect_keys) et que j'ai trouvé que c'était quand même utile d'exposer la variable pour la donnée complexe "fichier".

Introduction de cet attribut "include_in_inspect" sans quoi le form_attachments_xxx n'apparait pas (parce qu'il a une méthode inspect_keys) et que j'ai trouvé que c'était quand même utile d'exposer la variable pour la donnée complexe "fichier".
@ -1219,2 +1221,4 @@
str: _('string'),
PicklableUpload: _('file'),
AttachmentSubstitutionProxy: _('file'),
NamedAttachmentsSubstitutionProxy: _('file'),
Author
Owner

Pour afficher le type dans l'inspecteur, sans distinguer.

Pour afficher le type dans l'inspecteur, sans distinguer.
@ -219,0 +228,4 @@
try:
return super().__getattr__(name)
except AttributeError:
return getattr(self[-1], name)
Author
Owner

On passe du self[0] qui visait le premier de la liste passée par .reverse() à self[-1] maintenant que la liste n'est plus inversée.

On passe du `self[0]` qui visait le premier de la liste passée par .reverse() à `self[-1]` maintenant que la liste n'est plus inversée.
@ -219,0 +234,4 @@
yield from self[-1].inspect_keys()
if len(self.parts) > 1:
# only advertise indexed keys if there are multiple elements
yield from [str(x) for x in range(len(self.parts))]
Author
Owner

Pour afficher uniquement form_attachments_xxx quand il y a un seul fichier attaché pour ce nom de variable, et form_attachments_xxx_0_... form_attachments_xxx_1_... etc. en plus quand il y a plusieurs fichiers attachés.

Pour afficher uniquement form_attachments_xxx quand il y a un seul fichier attaché pour ce nom de variable, et form_attachments_xxx_0_... form_attachments_xxx_1_... etc. en plus quand il y a plusieurs fichiers attachés.
@ -219,3 +238,3 @@
def __getitem__(self, i):
return AttachmentSubstitutionProxy(self.formdata, self.parts[i])
if isinstance(i, int) or (isinstance(i, str) and misc.is_ascii_digit(i)):
Author
Owner

Les clés retournées par inspect_keys doivent être des chaines pour être interprétées correctement par CompatibilityNamesDict, ici donc on gère soit les entiers soit les chaines avec un entier (positif).

Les clés retournées par inspect_keys doivent être des chaines pour être interprétées correctement par CompatibilityNamesDict, ici donc on gère soit les entiers soit les chaines avec un entier (positif).
tnoel approved these changes 2023-09-18 09:57:24 +02:00
fpeters merged commit bc13337541 into main 2023-09-18 10:49:48 +02:00
fpeters deleted branch wip/16507-form-attachments-in-inspect 2023-09-18 10:49:49 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#684
No description provided.