hot-glue 0.0.8 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ae1088aa0a809a10fb6332fa7ca062e1af6aa39a1c8541dabdf4d8451c5ba04d
-  data.tar.gz: a3ff6d5d8a171e1d1d361df692f6255f855720609e692c21573d6311fbeb41e2
+  metadata.gz: 831970726374d97ef6b5380d122033384777a520ba5fcaeebc9ee79717b82ef1
+  data.tar.gz: f45f7a48e78a6bfd62973a11d5b55b24d7ba7a7979855940429ea00398e3ef1f
 SHA512:
-  metadata.gz: f65f042a47123cb7a7f69222ddf6618ca085f8cb38bf372179f3bb3da6e47fb07cb383ecba4e0c22642d775db004c0ab111fc99667f575fde62869f7d4c4efba
-  data.tar.gz: e6c36f5bb8a516ee2e74250000174eede61f39c38c46fb74ac963cfa958ac2c5ba2bd3323832b1359ef452e532b7e01b24e04ff68bb67b28df781ccf04e958a4
+  metadata.gz: 23fa8de6cc6460b36d3a867c502a708c6ea825575f25240d28f11f0c0f5a333fc6fe583fa95668d517555bbef96177407432a8c98ca41a29caeca82fa37e765a
+  data.tar.gz: a1c9db09d05ab741a85906e31b9cc5336c73f4108f550edbd14166d00f0623b65b1ea8c97a8b9c95c39de55a72bc006cb657ab3d5cddff17258983eb26183ccc

data/.github/FUNDING.yml

@@ -1,2 +1,2 @@
 github: [jasonfb]
-custom: ["https://jfb.teachable.com"]
+custom: ["https://www.instagram.com/jfbcodes/"]

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    hot-glue (0.0.7)
+    hot-glue (0.0.8)
       ffaker (~> 2.16)
       haml-rails (~> 2.0)
       kaminari (~> 1.2)
@@ -85,7 +85,7 @@ GEM
     erubi (1.10.0)
     erubis (2.7.0)
     ffaker (2.18.0)
-    ffi (1.14.2)
+    ffi (1.15.0)
     globalid (0.4.2)
       activesupport (>= 4.2.0)
     haml (5.2.1)

data/README.md

@@ -313,6 +313,8 @@ If you do not want inline editing of your list items but instead to fall back to
 
 # VERSION HISTORY
 
+#### 2021-03-24 - v0.0.9 - fixes in the automatic field label detection; cleans up junk in spec output
+
 #### 2021-03-21 - v0.0.8 - show only flag; more specific spec coverage in generator spec
 
 #### 2021-03-20 - v0.0.7 - adds lots of spec coverage; cleans up generated cruft code on each run;  adds no-delete, no-create; a working --big-edit with basic data-turbo false to disable inline editing

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -55,7 +55,7 @@ module HotGlue
     class_option :include, type: :string, default: ""
     class_option :god, type: :boolean, default: false
     class_option :gd, type: :boolean, default: false # alias for god
-    class_option :spacs_only, type: :boolean, default: false
+    class_option :specs_only, type: :boolean, default: false
     class_option :no_specs, type: :boolean, default: false
     class_option :no_delete, type: :boolean, default: false
     class_option :no_create, type: :boolean, default: false
@@ -104,6 +104,7 @@ module HotGlue
 
       @god = options['god'] || options['gd'] || false
       @specs_only = options['specs_only'] || false
+
       @no_specs = options['no_specs'] || false
       @no_delete = options['no_delete'] || false
 
@@ -208,18 +209,16 @@ module HotGlue
             end
 
             assoc_class = eval(assoc.class_name)
