ditty 0.8.0 → 0.10.2

data/lib/ditty/controllers/{main.rb → main_controller.rb}

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
 
-require 'ditty/controllers/application'
+require 'ditty/controllers/application_controller'
 require 'ditty/models/role'
 require 'ditty/models/user'
+require 'ditty/services/settings'
 
 module Ditty
-  class Main < Application
+  class MainController < ApplicationController
     set track_actions: true
 
     before '/' do
@@ -17,6 +18,9 @@ module Ditty
 
     # Home Page
     get '/' do
+      home_page = Services::Settings['ditty.home_page']
+      redirect "#{settings.map_path}#{home_page}" if home_page
+
       authenticate!
       haml :index, locals: { title: 'Home' }
     end

data/lib/ditty/controllers/roles_controller.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'ditty/controllers/component_controller'
+require 'ditty/models/role'
+require 'ditty/policies/role_policy'
+
+module Ditty
+  class RolesController < ::Ditty::ComponentController
+    SEARCHABLE = %i[name].freeze
+
+    set model_class: Role
+
+    helpers do
+      def parent_options(entity)
+        return policy_scope(::Ditty::Role) if entity.new?
+
+        policy_scope(::Ditty::Role)
+          .exclude(id: [entity.id] + entity.descendants.map(&:id))
+          .order(:name)
+      end
+    end
+  end
+end

data/lib/ditty/controllers/user_login_traits_controller.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require 'ditty/controllers/component_controller'
+require 'ditty/models/user_login_trait'
+require 'ditty/policies/user_login_trait_policy'
+
+module Ditty
+  class UserLoginTraitsController < ::Ditty::ComponentController
+    SEARCHABLE = %i[platform device browser ip_address].freeze
+    FILTERS = [
+      { name: :user, field: 'user.email' },
+      { name: :platform },
+      { name: :device },
+      { name: :browser }
+    ].freeze
+
+    set base_path: '/login-traits'
+    set model_class: UserLoginTrait
+    set heading: 'Login'
+    # set track_actions: true
+
+    helpers do
+      def user_options
+        policy_scope(::Ditty::User).as_hash(:email, :email)
+      end
+
+      def platform_options
+        policy_scope(::Ditty::UserLoginTrait).select(:platform).distinct.as_hash(:platform, :platform)
+      end
+
+      def device_options
+        policy_scope(::Ditty::UserLoginTrait).select(:device).distinct.as_hash(:device, :device)
+      end
+
+      def browser_options
+        policy_scope(::Ditty::UserLoginTrait).select(:browser).distinct.as_hash(:browser, :browser)
+      end
+    end
+
+    def list
+      return super if params[:sort]
+
+      super.order(:updated_at).reverse
+    end
+  end
+end

data/lib/ditty/controllers/{users.rb → users_controller.rb}

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'ditty/controllers/component'
+require 'ditty/controllers/component_controller'
 require 'ditty/models/user'
 require 'ditty/models/user_login_trait'
 require 'ditty/policies/user_policy'
@@ -8,7 +8,7 @@ require 'ditty/models/identity'
 require 'ditty/policies/identity_policy'
 
 module Ditty
-  class Users < Ditty::Component
+  class UsersController < ::Ditty::ComponentController
     SEARCHABLE = %i[name surname email].freeze
 
     set model_class: User
@@ -28,8 +28,8 @@ module Ditty
 
       locals = { title: heading(:new) }
 
-      user_params = permitted_attributes(User, :create)
-      identity_params = permitted_attributes(Identity, :create)
+      user_params = permitted_parameters(User, :create)
+      identity_params = permitted_parameters(Identity, :create)
       user_params['email'] = identity_params['username']
       roles = user_params.delete('role_id')
 
@@ -38,7 +38,7 @@ module Ditty
 
       DB.transaction(isolation: :serializable) do
         begin
-          identity.save
+          identity.save_changes
         rescue Sequel::ValidationFailed
           raise unless request.accept? 'text/html'
 
@@ -46,7 +46,7 @@ module Ditty
           locals = { title: heading(:new), entity: user, identity: identity }
           return haml(:"#{view_location}/new", locals: locals)
         end
-        user.save
+        user.save_changes
         user.add_identity identity
 
         roles&.each do |role_id|
@@ -55,7 +55,7 @@ module Ditty
         user.check_roles
       end
 
-      broadcast(:component_create, target: self)
+      broadcast(:component_create, target: self, entity: user)
       create_response(user)
     end
 
@@ -65,10 +65,10 @@ module Ditty
       halt 404 unless entity
       authorize entity, :update
 
-      values = permitted_attributes(settings.model_class, :update)
+      values = permitted_parameters(settings.model_class, :update)
       roles  = values.delete('role_id')
       entity.set values
