Homec4science

Add "does not match regexp" to Herald

Authored by epriestley <git@epriestley.com> on Feb 12 2016, 00:17.

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

Committed
epriestley <git@epriestley.com>Feb 12 2016, 00:29
Pushed
aubortJan 31 2017, 17:16
Parents
rPH86611708191c: Fix a couple typos in "why so many databases?" document
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH8934dee5437b: Add "does not match regexp" to Herald (authored by epriestley <git@epriestley.com>).Feb 12 2016, 00:29