From 25de4bcf2581fe075e4c26298eebae70d0d855ec Mon Sep 17 00:00:00 2001
From: alexdbondoc17 <alexdbondoc17@gmail.com>
Date: Thu, 24 Feb 2022 11:20:44 +0000
Subject: [PATCH] Initial viewing updates

---
 .../document_advanced_search_component.html.erb        |  4 ++--
 app/components/subject_index_form_component.rb         |  4 ++++
 .../subject_index_form_component.html.erb              |  6 ++++--
 app/controllers/concerns/doctrine_search.rb            |  8 ++++++++
 app/controllers/reports_controller.rb                  |  2 +-
 app/models/cdao/subject.rb                             |  2 +-
 app/views/reports/index.html.erb                       | 10 ++++++++--
 app/views/subject_indexes/show.html.erb                |  5 ++++-
 8 files changed, 32 insertions(+), 9 deletions(-)

diff --git a/app/components/document_advanced_search_component/document_advanced_search_component.html.erb b/app/components/document_advanced_search_component/document_advanced_search_component.html.erb
index 12d8fb6..0d0d2d1 100644
--- a/app/components/document_advanced_search_component/document_advanced_search_component.html.erb
+++ b/app/components/document_advanced_search_component/document_advanced_search_component.html.erb
@@ -15,7 +15,7 @@
 
   <div class="mb-3">
     <label for="exampleFormControlInput1" class="form-label text-white">Year</label>
-    <%= select_tag :year, options_for_select(years, params[:year]), class: "form-select" %>
+    <%= select_tag :year, options_for_select(years, params[:year]), class: "form-select", include_blank: true %>
   </div>
 
   <div class="mb-3">
@@ -27,4 +27,4 @@
     <button  type="button" class="btn btn-success  me-2 mb-3">Refresh</button>
     <button type="submit" class="btn btn-danger mb-3">Search</button>
   </div>
-<% end %>
\ No newline at end of file
+<% end %>
diff --git a/app/components/subject_index_form_component.rb b/app/components/subject_index_form_component.rb
index e094aa2..e4a2333 100644
--- a/app/components/subject_index_form_component.rb
+++ b/app/components/subject_index_form_component.rb
@@ -7,6 +7,10 @@ class SubjectIndexFormComponent < BaseComponent
 		@parent_id = @opts[:parent_id]
 	end
 
+	def parent
+		Cdao::Subject.find(parent_id)
+	end
+
 	def current_library
 		Cdao::Library.first
 	end
diff --git a/app/components/subject_index_form_component/subject_index_form_component.html.erb b/app/components/subject_index_form_component/subject_index_form_component.html.erb
index 27aa534..5ded64e 100644
--- a/app/components/subject_index_form_component/subject_index_form_component.html.erb
+++ b/app/components/subject_index_form_component/subject_index_form_component.html.erb
@@ -7,11 +7,13 @@
 		</div>
 	</div>
 
-	<% if parent_id.present? %>
+	<% if parent_id.present? || subject_index.persisted? %>
 		<div cass="row">
 			<div class="col-sm-12">
 				<strong> <%= label_tag :subjects %> </strong>
-				<select class="form-control subjects-selectize ps-0 pe-0" name="parent_id" id="parent_id" value="<%= parent_id %>"> </select>
+				<select class="form-control subjects-selectize ps-0 pe-0" name="parent_id" id="parent_id" value="<%= parent_id %>">
+					<option value="<%= parent_id %>" selected="selected"> <%= parent.name %> </option>
+				</select>
 			</div>
 		</div>
 	<% end %>
diff --git a/app/controllers/concerns/doctrine_search.rb b/app/controllers/concerns/doctrine_search.rb
index ff23f14..efa4e22 100644
--- a/app/controllers/concerns/doctrine_search.rb
+++ b/app/controllers/concerns/doctrine_search.rb
@@ -13,6 +13,14 @@ module DoctrineSearch
         with(:created_at).between(Date.parse(search_params[:created_at])..Time.zone.today.to_date)
       end
 
