Page MenuHomePhabricator

Allow atoms to be queried by book
ClosedPublic

Authored by joshuaspence on Jun 16 2015, 10:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 21, 6:52 AM
Unknown Object (File)
Sun, Mar 10, 12:19 PM
Unknown Object (File)
Sun, Mar 10, 12:18 PM
Unknown Object (File)
Sun, Mar 3, 4:49 AM
Unknown Object (File)
Sun, Mar 3, 4:49 AM
Unknown Object (File)
Sun, Mar 3, 4:48 AM
Unknown Object (File)
Feb 23 2024, 2:05 PM
Unknown Object (File)
Feb 12 2024, 4:26 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4558: Make Diviner useful for third-parties
Commits
Restricted Diffusion Commit
rP9921cbc41ac3: Allow atoms to be queried by book
Summary

Ref T4558. Allows querying for atoms from specified books. Depends on D13091.

Test Plan

Poked around at /diviner/query/.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6873
Build 6895: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Allow atoms to be queried by book.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence added a parent revision: D13091: Modernize Diviner.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/diviner/markup/DivinerSymbolRemarkupRule.php
117–121 ↗(On Diff #32187)

I think we're generally better off not doing this, and that preserving git blame / having smaller diffs is more valuable on the balance than having statements aligned.

In a perfect world, git blame would ignore these changes (maybe as arc blame), and Differential would be smart enough to tell that you were only doing an alignment change, even though you were changing intraline whitespace, and ignore it. But I don't expect us to get there for a long time.

(In this specific case 'book' is not correctly aligned.)

src/applications/diviner/query/DivinerBookQuery.php
192

This is maybe a little confusing because name can be zebrafacts ("Z") and the title can be A Collection of Fine Facts about Zebras ("A" or "C", depending on how smart we want to be), which puts the book at totally separate ends of the list.

Ideally we would probably define a sortableTitle column, strip common articles ("A", "An", "The") in some central algorithm, dump the rest of the title in there in a normalized form, then sort by that.

This can happen some time far in the future, though.

This revision is now accepted and ready to land.Jun 16 2015, 1:58 PM
This revision was automatically updated to reflect the committed changes.