Files
Kabiru Mwenja 59a631e983 Scope dirty-state reset to identifier and sequence_number
clear_changes_information would also clobber any unrelated dirty
attributes the work package happened to be carrying at this point.
The intent is narrower: only the two attributes we just synced from
the raw-SQL UPDATE should be marked clean. Use clear_attribute_changes
with the explicit attribute list instead.
2026-05-05 15:55:28 +03:00

188 lines
6.8 KiB
Ruby

# frozen_string_literal: true
#-- copyright
# OpenProject is an open source project management software.
# Copyright (C) the OpenProject GmbH
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License version 3.
#
# OpenProject is a fork of ChiliProject, which is a fork of Redmine. The copyright follows:
# Copyright (C) 2006-2013 Jean-Philippe Lang
# Copyright (C) 2010-2013 the ChiliProject Team
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License
# as published by the Free Software Foundation; either version 2
# of the License, or (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
#
# See COPYRIGHT and LICENSE files for more details.
#++
class WorkPackages::UpdateService < BaseServices::Update
include ::WorkPackages::Shared::UpdateAncestors
include Attachments::ReplaceAttachments
include Types::ApplyPatterns
attr_accessor :cause_of_rescheduling
def initialize(user:, model:, contract_class: nil, contract_options: {}, cause_of_rescheduling: nil)
super(user:, model:, contract_class:, contract_options:)
self.cause_of_rescheduling = cause_of_rescheduling || model
end
private
def after_perform(service_call)
# TODO: code smell here: saving the automatically generated subject depends
# on running the UpdateAncestorsService right after. The subject gets saved
# only thanks to this. If the UpdateAncestorsService is not run, the subject
# is not saved. That's an odd coupling.
apply_patterns(service_call.result, save: false)
update_related_work_packages(service_call)
cleanup(service_call.result)
service_call
end
def update_related_work_packages(service_call)
work_package = service_call.result
changed_attributes = work_package.changed_attribute_keys_before_last_save
update_ancestors(work_package, changed_attributes).tap do |ancestor_service_call|
ancestor_service_call.dependent_results.each do |ancestor_dependent_service_call|
service_call.add_dependent!(ancestor_dependent_service_call)
end
end
# update saved changes as they might have changed due to the ancestors updates
changed_attributes += work_package.changed_attribute_keys_before_last_save
changed_attributes.uniq!
update_related(work_package, changed_attributes).each do |related_service_call|
service_call.add_dependent!(related_service_call)
end
end
def update_related(work_package, changed_attributes)
consolidated_calls(update_descendants(work_package) + reschedule_related(work_package, changed_attributes))
.each { |dependent_call| dependent_call.result.save(validate: false) }
end
def update_descendants(work_package)
if work_package.saved_change_to_project_id?
attributes = { project: work_package.project }
work_package.descendants.map do |descendant|
set_descendant_attributes(attributes, descendant)
end
else
[]
end
end
def set_descendant_attributes(attributes, descendant)
WorkPackages::SetAttributesService
.new(user:,
model: descendant,
contract_class: WorkPackages::UpdateDependentContract)
.call(attributes)
end
def cleanup(work_package)
if work_package.saved_change_to_project_id?
moved_work_packages = [work_package] + work_package.descendants
delete_relations(moved_work_packages)
move_time_entries(moved_work_packages, work_package.project_id)
move_work_package_memberships(moved_work_packages, work_package.project_id)
update_semantic_ids(moved_work_packages) if Setting::WorkPackageIdentifier.semantic?
end
if work_package.saved_change_to_type_id?
reset_custom_values(work_package)
end
end
def update_semantic_ids(work_packages)
return if work_packages.empty?
# reserve_semantic_id_block! writes via raw SQL UPDATE, so the in-memory
# records still carry the nil identifier left by SetAttributesService.
# Apply the returned assignments in-memory so callers (HAL representers,
# redirect helpers) see the freshly allocated semantic id without N reloads.
assignments = work_packages.first.project.reserve_semantic_id_block!(work_packages.map(&:id))
work_packages.each do |wp|
next unless (identifier = assignments[wp.id])
wp.assign_attributes(identifier:, sequence_number: identifier.split("-").last.to_i)
wp.clear_attribute_changes(%i[identifier sequence_number])
end
end
def delete_relations(work_packages)
unless Setting.cross_project_work_package_relations?
Relation
.of_work_package(work_packages)
.destroy_all
end
end
def move_time_entries(work_packages, project_id)
TimeEntry
.on_work_packages(work_packages)
.update_all(project_id:)
end
def move_work_package_memberships(work_packages, project_id)
Member
.where(entity: work_packages)
.update_all(project_id:)
end
def reset_custom_values(work_package)
work_package.reset_custom_values!
end
def reschedule_related(work_package, changed_attributes)
work_packages_to_reschedule = [work_package]
# if parent changed, the former parent needs to be rescheduled too.
if parent_just_changed?(work_package)
former_parent = WorkPackage.visible(user).find_by(id: work_package.parent_id_before_last_save)
work_packages_to_reschedule << former_parent if former_parent
end
WorkPackages::SetScheduleService
.new(user:, work_package: work_packages_to_reschedule, initiated_by: cause_of_rescheduling)
.call(changed_attributes)
.dependent_results
end
def parent_just_changed?(work_package)
work_package.saved_change_to_parent_id? && work_package.parent_id_before_last_save
end
# When multiple services change a work package, we still only want one update to the database due to:
# * performance
# * having only one journal entry
# * stale object errors
# we thus consolidate the results so that one instance contains the changes made by all the services.
def consolidated_calls(service_calls)
service_calls
.group_by { |sc| sc.result.id }
.map do |(_, same_work_package_calls)|
same_work_package_calls.pop.tap do |master|
same_work_package_calls.each do |sc|
master.result.attributes = sc.result.changes.transform_values(&:last)
end
end
end
end
end