entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

misc: autoconvert HEIC files (#88586) #1308

Merged
fpeters merged 1 commits from wip/88586-heic-auto-convert into main 2024-03-29 08:33:10 +01:00
Conversation 0 Commits 1 Files Changed 3 +55 -6
fpeters commented 2024-03-25 08:21:58 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2024-03-25 08:21:59 +01:00
gitea/wcs/pipeline/head There was a failure building this commit Details
e973b4bdbe misc: autoconvert HEIC files (#88586)
fpeters force-pushed wip/88586-heic-auto-convert from e973b4bdbe to 51c7dec65b 2024-03-25 08:29:38 +01:00 Compare
fpeters force-pushed wip/88586-heic-auto-convert from 51c7dec65b to c933433a24 2024-03-25 10:03:40 +01:00 Compare
fpeters force-pushed wip/88586-heic-auto-convert from c933433a24 to 8fd3cdf6b1 2024-03-25 10:09:57 +01:00 Compare
fpeters reviewed 2024-03-25 10:16:49 +01:00
wcs/qommon/form.py
@ -1047,12 +1048,6 @@ class FileWithPreviewWidget(CompositeWidget):
self.value.content_type = filetype
if self.max_file_size and hasattr(self.value, 'file_size'):
fpeters commented 2024-03-25 10:15:36 +01:00
Author
Owner
Copy Link

Je déplace la vérification de la taille après la possible transformation du fichier.

Je déplace la vérification de la taille après la possible transformation du fichier.
wcs/qommon/form.py
@ -1099,2 +1094,4 @@
self.set_error(_('forbidden file type'))
if self.value.content_type in ('image/heic', 'image/heif') and not get_publisher().has_site_option(
'do-no-transform-heic-files'
fpeters commented 2024-03-25 10:16:07 +01:00
Author
Owner
Copy Link

Je ne pense pas que ça serve mais au cas où quelqu'un demanderait à un moment de laisser les HEIC passer le flag est déjà là.

Je ne pense pas que ça serve mais au cas où quelqu'un demanderait à un moment de laisser les HEIC passer le flag est déjà là.
wcs/qommon/form.py
@ -1101,0 +1102,4 @@
# libheic will automatically switch image orientation so we need to remove
# EXIF profile to avoid it being applied a second time.
# (graphicsmagick >= 1.3.41 have heif:ignore-transformations=false to avoid
# that).
fpeters commented 2024-03-25 10:16:44 +01:00
Author
Owner
Copy Link

Il y a 1.3.40 dans bookworm donc on doit faire avec.

Il y a 1.3.40 dans bookworm donc on doit faire avec.
fpeters changed title from WIP: misc: autoconvert HEIC files (#88586) to misc: autoconvert HEIC files (#88586) 2024-03-25 10:16:56 +01:00
tnoel requested changes 2024-03-25 10:26:43 +01:00
Dismissed
wcs/qommon/form.py
@ -1101,0 +1124,4 @@
if self.max_file_size and hasattr(self.value, 'file_size'):
# validate file size
if self.value.file_size > self.max_file_size_bytes:
self.set_error(_('over file size limit (%s)') % self.max_file_size)
tnoel commented 2024-03-25 10:26:40 +01:00
Owner
Copy Link

Ici on pourrait se retrouver avec une conversion qui a fait grossir le fichier, et l'utilisateur averti ne comprendrait pas le pépin. Il s'agirait de dire qu'il y a eu conversion parce que le format HEIC n'est pas pris en charge nativement... Genre,

if convert_from_heic:
  self.set_error(_('over file size limit (%s) (after JPEG conversion , HEIC is not supported)') % self.max_file_size)

Ceci étant je me dis que l'utilisateur non averti ne va rien comprendre à ce message d'erreur...

Je ne valide donc pas, mais si tu penses que ça va juste embrouiller la plupart des personnes, je n'insisterai pas.

Ici on pourrait se retrouver avec une conversion qui a fait grossir le fichier, et l'utilisateur averti ne comprendrait pas le pépin. Il s'agirait de dire qu'il y a eu conversion parce que le format HEIC n'est pas pris en charge nativement... Genre, if convert_from_heic: self.set_error(_('over file size limit (%s) (after JPEG conversion , HEIC is not supported)') % self.max_file_size) Ceci étant je me dis que l'utilisateur non averti ne va rien comprendre à ce message d'erreur... Je ne valide donc pas, mais si tu penses que ça va juste embrouiller la plupart des personnes, je n'insisterai pas.
fpeters commented 2024-03-25 10:29:59 +01:00
Author
Owner
Copy Link

Ceci étant je me dis que l'utilisateur non averti ne va rien comprendre à ce message d'erreur...

Oui je pense que c'est compliqué à comprendre et comme en local la conversion jpeg réduisait la taille je ne m'inquièterais pas.

> Ceci étant je me dis que l'utilisateur non averti ne va rien comprendre à ce message d'erreur... Oui je pense que c'est compliqué à comprendre et comme en local la conversion jpeg réduisait la taille je ne m'inquièterais pas.
tnoel marked this conversation as resolved
fpeters requested review from tnoel 2024-03-25 16:17:19 +01:00
tnoel approved these changes 2024-03-25 16:37:26 +01:00
fpeters merged commit c5c8c0fe9d into main 2024-03-29 08:33:10 +01:00
fpeters deleted branch wip/88586-heic-auto-convert 2024-03-29 08:33:10 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#1308
No description provided.
Delete Branch "wip/88586-heic-auto-convert"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?