trusty-cms 4.3.5 → 5.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93c5757a5dcb67c36520c6ddb0cd237065fb7030fc7f2cd2f2e07f63a78224c4
-  data.tar.gz: c44066af31851544cba8ae7ed6d055b59d10a1d84cc3836f1168ebd817853dfb
+  metadata.gz: f0fc7adbc55517598d8eb365fe2518df17faab505fece52ec5dcb3374459b45c
+  data.tar.gz: 9de9c0c248cbe41a2c4b55592c9a3412f5cc50279e680de125c7ced575c0a0ac
 SHA512:
-  metadata.gz: e37967a7f2db311cf7144e90da17c5c3f0ad4e5ab8bd3eefabbeb3cc86204bbe04975b1e9e6de09daf8f4e0eeb13640982f5205440d6874488f25f365ab97c77
-  data.tar.gz: cf4e4e7fd6d0a475cd4f704ee607abef8d83b935bdf161b3d0e61c9e66112d40c14d58a92395021b7867ad206061a27497482bb9c0c3cebc4897b5e64d1559ef
+  metadata.gz: 737f21b157db73ad8763d0bcb2576e8f6ff9ddff69d3be98713aae5903c71432e48db36df8f16b4def7e226f18d2f8fe4d4905ef80623f5638a4b245f2ebbe26
+  data.tar.gz: f472cff774a907272ac9a4a60a9a73ac9ef081fe4cd29411abf55238e1ef62640384f00a5be9c7f7fe68e968b05c9fa680438614378501e0cc7e7eebb50e4581

data/Gemfile

@@ -12,10 +12,11 @@ gem 'trustygems', '~> 0.2.0'
 gemspec
 
 group :development, :test do
+  gem 'activestorage-validator'
   gem 'acts_as_list'
   gem 'database_cleaner'
   gem 'factory_bot_rails', '6.2.0'
-  gem 'kt-paperclip', '~> 7.0.0'
+  gem 'file_validators'
   gem 'launchy', '~> 2.5.0'
   gem 'mysql2'
   gem 'poltergeist', '~> 1.18.1'

data/Gemfile.lock

@@ -1,8 +1,9 @@
 PATH
   remote: .
   specs:
-    trusty-cms (4.3.5)
+    trusty-cms (5.0.0)
       RedCloth (= 4.3.2)
+      activestorage-validator
       acts_as_list (>= 0.9.5, < 1.1.0)
       acts_as_tree (~> 2.9.1)
       ckeditor (>= 4.2.2, < 4.4.0)
@@ -12,6 +13,7 @@ PATH
       haml
       haml-rails (~> 2.0)
       highline (>= 1.7.8, < 2.1.0)
+      image_processing
       kraken-io
       mini_racer
       mysql2
@@ -86,6 +88,8 @@ GEM
       activesupport (= 6.1.4.1)
       marcel (~> 1.0.0)
       mini_mime (>= 1.1.0)