+      if search_params[:created_at_start].present? && search_params[:created_at_end].blank?
+        with(:created_at).between(Date.parse(search_params[:created_at_start])..Time.zone.today.to_date)
+      end
+
+      if search_params[:created_at_start].present? && search_params[:created_at_end].present?
+        with(:created_at).between(Date.parse(search_params[:created_at_start])..Date.parse(search_params[:created_at_end]))
+      end
+
       without(:id).any_of(search_params[:exclude_ids]) if search_params[:exclude_ids].present?
 
       paginate page: search_params[:page] || 1, per_page: search_params[:per_page] || 20
diff --git a/app/controllers/reports_controller.rb b/app/controllers/reports_controller.rb
index 47b63d5..46e387b 100644
--- a/app/controllers/reports_controller.rb
+++ b/app/controllers/reports_controller.rb
@@ -12,6 +12,6 @@ class ReportsController < ApplicationController
   private
 
     def search_params
-      params.permit(:created_at, :q, :page, :per_page, exclude_ids: [], subject_ids: [], user_ids: [])
+      params.permit(:created_at, :created_at_start, :created_at_end, :q, :page, :per_page, exclude_ids: [], subject_ids: [], user_ids: [])
     end
 end
diff --git a/app/models/cdao/subject.rb b/app/models/cdao/subject.rb
index a170873..25da037 100644
--- a/app/models/cdao/subject.rb
+++ b/app/models/cdao/subject.rb
@@ -41,7 +41,7 @@ class Cdao::Subject < Cdao::Base
   end
 
   def lineage_name
-    path.map(&:name).join("; ")
+    path.map(&:name).join(" > ")
   end
 
   def name_with_product_names
diff --git a/app/views/reports/index.html.erb b/app/views/reports/index.html.erb
index 555fd7d..35a9f1e 100644
--- a/app/views/reports/index.html.erb
+++ b/app/views/reports/index.html.erb
@@ -11,9 +11,15 @@
           <select class="form-control subject-ids-selectize p-0" name="subject_ids[]" id="subject_ids_" multiple="true"> </select>
         </div>
         <div class="mb-3">
-          <label for="exampleFormControlInput1" class="form-label text-white"> Date Created At </label>
-          <%= text_field_tag :created_at, params[:created_at], class: "form-control  default-flatpickr" %>
+          <label for="exampleFormControlInput1" class="form-label text-white"> Date Created (Start) </label>
+          <%= text_field_tag :created_at_start, params[:created_at_start], class: "form-control  default-flatpickr" %>
         </div>
+
+        <div class="mb-3">
+          <label for="exampleFormControlInput1" class="form-label text-white"> Date Created (End) </label>
+          <%= text_field_tag :created_at_end, params[:created_at_end], class: "form-control  default-flatpickr" %>
+        </div>
+
         <hr class="simple">
         <div class="mb-3">
           <label for="exampleFormControlInput1" class="form-label text-white"> Users </label>
diff --git a/app/views/subject_indexes/show.html.erb b/app/views/subject_indexes/show.html.erb
index 935e905..ccd954c 100644
--- a/app/views/subject_indexes/show.html.erb
+++ b/app/views/subject_indexes/show.html.erb
@@ -25,7 +25,10 @@
 
       <div class="col-sm-2 d-flex justify-content-end ">
         <a class="btn btn-secondary me-3 btn-sm" href="<%= edit_subject_index_path(@subject_index) %>"> Edit </a>
-        <a class="btn btn-danger btn-sm" href="<%= subject_index_path(@subject_index) %>" data-confirm="Are you sure to delete this record?" data-method="DELETE"> Delete </a>
+        
+        <% if !@subject_index.has_children? %>
+          <a class="btn btn-danger btn-sm" href="<%= subject_index_path(@subject_index) %>" data-confirm="Are you sure to delete this record?" data-method="DELETE"> Delete </a>
+        <% end %>
       </div>
     </div>
   </div>