Version 6.2.0

Author: luigifab

README.md

@@ -6,7 +6,7 @@ For more information, go to https://www.luigifab.fr/apijs and https://www.luigif
 
 This repository is a releases mirror. To install the plugin, please read the documentation.
 
-- Current version: 6.1.0 (05/05/2020) (archive regenerated on 02/07/2021)
+- Current version: 6.2.0 (12/07/2020) (archive regenerated on 02/07/2021)
 - Compatibility: Redmine 1.4 / 2.x / 3.x / 4.0 / 4.1 / 4.2, Python 3.3 / 3.4 / 3.5 / 3.6 / 3.7 / 3.8 / 3.9
 - Client compatibility: Firefox 36+, Chrome 32+, Opera 19+, Edge 16+, Safari 9+
 - Translations: English (en), French (fr-FR/fr-CA), German (de), Italian (it), Portuguese (pt-PT/pt-BR), Spanish (es)

src/app/controllers/apijs_controller.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 # Created J/12/12/2013
-# Updated D/03/05/2020
+# Updated J/02/07/2020
 #
 # Copyright 2008-2020 | Fabrice Creuzot (luigifab) <code~luigifab~fr>
 # https://www.luigifab.fr/redmine/apijs
@@ -41,29 +41,33 @@ def find_project
   end
 
 
-  # #### Gestion de l'image miniature (photo ou vidéo) ################################ public ### #
-  # » Vérifie si l'utilisateur a accès au projet avant l'envoi de la miniature au format JPG ou PNG
+  # #### Gestion de l'image miniature (photo ou vidéo) ################################# #
+  # » Vérifie si l'utilisateur a accès au projet avant l'envoi de la miniature
   # » Utilise un script python pour générer l'image thumb (taille 200x150)
+  # » Utilise un script python pour générer l'image srcset (taille 400x300)
   # » Utilise un script python pour générer l'image show (taille 1200x900)
   # » Enregistre les commandes et leurs résultats dans le log
   # » Téléchargement d'une image avec mise en cache (inline/stale)
   def thumb
 
     source = @attachment.diskfile
-    img_thumb = File.extname(@attachment.filename).downcase == '.png' ? '.png' : '.jpg'
-    img_thumb = File.join(APIJS_ROOT, 'thumb', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + img_thumb)
+    img_thumb = File.join(APIJS_ROOT, 'thumb', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + @attachment.getExt)
 
     # génération de l'image thumb
     if File.file?(source) && !File.file?(img_thumb)
-      cmd = @attachment.getCmd(source, img_thumb, 200, 150)
+      cmd = @attachment.getCmd(source, img_thumb, 200, 150, true)
       logger.info 'APIJS::ApijsController#thumb: ' + cmd + ' (' + `#{cmd}`.gsub(/^\s+|\s+$/, '') + ')'
     end
-    # génération de l'image show
+    # génération des images srcset et show
     if @attachment.isPhoto? && Setting.plugin_redmine_apijs['create_all'] == '1' && File.file?(source)
-      img_show = File.extname(@attachment.filename).downcase == '.png' ? '.png' : '.jpg'
-      img_show = File.join(APIJS_ROOT, 'show', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + img_show)
+      img_srcset = File.join(APIJS_ROOT, 'srcset', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + @attachment.getExt)
+      unless File.file?(img_srcset)
+        cmd = @attachment.getCmd(source, img_srcset, 400, 300, true)
+        logger.info 'APIJS::ApijsController#thumb: ' + cmd + ' (' + `#{cmd}`.gsub(/^\s+|\s+$/, '') + ')'
+      end
+      img_show = File.join(APIJS_ROOT, 'show', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + @attachment.getExt)
       unless File.file?(img_show)
-        cmd = @attachment.getCmd(source, img_show, 1200, 900)
+        cmd = @attachment.getCmd(source, img_show, 1200, 900, false)
         logger.info 'APIJS::ApijsController#thumb: ' + cmd + ' (' + `#{cmd}`.gsub(/^\s+|\s+$/, '') + ')'
       end
     end