-      entity.save # Will trigger a Sequel::ValidationFailed exception if the model is incorrect
+      entity.save_changes # Will trigger a Sequel::ValidationFailed exception if the model is incorrect
 
       if roles
         entity.remove_all_roles
@@ -94,9 +94,10 @@ module Ditty
         return redirect back
       end
 
-      values = permitted_attributes(Identity, :create)
+      values = permitted_parameters(Identity, :create)
       identity.set values
-      if identity.valid? && identity.save
+      if identity.valid?
+        identity.save_changes
         broadcast(:identity_update_password, target: self)
         flash[:success] = 'Password Updated'
         redirect back
@@ -129,6 +130,7 @@ module Ditty
       halt 404 unless entity
       authorize entity, :read
 
+      flash[:redirect_to] = request.path
       haml :"#{view_location}/profile", locals: { entity: entity, identity: entity.identity.first, title: 'My Account' }
     end
   end

data/lib/ditty/db.rb

@@ -8,7 +8,7 @@ require 'active_support/core_ext/object/blank'
 pool_timeout = (ENV['DB_POOL_TIMEOUT'] || 5).to_i
 
 if defined? DB
-  Ditty::Services::Logger.instance.warn 'Database connection already set up'
+  ::Ditty::Services::Logger.warn 'Database connection already set up'
 elsif ENV['DATABASE_URL'].blank? == false
   # Delete DATABASE_URL from the environment, so it isn't accidently
   # passed to subprocesses.  DATABASE_URL may contain passwords.
@@ -18,14 +18,16 @@ elsif ENV['DATABASE_URL'].blank? == false
   )
 
   DB.sql_log_level = (ENV['SEQUEL_LOGGING_LEVEL'] || :debug).to_sym
-  DB.loggers << Ditty::Services::Logger.instance if ENV['DB_DEBUG'].to_i == 1
+  DB.loggers << ::Ditty::Services::Logger if ENV['DB_DEBUG'].to_i == 1
   DB.extension(:pagination)
   DB.extension(:schema_caching)
   DB.load_schema_cache?('./config/schema.dump')
 
-  Sequel::Model.plugin :validation_helpers
-  Sequel::Model.plugin :update_or_create
+  Sequel::Model.plugin :auto_validations
+  Sequel::Model.plugin :string_stripper
   Sequel::Model.plugin :timestamps, update_on_create: true
+  Sequel::Model.plugin :update_or_create
+  Sequel::Model.plugin :validation_helpers
 else
-  Ditty::Services::Logger.instance.error 'No database connection set up'
+  ::Ditty::Services::Logger.error 'No database connection set up'
 end

data/lib/ditty/emails/base.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'haml'
-require 'ditty/components/app'
+require 'ditty/components/ditty'
 
 module Ditty
   module Emails
@@ -23,7 +23,10 @@ module Ditty
           @locals[param] ||= options[param]
           mail.send(param, options[param])
         end
-        mail.body content
+        html = content
+        mail.html_part do
+          body html
+        end
         mail.deliver!
       end
 
@@ -34,50 +37,52 @@ module Ditty
       end
 
       def respond_to_missing?(method, _include_private = false)
-        mail.respond_to? method
+        return true if mail.respond_to?(method)
+
+        super
       end
 
       private
 
-      def content
-        result = Haml::Engine.new(content_haml).render(Object.new, locals)
-        return result unless options[:layout]
+        def content
+          result = Haml::Engine.new(content_haml).render(Object.new, locals)
+          return result unless options[:layout]
 
-        Haml::Engine.new(layout_haml).render(Object.new, locals.merge(content: result))
-      end
+          Haml::Engine.new(layout_haml).render(Object.new, locals.merge(content: result))
+        end
 
-      def content_haml
-        read_template(options[:view])
-      end
+        def content_haml
+          read_template(options[:view])
+        end
 
-      def layout_haml
-        read_template("layouts/#{options[:layout]}") if options[:layout]
-      end
+        def layout_haml
+          read_template("layouts/#{options[:layout]}") if options[:layout]
+        end
 
-      def read_template(template)
-        File.read(find_template("emails/#{template}"))
-      end
+        def read_template(template)
+          File.read(find_template("emails/#{template}"))
+        end
 
-      def base_options
-        { subject: '(No Subject)', from: 'no-reply@ditty.io', view: :base, content_type: 'text/html; charset=UTF-8' }
-      end
+        def base_options
+          { subject: '(No Subject)', from: 'no-reply@ditty.io', view: :base, content_type: 'text/html; charset=UTF-8' }
+        end
 
-      def find_template(file)
-        template = File.expand_path("./views/#{file}.haml")
-        return template if File.file? template
+        def find_template(file)
+          template = File.expand_path("./views/#{file}.haml")
+          return template if File.file? template
 
-        template = File.expand_path("./#{file}.haml", App.view_folder)
-        return template if File.file? template
+          template = File.expand_path("./#{file}.haml", ::Ditty::Ditty.view_folder)
+          return template if File.file? template
 
