Merge pull request #9 from Vizzuality/feature/litigation-sides-edit

Feature/litigation sides edit

Gemfile

@@ -31,6 +31,7 @@ group :development, :test do
   gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
   gem 'capybara'
   gem 'factory_bot_rails'
+  gem 'rails-controller-testing'
   gem 'rspec-collection_matchers'
   gem 'rspec-rails', '~> 3.8'
 end

Gemfile.lock

@@ -235,6 +235,10 @@ GEM
       bundler (>= 1.3.0)
       railties (= 5.2.3)
       sprockets-rails (>= 2.0.0)
+    rails-controller-testing (1.0.4)
+      actionpack (>= 5.0.1.x)
+      actionview (>= 5.0.1.x)
+      activesupport (>= 5.0.1.x)
     rails-dom-testing (2.0.3)
       activesupport (>= 4.2.0)
       nokogiri (>= 1.6)
@@ -390,6 +394,7 @@ DEPENDENCIES
   pg (>= 0.18, < 2.0)
   puma (~> 3.11)
   rails (~> 5.2.3)
+  rails-controller-testing
   rspec-collection_matchers
   rspec-rails (~> 3.8)
   rubocop

app/admin/admin_users.rb

@@ -1,5 +1,5 @@
 ActiveAdmin.register AdminUser do
-  menu priority: 2
+  menu priority: 2, parent: 'Administration'
 
   permit_params :email, :password, :password_confirmation
 

app/admin/companies.rb

@@ -1,5 +1,5 @@
 ActiveAdmin.register Company do
-  menu priority: 4
+  menu priority: 0, parent: 'TPI'
 
   permit_params :name, :isin, :sector_id, :location_id, :headquarter_location_id, :ca100,
                 :size
@@ -94,6 +94,20 @@
           end
         end
       end
+
+      tab :litigations do
+        panel 'Connected Litigations' do
+          if resource.litigations.empty?
+            div class: 'padding-20' do
+              'No Litigations connected with this company'
+            end
+          else
+            table_for resource.litigations.decorate do
+              column :title, &:title_link
+            end
+          end
+        end
+      end
     end
   end
 

app/admin/cp_assessments.rb

@@ -1,7 +1,7 @@
 ActiveAdmin.register CP::Assessment do
   config.sort_order = 'publication_date_desc'
 
-  menu priority: 6, parent: 'Assessments', label: 'Carbon Performance Assessments'
+  menu parent: 'TPI', priority: 2, label: 'Carbon Performance Assessments'
 
   decorate_with CP::AssessmentDecorator
 

app/admin/dashboard.rb

@@ -1,5 +1,5 @@
 ActiveAdmin.register_page 'Dashboard' do
-  menu priority: 1, label: 'Dashboard'
+  menu priority: 0, label: 'Dashboard'
 
   content do
     div class: 'blank_slate_container' do

app/admin/documents.rb

@@ -1,5 +1,5 @@
 ActiveAdmin.register Document do
-  menu priority: 3
+  menu priority: 6
 
   decorate_with DocumentDecorator
 

app/admin/legislations.rb

@@ -1,6 +1,8 @@
 ActiveAdmin.register Legislation do
   decorate_with LegislationDecorator
 
+  menu parent: 'Laws', priority: 1
+
   permit_params :title, :description, :framework, :location_id, :law_id
 
   filter :title_contains, label: 'Title'

app/admin/litigations.rb

@@ -1,9 +1,12 @@
 ActiveAdmin.register Litigation do
-  menu priority: 3
+  menu parent: 'Laws', priority: 2
 
   decorate_with LitigationDecorator
 
-  permit_params :title, :location_id, :document_type, :summary, :core_objective,
+  permit_params :title, :location_id, :jurisdiction_id, :document_type, :summary, :core_objective,
+                litigation_sides_attributes: [
+                  :id, :_destroy, :name, :side_type, :party_type, :connected_with
+                ],
                 documents_attributes: [
                   :id, :_destroy, :name, :external_url, :type, :file
                 ]