@@ -74,25 +78,51 @@ def thumb
     # envoie de l'image avec mise en cache
     elsif File.file?(img_thumb) && stale?(etag: img_thumb)
       send_file(img_thumb, filename: filename_for_content_disposition(@attachment.filename),
-        type: File.extname(img_thumb).downcase == '.png' ? 'image/png' : 'image/jpeg', disposition: 'inline')
+        type: Redmine::MimeType.of(img_thumb), disposition: 'inline')
+    end
+  end
+
+
+  # #### Gestion de l'image miniature 2x (photo ou vidéo) ############################## #
+  # » Vérifie si l'utilisateur a acc��s au projet avant l'envoi de l'aperçu
+  # » Utilise un script python pour générer l'image srcset (taille 400x300)
+  # » Enregistre les commandes et leurs résultats dans le log
+  # » Téléchargement d'une image avec mise en cache (inline/stale)
+  def srcset
+
+    source = @attachment.diskfile
+    img_srcset = File.join(APIJS_ROOT, 'srcset', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + @attachment.getExt)
+
+    # génération de l'image srcset
+    if File.file?(source) && !File.file?(img_srcset)
+      cmd = @attachment.getCmd(source, img_srcset, 400, 300, true)
+      logger.info 'APIJS::ApijsController#srcset: ' + cmd + ' (' + `#{cmd}`.gsub(/^\s+|\s+$/, '') + ')'
+    end
+
+    # vérification d'accès
+    if !User.current.allowed_to?({controller: 'projects', action: 'show'}, @project)
+      deny_access
+    # envoie de l'image avec mise en cache
+    elsif File.file?(img_srcset) && stale?(etag: img_srcset)
+      send_file(img_srcset, filename: filename_for_content_disposition(@attachment.filename),
+        type: Redmine::MimeType.of(img_srcset), disposition: 'inline')
     end
   end
 
 
-  # #### Gestion de l'image aperçu (photo) ############################################ public ### #
-  # » Vérifie si l'utilisateur a accès au projet avant l'envoi de l'aperçu au format JPG ou PNG
+  # #### Gestion de l'image aperçu (photo) ############################################# #
+  # » Vérifie si l'utilisateur a accès au projet avant l'envoi de l'aperçu
   # » Utilise un script python pour générer l'image show (taille 1200x900)
   # » Enregistre les commandes et leurs résultats dans le log
   # » Téléchargement d'une image avec mise en cache (inline/stale)
   def show
 
     source = @attachment.diskfile
-    img_show = File.extname(@attachment.filename).downcase == '.png' ? '.png' : '.jpg'
-    img_show = File.join(APIJS_ROOT, 'show', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + img_show)
+    img_show = File.join(APIJS_ROOT, 'show', @attachment.created_on.strftime('%Y-%m').to_s, @attachment.id.to_s + @attachment.getExt)
 
     # génération de l'image show
     if File.file?(source) && !File.file?(img_show)
-      cmd = @attachment.getCmd(source, img_show, 1200, 900)
+      cmd = @attachment.getCmd(source, img_show, 1200, 900, false)
       logger.info 'APIJS::ApijsController#show: ' + cmd + ' (' + `#{cmd}`.gsub(/^\s+|\s+$/, '') + ')'
     end
 
@@ -102,12 +132,12 @@ def show
     # envoie de l'image avec mise en cache
     elsif File.file?(img_show) && stale?(etag: img_show)
       send_file(img_show, filename: filename_for_content_disposition(@attachment.filename),
-        type: File.extname(img_show).downcase == '.png' ? 'image/png' : 'image/jpeg', disposition: 'inline')
+        type: Redmine::MimeType.of(img_show), disposition: 'inline')
     end
   end
 
 
-  # #### Gestion du téléchargement des fichiers ####################################### public ### #
+  # #### Gestion du téléchargement des fichiers ######################################## #
   # » Vérifie si l'utilisateur a accès au projet avant
   # » Téléchargement d'une vidéo en 206 Partial Content (inline)
   # » Téléchargement d'une image avec mise en cache (inline/stale) ou téléchargement d'un fichier (attachment)
@@ -171,7 +201,7 @@ def download
   end
 
 
-  # #### Modification d'une description ############################################### public ### #
+  # #### Modification d'une description ################################################ #
   # » Vérifie si l'utilisateur a accès au projet et à la modification
   # » Renvoie l'id du fichier suivi de la description en cas de modification réussie
   # » Supprime certains caractères de la description avant son enregistrement
@@ -197,7 +227,7 @@ def editdesc
   end
 
 
-  # #### Suppression d'un fichier ##################################################### public ### #
+  # #### Suppression d'un fichier ###################################################### #
   # » Vérifie si l'utilisateur a accès au projet et à la suppresion
   # » Renvoie l'id du fichier suivi en cas de suppression réussie
   def delete

src/app/views/application/_browser.html.erb

@@ -1,6 +1,6 @@
 <%
 # Created S/17/08/2019
-# Updated V/01/05/2020
+# Updated L/25/05/2020
 #
 # Copyright 2008-2020 | Fabrice Creuzot (luigifab) <code~luigifab~fr>
 # https://www.luigifab.fr/redmine/apijs
@@ -20,15 +20,15 @@ data = (Useragentparser.new).parse
 
 if data['browser'] && data['version']
   data['version'] = Thread.current[:request].query_parameters[:browser] ? 1 : data['version'].to_f
-  if (data['browser'] == 'MSIE') && (data['version'] < 13)
+  if (data['browser'] == 'MSIE') && (data['version'] < 15)
     text = raw l(:apijs_browser_warning, { n: 'Internet Explorer', v: data['version'], l: 'href="https://www.microsoft.com/windows/microsoft-edge"' })
