hot-glue 0.0.5 → 0.1.0

data/Rakefile

@@ -14,7 +14,7 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-APP_RAKEFILE = File.expand_path("test/dummy/Rakefile", __dir__)
+APP_RAKEFILE = File.expand_path("spec/dummy/Rakefile", __dir__)
 load 'rails/tasks/engine.rake'
 
 load 'rails/tasks/statistics.rake'

data/bin/rails

@@ -3,8 +3,8 @@
 # installed from the root of your application.
 
 ENGINE_ROOT = File.expand_path('..', __dir__)
-ENGINE_PATH = File.expand_path('../lib/common_core_js/engine', __dir__)
-APP_PATH = File.expand_path('../test/dummy/config/application', __dir__)
+ENGINE_PATH = File.expand_path('../lib/hotglue/engine', __dir__)
+APP_PATH = File.expand_path('../spec/dummy/application', __dir__)
 
 # Set up gems listed in the Gemfile.
 ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__)

data/config/database.yml

@@ -0,0 +1,11 @@
+development:
+  adapter: sqlite3
+  database: ../spec/dummy/db/development.sqlite3
+  pool: 5
+  timeout: 5000
+
+test:
+  adapter: sqlite3
+  database: ../spec/dummy/db/test.sqlite3
+  pool: 5
+  timeout: 5000

data/db/migrate/20210306212711_create_abcs.rb

@@ -0,0 +1,11 @@
+class CreateAbcs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :abcs do |t|
+      t.integer :xxx
+      t.string :yyy
+      t.integer :def_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223300_create_defs.rb

@@ -0,0 +1,9 @@
+class CreateDefs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :defs do |t|
+      t.integer :user_id
+      t.string :name
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223305_create_ghis.rb

@@ -0,0 +1,9 @@
+class CreateGhis < ActiveRecord::Migration[6.1]
+  def change
+    create_table :ghis do |t|
+      t.integer :def_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223309_create_jkls.rb

@@ -0,0 +1,9 @@
+class CreateJkls < ActiveRecord::Migration[6.1]
+  def change
+    create_table :jkls do |t|
+      t.integer :hgi_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223701_devise_create_users.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+class DeviseCreateUsers < ActiveRecord::Migration[6.1]
+  def change
+    create_table :users do |t|
+      ## Database authenticatable
+      t.string :email,              null: false, default: ""
+      t.string :encrypted_password, null: false, default: ""
+
+      ## Recoverable
+      t.string   :reset_password_token
+      t.datetime :reset_password_sent_at
+
+      ## Rememberable
+      t.datetime :remember_created_at
+
+      ## Trackable
+      # t.integer  :sign_in_count, default: 0, null: false
+      # t.datetime :current_sign_in_at
+      # t.datetime :last_sign_in_at
+      # t.string   :current_sign_in_ip
+      # t.string   :last_sign_in_ip
+
+      ## Confirmable
+      # t.string   :confirmation_token
+      # t.datetime :confirmed_at
+      # t.datetime :confirmation_sent_at
+      # t.string   :unconfirmed_email # Only if using reconfirmable
+
+      ## Lockable
+      # t.integer  :failed_attempts, default: 0, null: false # Only if lock strategy is :failed_attempts
+      # t.string   :unlock_token # Only if unlock strategy is :email or :both
+      # t.datetime :locked_at
+
+
+      t.timestamps null: false
+    end
+
+    add_index :users, :email,                unique: true
+    add_index :users, :reset_password_token, unique: true
+    # add_index :users, :confirmation_token,   unique: true
+    # add_index :users, :unlock_token,         unique: true
+  end
+end

data/db/migrate/20210306225506_create_xyzs.rb

@@ -0,0 +1,9 @@
+class CreateXyzs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :xyzs do |t|
+      t.integer :nothing_id
+
+      t.timestamps
+    end
+  end
+end

data/db/schema.rb

@@ -0,0 +1,60 @@
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# This file is the source Rails uses to define your schema when running `bin/rails
+# db:schema:load`. When creating a new database, `bin/rails db:schema:load` tends to
+# be faster and is potentially less error prone than running all of your
+# migrations from scratch. Old migrations may fail to apply correctly if those
+# migrations use external dependencies or application code.
+#
+# It's strongly recommended that you check this file into your version control system.
+
+ActiveRecord::Schema.define(version: 2021_03_06_225506) do
+
+  create_table "abcs", force: :cascade do |t|
+    t.integer "xxx"
+    t.string "yyy"
+    t.integer "def_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "defs", force: :cascade do |t|
+    t.integer "user_id"
+    t.string "name"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "ghis", force: :cascade do |t|
+    t.integer "def_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "jkls", force: :cascade do |t|
+    t.integer "hgi_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "users", force: :cascade do |t|
+    t.string "email", default: "", null: false
+    t.string "encrypted_password", default: "", null: false
+    t.string "reset_password_token"
+    t.datetime "reset_password_sent_at"
+    t.datetime "remember_created_at"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+    t.index ["email"], name: "index_users_on_email", unique: true
+    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
+  end
+
+  create_table "xyzs", force: :cascade do |t|
+    t.integer "nothing_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+end

