bullet_train-super_scaffolding 1.0.34 → 1.0.37

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e89abdab9630d8af6498af03edec21ce70c456c1579d5c35bf7e16ed3a0383c3
-  data.tar.gz: 57f3be888b988cc3d0ff747776cb319ec1bc3399f9e55209d94f132c2ef3d6f1
+  metadata.gz: 1ca43f2f490c937c87fcc1493b3e194d56f2e39bccd57e01ba7eda9bbaa230fa
+  data.tar.gz: f027670522cf4faf29a93bf237874ccffcee91f88fece103522ec2e0c0e21508
 SHA512:
-  metadata.gz: cf98aac64c6ee99b4b9778b8d371a770c26d8a1d89925e8e3144aa5b6eee2180890cff7732e700b78c764ff9871a4dc1543db7b3e69e25cc763eb9a16cb34a98
-  data.tar.gz: d7d0935c5280a7b482e260fe857fba7b7a1dd99076505978ff7d47505921d61abfb041fed3737295f96fb1ffcddc1d368703acd67e2799e01538ac7798887d65
+  metadata.gz: 27dba84c5f1fa859f612c33897b747eb5cdcb850d6774dd1a95846c7f7dcf646630d8e8093e023e3084342cc864f2a58a669605147de16fa02fde8b74fcf2cdc
+  data.tar.gz: 647922cfdf4696130bf23c1fe15544792ca257cc358acd98baa1f2932f09521d3457ba429fb3c8ff2e6b3a9a3f9d1e53c754b1b92d0f708ee37d6012da49a10f

data/app/controllers/api/v1/scaffolding/completely_concrete/tangible_things_endpoint.rb

@@ -16,6 +16,7 @@ class Api::V1::Scaffolding::CompletelyConcrete::TangibleThingsEndpoint < Api::V1
       optional :date_field_value, type: Date, allow_blank: true, desc: Api.heading(:date_field_value)
       optional :date_and_time_field_value, type: DateTime, allow_blank: true, desc: Api.heading(:date_and_time_field_value)
       optional :email_field_value, type: String, allow_blank: true, desc: Api.heading(:email_field_value)
+      optional :file_field_value, type: String, allow_blank: true, desc: Api.heading(:file_field_value)
       optional :password_field_value, type: String, allow_blank: true, desc: Api.heading(:password_field_value)
       optional :phone_field_value, type: String, allow_blank: true, desc: Api.heading(:phone_field_value)
       optional :option_value, type: String, allow_blank: true, desc: Api.heading(:option_value)

data/app/serializers/api/v1/scaffolding/completely_concrete/tangible_thing_serializer.rb

@@ -1,5 +1,6 @@
 class Api::V1::Scaffolding::CompletelyConcrete::TangibleThingSerializer < Api::V1::ApplicationSerializer
   set_type "scaffolding/completely_concrete/tangible_thing"
+  singleton_class.include Rails.application.routes.url_helpers
 
   attributes :id,
     :absolutely_abstract_creative_concept_id,
@@ -25,5 +26,12 @@ class Api::V1::Scaffolding::CompletelyConcrete::TangibleThingSerializer < Api::V
     :created_at,
     :updated_at
 
+  # 🚅 skip this section when scaffolding.
+  attribute :file_field_value do |object|
+    rails_blob_path(object.file_field_value, disposition: "attachment", only_path: true) if object.file_field_value.attached?
+  end
+  # 🚅 stop any skipping we're doing now.
+  # 🚅 super scaffolding will insert file-related logic above this line.
+
   belongs_to :absolutely_abstract_creative_concept, serializer: Api::V1::Scaffolding::AbsolutelyAbstract::CreativeConceptSerializer
 end

data/lib/bullet_train/super_scaffolding/version.rb

@@ -1,5 +1,5 @@
 module BulletTrain
   module SuperScaffolding
-    VERSION = "1.0.34"
+    VERSION = "1.0.37"
   end
 end

data/lib/scaffolding/block_manipulator.rb

@@ -107,6 +107,7 @@ class Scaffolding::BlockManipulator
     insert_line(block_content[1], block_end + 1)
   end
 
+  # TODO: Delete this because it only really needs to be in the FileManipulator
   def write
     File.write(@filepath, @lines.join)
   end

data/lib/scaffolding/class_names_transformer.rb