-  elsif (data['browser'] == 'Edge') && (data['version'] < 14)
+  elsif (data['browser'] == 'Edge') && (data['version'] < 16)
     text = raw l(:apijs_browser_warning, { n: 'Edge', v: data['version'], l: 'href="https://www.microsoft.com/windows/microsoft-edge"' })
-  elsif (data['browser'] == 'Firefox') && (data['version'] < 27)
+  elsif (data['browser'] == 'Firefox') && (data['version'] < 36)
     text = raw l(:apijs_browser_warning, { n: 'Firefox', v: data['version'], l: 'href="https://www.mozilla.org/firefox"' })
-  elsif (data['browser'] == 'Chrome') && (data['version'] < 29)
+  elsif (data['browser'] == 'Chrome') && (data['version'] < 31)
     text = raw l(:apijs_browser_warning, { n: 'Chrome', v: data['version'], l: 'href="https://www.google.com/chrome"' })
-  elsif (data['browser'] == 'Opera') && (data['version'] < 17)
+  elsif (data['browser'] == 'Opera') && (data['version'] < 19)
     text = raw l(:apijs_browser_warning, { n: 'Opera', v: data['version'], l: 'href="https://www.opera.com/"' })
   elsif (data['browser'] == 'Safari') && (data['version'] < 9)
     text = raw l(:apijs_browser_warning, { n: 'Safari', v: data['version'], l: 'href="https://www.apple.com/safari/"' })
@@ -38,7 +38,7 @@ end
 <% if text.length > 0 %>
 	<% if pos == "bottom" %>
 		<div class="flash error apijs" id="oldBrowser" style="position:absolute; top:25px; left:33%; width:31%; z-index:9000;"><%= text %></div>
-		<script>var e = document.getElementById('oldBrowser'); document.body.insertBefore(e, document.getElementById('wrapper')); e.removeAttribute('style');</script>
+		<script>var e = document.getElementById('oldBrowser'); document.querySelector('body').insertBefore(e, document.getElementById('wrapper')); e.removeAttribute('style');</script>
 	<% else %>
 		<div class="flash error apijs" id="oldBrowser"><%= text %></div>
 	<% end %>

src/app/views/attachments/_links.html.erb

@@ -1,6 +1,6 @@
 <%
 # Created L/21/05/2012
-# Updated S/02/05/2020
+# Updated D/10/05/2020
 #
 # Copyright 2008-2020 | Fabrice Creuzot (luigifab) <code~luigifab~fr>
 # https://www.luigifab.fr/redmine/apijs
@@ -28,23 +28,23 @@ permission_delete        = User.current.allowed_to?(:delete_attachments, @projec
 permission_edit          = User.current.allowed_to?(:edit_attachments, @project)
 
 @slideshowi = 0 unless defined? @slideshowi
-showAlbum = showList = false
+show_album = show_list = false
 
 if setting_show_album
   attachments.each { |attachment|
     if (attachment.isPhoto? || attachment.isVideo?) && !attachment.isExcluded?
-      showAlbum = true
+      show_album = true
     else
-      showList = true
+      show_list = true
     end
   }
 else
-  showList = true
+  show_list = true
 end
 %>
 
 <% if setting_enabled %>
-	<% if showAlbum %>
+	<% if show_album %>
 		<p class="apijs section"><strong><%= l(:apijs_title_album) %></strong></p>
 		<div class="attachments album" id="slideshow.<%= @slideshowi %>">
 			<% slideshowj = 0 %>
@@ -54,7 +54,7 @@ end
 					<dl id="attachmentId<%= attachment.id %>">
 						<dt>
 							<a href="<%= (attachment.isVideo?) ? attachment.getDownloadUrl + '?stream=1' : attachment.getShowUrl %>" type="<%= attachment.getMimeType %>" onclick="return false" id="slideshow.<%= @slideshowi %>.<%= slideshowj %>">
-								<img src="<%= attachment.getThumbUrl %>" width="200" height="150" alt="<%= h(description) %>"/>
+								<img src="<%= attachment.getThumbUrl %>" srcset="<%= attachment.getSrcsetUrl %> 2x" width="200" height="150" alt="<%= h(description) %>"/>
 								<input type="hidden" value="<%= (setting_show_filename) ? attachment.filename : 'false' %>|<%= (setting_show_exifdate) ? format_time(attachment.created_on) : 'false' %>|<%= h(description) %>"/>
 							</a>
 						</dt>
@@ -85,7 +85,7 @@ end
 			<% @slideshowi += 1 if slideshowj > 0 %>
 		</div>
 	<% end %>
-	<% if showList %>
+	<% if show_list %>
 		<p class="apijs section"><strong><%= l(:apijs_title_files) %></strong></p>
 		<ul class="attachments" id="slideshow.<%= @slideshowi %>">
 			<% slideshowj = 0 %>