Page MenuHomePhabricator

MetaMTA - more progress to mail app
ClosedPublic

Authored by btrahan on Jun 23 2015, 7:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 6 2024, 11:11 AM
Unknown Object (File)
Feb 6 2024, 10:11 AM
Unknown Object (File)
Feb 3 2024, 5:19 AM
Unknown Object (File)
Feb 2 2024, 8:57 AM
Unknown Object (File)
Jan 7 2024, 7:03 AM
Unknown Object (File)
Dec 20 2023, 11:28 PM
Unknown Object (File)
Dec 18 2023, 2:14 PM
Unknown Object (File)
Dec 17 2023, 12:20 AM
Subscribers

Details

Summary

Ref T5791. This diff adds a "sensitive" flag to PhabricatorMetaMTAMail, defaults it to true in the constructor, and then sets it to false in teh application transaction editor. Assumption here is that sensitive emails are basically all the emails that don't flow through the application transaction editor.

This diff also gets a basic "mail view" page up and going.

This diff also fixes a bug writing recipient edges; the actor was being included.

This bug also fixes a querying bug; we shouldn't do the automagic join of $viewer is recipient or $viewer is actor if folks are querying for recipients or actors already. The bug manifested itself as having the "inbox" be inbox + outbox.

Test Plan

viewd list of messages. viewed message detail.

Diff Detail

Repository
rP Phabricator
Branch
T5791
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6963
Build 6989: [Placeholder Plan] Wait for 30 Seconds
Build 6988: arc lint + arc unit

Event Timeline

btrahan retitled this revision from to MetaMTA - more progress to mail app.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

We may eventually want to do the join twice and potentially end up with something like this:

LEFT JOIN edge viewer_recipient ON ...
JOIN edge query_recipient ON ...
(authorPHID = you OR viewer_recipient.dst = you) AND authorPHID in (%Ls) AND query_recipient.dst IN (%Ls)

But that would only matter for performance (more filtering in SQL instead of PHP) which might never really be an issue. I don't anticipate this UI getting enormous amounts of use.

This revision is now accepted and ready to land.Jun 23 2015, 7:44 PM
This revision was automatically updated to reflect the committed changes.