Issue #448

Gemfile.lock

@@ -462,6 +462,7 @@ GEM
       sprockets (>= 3.0.0)
     sqlite3 (1.6.8)
       mini_portile2 (~> 2.8.0)
+    sqlite3 (1.6.8-x86_64-linux)
     ssrf_filter (1.1.2)
     stringio (3.1.1)
     strscan (3.1.0)
@@ -497,6 +498,7 @@ GEM
 PLATFORMS
   ruby
   x86-mingw32
+  x86_64-linux
 
 DEPENDENCIES
   active_scaffold!

app/controllers/affiliation_controller.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+class AffiliationController < ApplicationController
+  authorize_resource
+
+  active_scaffold :affiliation do |config|
+    columns = [:professor, :institution, :start_date, :end_date]
+
+    config.list.columns = columns
+    config.create.columns = columns
+    config.update.columns = columns
+    config.show.columns = columns
+
+    config.columns[:professor].form_ui = :record_select
+    config.columns[:institution].form_ui = :record_select
+    config.columns[:start_date].form_ui = :date_picker
+    config.columns[:end_date].form_ui = :date_picker
+  end
+  record_select(
+    per_page: 10,
+    search_on: [:name],
+    order_by: "name",
+    full_text_search: true
+  )
+end

app/controllers/concerns/shared_pdf_concern.rb

@@ -41,7 +41,7 @@ def render_enrollments_academic_transcript_pdf(enrollment)
       .order("course_classes.year", "course_classes.semester")
 
     accomplished_phases = enrollment.accomplishments.order(:conclusion_date)
-
+    program_level = ProgramLevel.date(enrollment.thesis_defense_date).last
     render_to_string(
       template: "enrollments/academic_transcript_pdf",
       type: "application/pdf",
@@ -50,6 +50,7 @@ def render_enrollments_academic_transcript_pdf(enrollment)
         enrollment: enrollment,
         class_enrollments: class_enrollments,
         accomplished_phases: accomplished_phases,
+        program_level: program_level
       }
     )
   end

app/controllers/enrollments_controller.rb

@@ -214,6 +214,7 @@ def to_pdf
 
   def academic_transcript_pdf
     enrollment = Enrollment.find(params[:id])
+
     respond_to do |format|
       format.pdf do
         title = I18n.t("pdf_content.enrollment.academic_transcript.title")

app/controllers/professors_controller.rb

@@ -29,7 +29,7 @@ class ProfessorsController < ApplicationController
     config.columns[:civil_status].options = {
       options: [["Solteiro(a)", "solteiro"], ["Casado(a)", "casado"]]
     }
-    config.columns[:institution].form_ui = :record_select
+    config.nested.add_link(:affiliations)
     config.columns[:sex].form_ui = :select
     config.columns[:sex].options = { options: [["Masculino", "M"],
                                                ["Feminino", "F"]] }
@@ -53,9 +53,9 @@ class ProfessorsController < ApplicationController
       :address, :zip_code, :telephone1, :telephone2, :cpf,
       :identity_expedition_date, :identity_issuing_body,
       :identity_issuing_place, :identity_number, :enrollment_number,
-      :siape, :institution, :scholarships, :academic_title_level,
+      :siape, :scholarships, :academic_title_level,
       :academic_title_institution, :academic_title_country,
-      :academic_title_date, :obs, :professor_research_areas,
+      :academic_title_date, :obs, :professor_research_areas, :affiliations
     ]
 
     config.create.columns = form_columns

app/helpers/enrollments_pdf_helper.rb

@@ -132,12 +132,12 @@ def grades_report_header(pdf, options = {})
 
   def enrollment_header(pdf, options = {})
     enrollment ||= options[:enrollment]
+    program_level ||= options[:program_level]
     pdf.bounding_box([0, pdf.cursor - 3], width: 560) do
       pdf.font("FreeMono", size: 8) do
         pdf.line_width 0.5
-
         common_header_part1(pdf, enrollment, [
-          "#{i18n_eht(:program_level)} <b>#{CustomVariable.program_level} </b>"
+          "#{i18n_eht(:program_level)} <b>#{program_level.level}</b>"
         ])
 
         common_header_part(pdf) do
@@ -614,10 +614,13 @@ def thesis_table(curr_pdf, options = {})
           data_table_rows_defense_committee += [[
             "#{I18n.t("pdf_content.enrollment.thesis.defense_committee")} "
           ]]
+          # TODO: modificar o institution para a affiliation
           thesis_desense_committee.each do |professor|
+            affiliation = professor.affiliations.date(thesis_defense_date).last
+            binding.pry
             data_table_rows_defense_committee += [[
               "<b>#{professor.name} / #{rescue_blank_text(
-                professor.institution, method_call: :name
+                affiliation.institution, method_call: :name
               )}</b>"
             ]]
           end