@@ -33,6 +36,7 @@
           row :title
           row :slug
           row :location
+          row :jurisdiction
           row :document_type
           row :citation_reference_number
           row :summary
@@ -45,7 +49,7 @@
 
       tab :sides do
         panel 'Litigation Sides' do
-          table_for resource.litigation_sides.order(:side_type).decorate do
+          table_for resource.litigation_sides.decorate do
             column :side_type
             column :name
             column :party_type

app/admin/locations.rb

@@ -1,5 +1,5 @@
 ActiveAdmin.register Location do
-  menu priority: 3
+  menu priority: 1
 
   decorate_with LocationDecorator
 

app/admin/mq_assessments.rb

@@ -1,7 +1,7 @@
 ActiveAdmin.register MQ::Assessment do
   config.sort_order = 'publication_date_desc'
 
-  menu priority: 6, parent: 'Assessments', label: 'Management Quality Assessments'
+  menu parent: 'TPI', priority: 1, label: 'Management Quality Assessments'
 
   decorate_with MQ::AssessmentDecorator
 

app/assets/stylesheets/active_admin.scss

@@ -1,4 +1,5 @@
 @import 'admin/settings';
+@import 'admin/mixins';
 @import 'trix/dist/trix';
 @import 'activeadmin_addons/all';
 
@@ -109,6 +110,13 @@ form {
     list-style: disc;
     color: $am-theme-error;
   }
+
+  fieldset.inputs {
+    > legend {
+      font-size: 18px;
+      font-weight: bold;
+    }
+  }
 }
 
 // Comments
@@ -141,24 +149,22 @@ form {
   }
 }
 
