From 47fe6ccf6c95af64e2adbc06f18abc9772a30019 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Wed, 22 May 2024 13:57:47 +0200
Subject: [PATCH] fix(gallery): keep variants but rollback pdf previews

---
 app/assets/images/pdf-placeholder.png            | Bin 0 -> 3581 bytes
 .../dossiers/pieces_jointes.html.haml            |   4 ++--
 .../champs/piece_justificative/_show.html.haml   |   4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)
 create mode 100644 app/assets/images/pdf-placeholder.png

diff --git a/app/assets/images/pdf-placeholder.png b/app/assets/images/pdf-placeholder.png
new file mode 100644
index 0000000000000000000000000000000000000000..58da75f9f728e70eb30af89f205a263ddc1aa1bb
GIT binary patch
literal 3581
zcmeH~S5%YP7RQqqAv6K02Eh<i6j6$_p#~<Pj4(7QBa<K&N+L}{5r_n37Qq-Ba}kgL
zAr8H%^Z+V|8KslZLPxJaC;=gq(B@;^$Gh&^eVB)NIO|*c>~EcOzWwd}|Lv3P{PP7V
zakw}L1d_79h;{{mz)$uEOaz!Ay)|kF9%7*ve+dVH6i@6Au)XW?C14U9?s~xnRMvTP
z7BHax){fR7(5uH1+r9^Zm7dw7tuIG|7e>!It4@!zCKt}TC+DRO=EzYG8OgeE6kXi?
z9@yj{?*&2w464v_FT7k^B?GR#UrQgK&5VC$lv&}^IlPD+XixVeeRckDx7KHE>{w@4
z*IqCv=Z`%+2!TL=&`>CeOajTs%7Bm%2nvk?kzp{9EEtSH{^R@`2I2el1L3!c#L-`|
zSR+qQ&yFXhrKN^v&YY1V&`7#UEEcO{xF+O+?2-B;>0?GrCbNUb!yUl*iBr<eOiYvy
zqRn?*tdbfV8+!sI>m5*Og={vvhf<ylBO(y(oKK(ZTWXSF7)?a`@02e$kT%c7VA^Tq
zZg&Xe5(+(o1SgzOM-r%{cZX$V>(QvIpU5!Ga?9rC=D1Y*pKZ@|_^S~S4J@HBhHV@n
zck{FAoVMA8^a^?2tyADlZuD^^Qu~aXSH%Yh|5xo$Xlf?8q67*xMWMayAP`k`q_zq;
z;gGCsYBCAMBE#BjP&xO}i8m35ZWm;8t{9AvMy|LByr_xPen3J=lVKFVC5LQo5QD9f
z$+}Juh=m5SodUS18jv@~5#4SIvS}xPM4n1cl;mQL)p<Kz8sTzrq^^gki9=;&W$+2S
z9YV6dqoafN)(GR2)*3!$b$9zzfc8PKZv_`U$<B9jyr#&;ii(PM%+3ZLpa4;Pk6ZDM
zYyDVuawCJJ>lKt&@!2PSrw^1Z2D{%EsY>rxBXvO&-{VxCP$(3U4F{BRNwsywmRQqC
zgrvvq!JJX2rF4?+#VS(Pd{NBSTF`;*A5qFL2^Ngp>w2oY>IlhRA~G~CZUh8rMvZDB
z1W{BQ6iT^ZOC;x!TmoHAM#LY^NaA+lwJ7_>bW4(ShXLo2_WEzSHz3UunKYUq+u7{3
z!%aQF%;AvL>k^iX3{@iu1kO7d;E4K^Q8!IZ&5o^gzKSv&?Cb0oA^&M|k}i-DxqwoB
zSpTkzw@bYuJv2NlCo5tL%os5q_x1Jp1qGD`GN#m#lF1}p<siuGrY6gntW99|7e?qr
zby*n^`obIYEK4G{q@?5_vNx5iD-UT#R30I6IPR?s^E=;P4h;?laEqen=jR7$w&h~5
zMo*=cYPP2jcVy&Nqnazumlz*E{#0`{mDC*@q%&d2+o)MsSlFZf^lW`kFyP}D*kcB<
zDX$u>_hvi}Ih$z}y_7RFI!e{h&+65O)19D+SUfY@GO&>`6_!or{N5jJFg3(mO(1pS
zgYJGQkG0|%ej;kPY6V2Vt}k%DU)2I~f2Q;StJ3b6Plg@(r&b11U+t-v1|1Kqgk!E0
zU0dRhdZvtwj8qDR_8&E#i6_g7VSw7R;H()7m2|Kp!n!NW0*#3|>&r??N(`$3mfvKo
zTR?>aG;u6&t9B@aYN;i|A8S+@g_*QnjVqk$S(}mfIu52safAP*`q;AR;YVcNGc}7C
z4(d4!*68Zy8Psq2@$Irs%cBC*cnP4mW~j?V#h|^;*X9EwLQ%|hCB??&>FoH($TPP}
zTSlV1b=zN%x+M&EE_a>eYcZtjFQR4|mWlEhzn%N%4$GMGXJ!gkqQZs~Y?XdN=d*D=
zaYxj+mzi}qIIbbprCjJmo7&=6QN!kW!E3_sMhD*>TZoOnyNP0$xzI#W@T3s_gZbU<
z+CC|gV%w|)n4ZF0Txw})@xy8<`ZR&S^!vEa(}KBy^H$iGHZGK5L81*qWJ3o`chgdf
z7}zLL?7(aBhTixyLcN}oK0y;Z({7sMDqZ}qrDxq^%E_40gXKrY$Fqf%?zCoaok`Lf
zH1X$xQ4jV5R&+)Y_K{OkCaGJ@+uOUNv$L}ii9D){Sw!c{;gmQbZ+H4%;xc%Tt#*}5
z!^xr!NwP4S%bdK5ic0Ar$C9@zD_vH@E(gye5QNX)zWIemMjF*W4SfcG#i@MW8kVN&
z=0)gSTU#q-I}4UG@?uuLQKR2I6ob*+7?kO*?7itP3J*QYu{-=D)9-GjH|M6{&VB`S
zfFXDSkJ$d+rju0&4nK(?cu);^wS?i@zIwo?6-B3&-D`a%^1tXAb_{ljuI3gS6T@91
zzKKyYuRMX!{w?N4P$Owk_qT)p#oWa7?_NW#%WwSqp^52Ne0)N@B&}lo909SeC)Yjc
z@Ui$l&_(-YYjS&gn=j%wzp)X$daat*(<C7KIN@%nK@;ohnwo;o>FB7o>S+LI01!|#
z-E!U-!21XI|B>yRuM&O$pb{Vu>~FHK27p%d6z`hPtsgC3k6D^{KSPWc&gS>T=g$J!
zxPPcTdQ%ORN)`ELi%Mex==o;Z-`}4h5D0qkSpE+ldMa<@ch;N6-o}sriopO{HvX6d
z2)oU6-1bJ;s;Q-AaAZ`}1tr0qLJG0904r$H!8Qaa=+DUEfnS1YkBW@UXlrZpljZH;
zaq~cSx~bUimlw@rPBb<)R-%y<zp>=IYN4%lqv8~>syn*Z_IqF<X=itf@8*q9I9DOk
z+*@Ef0U&iRZu{G}yt>+2<<dJNM!m}AluFEC8OFmY%e+poA{bLB5Z3T>Bi$`xRs*z{
zRbvfz*zfe^WKur|NlZmpmzi)fSV6;ByjEcl3P)M2$rXeMvIKmm2ZI}PILEJj#hyKp
zUvxUc!$HS^$~zKaO<bEFD$TYsGsFA+am|@oio5LmK3zU(dCGK^M(RGDQr+H9U^g@+
z!F?Uz>YRXom$9+?R8Dxn0+RW|(~FK#)&PTO`sFSkF`WALm#3@X2@Z2<hXJ(O<Y#7P
z0(xbuZ934So?@q7E$H<JYMm3ZnsV;XDq`dqPHAA>PQBZ+dik|MFmpUN(y%X=plWXN
z>7~S`wj<XxE}O~rX^Oj(Utaw5d90lm6zm(wX{~+)wdZse*p3?O1%CbVP+Qn8uShet
zREToH!SSvQ=)@*KhDy(ktXTG{ZX})7HZ9I8H8M8l6L0)!-~hkLfgKbAn)N$8(3~hF
z5I|D~Suhi+6vfYHf&(WOg?s+1t+_DRrR%-t8(CdwpO2KAH2Js}TbZA|-NGx{p^1O!
zSyoiwQy0eM)08MPq0(QbpN#nRzOl=hgF<b_V-H_*tU23eD%F0Egqm4e3Y}UkwjFx`
zS-C0&y;Ag)wY$6fp|BWEB!pRra%E&{8R9Hv6yL6Eb7M?zAtGKJh`Mg{xS3u}ULg<9
zL$R3V<BYsA9#&z-4D8#jb!Zm+Xxx?U(Qm0N4o)~-_yXaG(K`--tY0=pS;p^f_cLec
z<xuD%eQ{-w%e}3pa3APLdN+P8yD}HbK>T<Tv7~BPU*Ji@yT6a#-dJ`$sxD1g`%fbU
ms{DFN?w?-#?|C8FlUCF7{EF68^9F8fAbZ=N(PcJQ@BIaB4wH)j

literal 0
HcmV?d00001

diff --git a/app/views/instructeurs/dossiers/pieces_jointes.html.haml b/app/views/instructeurs/dossiers/pieces_jointes.html.haml
index 1d08707a3..58a867ccd 100644
--- a/app/views/instructeurs/dossiers/pieces_jointes.html.haml
+++ b/app/views/instructeurs/dossiers/pieces_jointes.html.haml
@@ -11,7 +11,7 @@
           - if blob.content_type.in?(AUTHORIZED_PDF_TYPES)
             = link_to blob.url, id: blob.id, data: { iframe: true, src: blob.url }, class: 'gallery-link', type: blob.content_type, title: "#{champ.libelle} -- #{blob.filename}" do
               .thumbnail
-                = image_tag(attachment.representation(resize_to_limit: [400, 400]).processed.url, loading: :lazy)
+                = image_tag("pdf-placeholder.png")
                 .fr-btn.fr-btn--tertiary.fr-btn--icon-left.fr-icon-eye{ role: :button }
                   Visualiser
             .champ-libelle
@@ -21,7 +21,7 @@
           - elsif blob.content_type.in?(AUTHORIZED_IMAGE_TYPES)
             = link_to image_url(blob.url), title: "#{champ.libelle} -- #{blob.filename}", data: { src: blob.url }, class: 'gallery-link' do
               .thumbnail
-                = image_tag(attachment.representation(resize_to_limit: [400, 400]).processed.url, loading: :lazy)
+                = image_tag(attachment.variant(resize_to_limit: [400, 400]).processed.url, loading: :lazy)
                 .fr-btn.fr-btn--tertiary.fr-btn--icon-left.fr-icon-eye{ role: :button }
                   Visualiser
             .champ-libelle
diff --git a/app/views/shared/champs/piece_justificative/_show.html.haml b/app/views/shared/champs/piece_justificative/_show.html.haml
index 670e9e011..734aeadd8 100644
--- a/app/views/shared/champs/piece_justificative/_show.html.haml
+++ b/app/views/shared/champs/piece_justificative/_show.html.haml
@@ -11,14 +11,14 @@
           - if blob.content_type.in?(AUTHORIZED_PDF_TYPES)
             = link_to blob.url, id: blob.id, data: { iframe: true, src: blob.url }, class: 'gallery-link', type: blob.content_type, title: "#{champ.libelle} -- #{blob.filename}" do
               .thumbnail
-                = image_tag(attachment.representation(resize_to_limit: [400, 400]).processed.url, loading: :lazy)
+                = image_tag("pdf-placeholder.png")
                 .fr-btn.fr-btn--tertiary.fr-btn--icon-left.fr-icon-eye{ role: :button }
                   = 'Visualiser'
 
           - elsif blob.content_type.in?(AUTHORIZED_IMAGE_TYPES)
             = link_to image_url(blob.url), title: "#{champ.libelle} -- #{blob.filename}", data: { src: blob.url }, class: 'gallery-link' do
               .thumbnail
-                = image_tag(attachment.representation(resize_to_limit: [400, 400]).processed.url, loading: :lazy)
+                = image_tag(attachment.variant(resize_to_limit: [400, 400]).processed.url, loading: :lazy)
                 .fr-btn.fr-btn--tertiary.fr-btn--icon-left.fr-icon-eye{ role: :button }
                   = 'Visualiser'
           - else