Differential D326

Case insensitive search for words in Phabricator stories
ClosedPublic
Actions

Authored by dereckson on Mar 5 2016, 05:07.

Details

Reviewers

dereckson

Maniphest Tasks

T773: Words matching shouldn't be case sensitive in PhabricatorPayloadAnalyzer

Commits

rNOTIF2117ad551bef: Case insensitive search for words in Phabricator stories

Summary

When a Phabricator story contains some relevant keywords,
it could be matched to a specific group.

We want to be able to match regardless of the case.

Fixes T773.

Test Plan

A test has been added to address the bug.

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 772.Mar 5 2016, 05:07

dereckson added a task: T773: Words matching shouldn't be case sensitive in PhabricatorPayloadAnalyzer.

dereckson retitled this revision from to Case insensitive search for words in Phabricator stories.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

Build started: https://ci.nasqueron.org/job/test-notifications-phab/116/ (console: https://ci.nasqueron.org/job/test-notifications-phab/116/console)

Build is green https://ci.nasqueron.org/job/test-notifications-phab/116/ for more details.

dereckson accepted this revision.Mar 5 2016, 05:23

dereckson edited edge metadata.

This revision is now accepted and ready to land.Mar 5 2016, 05:23

Closed by commit rNOTIF2117ad551bef: Case insensitive search for words in Phabricator stories (authored by dereckson, committed by dereckson). · Explain WhyMar 5 2016, 05:24

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Analyzers/

Phabricator/

PhabricatorGroupMapping.php

2 lines

tests/

Analyzers/

Phabricator/

PhabricatorGroupMappingTest.php

12 lines

Diff 773

View Options

app/Analyzers/Phabricator/PhabricatorGroupMapping.php