data/lib/generators/hot_glue/install_generator.rb

@@ -10,9 +10,9 @@ module HotGlue
 
     def initialize(*args) #:nodoc:
       super
-      # copy_file "common_core.js", "app/javascript/common_core.js"
-      # copy_file "common_core.scss", "app/assets/stylesheets/common_core.scss"
-      copy_file "_flash_notices.haml", "app/views/layouts/_flash_notices.haml"
+      # copy_file "hot_glue.js", "#{'spec/dummy/' if Rails.env.test?}app/javascript/hot_glue.js"
+      # copy_file "hot_glue.scss", "#{'spec/dummy/' if Rails.env.test?}app/assets/stylesheets/hot_glue.scss"
+      copy_file "_flash_notices.haml", "#{'spec/dummy/' if Rails.env.test?}app/views/layouts/_flash_notices.haml"
     end
   end
 end

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -3,7 +3,35 @@ require 'ffaker'
 
 
 module HotGlue
+  class Error < StandardError
+  end
+
+
+
+
+
   module GeneratorHelper
+    def derrive_reference_name thing_as_string
+      assoc_class = eval(thing_as_string)
+
+      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"
+      else
+        raise("this should have been caught by the checker in the initializer")
+        # puts "*** 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)"
+      end
+      display_column
+    end
+
+
     def text_area_output(col, field_length, col_identifier )
       lines = field_length % 40
       if lines > 5
@@ -46,12 +74,16 @@ module HotGlue
     class_option :auth, type: :string, default: nil
     class_option :auth_identifier, type: :string, default: nil
     class_option :exclude, type: :string, default: ""
+    class_option :include, type: :string, default: ""
     class_option :god, type: :boolean, default: false
-    class_option :spacs_only, type: :boolean, default: false
+    class_option :gd, type: :boolean, default: false # alias for god
+    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
     class_option :no_paginate, type: :boolean, default: false
+    class_option :big_edit, type: :boolean, default: false
+    class_option :show_only, type: :string, default: ""
 
     def initialize(*meta_args) #:nodoc:
       super
@@ -59,55 +91,61 @@ module HotGlue
       begin
         object = eval(class_name)
       rescue StandardError => e
-        puts "*** Oops: It looks like there is no object for #{class_name}. Please define the object + database table first."
-        exit
+        message = "*** Oops: It looks like there is no object for #{class_name}. Please define the object + database table first."
+        raise(HotGlue::Error, message)
+      end
+
+      if @specs_only && @no_specs
+        raise(HotGlue::Error, "*** Oops: You seem to have specified both the --specs-only flag and --no-specs flags. this doesn't make any sense, so I am aborting. sorry.")
       end
-      args = meta_args[0]
-      @singular = args.first.tableize.singularize # should be in form hello_world
 
 
+      args = meta_args[0]
+      @singular = args.first.tableize.singularize # should be in form hello_world
       @plural = options['plural'] || @singular + "s" # supply to override; leave blank to use default
       @auth = options['auth'] || "current_user"
-      @auth_identifier = options['auth'] || (!@auth.nil? && @auth.gsub("current_", "")) || nil
+      @auth_identifier = options['auth_identifier'] || (!@auth.nil? && @auth.gsub("current_", "")) || nil
 
 
-      @nest = options['auth'] || nil
+      @nest = (!options['nest'].empty? && options['nest']) || nil
       @namespace = options['namespace'] || nil
 
       @singular_class = @singular.titleize.gsub(" ", "")
-
       @exclude_fields = []
+      @exclude_fields += options['exclude'].split(",").collect(&:to_sym)
 
+      if !options['include'].empty?
+        @include_fields = []
+        @include_fields += options['include'].split(",").collect(&:to_sym)
+      end
 
 
-      @exclude_fields += options['exclude'].split(",").collect(&:to_sym)
+      @show_only = []
+      if !options['show_only'].empty?
+        @show_only += options['show_only'].split(",").collect(&:to_sym)
+      end
 
       auth_assoc = @auth.gsub("current_","")
 
