sliders 0.1.2 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 92e334f7082b4b266da898f0d116abeb4aed0f62ac1e7a1103aaeb336a40a025
-  data.tar.gz: f87cc0934a9da7fe46da857b1938f224f173d1274e10c7512eb81e43861e09b0
+  metadata.gz: 3b733addcf7127778b4c2133e774a5bf4353b7cfd938e220ae237cdb39c83728
+  data.tar.gz: 151a841d645cafd167295b08991603d5ed382f7f5ce9bc6123eb228c677fd027
 SHA512:
-  metadata.gz: 0326d5f40aba0722bbf72da10879b04353abbc64d81d2af0ec8b975ee6a2161bd2a7ac7c8ff29430bed9cccfd04b6994bf2755fd88e3162bb5dced50cfe96bf7
-  data.tar.gz: 60e293a699f2d1d7b30727a4a10aac8839357ed70f42d7aea9488da9ec4f58eb00190395ff0040cb94e3db8ca112facbbf352785ef748574cc5acc068669b2f7
+  metadata.gz: d36018593a31a9c4091ea9cf80c3049d3d7063477c8574e2e8278145c232af159858cffaf004db65fca0dc4caff33adfd03aed34704b3fb20b9ee4d111a427e3
+  data.tar.gz: 30290c6d3318698f30f1a6cb4b3793998c981364f33252b614b27811a466131a5729d9d61009ee3b5f8115704563a63db15e9b1573cc2e153095a468fbb9ff47

data/README.md

@@ -70,38 +70,16 @@ This creates the `app/sliders/` folder and a module and folder for your feature.
 
 Now you can just use regular generators to create thing in `MyFeature` namespace and *sliders* will create the files in your slider folder.
 
-Generators that will now create files in slider folder:
-- controllers
-- scaffold
-- helpers
-- mailers
-- jobs
-- active record model
+The folder *sliders* is not used for namespace, just like the folders in your slider is not used. So a file in `app/sliders/my_feature/models/admin.rb` will be for the constant `MyFeature::Admin`.
+
+### Fixtures
+Load fixtures from slides by adding `Sliders.add_fixtures` before `module ActiveSupport` in `test_helper.rb`
 
 ## Contributing
-Feel free to open PR with fixes for generators for that creates files outside of sliders folder.
-
-Known generators that will not do what you expect:
-- view
-  - controller
-  - mailer
-  - scaffold
-- action cable
-- action mailbox
-- tests
-  - controller
-  - model
-  - scaffold
-  - mailer
-  - job
-  - integration
-  - helper
-  - system?
-- fixtures (part of model tests)
-- components?
-
-Other things we need to fix:
-- loading fixtures from slider fixture foldeer
+PR with fixes or improvements are most welcome.
+
+We have tested *sliders* with most of Rails built-in generators, only a few of them have actual test cases.
+PR with tests for more generators are very welcome - also generators for often used gems like eg. ViewComponent or Rspec.
 
 ## License
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/config/initializers/generators.rb