app/models/affiliation.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+class Affiliation < ApplicationRecord
+  has_paper_trail
+
+  belongs_to :institution, optional: true
+  belongs_to :professor, optional: true
+
+  validates :start_date, presence: true
+  validates :end_date, presence: false
+  validates_uniqueness_of :start_date, scope: [:professor_id],
+                          allow_nil: true, allow_blank: true,
+                          message: "A afiliação só pode ser iniciada em uma data por professor"
+  validate :uniqueness_end_date
+
+  scope :date, ->(date) { where("start_date <= ? AND (end_date > ? OR end_date IS null)", date, date) }
+  scope :professor, ->(professor) { where(professor_id: professor.id) }
+  scope :date_professor, ->(professor, date) { professor(professor).date(date) }
+
+  private
+
+  def uniqueness_end_date
+    exists = Affiliation.where(professor_id: professor_id, end_date: end_date).where.not(id: id).exists?
+    if exists
+      errors.add(:end_date,"Apenas uma afiliação pode estar ativa por professor e só pode ter uma data de fim por professor")
+    end
+    exists
+  end
+
+end

app/models/custom_variable.rb

@@ -12,7 +12,6 @@ class CustomVariable < ApplicationRecord
   VARIABLES = {
     "single_advisor_points" => :text,
     "multiple_advisor_points" => :text,
-    "program_level" => :text,
     "identity_issuing_country" => :text,
     "class_schedule_text" => :text,
     "redirect_email" => :text,
@@ -42,11 +41,6 @@ def self.multiple_advisor_points
     config.blank? ? 0.5 : config.value.to_f
   end
 
-  def self.program_level
-    config = CustomVariable.find_by_variable(:program_level)
-    config.blank? ? nil : config.value.to_i
-  end
-
   def self.identity_issuing_country
     config = CustomVariable.find_by_variable(:identity_issuing_country)
     config.blank? ? "" : config.value

app/models/institution.rb

@@ -8,7 +8,9 @@ class Institution < ApplicationRecord
   has_paper_trail
 
   has_many :majors, dependent: :restrict_with_exception
-  has_many :professors, dependent: :restrict_with_exception
+  has_many :affiliations, dependent: :destroy
+  has_many :professors, through: :affiliations, dependent: :destroy
+
 
   validates :name, presence: true, uniqueness: true
 

app/models/professor.rb

@@ -18,9 +18,11 @@ class Professor < ApplicationRecord
     dependent: :restrict_with_exception
   has_many :thesis_defense_committee_enrollments,
     source: :enrollment, through: :thesis_defense_committee_participations
+  has_many :affiliations, dependent: :restrict_with_exception
+  has_many :institutions, through: :affiliations, dependent: :restrict_with_exception
+  accepts_nested_attributes_for :affiliations, allow_destroy: true, reject_if: :all_blank
 
   belongs_to :city, optional: true
-  belongs_to :institution, optional: true
   belongs_to :academic_title_country,
     optional: true,
     class_name: "Country",

app/models/program_level.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+class ProgramLevel < ApplicationRecord
+  has_paper_trail
+
+  validates :level, presence: true, on: [:create, :update]
+  validates :start_date, presence: true, on: [:create, :update]
+  validates :end_date, presence: false
+
+  scope :date, -> (date) { where("program_levels.start_date <= ? AND program_levels.end_date > ? OR program_levels.end_date is null", date, date)}
+
+end

app/views/enrollments/_show_defense_committee_table.html.erb

@@ -2,7 +2,7 @@
   <thead>
     <tr>
       <th><%= I18n.t("activerecord.attributes.professor.name") %></th>
-      <th><%= I18n.t("activerecord.attributes.professor.institution") %></th>
+      <th><%= I18n.t("activerecord.attributes.professor.institution.one") %></th>
     </tr>
   </thead>
   <tbody class="records">
@@ -12,7 +12,7 @@
       <% tr_class = count.even? ? "even-record" : "" %>
       <tr class="record <%= tr_class %>">
         <td><%= professor.name %></td>
-        <td><%= rescue_blank_text(professor.institution, method_call: :name) %></td>
+        <td><%= rescue_blank_text(professor.affiliations.date(thesis_defense_date), method_call: :name) %></td>
       </tr>
     <% end %>
   </tbody>

app/views/enrollments/academic_transcript_pdf.pdf.prawn

@@ -15,7 +15,7 @@ new_document(
 ) do |pdf|
   enrollment_student_header(pdf, enrollment: @enrollment)
 
-  enrollment_header(pdf, enrollment: @enrollment)
+  enrollment_header(pdf, enrollment: @enrollment, program_level: @program_level)
 
   transcript_table(pdf, class_enrollments: @class_enrollments)
 

app/views/student_enrollment/_show_dismissal.html.erb

@@ -34,7 +34,8 @@
     <% unless thesis_defense_committee_professors.empty? %>
       <h4><%= t "activerecord.attributes.enrollment.thesis_defense_committee_professors" %></h4>
       <%= render partial: "enrollments/show_defense_committee_table", locals: {
-        thesis_defense_committee_professors: thesis_defense_committee_professors
+        thesis_defense_committee_professors: thesis_defense_committee_professors,
+        thesis_defense_date: enrollment.thesis_defense_date,
       } %>
     <% end %>
   </details>

config/application.rb

@@ -35,9 +35,11 @@ class Application < Rails::Application
     # config.autoload_paths += %W(#{config.root}/extras)
     config.autoload_paths << "#{config.root}/lib"
 
+
     # Only load the plugins named here, in the order given (default is alphabetical).
     # :all can be used as a placeholder for all plugins not explicitly named.
     # config.plugins = [ :exception_notification, :ssl_requirement, :all ]
+    config.active_record.yaml_column_permitted_classes = [Symbol, Date, Time]
 
     # Activate observers that should always be running.
     # config.active_record.observers = :cacher, :garbage_collector, :forum_observer

config/locales/affiliation.pt-BR.yml

@@ -0,0 +1,17 @@
+# Copyright (c) Universidade Federal Fluminense (UFF).
+# This file is part of SAPOS. Please, consult the license terms in the LICENSE file.
+
+pt-BR:
+  activerecord:
+    attributes:
+      affiliation:
+        institution: "Instituição"
+        professor: "Professor"
+        start_date: "Data de início"
+        end_date: "Data de fim"
+        active: "Ativo"
+
+    models:
+      affiliation:
+        one: "Afiliação"
+        other: "Afiliações"

config/locales/institution.pt-BR.yml

@@ -8,6 +8,7 @@ pt-BR:
         code: "Sigla"
         majors: "Cursos"
         name: "Nome"
+        affiliations: "Afiliações"
 
     models:
       institution:

config/locales/professor.pt-BR.yml

@@ -21,7 +21,10 @@ pt-BR:
         identity_issuing_body: "Órgão Expeditor"
         identity_issuing_place: "Local de Expedição"
         identity_number: "Número da identidade"
-        institution: "Instituição"
+        affiliations: "Afiliações"
+        institution:
+          one: "Instituição"
+          other: "Instituições"
         name: "Nome"
         neighborhood: "Bairro"
         professor_research_areas: "Áreas de Pesquisa"

config/routes.rb

@@ -160,6 +160,11 @@
     end
   end
 
+  resources :affiliation do
+    concerns :active_scaffold
+    record_select_routes
+  end
+
   resources :professors do
     concerns :active_scaffold
     record_select_routes

db/migrate/20240524135850_create_affiliation.rb

@@ -0,0 +1,59 @@
+class CreateAffiliation < ActiveRecord::Migration[7.0]
+  def up
+    create_table :affiliations do |t|
+      t.belongs_to :professor, index: true
+      t.belongs_to :institution, index: true
+      t.date :start_date
+      t.date :end_date
+
+      t.timestamps
+    end
+    Professor.all.each do |professor|
+      start_date = professor.updated_at
+      end_date = nil
+      institution_id = professor.institution_id
+      institutions = []
+      affiliation = Affiliation.create(
+        professor: professor,
+        institution_id: institution_id,
+        start_date: start_date
+      )
+      professor = professor.paper_trail.previous_version
+      while professor.present?
+        if professor.institution_id != institution_id
+          if institution_id.present? # Não armazena afiliação em branco
+            institutions << { institution_id:, start_date:, end_date: }
+          end
+          end_date = start_date
+          start_date = professor.updated_at
+          institution_id = professor.institution_id
+          affiliation = Affiliation.create(
+            professor: professor,
+            institution_id: institution_id,
+            start_date: start_date,
+            end_date: end_date
+          )
+        else
+          # Atualiza data de início para data da primeira mudança em que instituição foi definida com esse valor
+          start_date = professor.updated_at
+          affiliation.update(start_date: start_date)
+        end
+        professor = professor.paper_trail.previous_version
+      end
+      if institution_id.present? # Não armazena afiliação em branco
+        institutions << { institution_id:, start_date:, end_date: }
+      end
+    end
+
+    remove_column :professors, :institution_id
+  end
+  def down
+    add_column :professors, :institution_id, :integer
+    add_index :professors, :institution_id
+    Professor.all.each do |p|
+      affiliation = Affiliation.professor(p).where(active: true).last
+      p.update(institution_id: affiliation.institution_id)
+    end
+    drop_table :affiliations
+  end
+end