-
       @god = options['god'] || options['gd'] || false
-      @specs_only = options['specs-only'] || false
-      @no_specs = options['no-specs'] || false
-      @no_delete = options['no-delete'] || false
-      @no_create = options['no-create'] || false
-      @no_paginate = options['no-paginate'] || false
+      @specs_only = options['specs_only'] || false
 
-      if @specs_only && @no_specs
-        puts "*** Oops: You seem to have specified both the --specs-only flag and --no-specs flags. this doesn't make any sense, so I am aborting. sorry."
-        exit
-      end
+      @no_specs = options['no_specs'] || false
+      @no_delete = options['no_delete'] || false
+
+      @no_create = options['no_create'] || false
+      @no_paginate = options['no_paginate'] || false
+      @big_edit = options['big_edit']
 
       if @god
         @auth = nil
       end
 
-
       # when in self auth, the object is the same as the authenticated object
       if @auth && auth_identifier == @singular
         @self_auth = true
       end
 
-
       @nested_args = []
       if !@nest.nil?
         @nested_args = @nest.split("/")
@@ -134,42 +172,101 @@ module HotGlue
         end
       end
 
-                                 # create the columns
+
       if !@object_owner_sym.empty?
         auth_assoc_field = auth_assoc + "_id"
         assoc = eval("#{singular_class}.reflect_on_association(:#{@object_owner_sym})")
+
         if assoc
           ownership_field = assoc.name.to_s + "_id"
+        elsif !@nest
+            exit_message = "*** Oops: It looks like is no association from current_#{@object_owner_sym} to a class called #{@singular_class}. If your user is called something else, pass with flag auth=current_X where X is the model for your users as lowercase. Also, be sure to implement current_X as a method on your controller. (If you really don't want to implement a current_X on your controller and want me to check some other method for your current user, see the section in the docs for auth_identifier.) To make a controller that can read all records, specify with --god."
+
         else
-          if @auth
-            puts "*** Oops: It looks like is no association from current_#{@object_owner_sym} to a class called #{singular_class}. If your user is called something else, pass with flag auth=current_X where X is the model for your users as lowercase. Also, be sure to implement current_X as a method on your controller. (If you really don't want to implement a current_X on your controller and want me to check some other method for your current user, see the section in the docs for auth_identifier.) To make a controller that can read all records, specify with --god."
+          if @god
+
+            exit_message= "*** Oops:  god mode could not find the association(?). something is wrong."
           else
-            puts "*** Oops:  god mode could not find the association(?). something is wrong."
+            @auth_check = "current_user"
+            @nested_args.each do |arg|
+
+              if !@auth_check.method("#{arg}s")
+                exit_message= "*** Oops:  your nesting chain does not have a assocation for #{arg}s on #{@auth_check}  something is wrong."
+              end
+              byebug
+              puts ""
+            end
           end
-          exit
+
+          raise(HotGlue::Error, exit_message)
         end
       end
 
-      @exclude_fields.push :id, :created_at, :updated_at, :encrypted_password,
-                         :reset_password_token,
-                         :reset_password_sent_at, :remember_created_at,
-                         :confirmation_token, :confirmed_at,
-                         :confirmation_sent_at, :unconfirmed_email
+      if !@include_fields
+        @exclude_fields.push :id, :created_at, :updated_at, :encrypted_password,
+                             :reset_password_token,
+                             :reset_password_sent_at, :remember_created_at,
+                             :confirmation_token, :confirmed_at,
+                             :confirmation_sent_at, :unconfirmed_email
+
+        @exclude_fields.push(auth_assoc_field.to_sym) if !auth_assoc_field.nil?
+        @exclude_fields.push(ownership_field.to_sym) if !ownership_field.nil?
 
-      @exclude_fields.push(auth_assoc_field.to_sym) if !auth_assoc_field.nil?
-      @exclude_fields.push(ownership_field.to_sym) if !ownership_field.nil?
 
-      begin
         @columns = object.columns.map(&:name).map(&:to_sym).reject{|field| @exclude_fields.include?(field) }
-      rescue StandardError => e
-        puts "Ooops... #{e} it looks like is no object for #{class_name}. Please create the database table with fields first. "
-        exit
+
+      else
+        @columns = object.columns.map(&:name).map(&:to_sym).reject{|field| !@include_fields.include?(field) }
+
       end
 