+    activestorage-validator (0.1.4)
+      rails (>= 5.2.0)
     activesupport (6.1.4.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
@@ -144,6 +148,9 @@ GEM
       factory_bot (~> 6.2.0)
       railties (>= 5.0.0)
     ffi (1.15.4)
+    file_validators (3.0.0)
+      activemodel (>= 3.2)
+      mime-types (>= 1.0)
     globalid (0.5.2)
       activesupport (>= 5.0)
     haml (5.2.2)
@@ -166,18 +173,15 @@ GEM
       multi_xml (>= 0.5.2)
     i18n (1.8.10)
       concurrent-ruby (~> 1.0)
+    image_processing (1.12.1)
+      mini_magick (>= 4.9.5, < 5)
+      ruby-vips (>= 2.0.17, < 3)
     json (2.5.1)
     kraken-io (0.1.3)
       activesupport
       httparty
       json
       multipart-post
-    kt-paperclip (7.0.0)
-      activemodel (>= 4.2.0)
-      activesupport (>= 4.2.0)
-      marcel (~> 1.0.1)
-      mime-types
-      terrapin (~> 0.6.0)
     launchy (2.5.0)
       addressable (~> 2.7)
     libv8-node (15.14.0.1-x86_64-darwin-18)
@@ -188,11 +192,12 @@ GEM
       nokogiri (>= 1.5.9)
     mail (2.7.1)
       mini_mime (>= 0.1.1)
-    marcel (1.0.1)
+    marcel (1.0.2)
     method_source (1.0.0)
     mime-types (3.3.1)
       mime-types-data (~> 3.2015)
     mime-types-data (3.2021.0225)
+    mini_magick (4.11.0)
     mini_mime (1.1.1)
     mini_racer (0.4.0)
       libv8-node (~> 15.14.0.0)
@@ -201,9 +206,9 @@ GEM
     multipart-post (2.1.1)
     mysql2 (0.5.3)
     nio4r (2.5.8)
-    nokogiri (1.12.5-x86_64-darwin)
+    nokogiri (1.12.4-x86_64-darwin)
       racc (~> 1.4)
-    nokogiri (1.12.5-x86_64-linux)
+    nokogiri (1.12.4-x86_64-linux)
       racc (~> 1.4)
     orm_adapter (0.5.0)
     poltergeist (1.18.1)
@@ -281,6 +286,8 @@ GEM
       rspec-mocks (~> 3.10)
       rspec-support (~> 3.10)
     rspec-support (3.10.2)
+    ruby-vips (2.1.3)
+      ffi (~> 1.12)
     ruby_parser (3.17.0)
       sexp_processor (~> 4.15, >= 4.15.1)
     sass-rails (6.0.0)
@@ -336,10 +343,11 @@ PLATFORMS
   x86_64-linux
 
 DEPENDENCIES
+  activestorage-validator
   acts_as_list
   database_cleaner
   factory_bot_rails (= 6.2.0)
-  kt-paperclip (~> 7.0.0)
+  file_validators
   launchy (~> 2.5.0)
   mysql2
   poltergeist (~> 1.18.1)

data/app/assets/stylesheets/admin/partials/_content.scss

@@ -6,6 +6,7 @@ main {
 
 .reversed main {
   .content {
+    line-height: 1.5;
     padding: 1em;
   }
 }

data/app/controllers/admin/assets_controller.rb

@@ -1,6 +1,9 @@
 class Admin::AssetsController < Admin::ResourceController
   paginate_models(per_page: 50)
   COMPRESS_FILE_TYPE = ['image/jpeg', 'image/png', 'image/gif', 'image/svg+xml'].freeze
+  before_action do
+    ActiveStorage::Current.host = request.base_url
+  end
 
   def index
     assets = Asset.order('created_at DESC')
@@ -67,6 +70,7 @@ class Admin::AssetsController < Admin::ResourceController
   private
 
   def compress(uploaded_asset)
+    require 'open-uri'
     data = $kraken.upload(uploaded_asset.tempfile.path, 'lossy' => true)
     File.write(uploaded_asset.tempfile, URI.open(data.kraked_url).read, mode: 'wb')
     uploaded_asset
@@ -81,4 +85,4 @@ class Admin::AssetsController < Admin::ResourceController
   def asset_params
     params.permit(:id, :for_attachment, asset: [:caption, :for_attachment, asset: []])
   end
-end
+end

data/app/models/asset.rb

@@ -1,6 +1,4 @@
 class Asset < ActiveRecord::Base
-  include Paperclip::Glue
-
   has_many :page_attachments, dependent: :destroy
   has_many :pages, through: :page_attachments
   has_site if respond_to? :has_site
@@ -33,52 +31,43 @@ class Asset < ActiveRecord::Base
     end
   }
 
-  has_attached_file :asset,
-                    styles: lambda { |attachment|
-                      AssetType.for(attachment).paperclip_styles
-                    },
-                    processors: lambda { |asset|
-                      asset.paperclip_processors
-                    },
-                    whiny: false,
-                    storage: TrustyCms.config['paperclip.storage'],
-                    path: TrustyCms.config['paperclip.path'],
-                    url: TrustyCms.config['paperclip.url'],
-                    fog_credentials: TrustyCmsClippedExtension::Cloud.credentials,
-                    fog_directory: TrustyCms.config['paperclip.fog.directory'],
-                    fog_public: TrustyCms.config['paperclip.fog.public?'] || true,
-                    fog_host: TrustyCmsClippedExtension::Cloud.host,
-                    fog_file: {
-                      'Cache-Control' => 'max-age=31536000',
-                    }
-
-  validates_attachment_content_type :asset, content_type: ['application/zip', 'image/jpg', 'image/jpeg', 'image/png', 'image/gif', 'application/pdf', 'application/javascript', 'text/javascript', 'text/css']
-
+  has_one_attached :asset
+  validates :asset,
+            presence: true,
+            blob:
+              {
+                content_type: %w[application/zip image/jpg image/jpeg image/png image/gif application/pdf text/css],
+                size_range: 1..5.megabytes,
+              }
   before_save :assign_title
   before_save :assign_uuid
 
-  after_post_process :read_dimensions
-
-  validates_attachment_presence :asset, message: 'You must choose a file to upload!'
-  if TrustyCms.config['paperclip.skip_filetype_validation'] != 'true' && TrustyCms.config['paperclip.content_types']
-    validates_attachment_content_type :asset, content_type: TrustyCms.config['paperclip.content_types'].gsub(' ', '').split(',')
-  else
-    validates_attachment_presence :asset, message: 'Your uploaded file must have an extension in its name!'
-  end
-  validates_attachment_size :asset, less_than: (TrustyCms.config['assets.max_asset_size'] || 5).to_i.megabytes
-
   def asset_type
     AssetType.for(asset)
   end
-  delegate :paperclip_processors, :paperclip_styles, :style_dimensions, :style_format, to: :asset_type
+  delegate :paperclip_processors, :paperclip_styles, :active_storage_styles, :style_dimensions, :style_format,
+           to: :asset_type
 
   def thumbnail(style_name = 'original')
-    return asset.url if style_name.to_sym == :original
-    return asset.url(style_name.to_sym) if style?(style_name)
+    return asset.url if style_name == 'original'
+    return asset_variant(style_name).processed.url if asset.variable?
 
     asset_type.icon(style_name)
   end
 
+  def asset_variant(style_name)
+    case style_name
+    when 'thumbnail'
+      asset.variant(gravity: 'Center', resize: '100x100^', crop: '100x100+0+0')
+    when 'normal'
+      asset.variant(gravity: 'Center', resize: '640x640^')
+    when 'small'
+      asset.variant(gravity: 'Center', resize: '320x320^')
+    when 'icon'
+      asset.variant(gravity: 'Center', resize: '50x50^')
+    end
+  end
+
   def style?(style_name = 'original')
     style_name == 'original' || paperclip_styles.keys.include?(style_name.to_sym)
   end
@@ -110,7 +99,10 @@ class Asset < ActiveRecord::Base
   end
 
   def geometry(style_name = 'original')
-    raise Paperclip::StyleError, "Requested style #{style_name} is not defined for this asset." unless style?(style_name)
+    unless style?(style_name)
+      raise Paperclip::StyleError,
+            "Requested style #{style_name} is not defined for this asset."
+    end
 
     @geometry ||= {}
     begin
@@ -118,8 +110,9 @@ class Asset < ActiveRecord::Base
                                   original_geometry
                                 else
                                   style = asset.styles[style_name.to_sym]
-                                  original_geometry.transformed_by(style.geometry) # this can return dimensions for fully specified style sizes but not for relative sizes when there are no original dimensions
-      end
+                                  original_geometry.transformed_by(style.geometry)
+                                  # this can return dimensions for fully specified style sizes but not for relative sizes when there are no original dimensions
+                                end
     rescue Paperclip::TransformationError => e
       Rails.logger.warn "geometry transformation error: #{e}"
       original_geometry # returns a blank geometry if the real geometry cannot be calculated
@@ -174,19 +167,17 @@ class Asset < ActiveRecord::Base
   # original file and calculate thumbnail dimensions later, on demand.
 
   def read_dimensions
-    if image?
-      if file = asset.queued_for_write[:original]
-        geometry = Paperclip::Geometry.from_file(file)
-        self.original_width = geometry.width
-        self.original_height = geometry.height
-        self.original_extension = File.extname(file.path)
-      end
+    if image? && file = asset.queued_for_write[:original]
+      geometry = Paperclip::Geometry.from_file(file)
+      self.original_width = geometry.width
+      self.original_height = geometry.height
+      self.original_extension = File.extname(file.path)
     end
     true
   end
 
   def assign_title
-    self.title = asset_file_name.downcase.sub(original_extension, '').sub('.', '')
+    self.title = asset.filename.base
   end
 
   def assign_uuid
@@ -235,7 +226,7 @@ class Asset < ActiveRecord::Base
 
   # this is a convenience for image-pickers
   def self.thumbnail_options
-    asset_sizes = thumbnail_sizes.collect do |k, v|
+    asset_sizes = thumbnail_sizes.map do |k, v|
       size_id = k
       size_description = "#{k}: "
       size_description << (v.is_a?(Array) ? v.join(' as ') : v)

data/app/models/asset_type.rb

@@ -104,6 +104,10 @@ class AssetType
     @paperclip_styles
   end
 
+  def active_storage_styles
+    @active_storage_styles ||= normalize_style_rules(configured_styles.merge(styles))
+  end
+
   # Takes a motley collection of differently-defined styles and renders them into the standard hash-of-hashes format.
   # Solitary strings are assumed to be  #
   def normalize_style_rules(styles = {})
@@ -173,8 +177,8 @@ class AssetType
   # class methods
 
   def self.for(attachment)
-    extension = File.extname(attachment.original_filename).sub(/^\.+/, '')
-    from_extension(extension) || from_mimetype(attachment.instance_read(:content_type)) || catchall
+    extension = attachment.record.original_extension
+    from_extension(extension) || from_mimetype(attachment.content_type) || catchall
   end
 
   def self.from_extension(extension)

data/app/views/admin/assets/edit.html.haml

@@ -16,10 +16,12 @@
               = "#{t("clipped_extension.filename")}: #{@asset.asset_file_name unless @asset.new_record?}"
             %br
             %label.url
-              = "#{t("clipped_extension.asset_url")}: #{image_path @asset.thumbnail unless @asset.new_record?}"
+              = "#{t("clipped_extension.asset_url")}: #{image_path @asset.asset.url unless @asset.new_record?}"
+            %br
+            %label.id
+              = "#{t("clipped_extension.asset_id")}: #{@asset.asset.id unless @asset.new_record?}"
           %p.asset
-            - display_size = TrustyCms::config['assets.display_size'] || 'normal'
-            = image_tag @asset.thumbnail(display_size.to_sym), :class => 'preview'
+            = image_tag @asset.thumbnail("normal"), :class => 'preview'
 
           = updated_stamp @asset
         

data/config/locales/en.yml

@@ -66,6 +66,7 @@ en:
     asset_page_attachment_note: 'Attached assets. These can be inserted with drag and drop or shown with the <code>&lt;r:assets:each /&gt;</code> tag.'
     asset_note: 'Attached assets. '
     asset_url: 'Asset URL'
+    asset_id: 'Asset ID'
     assets: 'Assets'
     assets_explanation: 'Assets and images can be uploaded and then attached or inserted into pages, snippets and layouts. Click on an asset name above to edit it or click <code>Remove</code> to delete it.'
     assets_title: 'Assets'

data/lib/trusty_cms.rb

@@ -2,6 +2,6 @@ TRUSTY_CMS_ROOT = File.expand_path(File.join(File.dirname(__FILE__), '..')) unle
 
 unless defined? TrustyCms::VERSION
   module TrustyCms
-    VERSION = '4.3.5'.freeze
+    VERSION = '5.0.0'.freeze
   end
 end

data/trusty_cms.gemspec

@@ -23,7 +23,7 @@ a general purpose content managment system--not merely a blogging engine.'
   s.summary = 'A no-fluff content management system designed for small teams.'
   s.license = 'MIT'
   s.test_files = Dir['spec/**/*']
-
+  s.add_dependency 'activestorage-validator'
   s.add_dependency 'acts_as_list',    '>= 0.9.5', '< 1.1.0'
   s.add_dependency 'acts_as_tree',    '~> 2.9.1'
   s.add_dependency 'ckeditor',        '>= 4.2.2', '< 4.4.0'
@@ -33,6 +33,7 @@ a general purpose content managment system--not merely a blogging engine.'
   s.add_dependency 'haml'
   s.add_dependency 'haml-rails',      '~> 2.0'
   s.add_dependency 'highline',        '>= 1.7.8', '< 2.1.0'
+  s.add_dependency 'image_processing'
   s.add_dependency 'kraken-io'
   s.add_dependency 'mini_racer'
   s.add_dependency 'mysql2'

data/vendor/extensions/clipped-extension/lib/clipped/engine.rb

@@ -1,7 +1,7 @@
 require 'acts_as_list'
 require 'uuidtools'
 require 'trusty_cms_clipped_extension/cloud'
-require 'paperclip'
+require 'active_storage/engine'
 require 'will_paginate/array'
 module Clipped
   class Engine < Rails::Engine

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: trusty-cms
 version: !ruby/object:Gem::Version
-  version: 4.3.5
+  version: 5.0.0
 platform: ruby
 authors:
 - TrustyCms CMS dev team
@@ -10,6 +10,20 @@ bindir: bin
 cert_chain: []
 date: 2021-10-05 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: activestorage-validator
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: acts_as_list
   requirement: !ruby/object:Gem::Requirement
@@ -160,6 +174,20 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: 2.1.0
+- !ruby/object:Gem::Dependency
+  name: image_processing
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: kraken-io
   requirement: !ruby/object:Gem::Requirement
@@ -793,7 +821,6 @@ files:
 - config/initializers/active_record_extensions.rb
 - config/initializers/assets.rb
 - config/initializers/devise.rb
-- config/initializers/interpolation.rb
 - config/initializers/kraken.rb
 - config/initializers/response_cache_timeout.rb
 - config/initializers/secret_token.rb
@@ -8541,7 +8568,6 @@ files:
 - spec/rails_helper.rb
 - spec/spec_helper.rb
 - spec/support/custom_actions.rb
-- trusty-cms-5.0.0.gem
 - trusty_cms.gemspec
 - vendor/extensions/clipped-extension/clipped_extension.rb
 - vendor/extensions/clipped-extension/lib/admin/assets_helper.rb

data/config/initializers/interpolation.rb

@@ -1,6 +0,0 @@
-require 'paperclip'
-
-Paperclip.interpolates :no_original_style do |attachment, style|
-  style ||= :original
-  style == attachment.default_style ? nil : "_#{style}"
-end