crud 0.0.4 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c13137f3cd61c4548411788c62812424776b992b
-  data.tar.gz: 170567c4d0ae6a5a5f8a9c91634ab0be6a7e2c3d
+SHA256:
+  metadata.gz: c72cddfbc0bacb84368d337f7fae090ec6a90cfb1426551dc627adecab354b58
+  data.tar.gz: 747abf05a99d8295c8a4ffc34904c5b3f97b2b1f076d72b347f36dc487843e3a
 SHA512:
-  metadata.gz: 9ce6de70b3b1ccf93869be5a7e8969a8d3dc4c049f9288c62a13c14638657f65c226dfc83993590599cae500e458287f7cf5e1cff799a1766cea1994cf2ac301
-  data.tar.gz: aa2ca0f928033540a88d1a639ce78b407cf2b4535827c9c63dcba36bce787acca965970d87e02b5482820cf29f147e074e95cb5f04e58c4c472aea56fb5ec930
+  metadata.gz: 40bb0c99d245f7d44ffb7b290b915deffc792270df46864af6afea5ba412c3e0647ae05a0dd695a57f9debd29f159b2450e3601c8e5afac96d27a23a2a7d8cd7
+  data.tar.gz: f99011fdc613bd7746717fcddb61b7ed483fea668e28fa9dfca88c4c7f226e483c370558b3967ce8471d2106e9c12d616a5561005c1a33c55e3ba9d1fe91818c

data/Rakefile

@@ -54,7 +54,7 @@ task :cucumber => 'app:cucumber'
 
 # ci_reporter tasks
 require 'rubygems'
-require 'ci/reporter/rake/test_unit' # use this if you’re using Test::Unit
+#require 'ci/reporter/rake/test_unit' # use this if you’re using Test::Unit
 require 'ci/reporter/rake/rspec'     # use this if you’re using RSpec
 require 'ci/reporter/rake/cucumber'  # use this if you're using Cucumber
 

data/app/controllers/crud/application_controller.rb

@@ -0,0 +1,5 @@
+module Crud
+  class ApplicationController < ActionController::Base
+    protect_from_forgery with: :exception
+  end
+end

data/app/controllers/crud/crud_controller.rb

@@ -2,12 +2,22 @@ require_dependency "crud/crud_base_controller"
 
 module Crud
   class CrudController < CrudBaseController
-    around_filter :catch_not_found
 
-    before_filter :is_allowed_to_view?, :only => [:index, :show]
-    before_filter :is_allowed_to_update?, :only => [:new, :create, :edit, :update, :destroy, :delete]
-    before_filter :get_klass_info_from_params
-    before_filter :get_visible_attributes
+    if Rails.version.to_f >= 5.1
+      around_action :catch_not_found
+
+      before_action :get_klass_data, :only => [:show, :edit, :update, :destroy, :delete]
+      before_action :get_klass_info_from_params
+      before_action :get_visible_attributes
+    else
+      around_filter :catch_not_found
+
+      before_filter :is_allowed_to_view?, :only => [:index, :show]
+      before_filter :is_allowed_to_update?, :only => [:new, :create, :edit, :update, :destroy, :delete]
+      before_filter :get_klass_data, :only => [:show, :edit, :update, :destroy, :delete]
+      before_filter :get_klass_info_from_params
+      before_filter :get_visible_attributes
+    end
 
     # GET
     def index
@@ -43,8 +53,6 @@ module Crud
 
     # GET
     def show
-      @klass_data = @klass_info[:class].find(params[:id])
-
       respond_to do |format|
         format.html # index.html.erb
         format.json { render :json => @klass_data }
@@ -65,12 +73,11 @@ module Crud
 
     # GET
     def edit
-      @klass_data = @klass_info[:class].find(params[:id])
     end
 
     # POST
     def create
-      @klass_data = @klass_info[:class].new(params[@klass_info[:name_as_sym]])
+      @klass_data = @klass_info[:class].new(klass_params)
 
       respond_to do |format|
         if @klass_data.save
@@ -87,10 +94,8 @@ module Crud
 
     # PUT
     def update
-      @klass_data = @klass_info[:class].find(params[:id])
-
       respond_to do |format|
-        if @klass_data.update_attributes(params[@klass_info[:name_as_sym]])
+        if @klass_data.update_attributes(klass_params)
           format.html { redirect_to show_path(:class_name => @klass_info[:name], :id => @klass_data.id), :notice => "#{@klass_info[:name]} was successfully updated" }
           format.json { head :no_content }
           format.xml  { head :ok }
@@ -104,7 +109,6 @@ module Crud
 
     # DELETE
     def delete