+      @columns.each do |col|
+        if col.to_s.starts_with?("_")
+          @show_only << col
+        end
+
+        if object.columns_hash[col.to_s].type == :integer
+          if col.to_s.ends_with?("_id")
+            # guess the association name label
+            assoc_name = col.to_s.gsub("_id","")
+            assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
+
+
+            begin
+              eval(assoc.class_name)
+            rescue NameError => e
+              exit_message = "*** Oops: The model #{singular_class} is missing an association for #{assoc_name} or the model doesn't exist. TODO: Please implement a model for #{assoc_name.titlecase}; your model #{singular_class.titlecase} should have_many :#{assoc_name}s.  To make a controller that can read all records, specify with --god."
+              raise(HotGlue::Error, exit_message)
 
+            end
 
+
+            if assoc.nil?
+              exit_message= "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
+              raise(HotGlue::Error,exit_message)
+            end
+
+            assoc_class = eval(assoc.class_name)
+
+            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: 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
+        end
+      end
     end
 
+
+
+    #
     def formats
       [format]
     end
@@ -182,17 +279,22 @@ module HotGlue
       @default_colspan = @columns.size
 
       unless @specs_only
-        template "controller.rb", File.join("app/controllers#{namespace_with_dash}", "#{plural}_controller.rb")
-        if @namespace &&  defined?(controller_descends_from) == nil
-          template "base_controller.rb", File.join("app/controllers#{namespace_with_dash}", "base_controller.rb")
+        template "controller.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}app/controllers#{namespace_with_dash}", "#{plural}_controller.rb")
+        if @namespace
+          begin
+            eval(controller_descends_from)
+            puts "   skipping   base controller #{controller_descends_from}"
+          rescue NameError => e
+            template "base_controller.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}app/controllers#{namespace_with_dash}", "base_controller.rb")
+          end
         end
       end
 
       unless @no_specs
-        template "controller_spec.rb", File.join("spec/controllers#{namespace_with_dash}", "#{plural}_controller_spec.rb")
+        template "system_spec.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}spec/system#{namespace_with_dash}", "#{plural}_behavior_spec.rb")
       end
 
-      template "_errors.haml", File.join("app/views#{namespace_with_dash}", "_errors.haml")
+      template "_errors.haml", File.join("#{'spec/dummy/' if Rails.env.test?}app/views#{namespace_with_dash}", "_errors.haml")
     end
 
     def list_column_headings
@@ -268,9 +370,9 @@ module HotGlue
     end
 
 
-    def path_helper_full
-      "#{@namespace+"_" if @namespace}#{(@nested_args.join("_") + "_" if @nested_args.any?)}#{singular}_path"
-    end
+    # def path_helper_full
+    #   "#{@namespace+"_" if @namespace}#{(@nested_args.join("_") + "_" if @nested_args.any?)}#{singular}_path"
+    # end
 
     def path_helper_args
       if @nested_args.any?
@@ -382,19 +484,22 @@ module HotGlue
     end
 
 
+
+
+
     def copy_view_files
       return if @specs_only
       haml_views.each do |view|
         formats.each do |format|
           filename = cc_filename_with_extensions(view, "haml")
-          template filename, File.join("app/views#{namespace_with_dash}", controller_file_path, filename)
+          template filename, File.join("#{'spec/dummy/' if Rails.env.test?}app/views#{namespace_with_dash}", controller_file_path, filename)
         end
       end
 
       turbo_stream_views.each do |view|
         formats.each do |format|
           filename = cc_filename_with_extensions(view, 'turbo_stream.haml')
-          template filename, File.join("app/views#{namespace_with_dash}", controller_file_path, filename)
+          template filename, File.join("#{'spec/dummy/' if Rails.env.test?}app/views#{namespace_with_dash}", controller_file_path, filename)
         end
       end
     end
@@ -416,13 +521,21 @@ module HotGlue
     end
 
     def haml_views
-      res =  %w(index edit new _form _new_form _line _list _new_button _show _errors)
+      res =  %w(index edit _form _line _list _show _errors)
+
+      unless @no_create
+        res += %w(new _new_form _new_button)
+      end
 
       res
     end
 
     def turbo_stream_views
-      res = %w(create destroy edit update)
+      res = %w(create  edit update)
+      unless @no_delete
+        res << 'destroy'
+      end
+      res
     end
 
     def handler
