jobshop 0.0.157 → 0.0.163

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cfebb1f31d1fcbb60de9a9b6d74d38cf33db9dd8
-  data.tar.gz: 06d074ee3efe7275a4a2704b0c4ad1a512853259
+  metadata.gz: 72d9c251b2bcc65848a33510d25bdee8d14ca55b
+  data.tar.gz: b0e53706a8d59d0824affb1a416fb519786a520b
 SHA512:
-  metadata.gz: 342ba401730334f6e0b99959c2a90abb10b1f5a46dc5caf88c36df26515797d510944035742c133cd38e244f551df4235ed15bd6774cf0ac022594807be1f138
-  data.tar.gz: c6186e7da358301f38968e0a700bab329a4a98be4f57934163513b005cad73f31370184489f2ec10367a90cdfb0d0f6c5d74c100a437937b19aa3ab9ccd6f38d
+  metadata.gz: 3a442988c2fcc8dc15e7bca171754a7d1ef608cf55a1021354b0be3c189ec1eadac472ffbb33c69371a8a9caceff0513dde3d7624aadf69f07fa5c6cbfe15c53
+  data.tar.gz: 1aff3bac63832e1f71be8cb40ae5949b8f45966d9752a78f0c2e7a89e8f1e3b01c70b5c58c02ab2b4d034ee419a49fe72f7f562da98bd863046d728e0886fc2b

data/README.md

