Page MenuHomePhabricator

Support "only the first time" in Maniphest
ClosedPublic

Authored by epriestley on Feb 11 2014, 12:48 AM.
Tags
None
Referenced Files
F14067324: D8193.diff
Tue, Nov 19, 2:20 PM
F14008210: D8193.diff
Tue, Oct 29, 5:38 PM
F13984748: D8193.id18538.diff
Oct 20 2024, 3:38 PM
F13973060: D8193.diff
Oct 17 2024, 10:45 PM
F13973055: D8193.id.diff
Oct 17 2024, 10:42 PM
F13973052: D8193.id18535.diff
Oct 17 2024, 10:41 PM
F13973050: D8193.id18538.diff
Oct 17 2024, 10:40 PM
F13964992: D8193.id.diff
Oct 15 2024, 11:13 PM
Subscribers
Tokens
"Love" token, awarded by dctrwatson.

Details

Summary

Ref T4403. Implements "only the first time" for Maniphest rules, and fixes the trigger itself.

The trigger would never fire and block rules because it was comparing a string (like "first") to an int (like 0).

The "only" vs "every" stuff is contributed and I should have pushed back harder on this toInt / toString stuff. Maybe I'll just get rid of it; it purely causes confusion and problems.

Test Plan

Wrote an "only the first time" rule, ran it twice, it applied once.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped