Project

General

Profile

Bug #70194

en backoffice, lien vers le fichier déposé, via un formulaire workflow

Added by Frédéric Péters 4 months ago. Updated 4 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
12 October 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

L'adresse va être https://.../backoffice/management/test-basique/51/tempfile?t=7HG9zmOxciA:mFQbV9angu0Jkdo7xIjksCg7b3w et ça va être une 404. Il faudrait que soit elle fonctionne soit que ça lie vers https://.../test-basique/51/tempfile?t=7HG9zmOxciA:mFQbV9angu0Jkdo7xIjksCg7b3w

peut-être juste

--- a/wcs/backoffice/management.py
+++ b/wcs/backoffice/management.py
@@ -765,6 +765,7 @@ class FormPage(Directory):
         'map',
         'geojson',
         'actions',
+        'tempfile',
         ('export-spreadsheet', 'export_spreadsheet'),
         ('filter-options', 'filter_options'),
         ('save-view', 'save_view'),

Files

Associated revisions

Revision 36110d80 (diff)
Added by Frédéric Péters 4 months ago

backoffice: fix access to uploaded file in file widget (#70194)

Revision c3aedc55 (diff)
Added by Frédéric Péters 2 months ago

backoffice: allow backoffice submission with that single permission (#71694)

Method name got changed in 36110d80823da3312b334db5986278191bc601fb (#70194)
and the change was not made in those files.

History

#2

Updated by Frédéric Péters 4 months ago

peut-être juste

fail, il n'y a pas de classe de base entre /<slug formdef>/ et /backoffice/management/<slug formdef/>/, le patch crée ça (sous forme d'un FormPageMixin, avec juste la méthode tempfile qui nous intéresse). Pour que cette méthode fonctionne il faut aussi mini-harmonisation sur la méthode qui vérifie le droit d'accès, c'est le changement s/check_role/check_access/ du patch.

#3

Updated by Frédéric Péters 4 months ago

J'ai repris sous forme d'une classe de base plutôt qu'une classe Mixin random (qui peut donner l'impression de s'appliquer partout alors que non c'est très spécifiquement pour deux endroits).

#4

Updated by Lauréline Guérin 4 months ago

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

Updated by Frédéric Péters 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 36110d80823da3312b334db5986278191bc601fb
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Oct 12 21:06:16 2022 +0200

    backoffice: fix access to uploaded file in file widget (#70194)
#6

Updated by Transition automatique 4 months ago

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

Updated by Transition automatique about 2 months ago

Automatic expiration

Also available in: Atom PDF