@@ -160,6 +160,14 @@ class Scaffolding::ClassNamesTransformer
       parent_in_namespace_class_name.titlecase.pluralize
     when "Tangible Things"
       in_namespace_class_name.titlecase.pluralize
+    when "Creative concepts"
+      parent_in_namespace_class_name.titlecase.downcase.capitalize.pluralize
+    when "Tangible things"
+      in_namespace_class_name.titlecase.downcase.capitalize.pluralize
+    when "creative concepts"
+      parent_in_namespace_class_name.titlecase.downcase.pluralize
+    when "tangible things"
+      in_namespace_class_name.titlecase.downcase.pluralize
     when "creative-concepts"
       parent_in_namespace_class_name.underscore.gsub(/[\/_]/, "-").pluralize
     when "tangible-things"
@@ -173,6 +181,14 @@ class Scaffolding::ClassNamesTransformer
       parent_in_namespace_class_name.titlecase
     when "Tangible Thing"
       in_namespace_class_name.titlecase
+    when "Creative concept"
+      parent_in_namespace_class_name.titlecase.downcase.capitalize
+    when "Tangible thing"
+      in_namespace_class_name.titlecase.downcase.capitalize
+    when "creative concept"
+      parent_in_namespace_class_name.titlecase.downcase
+    when "tangible thing"
+      in_namespace_class_name.titlecase.downcase
     when "creative-concept"
       parent_in_namespace_class_name.underscore.gsub(/[\/_]/, "-")
     when "tangible-thing"

data/lib/scaffolding/file_manipulator.rb

@@ -0,0 +1,71 @@
+require "scaffolding/block_manipulator"
+
+# TODO: If we move this and the BlockManipulator into their own gems,
+# we can probably call these methods with something shorter without `Scaffolding::`.
+module Scaffolding::FileManipulator
+  # TODO: The block_manipulator shouldn't be an instance variable.
+  def self.find(lines, needle, within = nil, block_manipulator)
+    lines_within(lines, within, block_manipulator).each_with_index do |line, line_number|
+      return (within + (within ? 1 : 0) + line_number) if line.match?(needle)
+    end
+    nil
+  end
+
+  # TODO: The block_manipulator shouldn't be an instance variable.
+  def self.lines_within(lines, within, block_manipulator)
+    return lines unless within
+    lines[(within + 1)..(block_manipulator.find_block_end(starting_from: within, lines: lines) + 1)]
+  end
+
+  def self.replace_line_in_file(file, content, in_place_of)
+    target_file_content = File.read(file)
+
+    if target_file_content.include?(content)
+      puts "No need to update '#{file}'. It already has '#{content}'."
+    else
+      puts "Updating '#{file}'."
+      target_file_content.gsub!(in_place_of, content)
+      File.write(file, target_file_content)
+    end
+  end
+
+  # Pass in an array where this content should be inserted within the yml file.
+  # For example, to add content to admin.models pass in [:admin, :models]
+  def self.add_line_to_yml_file(file, content, location_array)
+    # First check that the given location array actually exists in the yml file:
+    yml = YAML.safe_load(File.read(file))
+    location_array.map!(&:to_s)
+
+    # TODO: Raise an error if we're returning nil.
+    return nil if yml.dig(*location_array).nil?
+
+    content += "\n" unless content[-1] == "\n"
+    # Find the location in the file where the location_array is
+    lines = File.readlines(file)
+    current_needle = location_array.shift.to_s
+    current_space = ""
+    insert_after = 1
+    lines.each_with_index do |line, index|
+      break if current_needle.nil?
+      if line.strip == current_needle + ":"
+        current_needle = location_array.shift.to_s
+        insert_after = index
+        current_space = line.match(/\s+/).to_s
+      end
+    end
+    new_lines = []
+    current_space += "  "
+    lines.each_with_index do |line, index|
+      new_lines << line
+      new_lines << current_space + content if index == insert_after
+    end
+    File.write(file, new_lines.join)
+  end
+
+  def self.write(file_name, lines)
+    puts "Updating '#{file_name}'."
+    File.open(file_name, "w+") do |file|
+      file.puts(lines.join.strip + "\n")
+    end
+  end
+end

data/lib/scaffolding/routes_file_manipulator.rb

@@ -82,6 +82,7 @@ class Scaffolding::RoutesFileManipulator
     results
   end
 