@@ -443,6 +556,16 @@ module HotGlue
       col_spaces_prepend = "    "
 
       res = @columns.map { |col|
+
+        if @show_only.include?(col)
+
+          "#{col_identifier}{class: \"form-group \#{'alert-danger' if #{singular}.errors.details.keys.include?(:#{col.to_s})}\"}
+    = @#{singular}.#{col.to_s}
+    %label.form-text
+      #{col.to_s.humanize}\n"
+        else
+
+
         type = eval("#{singular_class}.columns_hash['#{col}']").type
         limit = eval("#{singular_class}.columns_hash['#{col}']").limit
         sql_type = eval("#{singular_class}.columns_hash['#{col}']").sql_type
@@ -451,35 +574,17 @@ module HotGlue
         when :integer
           # look for a belongs_to on this object
           if col.to_s.ends_with?("_id")
-            # guess the association name label
-
-
             assoc_name = col.to_s.gsub("_id","")
             assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
             if assoc.nil?
-              puts "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
+              exit_message= "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
               exit
             end
+            display_column = derrive_reference_name(assoc.class_name)
 
 
-            assoc_class = eval(assoc.class_name)
-
-            if assoc_class.column_names.include?("name")
-              display_column = "name"
-            elsif assoc_class.column_names.include?("to_label")
-              display_column = "to_label"
-            elsif assoc_class.column_names.include?("full_name")
-              display_column = "full_name"
-            elsif assoc_class.column_names.include?("display_name")
-              display_column = "display_name"
-            elsif assoc_class.column_names.include?("email")
-              display_column = "email"
-            else
-              puts "*** 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)"
-            end
-
             "#{col_identifier}{class: \"form-group \#{'alert-danger' if #{singular}.errors.details.keys.include?(:#{assoc_name.to_s})}\"}
-#{col_spaces_prepend}= f.collection_select(:#{col.to_s}, #{assoc_class}.all, :id, :#{display_column}, {prompt: true, selected: @#{singular}.#{col.to_s} }, class: 'form-control')
+#{col_spaces_prepend}= f.collection_select(:#{col.to_s}, #{assoc.class_name}.all, :id, :#{display_column}, {prompt: true, selected: @#{singular}.#{col.to_s} }, class: 'form-control')
 #{col_spaces_prepend}%label.small.form-text.text-muted
 #{col_spaces_prepend}  #{col.to_s.humanize}"
 
@@ -529,7 +634,7 @@ module HotGlue
 #{col_spaces_prepend}= f.label(:#{col.to_s}, value: 'Yes', for: '#{singular}_#{col.to_s}_1')
       "
         end
-
+        end
       }.join("\n")
       return res
     end
@@ -556,28 +661,12 @@ module HotGlue
             assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
 
             if assoc.nil?
-              puts "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
-              exit
+              exit_message =  "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
+              raise(HotGlue::Error,exit_message)
             end
-            
-            assoc_class = eval(assoc.class_name)
 
-            if assoc_class.column_names.include?("name")
-              display_column = "name"
-            elsif assoc_class.column_names.include?("to_label")
-              display_column = "to_label"
-            elsif assoc_class.column_names.include?("full_name")
-              display_column = "full_name"
-            elsif assoc_class.column_names.include?("display_name")
-              display_column = "display_name"
-            elsif assoc_class.column_names.include?("email")
-              display_column = "email"
-            elsif assoc_class.column_names.include?("number")
-              display_column = "number"
+            display_column =  derrive_reference_name(assoc.class_name)
 
-            else
-              puts "*** 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, 5) email, or 6) number 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)"
-            end
 
             "#{col_identifer}
   = #{singular}.#{assoc.name.to_s}.try(:#{display_column}) || '<span class=\"content alert-danger\">MISSING</span>'.html_safe"
@@ -649,22 +738,25 @@ module HotGlue
 
     def display_class
       me = eval(singular_class)
+
       @display_class ||=
-        if me.column_names.include?("name")
+        if me.column_names.include?("name") || me.instance_methods(false).include?(:name)
+          # note that all class object respond_to?(:name) with the name of their own class
+          # this one is unique
           "name"
-        elsif me.column_names.include?("to_label")
+        elsif me.column_names.include?("to_label") || me.instance_methods(false).include?(:to_label)
           "to_label"
-        elsif me.column_names.include?("full_name")
+        elsif me.column_names.include?("full_name") || me.instance_methods(false).include?(:full_name)
           "full_name"
-        elsif me.column_names.include?("display_name")
+        elsif me.column_names.include?("display_name") || me.instance_methods(false).include?(:display_name)
           "display_name"
-        elsif me.column_names.include?("email")
+        elsif me.column_names.include?("email") || me.instance_methods(false).include?(:email)
           "email"
-        elsif me.column_names.include?("number")
-          display_column = "number"
-
+        elsif me.column_names.include?("number") || me.instance_methods(false).include?(:number)
+          "number"
         else
-          raise "*** 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)"
+          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)"
+          raise(HotGlue::Error, exit_message)
         end
     end
 
@@ -689,7 +781,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