Page MenuHomePhabricator

[lib] Refactor getThreadOtherUsers and all its occurrences
ClosedPublic

Authored by przemek on Nov 18 2022, 7:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 28, 10:49 AM
Unknown Object (File)
Thu, Nov 28, 8:57 AM
Unknown Object (File)
Sun, Nov 24, 8:41 PM
Unknown Object (File)
Fri, Nov 22, 7:59 AM
Unknown Object (File)
Fri, Nov 22, 7:59 AM
Unknown Object (File)
Fri, Nov 22, 7:54 AM
Unknown Object (File)
Fri, Nov 22, 7:45 AM
Unknown Object (File)
Oct 22 2024, 11:52 AM

Details

Summary

I refactored getThreadOtherUsers: changed its name so it matches methods around and
make it return (Relative)MemberInfo's instead of IDs. It was only used in thread-utils.js and
it was easy to changed places where it was called. I need it in that form in typeahead, so I figured
it's fine to extend that function instead of creating another almost identical one.

Test Plan

Tested in furhter diffs.
Visible and works correctly returning all users excluding ashoat and viewer.
Performed sanity check if I didn't break the function (traced one of the functions (getSingleOtherUser) that called it and it performed as expected (in Relationship Prompt))

Diff Detail

Repository
rCOMM Comm
Branch
feat/mention
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul retitled this revision from [lib] Refactor getThreadOtherUsers and all its occurances to [lib] Refactor getThreadOtherUsers and all its occurrences.Nov 18 2022, 1:43 PM
This revision is now accepted and ready to land.Nov 21 2022, 4:15 AM

Please include in the test plan a description of how existing functionality was tested.

Done. It would be nice to have unit tests covering utility functions like that. Probably will mention it during today's monthly planning.