-      @klass_data = @klass_info[:class].find(params[:id])
       @klass_data.delete
 
       respond_to do |format|
@@ -116,7 +120,6 @@ module Crud
 
     # DELETE
     def destroy
-      @klass_data = @klass_info[:class].find(params[:id])
       @klass_data.destroy
 
       respond_to do |format|
@@ -126,7 +129,24 @@ module Crud
       end
     end
 
-    private
+  private
+    # Use callbacks to share common setup or constraints between actions.
+    def get_klass_data
+      @klass_info = get_klass_info_from_params
+      @klass_data = @klass_info[:class].find(params[:id])
+    end
+
+    # Never trust parameters from the scary internet, only allow the white list through.
+    def klass_params
+      visible_attributes = Array.new
+      get_visible_attributes.each do |attribute|
+        visible_attributes << attribute[:column_name].to_sym
+      end
+      params.fetch(@klass_info[:name_as_sym], visible_attributes)
+
+      # Allow all visible attributes to be written to (bypass mass-assignment protection)
+      params.require(@klass_info[:name_as_sym]).permit(visible_attributes)
+    end
 
     def get_klass_info_from_params
       if params[:class_name]

data/app/controllers/crud/dashboard_controller.rb

@@ -2,8 +2,13 @@ require_dependency "crud/crud_base_controller"
 
 module Crud
   class DashboardController < CrudBaseController
-    before_filter :is_allowed_to_view?, :only => [:index]
-    before_filter :get_klass_list
+
+    if Rails.version.to_f >= 5.1
+      before_action :get_klass_list
+    else
+      before_filter :is_allowed_to_view?, :only => [:index]
+      before_filter :get_klass_list
+    end
 
     def index
     end

data/app/helpers/crud/application_helper.rb

@@ -0,0 +1,4 @@
+module Crud
+  module ApplicationHelper
+  end
+end

data/app/jobs/crud/application_job.rb

@@ -0,0 +1,4 @@
+module Crud
+  class ApplicationJob < ActiveJob::Base
+  end
+end

data/app/mailers/crud/application_mailer.rb

@@ -0,0 +1,6 @@
+module Crud
+  class ApplicationMailer < ActionMailer::Base
+    default from: 'from@example.com'
+    layout 'mailer'
+  end
+end

data/app/models/crud/application_record.rb

@@ -0,0 +1,5 @@
+module Crud
+  class ApplicationRecord < ActiveRecord::Base
+    self.abstract_class = true
+  end
+end

data/app/models/crud/klass_list.rb

@@ -30,16 +30,17 @@ module Crud
     private
 
     def create_klasses_from_model_names(model_names)
-      model_names.inject([]) do |output, model_name|
-        klass_name = model_name
+      output = Array.new
+      klass_names = model_names.select {|model_name| (model_name.constantize.is_a? Class) && !(model_name == 'ApplicationRecord')}
+      klass_names.each do |klass_name|
         begin
-          klass = klass_name.constantize
-          output << "::#{klass}".constantize if klass_name == klass.to_s
+          klass = Object.const_get(klass_name)
+          output << klass if klass_name == klass.to_s
         rescue Exception => e
           Rails.logger.debug e.message
         end
-        output
       end
+      output
     end
 
     def remove_klasses_without_table(klasses)

data/app/views/crud/crud/show.html.erb

@@ -1,5 +1,13 @@
 <%= render 'error_messages', :model => @klass_data %>
 
+<% if is_allowed_to_update? %>
+  <%= link_to 'Edit', edit_path(:class_name => @klass_info[:name], :id => @klass_data.id) %> |
+<% end %>
+<%= link_to "Back (List all #{@klass_info[:name].pluralize})", index_path(:class_name => @klass_info[:name]) %>
+
+<br/>
+<br/>
+
 <% @visible_attributes.each do |attribute| %>
   <p>
     <b><%= label(@klass_info[:name].to_sym, attribute[:column_name].to_sym) %>:</b>

data/app/views/layouts/crud/mailer.html.erb

@@ -0,0 +1,8 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+    <style>/* Email styles need to be inline */</style>
+  </head>
+  <body><%= yield %></body>
+</html>

data/app/views/layouts/crud/mailer.text.erb

@@ -0,0 +1 @@
+<%= yield %>

data/config/initializers/assets.rb

@@ -0,0 +1,3 @@
+Rails.application.config.assets.precompile += %w( crud/dashboard.css crud/dashboard.js )
+Rails.application.config.assets.precompile += %w( crud/pagination.css crud/pagination.js )
+Rails.application.config.assets.precompile += %w( crud/search-icon.png )

data/crud.gemspec