-            if assoc_class.respond_to?(:name)
-              # display_column = "name"
-            elsif assoc_class.respond_to?(:to_label)
-              # display_column = "to_label"
-            elsif assoc_class.respond_to?(:full_name)
-              # display_column = "full_name"
-            elsif assoc_class.respond_to?(:display_name)
-              # display_column = "display_name"
-            elsif assoc_class.respond_to?(:email)
-              # display_column = "email"
+
+            name_list = [:name, :to_label, :full_name, :display_name, :email]
+
+
+            if name_list.collect{ |field|
+              assoc_class.column_names.include?(field.to_s) ||  assoc_class.instance_methods.include?(field)
+            }.any?
+              # do nothing here
             else
-              exit_message= "*** Oops: Can't find any column to use as the display label for the #{assoc.name.to_s} association on the #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, or 5) email directly on your #{assoc.class_name} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
+              exit_message= "*** Oops: Missing a label for #{assoc.class_name.upcase}. Can't find any column to use as the display label for the #{assoc.name.to_s} association on the #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, or 5) email directly on your #{assoc.class_name.upcase} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
               raise(HotGlue::Error,exit_message)
             end
           end
@@ -729,18 +728,20 @@ module HotGlue
       me = eval(singular_class)
 
       @display_class ||=
-        if me.respond_to?("name")
+        if me.column_names.include?("name")
+          # note that all class object respond_to?(:name) with the name of their own class
+          # this one is unique
           "name"
-        elsif me.respond_to?("to_label")
+        elsif me.column_names.include?("to_label") || me.instance_methods(false).include?(:to_label)
           "to_label"
-        elsif me.respond_to?("full_name")
+        elsif me.column_names.include?("full_name") || me.instance_methods(false).include?(:full_name)
           "full_name"
-        elsif me.respond_to?("display_name")
+        elsif me.column_names.include?("display_name") || me.instance_methods(false).include?(:display_name)
           "display_name"
-        elsif me.respond_to?("email")
+        elsif me.column_names.include?("email") || me.instance_methods(false).include?(:email)
           "email"
-        elsif me.respond_to?("number")
-          display_column = "number"
+        elsif me.column_names.include?("number") || me.instance_methods(false).include?(:number)
+          "number"
 
         else
           exit_message = "*** Oops: Can't find any column to use as the display label on #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, 5) email, or 6) number directly on your #{singular_class} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
@@ -769,7 +770,8 @@ module HotGlue
 
 
   def paginate
-    "= paginate #{plural}"
+    "- if #{plural}.respond_to?(:total_pages)
+      = paginate #{plural}"
   end
 
   private # thor does something fancy like sending the class all of its own methods during some strange run sequence

data/lib/generators/hot_glue/templates/system_spec.rb.erb

@@ -7,102 +7,46 @@ describe "interaction for <%= controller_class_name %>", type: :feature do
 <%= objest_nest_factory_setup %>
 
   before(:each) do
-    @request.env["devise.mapping"] = Devise.mappings[:account]
 
-    sign_in <%= @auth %>, scope: :<%= @auth %>
   end
 
   describe "index" do
-    it "should respond" do
-      get :index, xhr: true, format: 'js', params: {
-          <%= objest_nest_params_by_id_for_specs %>
-      }
+    it "should show me the list" do
+
     end
   end
 
   describe "new" do
     it "should show form" do
-      get :new, xhr: true, format: 'js', params: {
-          <%= objest_nest_params_by_id_for_specs %>
-      }
+
     end
   end
 
   describe "create" do
     it "should create a new <%= singular %>" do
-      expect {
-        post :create, xhr: true, format: 'js', params: {
-          <%= (@nested_args.empty? ? "" : objest_nest_params_by_id_for_specs + ",") %>
-            <%= singular %>: {
-                <%= columns_spec_with_sample_data %>
-        }}
-      }.to change { <%= @singular_class %>.all.count }.by(1)
-      assert_response :ok
-    end
 
-    # it "should not create if there are errors" do
-    #   post :create, xhr: true, format: 'js',  params: {id: <%= singular %>.id,
-    #                                                    <%= singular %>: {skin_id: nil}}
-    #
-    #   expect(controller).to set_flash.now[:alert].to(/Oops, your <%= singular %> could not be saved/)
-    # end
+    end
   end
 
   describe "edit" do
     it "should return an editable form" do
