[#6441] Refactor: merge i18n strings by passing in a reason and object
This commit is contained in:
@@ -2,10 +2,7 @@ en:
|
||||
activerecord:
|
||||
errors:
|
||||
messages:
|
||||
cant_create_time_on_closed_deliverable: "Can't create a time entry on a closed deliverable"
|
||||
cant_create_time_on_closed_contract: "Can't create a time entry on a closed contract"
|
||||
cant_create_time_on_locked_deliverable: "Can't create a time entry on a locked deliverable"
|
||||
cant_create_time_on_locked_contract: "Can't create a time entry on a locked contract"
|
||||
cant_create_time_on_object: "Can't create a time entry on a %{reason} %{thing}"
|
||||
cant_assign_to_closed_deliverable: "Can't assign issue to a closed deliverable"
|
||||
cant_assign_to_locked_deliverable: "Can't assign issue to a locked deliverable"
|
||||
cant_update_locked_deliverable: "Can't update a locked deliverable"
|
||||
|
||||
@@ -13,15 +13,15 @@ module RedmineContracts
|
||||
|
||||
def validate_deliverable_status
|
||||
if issue.present? && issue.deliverable.present?
|
||||
errors.add_to_base(:cant_create_time_on_locked_deliverable) if issue.deliverable.locked?
|
||||
errors.add_to_base(:cant_create_time_on_closed_deliverable) if issue.deliverable.closed?
|
||||
errors.add_to_base("#{l(:"activerecord.errors.messages.cant_create_time_on_object", :reason => 'locked', :thing => 'deliverable')}") if issue.deliverable.locked?
|
||||
errors.add_to_base("#{l(:"activerecord.errors.messages.cant_create_time_on_object", :reason => 'closed', :thing => 'deliverable')}") if issue.deliverable.closed?
|
||||
end
|
||||
end
|
||||
|
||||
def validate_contract_status
|
||||
if issue.present? && issue.deliverable.present? && issue.deliverable.contract.present?
|
||||
errors.add_to_base(:cant_create_time_on_locked_contract) if issue.deliverable.contract.locked?
|
||||
errors.add_to_base(:cant_create_time_on_closed_contract) if issue.deliverable.contract.closed?
|
||||
errors.add_to_base("#{l(:"activerecord.errors.messages.cant_create_time_on_object", :reason => 'locked', :thing => 'contract')}") if issue.deliverable.contract.locked?
|
||||
errors.add_to_base("#{l(:"activerecord.errors.messages.cant_create_time_on_object", :reason => 'closed', :thing => 'contract')}") if issue.deliverable.contract.closed?
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
Reference in New Issue
Block a user