bullet_train-super_scaffolding 1.12.2 → 1.12.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be0df510fd098c114487239ac00e69b54856df38cef8655d61316f8119c1d9ce
-  data.tar.gz: 0aec8526362652c9003780d54cd2245e5b137ececb2d162997588517f37e51c8
+  metadata.gz: c84cb4d581618c07a651084cff60e510bf4df6bfa78bcbc6b7e3125aab92f1ba
+  data.tar.gz: bff2ba24ff94b0516a4ca36ad4836a87d1f5bfef5b00dfa5192da4f0128d6b63
 SHA512:
-  metadata.gz: 87e04a49f198d65ea4c71f67641b59c68387a61dfd064cf30ac42cfa3f987f612ca5bd9db5d0569f407f4d177b1a12c52f0e0e5b74a7d4042165dba526d011b5
-  data.tar.gz: 313288579b1ccb287d0026c65d77d11da4b24527321719ddfb6f046b553ecd7b293666e140a1f0598a00e817c9eb848045091e835119eb6295fd67bdda7e1246
+  metadata.gz: 61eca18fb9787f8a22313c8749effd04f629bac711ebfff419f43043b1228f9d325a88cc192885d5bc01f80e53ee7224347ed961c4b6584b22a2d119ae7d5f81
+  data.tar.gz: 22b0efedb45e2cf68b2751609647e5c49a89a71624788baf31b9d3f3406edbe111cc1afb2deb5877a226a3eb04c2f4b407143666de92f2351b3f026153189246

data/lib/bullet_train/super_scaffolding/version.rb

@@ -1,5 +1,5 @@
 module BulletTrain
   module SuperScaffolding
-    VERSION = "1.12.2"
+    VERSION = "1.12.3"
   end
 end

data/lib/scaffolding/transformer.rb

@@ -327,7 +327,19 @@ class Scaffolding::Transformer
 
     if target_file_content.include?(transformed_content) && !content_matches_model_name
       puts "No need to update '#{transformed_file_name}'. It already has '#{transformed_content}'." unless silence_logs?
-
+    elsif !target_file_content.include?(transform_hook)
+      unless options[:suppress_could_not_find_hook]
+        puts
+        puts "-------------------------------".yellow
+        puts "Heads up! We weren't able to find a super scaffolding hook where we expected it to be.".yellow
+        puts "In #{transformed_file_name}".yellow
+        puts "We expected to find a line like this:".yellow
+        puts transform_hook.yellow
+        puts
+        puts "See https://bullettrain.co/docs/super-scaffolding/targets for more details.".yellow
+        puts "-------------------------------".yellow
+        puts
+      end
     else
 
       new_target_file_content = []
@@ -967,10 +979,16 @@ class Scaffolding::Transformer
           "./app/controllers/account/scaffolding/completely_concrete/tangible_things_controller.rb",
           "./app/controllers/api/v1/scaffolding/completely_concrete/tangible_things_controller.rb"
         ].each do |file|
+          # We usually do StrongParams stuff in the API controller and the Account controller inherits it from there.
+          # We _used_ to do StrongParams stuff directly in the Account controller. So for backwards compatibility we
+          # need to try to make updates in the Account controller, but if we can't find the target hook there's no need
+          # to show an error and make the developer think that they've broken something.
+          suppress_could_not_find_hook = file.match("/account/")
+
           if attribute.is_ids? || attribute.is_multiple?
-            scaffold_add_line_to_file(file, "#{attribute.name}: [],", RUBY_NEW_ARRAYS_HOOK, prepend: true)
+            scaffold_add_line_to_file(file, "#{attribute.name}: [],", RUBY_NEW_ARRAYS_HOOK, prepend: true, suppress_could_not_find_hook: suppress_could_not_find_hook)
             if attribute.file_field? || attribute.active_storage_image?
-              scaffold_add_line_to_file(file, "#{attribute.name}_removal: [],", RUBY_NEW_ARRAYS_HOOK, prepend: true)
+              scaffold_add_line_to_file(file, "#{attribute.name}_removal: [],", RUBY_NEW_ARRAYS_HOOK, prepend: true, suppress_could_not_find_hook: suppress_could_not_find_hook)
             end
           elsif attribute.type == "address_field"
             address_strong_params = <<~RUBY
@@ -985,11 +1003,11 @@ class Scaffolding::Transformer
                 :postal_code
               ],
             RUBY
-            scaffold_add_line_to_file(file, address_strong_params, RUBY_NEW_ARRAYS_HOOK, prepend: true)
+            scaffold_add_line_to_file(file, address_strong_params, RUBY_NEW_ARRAYS_HOOK, prepend: true, suppress_could_not_find_hook: suppress_could_not_find_hook)
           else
-            scaffold_add_line_to_file(file, ":#{attribute.name},", RUBY_NEW_FIELDS_HOOK, prepend: true)
+            scaffold_add_line_to_file(file, ":#{attribute.name},", RUBY_NEW_FIELDS_HOOK, prepend: true, suppress_could_not_find_hook: suppress_could_not_find_hook)
             if attribute.file_field? || attribute.active_storage_image?
-              scaffold_add_line_to_file(file, ":#{attribute.name}_removal,", RUBY_NEW_FIELDS_HOOK, prepend: true)
+              scaffold_add_line_to_file(file, ":#{attribute.name}_removal,", RUBY_NEW_FIELDS_HOOK, prepend: true, suppress_could_not_find_hook: suppress_could_not_find_hook)
             end
           end
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-super_scaffolding
 version: !ruby/object:Gem::Version
-  version: 1.12.2
+  version: 1.12.3
 platform: ruby
 authors:
 - Andrew Culver
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-12-18 00:00:00.000000000 Z
+date: 2024-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: standard