@@ -7,7 +7,7 @@ require "crud/version"
 # Describe your gem and declare its dependencies:
 Gem::Specification.new do |s|
   s.name        = "crud"
-  s.version     = Crud::VERSION.dup
+  s.version     = Crud::VERSION::STRING.dup
   s.authors     = ["Everard Brown"]
   s.email       = ["Everard.Brown@htcl.eu"]
   s.homepage    = "https://github.com/htcl/crud"
@@ -22,8 +22,10 @@ Gem::Specification.new do |s|
   s.rdoc_options = ["--line-numbers", "--inline-source", "--title", "crud", "--main", "README.md"]
   s.extra_rdoc_files = ["CHANGELOG", "README.md", "lib/tasks/crud_tasks.rake", "lib/crud.rb"]
 
-  s.add_dependency 'rails'
+  s.add_dependency 'rails', '~> 5.0'
   s.add_dependency 'jquery-rails'
   s.add_dependency 'jquery-ui-rails'
   s.add_dependency 'will_paginate'
+
+  #s.add_development_dependency 'sqlite3'
 end

data/lib/crud/engine.rb

@@ -1,3 +1,5 @@
+require 'rails'
+
 module Crud
   class Engine < ::Rails::Engine
     isolate_namespace Crud
@@ -11,10 +13,10 @@ module Crud
     #config.autoload_paths << File.expand_path("../validators", __FILE__)
 
     config.generators do |g|
-      g.test_framework :rspec, :fixture_replacement => :factory_girl, :views => false, :helper => true
+      g.test_framework :rspec, :fixture_replacement => :factory_bot, :views => false, :helper => true
       g.view_specs false
       g.helper_specs true
-      g.fixture_replacement :factory_girl, :dir => 'spec/factories'
+      g.fixture_replacement :factory_bot, :dir => 'spec/factories'
     end
 
     # Load translations from config/locales/*.rb,yml are auto loaded.

data/lib/crud/version.rb

@@ -1,3 +1,19 @@
 module Crud
-  VERSION = "0.0.4"
+  # Returns the version of the currently loaded gem as a <tt>Gem::Version</tt>
+  def self.gem_version
+    Gem::Version.new VERSION::STRING
+  end
+
+  def self.version
+    VERSION::STRING.to_f
+  end
+
+  module VERSION
+    MAJOR = 0
+    MINOR = 1
+    PATCH = 0
+    PRE   = nil
+
+    STRING = [MAJOR, MINOR, PATCH, PRE].compact.join(".")
+  end
 end

metadata

@@ -1,69 +1,69 @@
 --- !ruby/object:Gem::Specification
 name: crud
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
 platform: ruby
 authors:
 - Everard Brown
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-17 00:00:00.000000000 Z
+date: 2018-04-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: jquery-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: jquery-ui-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: will_paginate
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Performs CRUD operations by reflection
@@ -90,11 +90,16 @@ files:
 - app/assets/stylesheets/crud/flash_notices.css
 - app/assets/stylesheets/crud/pagination.css
 - app/assets/stylesheets/dataTables/jquery.dataTables-override.css
+- app/controllers/crud/application_controller.rb
 - app/controllers/crud/crud_base_controller.rb
 - app/controllers/crud/crud_controller.rb
 - app/controllers/crud/dashboard_controller.rb
+- app/helpers/crud/application_helper.rb
 - app/helpers/crud/crud_helper.rb
 - app/helpers/crud/dashboard_helper.rb
+- app/jobs/crud/application_job.rb
+- app/mailers/crud/application_mailer.rb
+- app/models/crud/application_record.rb
 - app/models/crud/klass_info.rb
 - app/models/crud/klass_list.rb
 - app/models/crud/menus/development_menu.rb
@@ -110,7 +115,10 @@ files:
 - app/views/crud/crud/show.html.erb
 - app/views/crud/dashboard/index.html.erb
 - app/views/layouts/crud/application.html.erb
+- app/views/layouts/crud/mailer.html.erb
+- app/views/layouts/crud/mailer.text.erb
 - config/cucumber.yml
+- config/initializers/assets.rb
 - config/initializers/inflections.rb
 - config/initializers/rails_engine.rb
 - config/initializers/will_paginate.rb
@@ -130,28 +138,28 @@ licenses:
 metadata: {}
 post_install_message: 
 rdoc_options:
-- --line-numbers
-- --inline-source
-- --title
+- "--line-numbers"
+- "--inline-source"
+- "--title"
 - crud
-- --main
+- "--main"
 - README.md
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.7
+rubygems_version: 2.7.4
 signing_key: 
 specification_version: 4
-summary: Crud engine (v0.0.4)
+summary: Crud engine (v0.1.0)
 test_files: []