+  # TODO: Remove this and use the BlockManipulator
   def insert_before(new_lines, line_number, options = {})
     options[:indent] ||= false
     before = lines[0..(line_number - 1)]
@@ -90,6 +91,7 @@ class Scaffolding::RoutesFileManipulator
     self.lines = before + (options[:prepend_newline] ? ["\n"] : []) + new_lines + after
   end
 
+  # TODO: Remove this and use the BlockManipulator
   def insert_after(new_lines, line_number, options = {})
     options[:indent] ||= false
     before = lines[0..line_number]
@@ -164,21 +166,13 @@ class Scaffolding::RoutesFileManipulator
     namespace_block_start.present? ? {namespace => namespace_block_start} : {}
   end
 
-  def find(needle, within = nil)
-    lines_within(within).each_with_index do |line, line_number|
-      return (within + (within ? 1 : 0) + line_number) if line.match?(needle)
-    end
-
-    nil
-  end
-
   def find_in_namespace(needle, namespaces, within = nil, ignore = nil)
     if namespaces.any?
       namespace_lines = find_namespaces(namespaces, within)
       within = namespace_lines[namespaces.last]
     end
 
-    lines_within(within).each_with_index do |line, line_number|
+    Scaffolding::FileManipulator.lines_within(lines, within, block_manipulator).each_with_index do |line, line_number|
       # + 2 because line_number starts from 0, and within starts one line after
       actual_line_number = (within + line_number + 2)
 
@@ -232,7 +226,7 @@ class Scaffolding::RoutesFileManipulator
   # However, will not find namespace blocks inside namespace blocks.
   def top_level_namespace_block_lines(within)
     local_namespace_blocks = []
-    lines_within(within).each do |line|
+    Scaffolding::FileManipulator.lines_within(lines, within, block_manipulator).each do |line|
       # i.e. - Retrieve "foo" from "namespace :foo do"
       match_data = line.match(/(\s*namespace\s:)(.*)(\sdo$)/)
 
@@ -286,11 +280,6 @@ class Scaffolding::RoutesFileManipulator
     find_or_convert_resource_block(parts.last, options)
   end
 
-  def lines_within(within)
-    return lines unless within
-    lines[(within + 1)..(block_manipulator.find_block_end(starting_from: within, lines: lines) + 1)]
-  end
-
   def find_or_convert_resource_block(parent_resource, options = {})
     unless find_resource_block([parent_resource], options)
       if (resource_line_number = find_resource([parent_resource], options))
@@ -310,6 +299,7 @@ class Scaffolding::RoutesFileManipulator
     within
   end
 
+  # TODO: Remove this and use the BlockManipulator
   def insert(lines_to_add, within)
     insertion_line = block_manipulator.find_block_end(starting_from: within, lines: lines)
     result_line = insertion_line
@@ -343,7 +333,7 @@ class Scaffolding::RoutesFileManipulator
       # add the new resource within that namespace.
       line = "scope module: '#{parent_resource}' do"
       # TODO you haven't tested this yet.
