From fb2149d51f332faaf0b21d72c50f9ad8fa412a94 Mon Sep 17 00:00:00 2001
From: maki <makiniqa@sutty.nl>
Date: Fri, 7 Jun 2024 13:54:28 -0300
Subject: [PATCH] =?UTF-8?q?fix:=20documentaci=C3=B3n,=20valores=20por=20de?=
 =?UTF-8?q?fecto=20y=20ruta=20en=20routes.rb=20#102?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../disk_controller_decorator.rb              | 25 ++++++++++++++-----
 config/routes.rb                              |  3 +++
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/app/controllers/active_storage/disk_controller_decorator.rb b/app/controllers/active_storage/disk_controller_decorator.rb
index 295913bc..36afcd49 100644
--- a/app/controllers/active_storage/disk_controller_decorator.rb
+++ b/app/controllers/active_storage/disk_controller_decorator.rb
@@ -11,7 +11,7 @@ module ActiveStorage
       # Permitir incrustar archivos subidos (especialmente PDFs) desde
       # otros sitios.
       def show
-        original_show.tap do |_s|
+        original_show.tap do |_|
           response.headers.delete 'X-Frame-Options'
         end
       end
@@ -55,10 +55,15 @@ module ActiveStorage
       rescue ActiveStorage::IntegrityError
         head :unprocessable_entity
       end
-
-      def limits(max_upload_size, max_download_size)
+      
+      # Define una ruta que lleva a un JSON con los valores de los tamaños
+      # máximos de subida y descarga (respectivamente) que puede tener un
+      # archivo.
+      #
+      # @return [Integer] tamaño de subida que puede tener un archivo expresado en megabytes.
+      def limits
         ruby_hash = { max_upload_size: max_upload_size, max_download_size: max_download_size }
-        JSON.generate(ruby_hash)
+        render json: ruby_hash
       end
 
       private
@@ -72,12 +77,20 @@ module ActiveStorage
         ExceptionNotifier.notify_exception(exception, data: { params: params.to_hash })
       end
 
+      # Convierte a la variable de entorno que define el máximo tamaño de subida que puede tener un archivo
+      # en un entero expresado en megabytes.
+      #
+      # @return [Integer] tamaño máximo de subida que puede tener un archivo, expresado en megabytes.
       def max_upload_size
-        ENV['MAX_UPLOAD_SIZE'].to_i.megabytes
+        ENV.fetch('MAX_UPLOAD_SIZE', '111').to_i.megabytes
       end
 
+      # Convierte a la variable de entorno que define el máximo tamaño de descarga que puede tener un archivo
+      # en un entero expresado en megabytes.
+      #
+      # @return [Integer] tamaño máximo de descarga que puede tener un archivo, expresado en megabytes.
       def max_download_size
-        ENV['MAX_DOWNLOAD_SIZE'].to_i.megabytes
+        ENV.fetch('MAX_DOWNLOAD_SIZE', '111').to_i.megabytes
       end
     end
   end
diff --git a/config/routes.rb b/config/routes.rb
index 9d5c974a..f4371624 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -111,6 +111,9 @@ Rails.application.routes.draw do
     get 'i18n/edit', to: 'i18n#edit'
     post 'i18n', to: 'i18n#update'
 
+    # Ver límites de tamaño de archivos
+    get 'limits', to: 'limits'
+
     # Compilar el sitio
     post 'enqueue', to: 'sites#enqueue'
     post 'reorder_posts', to: 'sites#reorder_posts'