HomePhorge

Add "does not match regexp" to Herald

Description

Add "does not match regexp" to Herald

Summary:
Fixes T10330.

  • Anywhere we support "matches regexp", also allow "does not match regexp". Although you can sometimes write a clever negative regexp, these rules are better expressed with "does not match <simple regexp>" anyway, and sometimes no regexp will work.
  • Always allow "does not contain" when we support "contains".
  • Fix some JS issues with certain rules affecting custom fields.

Test Plan:

  • Wrote an "Affected files do not match regexp" rule that required every diff to touch "MANUALCHANGELOG.md".
  • Tried to diff without the file; rejected.
  • Tried to diff with the file; accepted.
  • Wrote a bunch of "contains" and "does not contain" rules against text fields and custom fields, then edited tasks to trigger/observe them.
  • Swapped the editor into custom text, user, remarkup, etc fields, no more JS errors.

{F1105172}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10330

Differential Revision: https://secure.phabricator.com/D15254

Details

Provenance
epriestleyAuthored on Feb 11 2016, 3:17 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP86611708191c: Fix a couple typos in "why so many databases?" document
Branches
Unknown
Tags
Unknown

Event Timeline