hot-glue 0.0.3 → 0.0.9

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_hgis.rb

@@ -0,0 +1,9 @@
+class CreateHgis < ActiveRecord::Migration[6.1]
+  def change
+    create_table :hgis 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 "hgis", 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,6 +3,12 @@ require 'ffaker'
 
 
 module HotGlue
+
+
+  class Error < StandardError
+  end
+
+
   module GeneratorHelper
     def text_area_output(col, field_length, col_identifier )
       lines = field_length % 40
@@ -38,153 +44,191 @@ module HotGlue
     include GeneratorHelper
 
 
-    def initialize(*meta_args) #:nodoc:
-    super
+    class_option :singular, type: :string, default: nil
+    class_option :plural, type: :string, default: nil
+    class_option :singular_class, type: :string, default: nil
+    class_option :nest, type: :string, default: ""
+    class_option :namespace, type: :string, default: nil
+    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 :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: ""
 
-    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
-    end
+    def initialize(*meta_args) #:nodoc:
+      super
 
+      begin
+        object = eval(class_name)
+      rescue StandardError => e
+        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[0].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
+      @nest = options['auth'] || 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
 
-    @plural = @singular + "s" # supply to override; leave blank to use default
-    @singular_class = @singular.titleize.gsub(" ", "")
-    @nest = nil
-    @namespace = nil
-    @nested_args = []
 
-    @auth = "current_user"
-    @auth_identifier = nil
-    @exclude_fields = []
+      @show_only = []
+      if !options['show_only'].empty?
+        @show_only += options['show_only'].split(",").collect(&:to_sym)
+      end
 
-    args[1..-1].each do |a|
-      var_name, var_value = a.split("=")
-      case (var_name)
+      auth_assoc = @auth.gsub("current_","")
 
-      when "plural"
-        @plural = var_value
-      when "nest"
-        @nest = var_value
-      when "namespace"
-        @namespace = var_value
-      when "auth"
-        @auth = var_value
-      when "auth_identifier"
-        @auth_identifier = var_value || ""
-      when "exclude"
-        @exclude_fields += var_value.split(",").collect(&:to_sym)
-      end
-    end
+      @god = options['god'] || options['gd'] || false
+      @specs_only = options['specs_only'] || false
 
-    auth_assoc = @auth.gsub("current_","")
+      @no_specs = options['no_specs'] || false
+      @no_delete = options['no_delete'] || false
 
-    @no_delete = false
-    @no_create = false
+      @no_create = options['no_create'] || false
+      @no_paginate = options['no_paginate'] || false
+      @big_edit = options['big_edit']
 
-    flags = meta_args[1]
-    flags.each do |f|
-      case (f)
-      when "--god"
+      if @god
         @auth = nil
-      when "--specs-only"
-        @specs_only = true
-      when "--no-specs"
-        @no_specs = true
-      when "--no-delete"
-        @no_delete = true
-      when "--no-create"
-        @no_create = true
-      when "--no-paginate"
-        @no_paginate = true
       end
-    end
 
-    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
+      # 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("/")
+        @nested_args_plural = {}
+        @nested_args.each do |a|
+          @nested_args_plural[a] = a + "s"
+        end
+      end
 
-                               # only used for the before_action
-    if @auth_identifier.nil? && !@auth.nil?
-      @auth_identifier = @auth.gsub("current_", "")
-    end
+                                 # the @object_owner will always be object that will 'own' the object
+                                 # for new and create
 
-                               # when in self auth, the object is the same as the authenticated object
-    if @auth && auth_identifier == @singular
-      @self_auth = true
-    end
+      if @auth && ! @self_auth && @nested_args.none?
+        @object_owner_sym = @auth.gsub("current_", "").to_sym
+        @object_owner_eval = @auth
+      else
 
-    if !@nest.nil?
-      @nested_args = @nest.split("/")
-      @nested_args_plural = {}
-      @nested_args.each do |a|
-        @nested_args_plural[a] = a + "s"
+        if @nested_args.any?
+          @object_owner_sym = @nested_args.last.to_sym
+          @object_owner_eval = "@#{@nested_args.last}"
+        else
+          @object_owner_sym = ""
+          @object_owner_eval = ""
+        end
       end
-    end
 
-                               # the @object_owner will always be object that will 'own' the object
-                               # for new and create
 
-    if @auth && ! @self_auth && @nested_args.none?
-      @object_owner_sym = @auth.gsub("current_", "").to_sym
-      @object_owner_eval = @auth
-    else
+      if !@object_owner_sym.empty?
+        auth_assoc_field = auth_assoc + "_id"
+        assoc = eval("#{singular_class}.reflect_on_association(:#{@object_owner_sym})")
 
-      if @nested_args.any?
-        @object_owner_sym = @nested_args.last.to_sym
-        @object_owner_eval = "@#{@nested_args.last}"
-      else
-        @object_owner_sym = ""
-        @object_owner_eval = ""
+        if assoc
+          ownership_field = assoc.name.to_s + "_id"
+        else
+          # if @auth
+            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
+          #   exit_message= "*** Oops:  god mode could not find the association(?). something is wrong."
+          # end
+          raise(HotGlue::Error, exit_message)
+        end
       end
-    end
 
+      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?
 
 
-                               # 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"
+        @columns = object.columns.map(&:name).map(&:to_sym).reject{|field| @exclude_fields.include?(field) }
+
       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."
-        else
-          puts "*** Oops:  god mode could not find the association(?). something is wrong."
-        end
-        exit
+        @columns = object.columns.map(&:name).map(&:to_sym).reject{|field| !@include_fields.include?(field) }
+
       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
+      @columns.each do |col|
+        if col.to_s.starts_with?("_")
+          @show_only << col
+        end
 
-    @exclude_fields += auth_assoc_field.to_sym if !auth_assoc_field.nil?
-    @exclude_fields +=  ownership_field.to_sym if !ownership_field.nil?
+        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
-      @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
-    end
 
+            begin
+              eval(assoc.class_name)
+            rescue NameError => e
+              exit_message = "*** Oops: The table #{singular_class} has an association for '#{assoc.name.to_s}', but I can't find an assoicated model for that association. TODO: Please implement a model for #{assoc.name.to_s} that belongs to #{singular_class} "
+              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
@@ -197,17 +241,19 @@ module HotGlue
       @default_colspan = @columns.size
 
       unless @specs_only
-        template "controller.rb", File.join("app/controllers#{namespace_with_dash}", "#{plural}_controller.rb")
+        template "controller.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}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 "base_controller.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}app/controllers#{namespace_with_dash}", "base_controller.rb")
         end
       end
 
       unless @no_specs
-        template "controller_spec.rb", File.join("spec/controllers#{namespace_with_dash}", "#{plural}_controller_spec.rb")
+        template "request_spec.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}spec/request#{namespace_with_dash}", "#{plural}_spec.rb")
+        template "system_spec.rb.erb", File.join("#{'spec/dummy/' if Rails.env.test?}spec/system#{namespace_with_dash}", "#{plural}_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
@@ -402,14 +448,14 @@ module HotGlue
       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
@@ -431,13 +477,21 @@ module HotGlue
     end
 
     def haml_views
-      res =  %w(index edit new _form _line _list _new_button _show)
+      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
@@ -458,6 +512,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
@@ -466,31 +530,29 @@ 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
 
 
             assoc_class = eval(assoc.class_name)
 
-            if assoc_class.column_names.include?("name")
+            if assoc_class.respond_to?("name")
               display_column = "name"
-            elsif assoc_class.column_names.include?("to_label")
+            elsif assoc_class.respond_to?("to_label")
               display_column = "to_label"
-            elsif assoc_class.column_names.include?("full_name")
+            elsif assoc_class.respond_to?("full_name")
               display_column = "full_name"
-            elsif assoc_class.column_names.include?("display_name")
+            elsif assoc_class.respond_to?("display_name")
               display_column = "display_name"
-            elsif assoc_class.column_names.include?("email")
+            elsif assoc_class.respond_to?("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)"
+              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
 
             "#{col_identifier}{class: \"form-group \#{'alert-danger' if #{singular}.errors.details.keys.include?(:#{assoc_name.to_s})}\"}
@@ -544,7 +606,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
@@ -571,24 +633,27 @@ 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")
+            if assoc_class.respond_to?("name")
               display_column = "name"
-            elsif assoc_class.column_names.include?("to_label")
+            elsif assoc_class.respond_to?("to_label")
               display_column = "to_label"
-            elsif assoc_class.column_names.include?("full_name")
+            elsif assoc_class.respond_to?("full_name")
               display_column = "full_name"
-            elsif assoc_class.column_names.include?("display_name")
+            elsif assoc_class.respond_to?("display_name")
               display_column = "display_name"
-            elsif assoc_class.column_names.include?("email")
+            elsif assoc_class.respond_to?("email")
               display_column = "email"
+            elsif assoc_class.respond_to?("number")
+              display_column = "number"
+
             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)"
+              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}
@@ -658,6 +723,32 @@ module HotGlue
     end
 
 
+
+    def display_class
+      me = eval(singular_class)
+
+      @display_class ||=
+        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.column_names.include?("to_label") || me.instance_methods(false).include?(:to_label)
+          "to_label"
+        elsif me.column_names.include?("full_name") || me.instance_methods(false).include?(:full_name)
+          "full_name"
+        elsif me.column_names.include?("display_name") || me.instance_methods(false).include?(:display_name)
+          "display_name"
+        elsif me.column_names.include?("email") || me.instance_methods(false).include?(:email)
+          "email"
+        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)"
+          raise(HotGlue::Error, exit_message)
+        end
+    end
+
     def destroy_action
       return false if @self_auth
       return !@no_delete
@@ -679,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