-        file
-      end
+          file
+        end
 
-      class << self
-        def deliver!(to = nil, options = {})
-          locals = options[:locals] || {}
-          new(options).deliver!(to, locals)
+        class << self
+          def deliver!(to = nil, options = {})
+            locals = options[:locals] || {}
+            new(options).deliver!(to, locals)
+          end
         end
-      end
     end
   end
 end

data/lib/ditty/generators/crud_generator.rb

@@ -0,0 +1,114 @@
+# frozen_string_literal: true
+
+require 'thor/group'
+require 'active_support/inflector'
+
+module Ditty
+  module Generators
+    class CrudGenerator < Thor::Group
+      include Thor::Actions
+      attr_reader :namespace, :folder, :views_folder, :controller_name, :model_name, :policy_name # , :file_name
+
+      desc 'Create a CRUD Endpoint including the Model, Controller, Policy, Views and GraphQL Types'
+      argument :name, type: :string, desc: 'Name of the Model, eg. MyApp::User'
+
+      # --no-views make views optional
+      class_option :views, type: :boolean, default: true, desc: 'Generate views for controller'
+
+      def self.source_root
+        File.dirname(__FILE__)
+      end
+
+      def setup
+        @namespace       = name.deconstantize
+        @folder          = namespace.underscore
+        @model_name      = name.demodulize
+        @views_folder    = File.join('views', model_name.pluralize.underscore)
+        @controller_name = "#{model_name.pluralize}Controller"
+        @policy_name     = "#{model_name}Policy"
+      end
+
+      def create_model
+        filename = File.join("lib/#{folder}/models", "#{model_name.underscore}.rb")
+        template '../templates/model.rb.erb', filename
+      rescue StandardError => e
+        puts "Could not generate model for #{model_name}: #{e.message}"
+      end
+
+      def create_controller
+        filename = File.join("lib/#{folder}/controllers", "#{controller_name.underscore}.rb")
+        template '../templates/controller.rb.erb', filename
+        # TODO: Insert the route into the component file
+        # insert_into_file 'config.ru', "use #{class_name}\n", after: 'run ApplicationController\n'
+      rescue StandardError => e
+        puts "Could not generate controller for #{model_name}: #{e.message}"
+      end
+
+      def create_policy
+        filename = File.join("lib/#{folder}/policies", "#{policy_name.underscore}.rb")
+        template '../templates/policy.rb.erb', filename
+      rescue StandardError => e
+        puts "Could not generate policy for #{model_name}: #{e.message}"
+      end
+
+      def create_type
+        filename = File.join("lib/#{folder}/types", "#{model_name.underscore}_type.rb")
+        template '../templates/type.rb.erb', filename
+      rescue StandardError => e
+        puts "Could not generate type for #{model_name}: #{e.message}"
+      end
+
+      def create_views
+        return unless options[:views]
+
+        directory '../templates/views', views_folder
+      end
+
+      private
+
+        def meta_columns
+          %i[id guid slug created_at updated_at]
+        end
+
+        def columns
+          require "#{folder}/models/#{model_name.underscore}"
+          name.constantize.columns
+        rescue StandardError
+          []
+        end
+
+        def schema
+          require "#{folder}/models/#{model_name.underscore}"
+          name.constantize.db_schema
+        rescue StandardError
+          []
+        end
+
+        def many_to_ones
+          DB.foreign_key_list(model_name.underscore.pluralize)
+        end
+
+        def name_column(table)
+          candidates = DB.schema(table.to_sym).to_h.keys - DB.foreign_key_list(table.to_sym).map do |e|
+                                                             e[:columns]
+                                                           end.flatten
+          (candidates - meta_columns).first
+        end
+
+        def graphql_types
+          @graphql_types ||= Hash.new('String').merge(
+            integer: 'Integer',
+            boolean: 'Boolean',
+            datetime: 'GraphQL::Types::ISO8601DateTime'
+          )
+        end
+
+        def input_types
+          @input_types ||= Hash.new('text').merge(
+            integer: 'number',
+            datetime: 'date'
+          )
+        end
+    end
+  end
+end

data/lib/ditty/generators/migration_generator.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require 'thor/group'
+require 'active_support/inflector'
+
+module Ditty
+  module Generators
+    class MigrationGenerator < Thor::Group
+      include Thor::Actions
+
+      attr_reader :namespace, :folder
+
+      desc 'Creates a new Sequel migration for the current project'
+      argument :name, type: :string, desc: 'Name of the migration'
+
+      def self.source_root
+        File.expand_path('../templates', __dir__)
+      end
+
+      def create_model
+        filename = File.join('migrations', "#{Time.now.strftime('%Y%m%d')}_#{name.underscore}.rb")
+        template '../templates/migration.rb.erb', filename
+      end
+    end
+  end
+end