bullet_train-super_scaffolding 1.7.10 → 1.7.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1cb0c43b459719314294d6a24a43e7054faa3d5e6cb1df3984182dae0a282bef
-  data.tar.gz: ccaf9c746b32982475c90daadf370fadc1c2633ddf5c31ff81cda454fcf95d23
+  metadata.gz: 16465881966c5cc7bfb8056943d036dea6ee5e2284a4000a2f6182c41c1eb22d
+  data.tar.gz: aa40db2f84412ec08860dd6387f096c5e53622526ab17700bb66be85e8f0dd6b
 SHA512:
-  metadata.gz: a936d0fdf8d8845179cf12682c51d4ad6016ee7d1a8ab1661b3127ad5d75ee67caa9d1007fb793ee68e59e00d2641d6851902f133b483f0b98fccd7e24e24838
-  data.tar.gz: 7de0542e5f1e40660015e6188e1dd28b7fe35678153516b138f923e76cd60f16f3998914a3bbfc5acf5e197439e8f62afe175704bb270a375199fe9bd31dce80
+  metadata.gz: 4c78f397e90c8c4175ab3b5c02fbd841ab8a6f1f6c6f2929bdf4d850f88822ac6ce50f0359b51a88ee2ffde1f2a445b3084d1d23433122fd454e65e361795e43
+  data.tar.gz: 033dd98dd593f934b77f39214f35876da85ae9a9716c7e11d4b7ca93e2ba169d5e217e63326551f941a77a69277cf7a8598d140babd2c60d001186791a48f2e6

data/lib/bullet_train/super_scaffolding/scaffolders/crud_scaffolder.rb

@@ -28,6 +28,8 @@ module BulletTrain
           end
 
           child = argv[0]
+          check_class_name_for_namespace_conflict(child)
+
           parents = argv[1] ? argv[1].split(",") : []
 
           # Raise an error if the developer skipped adding the parent and went straight to the attributes.

data/lib/bullet_train/super_scaffolding/scaffolders/join_model_scaffolder.rb

@@ -30,6 +30,8 @@ module BulletTrain
           end
 
           child = argv[0]
+          check_class_name_for_namespace_conflict(child)
+
           primary_parent = argv[1].split("class_name=").last.split(",").first.split("}").first
           secondary_parent = argv[2].split("class_name=").last.split(",").first.split("}").first
 

data/lib/bullet_train/super_scaffolding/version.rb

@@ -1,5 +1,5 @@
 module BulletTrain
   module SuperScaffolding
-    VERSION = "1.7.10"
+    VERSION = "1.7.11"
   end
 end

data/lib/scaffolding/attribute.rb

@@ -51,6 +51,13 @@ class Scaffolding::Attribute
     options[:required] || is_first_attribute?
   end
 
+  def association_class_name
+    if options[:class_name].present?
+      return options[:class_name].underscore.split("/").last
+    end
+    name.split("_id").first
+  end
+
   def is_association?
     is_belongs_to? || is_has_many?
   end

data/lib/scaffolding/script.rb

@@ -49,6 +49,39 @@ def get_untracked_files
   `git ls-files --other --exclude-standard`.split("\n")
 end
 
+# class_name is a potentially namespaced class like
+# "Tasks::Widget" or "Task::Widget". Here we ensure that
+# the namespace doesn't clobber an existing model. If it does
+# we suggest that the namespace could be pluralized.
+def check_class_name_for_namespace_conflict(class_name)
+  if class_name.include?("::")
+    parts = class_name.split("::") # ["Task", "Widget"]
+    # We drop the last segment because that's tne new model we're trying to create
+    parts.pop # ["Task"]
+    possible_conflicted_class_name = ""
+    parts.each do |part|
+      possible_conflicted_class_name += "::#{part}"
+      begin
+        klass = possible_conflicted_class_name.constantize
+        is_active_record_class = klass&.ancestors&.include?(ActiveRecord::Base)
+        is_aactive_hash_class = klass&.ancestors&.include?(ActiveHash::Base)
+        if klass && (is_active_record_class || is_aactive_hash_class)
+          problematic_namespace = possible_conflicted_class_name[2..]
+          puts "It looks like the namespace you gave for this model conflicts with an existing class: #{klass.name}".red
+          puts "You should use a namespace that doesn't clobber an existing class.".red
+          puts ""
+          puts "We reccomend using the pluralized version of the existing class.".red
+          puts ""
+          puts "For instance instead of #{problematic_namespace} use #{problematic_namespace.pluralize}".red
+          exit
+        end
+      rescue NameError
+        # this is good actually, it means we don't already have a class that will be clobbered
+      end
+    end
+  end
+end
+
 def check_required_options_for_attributes(scaffolding_type, attributes, child, parent = nil)
   tableized_parent = nil
 
@@ -169,7 +202,7 @@ def check_required_options_for_attributes(scaffolding_type, attributes, child, p
         is_active_record_class = class_name_constant&.ancestors&.include?(ActiveRecord::Base)
         unless File.exist?(file_name) && is_active_record_class
           puts ""
-          puts "Attributes that end with `_id` or `_ids` trigger awesome, powerful magic in Super Scaffolding. However, because no `#{attribute_options[:class_name]}` class was found defined in `#{file_name}`, you'll need to specify a `class_name` that exists to let us know what model class is on the other side of the association, like so:".red
+          puts "Attributes that end with `_id` or `_ids` trigger awesome, powerful magic in Super Scaffolding. However, because no `#{attribute_options[:class_name]}` class was found defined in your app, you'll need to specify a `class_name` that exists to let us know what model class is on the other side of the association, like so:".red
           puts ""
           puts "  bin/super-scaffold #{scaffolding_type} #{child}#{" " + parent if parent.present?} #{name}:#{type}{class_name=#{name.gsub(/_ids?$/, "").classify}}".red
           puts ""

data/lib/scaffolding/transformer.rb

@@ -758,7 +758,7 @@ class Scaffolding::Transformer
         end
 
         if attribute.is_association?
-          short = attribute.options[:class_name].underscore.split("/").last
+          short = attribute.association_class_name
           case attribute.type
           when "buttons", "options"
             field_attributes["\n  options"] = "@tangible_thing.#{valid_values}.map { |#{short}| [#{short}.id, #{short}.#{attribute.options[:label]}] }"
@@ -1362,12 +1362,12 @@ class Scaffolding::Transformer
 
       # update the factory generated by `rails g`.
       content = if transform_string(":absolutely_abstract_creative_concept") == transform_string(":scaffolding_absolutely_abstract_creative_concept")
-        transform_string("association :absolutely_abstract_creative_concept")
+        transform_string("    association :absolutely_abstract_creative_concept")
       else
-        transform_string("association :absolutely_abstract_creative_concept, factory: :scaffolding_absolutely_abstract_creative_concept")
+        transform_string("    association :absolutely_abstract_creative_concept, factory: :scaffolding_absolutely_abstract_creative_concept")
       end
 
-      scaffold_replace_line_in_file("./test/factories/scaffolding/completely_concrete/tangible_things.rb", content, "absolutely_abstract_creative_concept { nil }")
+      scaffold_replace_line_in_file("./test/factories/scaffolding/completely_concrete/tangible_things.rb", content, "    absolutely_abstract_creative_concept { nil }")
 
       add_has_many_association
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-super_scaffolding
 version: !ruby/object:Gem::Version
-  version: 1.7.10
+  version: 1.7.11
 platform: ruby
 authors:
 - Andrew Culver
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-05-30 00:00:00.000000000 Z
+date: 2024-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: standard