-// Documents
-
-.document-list {
+.nested-list {
   &__actions {
     padding: 16px;
   }
 }
 
 .inline-fields {
   display: flex;
-  justify-content: space-around;
+  justify-content: space-between;
   align-items: center;
 
   li {
     margin: 0;
 
     &.input:not(.error) {
-      input {
+      input, .select2, select {
         margin-bottom: 20px;
       }
     }
@@ -168,3 +174,25 @@ form {
     flex-grow: 1;
   }
 }
+
+.litigation-sides-fields {
+  li[id*=side_type] {
+    min-width: 150px;
+  }
+
+  li[id*=party_type] {
+    min-width: 250px;
+  }
+
+  li[id*=connected_with] {
+    min-width: 300px;
+  }
+
+  li[id*=_name] {
+    flex-grow: 1;
+  }
+
+  @include desktop-large {
+    width: 70%;
+  }
+}

app/assets/stylesheets/admin/mixins.scss

@@ -0,0 +1,17 @@
+@mixin tablet {
+  @media (min-width: $breakpoint-tablet) {
+    @content;
+  }
+}
+
+@mixin desktop {
+  @media (min-width: $breakpoint-desktop) {
+    @content;
+  }
+}
+
+@mixin desktop-large {
+  @media (min-width: $breakpoint-desktop-large) {
+    @content;
+  }
+}

app/assets/stylesheets/admin/settings.scss

@@ -7,3 +7,7 @@ $admin-red: red;
 $am-theme-primary: #00689f;
 $am-theme-accent: #7dd2f7;
 $am-theme-accent-fallback: $admin-red;
+
+$breakpoint-tablet: 768px;
+$breakpoint-desktop: 992px;
+$breakpoint-desktop-large: 1400px;

app/decorators/litigation_side_decorator.rb

@@ -1,11 +1,24 @@
 class LitigationSideDecorator < Draper::Decorator
   delegate_all
 
+  def name
+    return h.link_to model.name, connected_entity_url if connected_entity.present?
+
+    model.name
+  end
+
   def party_type
     model.party_type&.humanize
   end
 
   def side_type
     model.side_type&.humanize
   end
+
+  private
+
+  def connected_entity_url
+    return h.admin_company_path(connected_entity) if connected_entity.is_a?(Company)
+    return h.admin_location_path(connected_entity) if connected_entity.is_a?(Location)
+  end
 end

app/javascript/controllers/litigation_side_controller.js

@@ -0,0 +1,44 @@
+import { Controller } from 'stimulus';
+
+const ENTITY_TYPE_PARTY_TYPE_MAP = {
+  'Company': 'corporation',
+  'Location': 'government'
+};
+
+export default class extends Controller {
+  connect() {
+    $(this.connectedWithSelect).on('change', this._setDefaults.bind(this));
+  }
+
+  _setDefaults() {
+    if (this.connectedWithSelectedOption) {
+      const entityType = this.connectedWithSelectedOption.value.split('-')[0];
+      const partyType = ENTITY_TYPE_PARTY_TYPE_MAP[entityType];
+
+      if (partyType) {
+        $(this.partyTypeSelect).val(partyType);
+        $(this.partyTypeSelect).trigger('change');
+      }
+
+      this.nameInput.value = this.connectedWithSelectedOption.text;
+    }
+  }
+
+  get connectedWithSelect () {
+    return this.element.querySelector('select[id*="_connected_with"');
+  }
+
+  get connectedWithSelectedOption () {
+    return this.connectedWithSelect.selectedOptions &&
+      this.connectedWithSelect.selectedOptions.length &&
+      this.connectedWithSelect.selectedOptions[0];
+  }
+
+  get nameInput() {
+    return this.element.querySelector('input[id*="_name"');
+  }
+
+  get partyTypeSelect() {
+    return this.element.querySelector('select[id*="_party_type"');
+  }
+}

app/javascript/controllers/nested_list_controller.js

@@ -16,6 +16,10 @@ export default class extends Controller {
           .replace(/NEW_RECORD/g, new Date().getTime());
 
     this.linksTarget.insertAdjacentHTML('beforebegin', content);
+
+    // very nasty trick, using dynamic list instead of AA has_many forms
+    // many plugins listen to this event to reinitialize, for example select2 from activeadmin addons
+    $(document).trigger('has_many_add:after');
   }
 
   removeRecord(event) {

app/javascript/controllers/tabs_controller.js

@@ -0,0 +1,24 @@
+import { Controller } from 'stimulus';
+
+export default class extends Controller {
+  static targets = ['activeTab']
+
+  connect() {
+    $(this.element).tabs();
+
+    $(this.element.querySelectorAll('.nav-tabs a')).on('click', (event) => {
+      event.preventDefault();
+      this._changeActiveTabTo(event.target.hash);
+    });
+
+    const activeTab = this.activeTabTarget.value;
+    if (activeTab) {
+      $(this.element).find(`.nav-tabs a[href="${activeTab}"]`).click();
+    }
+  }
+
+  _changeActiveTabTo(hash) {
+    window.location.hash = hash;
+    this.activeTabTarget.value = hash;
+  }
+}

app/models/company.rb

@@ -28,6 +28,8 @@ class Company < ApplicationRecord
 
   has_many :mq_assessments, class_name: 'MQ::Assessment', inverse_of: :company
   has_many :cp_assessments, class_name: 'CP::Assessment', inverse_of: :company
+  has_many :litigation_sides, as: :connected_entity
+  has_many :litigations, through: :litigation_sides
 
   delegate :level, :status, :status_description_short,
            to: :latest_assessment, prefix: :mq, allow_nil: true
@@ -39,4 +41,8 @@ class Company < ApplicationRecord
   def latest_assessment
     mq_assessments.first
   end
+
+  def to_s
+    name
+  end
 end

app/models/litigation.rb

@@ -25,10 +25,12 @@ class Litigation < ApplicationRecord
   enum document_type: array_to_enum_hash(DOCUMENT_TYPES)
 
   belongs_to :location
-  has_many :litigation_sides
+  belongs_to :jurisdiction, class_name: 'Location'
+  has_many :litigation_sides, -> { order(:side_type) }, inverse_of: :litigation
   has_many :documents, as: :documentable, dependent: :destroy
 
   accepts_nested_attributes_for :documents, allow_destroy: true
+  accepts_nested_attributes_for :litigation_sides, allow_destroy: true
 
   validates_presence_of :title, :slug, :document_type
 end