Project

General

Profile

Actions

Bug #19354

closed

associated_type is set to a wrong value in audit_extensions.rb

Added by Shimon Shtein over 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Audit Log
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

When fixing STI types for :associated field in AuditExtensions#fix_auditable_type, the type is set to the one from :auditable object instead the one from :associated field.
It looks like a copy-paste error.

Actions #1

Updated by Dominic Cleal over 7 years ago

  • Category set to Audit Log

Specifically, I think this means:

self.associated_type = auditable.type if auditable_type == "Taxonomy" && auditable

Uses the auditable type, but sets associated_type.

Actions #2

Updated by Shimon Shtein over 7 years ago

exactly, thanks for clarifying!

Actions #3

Updated by The Foreman Bot over 7 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ori Rabin
  • Pull request https://github.com/theforeman/foreman/pull/4165 added
Actions #4

Updated by Tomer Brisker about 7 years ago

  • Status changed from Ready For Testing to Closed
  • Translation missing: en.field_release set to 240
Actions

Also available in: Atom PDF