@@ -0,0 +1,99 @@
+module SlidersCopyInvoke
+  extend ActiveSupport::Concern
+
+  included do
+    class << self
+      alias_method :original_invoke, :invoke
+      alias_method :original_run_after_generate_callback, :run_after_generate_callback
+    end
+  end
+end
+Rails::Generators.include SlidersCopyInvoke
+
+module SlidersInvoke
+  extend ActiveSupport::Concern
+
+  class_methods do
+    def invoke(namespace, args = ARGV, config = {})
+      # warn "SLIDERS INVOKE: #{namespace} #{args} #{config}"
+      original_invoke(namespace, args, config)
+
+      names = namespace.to_s.split(":")
+      if find_by_namespace(names.pop, names.any? && names.join(":"))
+        sliders_cleanup if config[:behavior] == :revoke
+        sliders_move if config[:behavior] == :invoke
+      end
+    end
+
+    private
+
+    def sliders_cleanup
+      loop_files(class_variable_get(:@@generated_files)) do |source_file, destination_file|
+        File.unlink destination_file if File.exist?(destination_file)
+      end
+    end
+
+    def sliders_move
+      loop_files(class_variable_get(:@@sliders_generated_files)) do |source_file, destination_file|
+        source_file = Rails.root.join(source_file)
+        FileUtils.mkdir_p(File.dirname(destination_file))
+        FileUtils.mv(source_file, destination_file)
+      end
+
+      class_variable_set(:@@sliders_generated_files, [])
+    end
+
+    def loop_files(source_files)
+      source_files.each do |source_file|
+        slider = Sliders.sliders.find { |slider| source_file.include? slider }
+        next unless slider
+
+        destination_file = file_in_slider_path(source_file, slider)
+        next unless destination_file
+
+        yield source_file, destination_file
+      end
+    end
+
+    def file_in_slider_path(source_file, slider)
+      path_parts = source_file.split("/")
+      return unless path_parts.include?(slider) || path_parts.include?("#{slider}.rb")
+      return unless path_parts.include?("app") || path_parts.include?("test")
+      return if path_parts.include?("sliders") && path_parts.include?("#{slider}.rb")
+
+      if path_parts.include?("#{slider}.rb")
+        # creating a file named like the slider should go to sliders folder
+        # overwriting our empty slider namespace file
+        return Rails.root.join("app", "sliders", path_parts.last)
+      end
+
+      # add sliders after app or test
+      path_parts.insert(path_parts.index("app") + 1, "sliders") if path_parts.include?("app")
+      path_parts.insert(path_parts.index("test") + 1, "sliders") if path_parts.include?("test")
+
+      # move slider namespace up or
+      slider_namespace_index = path_parts.index(slider)
+      path_parts[slider_namespace_index], path_parts[slider_namespace_index - 1] = path_parts[slider_namespace_index - 1], path_parts[slider_namespace_index]
+
+      # add slider name to filename if fixture
+      if path_parts.include?("fixtures") && !path_parts.last.include?("#{slider}_")
+        path_parts.insert(-2, slider)
+      end
+
+      Rails.root.join(path_parts.join("/"))
+    rescue StandardError => e
+      warn "===== Sliders Error ====="
+      warn "Error moving file: #{source_file} for slider: #{slider}"
+      warn "Error: #{e.message}"
+      warn "========================="
+      raise
+    end
+
+    def run_after_generate_callback
+      class_variable_set(:@@sliders_generated_files, [])
+      class_variable_set(:@@sliders_generated_files, class_variable_get(:@@generated_files)) if class_variable_defined?(:@@generated_files)
+      original_run_after_generate_callback
+    end
+  end
+end
+Rails::Generators.prepend SlidersInvoke

data/lib/generators/slider/slider_generator.rb

@@ -3,7 +3,7 @@ class SliderGenerator < Rails::Generators::NamedBase
 
   def create_slider_file
     template "module.rb", File.join("app/sliders", "#{file_name}.rb")
-    template ".keep", File.join("app/sliders", "#{file_name}", ".keep")
-    template ".keep", File.join("test/sliders", "#{file_name}", ".keep")
+    create_file File.join("app/sliders", "#{file_name}", ".keep"), ""
+    create_file File.join("test/sliders", "#{file_name}", ".keep"), ""
   end
 end

data/lib/sliders/version.rb

@@ -1,3 +1,3 @@
 module Sliders
-  VERSION = "0.1.2"
+  VERSION = "0.1.4"
 end

data/lib/sliders.rb

@@ -1,16 +1,8 @@
 require "sliders/version"
 require "sliders/engine"
 require "rails/generators"
-require "rails_ext/controller_generator"
-require "rails_ext/scaffold_controller_generator"
-require "rails_ext/helper_generator"
-require "rails_ext/job_generator"
-require "rails_ext/mailer_generator"
-require "rails_ext/active_record_model_generator"
 
 module Sliders
-  # Your code goes here...
-
   def self.sliders
     @sliders ||= []
   end
@@ -26,4 +18,13 @@ module Sliders
       end
     end
   end
+
+  def self.add_fixtures
+    sliders.each do |slider|
+      slider_path = Rails.root.join("test", "sliders", slider, "fixtures")
+      next unless Dir.exist?(slider_path)
+
+      ActiveSupport::TestCase.fixture_paths << slider_path
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sliders
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.4
 platform: ruby
 authors:
 - Simon Træls Ravn
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-08-27 00:00:00.000000000 Z
+date: 2025-06-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -37,16 +37,11 @@ files:
 - app/assets/config/sliders_manifest.js
 - app/controllers/concerns/fix_sliders_view_path.rb
 - config/initializers/autoloader.rb
+- config/initializers/generators.rb
 - config/routes.rb
 - lib/generators/slider/USAGE
 - lib/generators/slider/slider_generator.rb
 - lib/generators/slider/templates/module.rb
-- lib/rails_ext/active_record_model_generator.rb
-- lib/rails_ext/controller_generator.rb
-- lib/rails_ext/helper_generator.rb
-- lib/rails_ext/job_generator.rb
-- lib/rails_ext/mailer_generator.rb
-- lib/rails_ext/scaffold_controller_generator.rb
 - lib/sliders.rb
 - lib/sliders/engine.rb
 - lib/sliders/version.rb

data/lib/rails_ext/active_record_model_generator.rb

@@ -1,18 +0,0 @@
-require "rails/generators/active_record/model/model_generator"
-module SlidersActiveRecordModelGenerator
-  extend ActiveSupport::Concern
-
-  def create_model_file
-    return super unless (class_path & Sliders.sliders).any?
-
-    template "model.rb", File.join("app/sliders/", class_path[0], "models", class_path[1..], "#{file_name}.rb")
-  end
-
-  def create_module_file
-    return super unless (class_path & Sliders.sliders).any?
-
-    return if regular_class_path.empty? || regular_class_path.one?
-    template "module.rb", File.join("app/sliders/", class_path[0], "models", "#{class_path[1..].join('/')}.rb") if behavior == :invoke
-  end
-end
-ActiveRecord::Generators::ModelGenerator.prepend SlidersActiveRecordModelGenerator

data/lib/rails_ext/controller_generator.rb

@@ -1,11 +0,0 @@
-require "rails/generators/rails/controller/controller_generator"
-module SlidersControllerGenerator
-  extend ActiveSupport::Concern
-
-  def create_controller_files
-    return super unless (class_path & Sliders.sliders).any?
-
-    template "controller.rb", File.join("app/sliders/", class_path[0], "controllers", class_path[1..], "#{file_name}_controller.rb")
-  end
-end
-Rails::Generators::ControllerGenerator.prepend SlidersControllerGenerator

data/lib/rails_ext/helper_generator.rb

@@ -1,11 +0,0 @@
-require "rails/generators/rails/helper/helper_generator"
-module SlidersHelperGenerator
-  extend ActiveSupport::Concern
-
-  def create_helper_files
-    return super unless (class_path & Sliders.sliders).any?
-
-    template "helper.rb", File.join("app/sliders/", class_path[0], "helpers", class_path[1..], "#{file_name}_helper.rb")
-  end
-end
-Rails::Generators::HelperGenerator.prepend SlidersHelperGenerator

data/lib/rails_ext/job_generator.rb

@@ -1,11 +0,0 @@
-require "rails/generators/job/job_generator"
-module SlidersJobGenerator
-  extend ActiveSupport::Concern
-
-  def create_job_file
-    return super unless (class_path & Sliders.sliders).any?
-
-    template "job.rb", File.join("app/sliders/", class_path[0], "jobs", class_path[1..], "#{file_name}_job.rb")
-  end
-end
-Rails::Generators::JobGenerator.prepend SlidersJobGenerator

data/lib/rails_ext/mailer_generator.rb

@@ -1,11 +0,0 @@
-require "rails/generators/mailer/mailer_generator"
-module SlidersMailerGenerator
-  extend ActiveSupport::Concern
-
-  def create_mailer_file
-    return super unless (class_path & Sliders.sliders).any?
-
-    template "mailer.rb", File.join("app/sliders/", class_path[0], "mailers", class_path[1..], "#{file_name}_mailer.rb")
-  end
-end
-Rails::Generators::MailerGenerator.prepend SlidersMailerGenerator

data/lib/rails_ext/scaffold_controller_generator.rb

@@ -1,12 +0,0 @@
-require "rails/generators/rails/scaffold_controller/scaffold_controller_generator"
-module SlidersScaffoldControllerGenerator
-  extend ActiveSupport::Concern
-
-  def create_controller_files
-    return super unless (controller_class_path & Sliders.sliders).any?
-
-    template_file = options.api? ? "api_controller.rb" : "controller.rb"
-    template template_file, File.join("app/sliders/", controller_class_path[0], "controllers", controller_class_path[1..], "#{controller_file_name}_controller.rb")
-  end
-end
-Rails::Generators::ScaffoldControllerGenerator.prepend SlidersScaffoldControllerGenerator