@@ -6,7 +6,6 @@ Real-time production tracking and process evaluation, under your own roof.
 [![Code Climate](https://codeclimate.com/github/jobshop/jobshop/badges/gpa.svg)](https://codeclimate.com/github/jobshop/jobshop)
 [![Build Status](https://semaphoreci.com/api/v1/frankjmattia/jobshop/branches/master/shields_badge.svg)](https://semaphoreci.com/frankjmattia/jobshop)
 [![Test Coverage](https://codeclimate.com/github/jobshop/jobshop/badges/coverage.svg)](https://codeclimate.com/github/jobshop/jobshop/coverage)
-[![Dependency Status](https://gemnasium.com/badges/github.com/jobshop/jobshop.svg)](https://gemnasium.com/github.com/jobshop/jobshop)
 
 ## TL;DR
 

data/app/email_handlers/jobshop/rfq_handler.rb

@@ -0,0 +1,43 @@
+# frozen_string_literals: true
+
+module Jobshop
+  class RFQHandler
+    def initialize(message, organization:)
+      @from = message.from.first
+      @subject = message.subject
+      @content = message.text_part.decoded
+      @organization = organization
+    end
+
+    def process
+      new_rfq = @organization.rfqs.create({
+        from: @from,
+        subject: @subject,
+        content: @content
+      })
+
+      if new_rfq.persisted?
+        # TODO: Send response email to customer
+        # TODO: Send rfq to quoting queue.
+        return new_rfq
+      else
+        # TODO: There was some kind of failure which should be logged.
+        return false
+      end
+    end
+
+    # Whether or not to copy the original to a new location after processing.
+    def copy_after_processing?
+      true
+    end
+
+    # Whether or not to delete the original item after processing.
+    def delete_after_processing?
+      true
+    end
+
+    def processed_copy_prefix
+      "rfqs/processed"
+    end
+  end
+end

data/app/models/jobshop/collection.rb

@@ -1,20 +1,15 @@
+# frozen_string_literals
+
 module Jobshop
   class Collection < ApplicationRecord
-    self.primary_keys = [ :organization_id, :collection_id ]
+    self.primary_keys = %i[ organization_id collection_id ]
 
-    after_create do
-      self.class.connection.clear_query_cache
-      fresh_collection = self.class.unscoped {
-        self.class.find_by!(organization: organization, name: name)
-      }
-      @attributes = fresh_collection.instance_variable_get(:@attributes)
-      @new_record = false
-      self
-    end
+    after_initialize { self.collection_id ||= SecureRandom.uuid if new_record? }
 
     belongs_to :organization, inverse_of: :collections
+
     has_many :things, inverse_of: :collection,
-      foreign_key: [ :organization_id, :collection_id ]
+      foreign_key: %i[ organization_id collection_id ]
 
     validates :name, presence: true,
       uniqueness: { scope: :organization_id, case_sensitive: false }

data/app/models/jobshop/company.rb

@@ -1,21 +1,34 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Company < ApplicationRecord
-    self.primary_keys = [ :organization_id, :company_id ]
+    self.primary_keys = %i[ organization_id company_id ]
+
+    # Because we have a column named +type+, Rails wants to activate STI
+    # Disable it manually
+    self.inheritance_column = nil
 
-    after_create do
-      self.class.connection.clear_query_cache
-      @attributes = self.class.unscoped {
-        self.class.find_by!(organization: organization, name: name)
-      }.instance_variable_get(:@attributes)
-      @new_record = false
-      self
-    end
+    after_initialize { self.company_id ||= SecureRandom.uuid if new_record? }
 
     belongs_to :organization, inverse_of: :companies
-    belongs_to :created_by, class_name: "Jobshop::User",
-      foreign_key: [ :organization_id, :created_by_id ]
+
+    has_many :company_people, inverse_of: :company,
+      foreign_key: %i[ organization_id company_id ]
+
+    belongs_to :created_by, class_name: "Jobshop::Person",
+      foreign_key: %i[ organization_id created_by_id ]
+
     has_many :orders, inverse_of: :company,
-      foreign_key: [ :organization_id, :company_id ]
+      foreign_key: %i[ organization_id company_id ]
+
+    has_many :people, through: :company_people,
+      foreign_key: %i[ organization_id person_id ]
+
+    has_many :rfqs, inverse_of: :company,
+      foreign_key: %i[ organization_id company_id ]
+
+    belongs_to :type, class_name: "Jobshop::Company::Type",
+      foreign_key: %i[ organization_id type_id ], inverse_of: :companies
 
     validates :name, presence: true
   end

data/app/models/jobshop/company/type.rb

@@ -0,0 +1,17 @@
+# frozen_string_literals: true
+
+module Jobshop
+  class Company::Type < ApplicationRecord
+    self.primary_keys = %i[ organization_id type_id ]
+
+    after_initialize { self.type_id ||= SecureRandom.uuid if new_record? }
+
+    belongs_to :organization, inverse_of: :company_types
+
+    has_many :companies, inverse_of: :type,
+      foreign_key: %i[ organization_id type_id ]
+
+    validates :name,
+      uniqueness: { scope: :organization_id, case_sensitive: false }
+  end
+end

data/app/models/jobshop/company_person.rb

@@ -0,0 +1,15 @@
+# frozen_string_literals: true
+
+module Jobshop
+  class CompanyPerson < ApplicationRecord
+    self.primary_keys = %i[ organization_id company_id person_id ]
+
+    belongs_to :organization
+
+    belongs_to :company, inverse_of: :company_people,
+      foreign_key: %i[ organization_id company_id ]
+
+    belongs_to :person, inverse_of: :company_person,
+      foreign_key: %i[ organization_id person_id ]
+  end
+end

data/app/models/jobshop/inspection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literals: true
+
 module Jobshop
   module Inspection
     def self.table_name_prefix

data/app/models/jobshop/inspection/boolean_criterion.rb

@@ -1,21 +1,24 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Inspection::BooleanCriterion < ApplicationRecord
-    self.primary_keys = [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+    self.primary_keys = %i[ organization_id report_id criterion_id criterion_type ]
 
-    after_initialize do
-      self[:criterion_id] ||= SecureRandom.uuid if new_record?
-    end
+    after_initialize { self.criterion_id ||= SecureRandom.uuid if new_record? }
+    before_save(on: :create) { criterion.save unless criterion.persisted? }
 
     belongs_to :organization
+
     has_one :criterion, as: :criterion, autosave: true, dependent: :destroy,
-      foreign_key: [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+      foreign_key: %i[ organization_id report_id criterion_id criterion_type ],
+      inverse_of: :criterion
+
     belongs_to :report, class_name: "Jobshop::Inspection::Report",
-      foreign_key: [ :organization_id, :report_id ], inverse_of: :criteria
+      foreign_key: %i[ organization_id report_id ]
 
-    alias_method :criterion_without_build, :criterion
-    def criterion_with_build
-      criterion_without_build || build_criterion(criterion_id: criterion_id, report: report)
-    end; alias_method :criterion, :criterion_with_build
+    def criterion
+      super || build_criterion
+    end
 
     delegate :name, :name=, :position, :position=, to: :criterion
 

data/app/models/jobshop/inspection/criterion.rb

@@ -1,21 +1,18 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Inspection::Criterion < ApplicationRecord
-    self.primary_keys = [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+    self.primary_keys = %i[ organization_id report_id criterion_id criterion_type ]
 
-    after_create do
-      self.class.connection.clear_query_cache
-      @attributes = self.class.unscoped {
-        self.class.find_by!(organization: organization, criterion_id: criterion_id)
-      }.instance_variable_get(:@attributes)
-      @new_record = false
-      self
-    end
+    after_initialize { self.criterion_id ||= SecureRandom.uuid if new_record? }
 
     belongs_to :organization
+
     belongs_to :criterion, polymorphic: true, optional: true, dependent: :destroy,
-      foreign_key: [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+      foreign_key: %i[ organization_id report_id criterion_id criterion_type ]
+
     belongs_to :report, class_name: "Jobshop::Inspection::Report",
-      foreign_key: [ :organization_id, :report_id ], inverse_of: :criteria
+      foreign_key: %i[ organization_id report_id ], inverse_of: :criteria
 
     delegate :ascii, :pass?, :random, :unit, to: :criterion
   end

data/app/models/jobshop/inspection/deviation_criterion.rb

@@ -1,21 +1,24 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Inspection::DeviationCriterion < ApplicationRecord
-    self.primary_keys = [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+    self.primary_keys = %i[ organization_id report_id criterion_id criterion_type ]
 
-    after_initialize do
-      self.criterion_id ||= SecureRandom.uuid if new_record?
-    end
+    after_initialize { self.criterion_id ||= SecureRandom.uuid if new_record? }
+    before_save(on: :create) { criterion.save unless criterion.persisted? }
 
     belongs_to :organization
+
     belongs_to :report, class_name: "Jobshop::Inspection::Report",
-      foreign_key: [ :organization_id, :report_id ], inverse_of: :criteria
+      foreign_key: %i[ organization_id report_id ]
+
     has_one :criterion, as: :criterion, autosave: true, dependent: :destroy,
-      foreign_key: [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+      foreign_key: %i[ organization_id report_id criterion_id criterion_type ],
+      inverse_of: :criterion
 
-    alias_method :criterion_without_build, :criterion
-    def criterion_with_build
-      criterion_without_build || build_criterion
-    end; alias_method :criterion, :criterion_with_build
+    def criterion
+      super || build_criterion
+    end
 
     delegate :name, :name=, :position, :position=, to: :criterion
 

data/app/models/jobshop/inspection/limit_criterion.rb

@@ -1,16 +1,18 @@
 module Jobshop
   class Inspection::LimitCriterion < ApplicationRecord
-    self.primary_keys = [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+    self.primary_keys = %i[ organization_id report_id criterion_id criterion_type ]
 
-    after_initialize do
-      self.criterion_id = SecureRandom.uuid if new_record?
-    end
+    after_initialize { self.criterion_id ||= SecureRandom.uuid if new_record? }
+    before_save(on: :create) { criterion.save unless criterion.persisted? }
 
     belongs_to :organization
-    has_one :criterion, as: :criterion, autosave: true, dependent: :destroy, inverse_of: :criterion,
-      foreign_key: [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+
+    has_one :criterion, as: :criterion, autosave: true, dependent: :destroy,
+      foreign_key: %i[ organization_id report_id criterion_id criterion_type ],
+      inverse_of: :criterion
+
     belongs_to :report, class_name: "Jobshop::Inspection::Report",
-      foreign_key: [ :organization_id, :report_id ], inverse_of: :criteria
+      foreign_key: %i[ organization_id report_id ]
 
     def criterion
       super || build_criterion

data/app/models/jobshop/inspection/report.rb

@@ -1,25 +1,21 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Inspection::Report < ApplicationRecord
-    self.primary_keys = [ :organization_id, :report_id ]
+    self.primary_keys = %i[ organization_id report_id ]
 
-    after_initialize do
-      self.report_id ||= SecureRandom.uuid if new_record?
-    end
+    after_initialize { self.report_id ||= SecureRandom.uuid if new_record? }
 
-    after_create do
-      self.class.connection.clear_query_cache
-      @attributes = self.class.unscoped {
-        self.class.find_by!(organization: organization, report_id: report_id)
-      }.instance_variable_get(:@attributes)
-      @new_record = false
-      self
-    end
+    belongs_to :organization, inverse_of: :inspection_reports
 
-    belongs_to :organization
     has_many :criteria, -> { includes(:criterion).order(:position) },
-      foreign_key: [ :organization_id, :report_id ], inverse_of: :report
+      foreign_key: %i[ organization_id report_id ], inverse_of: :report
+
+    has_many :limit_criteria, inverse_of: :report,
+      foreign_key: %i[ organization_id report_id ]
+
     has_many :tuples, -> { order(:position) },  inverse_of: :report,
-      foreign_key: [ :organization_id, :report_id ]
+      foreign_key: %i[ organization_id report_id ]
 
     def ascii
       require "terminal-table"

data/app/models/jobshop/inspection/result.rb

@@ -1,22 +1,26 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Inspection::Result < ApplicationRecord
     FALSE_VALUES = [false, 0, "0", "f", "F", "false", "FALSE", "off", "OFF"]
 
-    self.primary_keys = [ :organization_id, :report_id, :criterion_id, :criterion_type, :position ]
+    self.primary_keys = %i[ organization_id report_id criterion_id criterion_type position ]
 
     belongs_to :organization
+
     belongs_to :report, class_name: "Jobshop::Inspection::Report",
-      foreign_key: [ :organization_id, :report_id ]
+      foreign_key: %i[ organization_id report_id ]
+
     belongs_to :criterion, polymorphic: true, optional: true, dependent: :destroy,
-      foreign_key: [ :organization_id, :report_id, :criterion_id, :criterion_type ]
+      foreign_key: %i[ organization_id report_id criterion_id criterion_type ]
+
     belongs_to :tuple, class_name: "Jobshop::Inspection::Tuple",
-      foreign_key: [ :organization_id, :report_id, :position ]
+      foreign_key: %i[ organization_id report_id position ]
 
     def value
       if self[:unit] == "boolean"
         !!self[:value]
       else
-#        binding.pry
         self[:value] && Unitwise(self[:value].truncate(4), self[:unit])
       end
     end

data/app/models/jobshop/inspection/tuple.rb

@@ -1,12 +1,16 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Inspection::Tuple < ApplicationRecord
-    self.primary_keys = [ :organization_id, :report_id, :position ]
+    self.primary_keys = %i[ organization_id report_id position ]
 
     belongs_to :organization
+
     belongs_to :report, class_name: "Jobshop::Inspection::Report", inverse_of: :tuples,
-      foreign_key: [ :organization_id, :report_id ]
+      foreign_key: %i[ organization_id report_id ]
+
     has_many :results, class_name: "Jobshop::Inspection::Result", inverse_of: :tuple,
-      foreign_key: [ :organization_id, :report_id, :position ]
+      foreign_key: %i[ organization_id report_id position ]
 
     has_many :criteria, through: :results
   end

data/app/models/jobshop/mailman.rb

@@ -0,0 +1,11 @@
+# frozen_string_literals: true
+
+module Jobshop
+  class Mailman < ApplicationRecord
+    self.primary_keys = %i[ organization_id address ]
+
+    belongs_to :organization, inverse_of: :mailmen
+
+    validates :address, presence: true
+  end
+end

data/app/models/jobshop/order.rb

@@ -1,23 +1,21 @@
+# frozen_string_literals: true
+
 module Jobshop
   class Order < ApplicationRecord
-    self.primary_keys = [ :organization_id, :order_id ]
-
-    after_create do
-      self.class.connection.clear_query_cache
-      @attributes = self.class.unscoped {
-        self.class.find_by!(organization: organization, number: number)
-      }.instance_variable_get(:@attributes)
-      @new_record = false
-      self
-    end
+    self.primary_keys = %i[ organization_id order_id ]
+
+    after_initialize { self.order_id ||= SecureRandom.uuid if new_record? }
 
     belongs_to :organization, inverse_of: :orders
-    belongs_to :created_by, class_name: "Jobshop::User",
-      foreign_key: [ :organization_id, :created_by_id ]
+
+    belongs_to :created_by, class_name: "Jobshop::Person",
+      foreign_key: %i[ organization_id created_by_id ]
+
     belongs_to :company, inverse_of: :orders,
-      foreign_key: [ :organization_id, :company_id ]
+      foreign_key: %i[ organization_id company_id ]
+
     has_many :order_lines, inverse_of: :order,
-      foreign_key: [ :organization_id, :order_id ]
+      foreign_key: %i[ organization_id order_id ]
 
     around_save :notify_on_save