-      get :edit, xhr: true, format: 'js',  params: {
-          <%= (@nested_args.empty? ? "" : objest_nest_params_by_id_for_specs + ",") %>
-          id: <%= singular %>.id
-      }
-      assert_response :ok
+
     end
   end
 
   describe "show" do
     it "should return a view form" do
-      get :show, xhr: true, format: 'js',  params: {
-          <%= (@nested_args.empty? ? "" : objest_nest_params_by_id_for_specs + ",") %>
-          id: <%= singular %>.id
-        }
-      assert_response :ok
+
     end
   end
 
   describe "update" do
     it "should update" do
-      put :update, xhr: true, format: 'js',
-          params: {
-            <%= (@nested_args.empty? ? "" : objest_nest_params_by_id_for_specs + ",") %>
-            id: <%= singular %>.id,
-            <%= singular %>: {
-                <%= columns_spec_with_sample_data %>
-            }}
-
-      assert_response :ok
     end
-
-    # it "should not update if invalid" do
-    #   put :update, xhr: true, format: 'js',
-    #       params: {
-    #         id: <%= singular %>.id,
-    #         <%= singular %>: {
-    #           <%= columns_spec_with_sample_data %>
-    #       }}
-    #
-    #   assert_response :ok
-    #
-    #   expect(controller).to set_flash.now[:alert].to(/Oops, your <%= singular %> could not be saved/)
-    # end
   end
 
-  describe "#destroy" do
+  describe "destroy" do
     it "should destroy" do
-      post :destroy, format: 'js', params: {
-          <%= (@nested_args.empty? ? "" : objest_nest_params_by_id_for_specs + ",") %>
-          id: <%= singular %>.id
-      }
-      assert_response :ok
-      expect(<%= @singular_class %>.count).to be(0)
     end
   end
 end

data/lib/hotglue/version.rb

@@ -1,3 +1,3 @@
 module HotGlue
-  VERSION = '0.0.8'
+  VERSION = '0.0.9'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hot-glue
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
 platform: ruby
 authors:
 - Jason Fleetwood-Boldt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-21 00:00:00.000000000 Z
+date: 2021-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -149,8 +149,8 @@ licenses:
 - Nonstandard
 metadata:
   source_code_uri: https://github.com/jasonfb/hot-glue
-  documentation_uri: https://jfb.teachable.com/p/gems-by-jason
-  homepage_uri: https://blog.jasonfleetwoodboldt.com/hot-glue/
+  documentation_uri: https://www.instagram.com/jfbcodes/
+  homepage_uri: https://jasonfleetwoodboldt.com/hot-glue/
 post_install_message: |
   ---------------------------------------------
   Welcome to Hot Glue - A Scaffold Building Companion for Hotwire + Turbo-Rails
@@ -159,11 +159,11 @@ post_install_message: |
 
       * Build plug-and-play scaffolding mixing HAML with the power of Hotwire and Turbo-Rails
       * Automatically Reads Your Models (make them before building your scaffolding!)
-      * Excellent for CRUD, lists with pagination, searching, sorting.
-      * Great for prototyping very.
-      * Plays nicely with Devise, Kaminari, Haml-Rails, Rspec.
-      * Create specs automatically along with the controllers.
-      * Nest your routes model-by-model for built-in poor man's authentication
+      * Excellent for CRUD, lists with pagination (coming soon: searching & sorting)
+      * Great for prototyping.
+      * Plays nicely with Devise, Kaminari, Haml-Rails, Rspec, FontAwesome
+      * Create specs automatically along with the generated controllers.
+      * Nest your routes model-by-model for built-in poor man's authentication.
       * Throw the scaffolding away when your app is ready to graduate to its next phase.
 
   see README for complete instructions.