-      unless (scope_within = find(/#{line}/, parent_within))
+      unless (scope_within = Scaffolding::FileManipulator.find(lines, /#{line}/, parent_within, block_manipulator))
         scope_within = insert([line, "end"], parent_within)
       end
 
@@ -399,11 +389,4 @@ class Scaffolding::RoutesFileManipulator
     return if concerns.empty?
     "concerns: #{concerns}"
   end
-
-  def write
-    puts "Updating '#{@filename}'."
-    File.open(@filename, "w+") do |file|
-      file.puts(lines.join.strip + "\n")
-    end
-  end
 end

data/lib/scaffolding/transformer.rb

@@ -1,10 +1,17 @@
 require "indefinite_article"
 require "yaml"
+require "scaffolding/file_manipulator"
 require "scaffolding/class_names_transformer"
 
 class Scaffolding::Transformer
   attr_accessor :child, :parent, :parents, :class_names_transformer, :cli_options, :additional_steps, :namespace, :suppress_could_not_find
 
+  def code_for_child_on_parent_show_page
+  end
+
+  def update_models_abstract_class
+  end
+
   def initialize(child, parents, cli_options = {})
     self.child = child
     self.parent = parents.first
@@ -20,6 +27,7 @@ class Scaffolding::Transformer
   RUBY_NEW_FIELDS_HOOK = "# 🚅 super scaffolding will insert new fields above this line."
   RUBY_ADDITIONAL_NEW_FIELDS_HOOK = "# 🚅 super scaffolding will also insert new fields above this line."
   RUBY_EVEN_MORE_NEW_FIELDS_HOOK = "# 🚅 super scaffolding will additionally insert new fields above this line."
+  RUBY_FILES_HOOK = "# 🚅 super scaffolding will insert file-related logic above this line."
   ENDPOINTS_HOOK = "# 🚅 super scaffolding will mount new endpoints above this line."
   ERB_NEW_FIELDS_HOOK = "<%#{RUBY_NEW_FIELDS_HOOK} %>"
   CONCERNS_HOOK = "# 🚅 add concerns above."
@@ -89,6 +97,10 @@ class Scaffolding::Transformer
       "tangible-things",
       "Creative Concepts",
       "Tangible Things",
+      "Creative concepts",
+      "Tangible things",
+      "creative concepts",
+      "tangible things",
 
       # just class name plural.
       "creative_concept",
@@ -97,6 +109,10 @@ class Scaffolding::Transformer
       "tangible-thing",
       "Creative Concept",
       "Tangible Thing",
+      "Creative concept",
+      "Tangible thing",
+      "creative concept",
+      "tangible thing",
 
       # Account namespace vs. others.
       ":account",
@@ -235,36 +251,6 @@ class Scaffolding::Transformer
     end
   end
 
-  # pass in an array where this content should be inserted within the yml file.  For example, to add content
-  # to admin.models pass in [:admin, :models]
-  def add_line_to_yml_file(file, content, location_array)
-    # First check that the given location array actually exists in the yml file:
-    yml = YAML.safe_load(File.read(file))
-    location_array.map!(&:to_s)
-    return nil if yml.dig(*location_array).nil? # Should we raise an error?
-    content += "\n" unless content[-1] == "\n"
-    # Find the location in the file where the location_array is
-    lines = File.readlines(file)
-    current_needle = location_array.shift.to_s
-    current_space = ""
-    insert_after = 1
-    lines.each_with_index do |line, index|
-      break if current_needle.nil?
-      if line.strip == current_needle + ":"
-        current_needle = location_array.shift.to_s
-        insert_after = index
-        current_space = line.match(/\s+/).to_s
-      end
-    end
-    new_lines = []
-    current_space += "  "
-    lines.each_with_index do |line, index|
-      new_lines << line
-      new_lines << current_space + content if index == insert_after
-    end
-    File.write(file, new_lines.join)
-  end
-
   def add_line_to_file(file, content, hook, options = {})
     increase_indent = options[:increase_indent]
     add_before = options[:add_before]
@@ -346,23 +332,11 @@ class Scaffolding::Transformer
     add_line_to_file(file, content, hook, options)
   end
 
-  def replace_line_in_file(file, content, in_place_of)
-    target_file_content = File.read(file)
-
-    if target_file_content.include?(content)
-      puts "No need to update '#{file}'. It already has '#{content}'."
-    else
-      puts "Updating '#{file}'."
-      target_file_content.gsub!(in_place_of, content)
-      File.write(file, target_file_content)
-    end
-  end
-
   def scaffold_replace_line_in_file(file, content, in_place_of)
     file = transform_string(file)
     # we specifically don't transform the content, we assume a builder function created this content.
     in_place_of = transform_string(in_place_of)
-    replace_line_in_file(file, content, in_place_of)
+    Scaffolding::FileManipulator.replace_line_in_file(file, content, in_place_of)
   end
 
   # if class_name isn't specified, we use `child`.
@@ -451,8 +425,8 @@ class Scaffolding::Transformer
     model_names = class_names || [child]
     role_file = "./config/models/roles.yml"
     model_names.each do |model_name|
-      add_line_to_yml_file(role_file, "#{model_name}: read", [:default, :models])
-      add_line_to_yml_file(role_file, "#{model_name}: manage", [:admin, :models])
+      Scaffolding::FileManipulator.add_line_to_yml_file(role_file, "#{model_name}: read", [:default, :models])
+      Scaffolding::FileManipulator.add_line_to_yml_file(role_file, "#{model_name}: manage", [:admin, :models])
     end
   end
 
@@ -480,7 +454,7 @@ class Scaffolding::Transformer
       current_parent = working_parents.pop
     end
 
-    setup_lines << current_transformer.transform_string("@tangible_thing = create(:scaffolding_completely_concrete_tangible_thing, #{previous_assignment})")
+    setup_lines << current_transformer.transform_string("@tangible_thing = build(:scaffolding_completely_concrete_tangible_thing, #{previous_assignment})")
 
     setup_lines
   end
@@ -615,7 +589,9 @@ class Scaffolding::Transformer
       is_id = name.match?(/_id$/)
       is_ids = name.match?(/_ids$/)
       # if this is the first attribute of a newly scaffolded model, that field is required.
-      is_required = attribute_options[:required] || (scaffolding_options[:type] == :crud && index == 0)
+      unless type == "file_field"
+        is_required = attribute_options[:required] || (scaffolding_options[:type] == :crud && index == 0)
+      end
       is_vanilla = attribute_options&.key?(:vanilla)
       is_belongs_to = is_id && !is_vanilla
       is_has_many = is_ids && !is_vanilla
@@ -653,6 +629,8 @@ class Scaffolding::Transformer
         "date_and_time"
       when "email_field"
         "email"
+      when "emoji_field"
+        "text"
       when "color_picker"
         "code"
       when "text_field"
@@ -661,8 +639,10 @@ class Scaffolding::Transformer
         "text"
       when "file_field"
         "file"
+      when "password_field"
+        "string"
       else
-        raise "Invalid attribute type: #{type}."
+        raise "Invalid field type: #{type}."
       end
 
       cell_attributes = if boolean_buttons
@@ -765,13 +745,14 @@ class Scaffolding::Transformer
         # field on the form.
         field_attributes = {method: ":#{name}"}
         field_options = {}
+        options = {}
 
         if scaffolding_options[:type] == :crud && index == 0
           field_options[:autofocus] = "true"
         end
 
         if is_id && type == "super_select"
-          field_options[:include_blank] = "t('.fields.#{name}.placeholder')"
+          options[:include_blank] = "t('.fields.#{name}.placeholder')"
           # add_additional_step :yellow, transform_string("We've added a reference to a `placeholder` to the form for the select or super_select field, but unfortunately earlier versions of the scaffolded locales Yaml don't include a reference to `fields: *fields` under `form`. Please add it, otherwise your form won't be able to locate the appropriate placeholder label.")
         end
 
@@ -796,12 +777,19 @@ class Scaffolding::Transformer
         end
 
         # https://stackoverflow.com/questions/21582464/is-there-a-ruby-hashto-s-equivalent-for-the-new-hash-syntax
-        if field_options.any?
+        if field_options.any? || options.any?
           field_options_key = if ["buttons", "super_select", "options"].include?(type)
+            if options.any?
+              field_attributes[:options] = "{" + field_options.map { |key, value| "#{key}: #{value}" }.join(", ") + "}"
+            end
+
             :html_options
           else
+            field_options.merge!(options)
+
             :options
           end
+
           field_attributes[field_options_key] = "{" + field_options.map { |key, value| "#{key}: #{value}" }.join(", ") + "}"
         end
 
@@ -1038,7 +1026,7 @@ class Scaffolding::Transformer
         # TODO The serializers can't handle these `has_rich_text` attributes.
         unless type == "trix_editor"
           scaffold_add_line_to_file("./app/views/account/scaffolding/completely_concrete/tangible_things/_tangible_thing.json.jbuilder", ":#{name},", RUBY_NEW_FIELDS_HOOK, prepend: true, suppress_could_not_find: true)
-          scaffold_add_line_to_file("./app/serializers/api/v1/scaffolding/completely_concrete/tangible_thing_serializer.rb", ":#{name},", RUBY_NEW_FIELDS_HOOK, prepend: true)
+          scaffold_add_line_to_file("./app/serializers/api/v1/scaffolding/completely_concrete/tangible_thing_serializer.rb", ":#{name},", RUBY_NEW_FIELDS_HOOK, prepend: true) unless type == "file_field"
 
           assertion = case type
           when "date_field"
@@ -1047,13 +1035,31 @@ class Scaffolding::Transformer
             "assert_equal DateTime.parse(tangible_thing_data['#{name}']), tangible_thing.#{name}"
           when "file_field"
             # TODO: If we want to use Cloudinary to handle our files, we should make sure we're getting a URL.
-            "assert_equal tangible_thing_data['#{name}']['record']['id'], tangible_thing.#{name}.record.id"
+            "assert tangible_thing_data['#{name}'].match?('foo.txt') unless response.status == 201"
           else
             "assert_equal tangible_thing_data['#{name}'], tangible_thing.#{name}"
           end
           scaffold_add_line_to_file("./test/controllers/api/v1/scaffolding/completely_concrete/tangible_things_endpoint_test.rb", assertion, RUBY_NEW_FIELDS_HOOK, prepend: true)
         end
 
+        # File fields are handled in a specific way when using the jsonapi-serializer.
+        if type == "file_field"
+          file_name = "./app/serializers/api/v1/scaffolding/completely_concrete/tangible_thing_serializer.rb"
+          content = <<~RUBY
+            attribute :#{name} do |object|
+              rails_blob_path(object.#{name}, disposition: "attachment", only_path: true) if object.#{name}.attached?
+            end
+
+          RUBY
+          hook = RUBY_FILES_HOOK
+          scaffold_add_line_to_file(file_name, content, hook, prepend: true)
+
+          # We also want to make sure we attach the dummy file in the endpoint test on setup
+          file_name = "./test/controllers/api/v1/scaffolding/completely_concrete/tangible_things_endpoint_test.rb"
+          content = "@#{child.underscore}.#{name} = Rack::Test::UploadedFile.new(\"test/support/foo.txt\")"
+          scaffold_add_line_to_file(file_name, content, hook, prepend: true)
+        end
+
         # scaffold_add_line_to_file("./test/controllers/api/v1/scaffolding/completely_concrete/tangible_things_controller_test.rb", "assert_equal tangible_thing_attributes['#{name.gsub('_', '-')}'], tangible_thing.#{name}", RUBY_NEW_FIELDS_HOOK, prepend: true)
 
         if attribute_assignment
@@ -1295,19 +1301,12 @@ class Scaffolding::Transformer
 
       add_has_many_association
 
-      # Adds file attachment to factory
-      attributes.each do |attribute|
-        attribute_name, partial_type = attribute.split(":")
-        if partial_type == "file_field"
-          content = "#{attribute_name} { Rack::Test::UploadedFile.new(\"test/support/foo.txt\") }"
-          scaffold_replace_line_in_file("./test/factories/scaffolding/completely_concrete/tangible_things.rb", content, "#{attribute_name} { nil }")
-        end
-      end
-
       if class_names_transformer.belongs_to_needs_class_definition?
         scaffold_replace_line_in_file("./app/models/scaffolding/completely_concrete/tangible_thing.rb", transform_string("belongs_to :absolutely_abstract_creative_concept, class_name: \"Scaffolding::AbsolutelyAbstract::CreativeConcept\"\n"), transform_string("belongs_to :absolutely_abstract_creative_concept\n"))
       end
 
+      update_models_abstract_class
+
       # add user permissions.
       add_ability_line_to_roles_yml
     end
@@ -1318,7 +1317,12 @@ class Scaffolding::Transformer
 
     # add children to the show page of their parent.
     unless cli_options["skip-parent"] || parent == "None"
-      scaffold_add_line_to_file("./app/views/account/scaffolding/absolutely_abstract/creative_concepts/show.html.erb", "<%= render 'account/scaffolding/completely_concrete/tangible_things/index', tangible_things: @creative_concept.completely_concrete_tangible_things, hide_back: true %>", "<%# 🚅 super scaffolding will insert new children above this line. %>", prepend: true)
+      scaffold_add_line_to_file(
+        "./app/views/account/scaffolding/absolutely_abstract/creative_concepts/show.html.erb",
+        code_for_child_on_parent_show_page || "<%= render 'account/scaffolding/completely_concrete/tangible_things/index', tangible_things: @creative_concept.completely_concrete_tangible_things, hide_back: true %>",
+        "<%# 🚅 super scaffolding will insert new children above this line. %>",
+        prepend: true
+      )
     end
 
     unless cli_options["skip-model"]
@@ -1405,7 +1409,7 @@ class Scaffolding::Transformer
         add_additional_step :yellow, "We weren't able to automatically add your `#{routes_namespace}` routes for you. In theory this should be very rare, so if you could reach out on Slack, you could probably provide context that will help us fix whatever the problem was. In the meantime, to add the routes manually, we've got a guide at https://blog.bullettrain.co/nested-namespaced-rails-routing-examples/ ."
       end
 
-      routes_manipulator.write
+      Scaffolding::FileManipulator.write("config/routes.rb", routes_manipulator.lines)
     end
 
     unless cli_options["skip-parent"]
@@ -1416,13 +1420,18 @@ class Scaffolding::Transformer
           icon_name = cli_options["sidebar"]
         else
           puts ""
-          puts "Hey, models that are scoped directly off of a Team (or nothing) are eligible to be added to the sidebar. Do you want to add this resource to the sidebar menu? (y/N)"
+          puts "Hey, models that are scoped directly off of a Team (or nothing) are eligible to be added to the sidebar."
+          puts "Do you want to add this resource to the sidebar menu? (y/N)"
           response = $stdin.gets.chomp
           if response.downcase[0] == "y"
             puts ""
-            puts "OK, great! Let's do this! By default these menu items appear with a puzzle piece, but after you hit enter I'll open two different pages where you can view other icon options. When you find one you like, hover your mouse over it and then come back here and and enter the name of the icon you want to use. (Or hit enter to skip this step.)"
+            puts "OK, great! Let's do this! By default these menu items appear as a #{font_awesome? ? "puzzle piece" : "gift icon"},"
+            puts "but after you hit enter I'll open #{font_awesome? ? "two different pages" : "a page"} where you can view other icon options."
+            puts "When you find one you like, hover your mouse over it and then come back here and"
+            puts "enter the name of the icon you want to use."
+            puts "(Or hit enter when choosing to skip this step.)"
             $stdin.gets.chomp
-            if `which open`.present?
+            if (TerminalCommands.macosx? && `which open`.present?) || TerminalCommands.linux?
               TerminalCommands.open_file_or_link("https://themify.me/themify-icons")
               if font_awesome?
                 TerminalCommands.open_file_or_link("https://fontawesome.com/icons?d=gallery&s=light")
@@ -1437,8 +1446,18 @@ class Scaffolding::Transformer
               puts ""
             end
             puts ""
-            puts "Did you find an icon you wanted to use? Enter the full CSS class here (e.g. 'ti ti-world'#{" or 'fal fa-puzzle-piece'" if font_awesome?}) or hit enter to just use the puzzle piece:"
-            icon_name = $stdin.gets.chomp
+
+            loop do
+              puts "Did you find an icon you wanted to use?"
+              puts "Enter the full CSS class here (e.g. 'ti ti-world'#{" or 'fal fa-puzzle-piece'" if font_awesome?}) or hit enter to just use the #{font_awesome? ? "puzzle piece" : "gift icon"}:"
+              icon_name = $stdin.gets.chomp
+              unless icon_name.match?(/ti\s.*/) || icon_name.match?(/fal\s.*/) || icon_name.strip.empty?
+                puts ""
+                puts "Please enter the full CSS class or hit enter."
+                next
+              end
+              break
+            end
             puts ""
             unless icon_name.length > 0 || icon_name.downcase == "y"
               icon_name = "fal fa-puzzle-piece ti ti-gift"

data/lib/scaffolding.rb

@@ -8,6 +8,7 @@ module Scaffolding
       "date_and_time_field",
       "date_field",
       "email_field",
+      "emoji_field",
       "file_field",
       "options",
       "password_field",

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-super_scaffolding
 version: !ruby/object:Gem::Version
-  version: 1.0.34
+  version: 1.0.37
 platform: ruby
 authors:
 - Andrew Culver
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-07-20 00:00:00.000000000 Z
+date: 2022-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: standard
@@ -166,6 +166,7 @@ files:
 - lib/scaffolding.rb
 - lib/scaffolding/block_manipulator.rb
 - lib/scaffolding/class_names_transformer.rb
+- lib/scaffolding/file_manipulator.rb
 - lib/scaffolding/oauth_providers.rb
 - lib/scaffolding/routes_file_manipulator.rb
 - lib/scaffolding/script.rb
@@ -177,7 +178,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/bullet-train-co/bullet_train-super_scaffolding
   source_code_uri: https://github.com/bullet-train-co/bullet_train-super_scaffolding
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -192,8 +193,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
-signing_key:
+rubygems_version: 3.1.6
+signing_key: 
 specification_version: 4
 summary: Bullet Train Super Scaffolding
 test_files: []