Page MenuHomePhabricator

Don't consider integer CustomFields as changed when the number doesn't change
ClosedPublic

Authored by epriestley on Feb 21 2014, 4:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 10, 6:00 AM
Unknown Object (File)
Thu, May 8, 6:08 AM
Unknown Object (File)
Thu, May 8, 6:08 AM
Unknown Object (File)
Thu, May 8, 6:08 AM
Unknown Object (File)
Tue, May 6, 9:10 AM
Unknown Object (File)
Tue, Apr 29, 2:33 AM
Unknown Object (File)
Sun, Apr 27, 7:53 AM
Unknown Object (File)
Apr 8 2025, 7:14 PM
Subscribers

Details

Summary

Fixes T4466. We do an excessively strict effect check now, which means that these fields changing from (string) "16" to (int) 16 generates a transaction. Instead, compare integer values if the field has data in it.

Test Plan

{F116261}

(Also made updates without changing the number, which did not appear in the transaction log anymore.)

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped