Page MenuHomePhabricator
Differential D11251

Continue after rejecting commits from a commit query
ClosedPublic
Actions

Authored by epriestley on Jan 6 2015, 3:36 PM.
Tags
None
Referenced Files
F14381911: D11251.diff
Sat, Dec 21, 5:39 AM
Unknown Object (File)
Thu, Dec 19, 1:05 AM
Unknown Object (File)
Tue, Dec 17, 2:50 AM
Unknown Object (File)
Sun, Dec 8, 6:45 PM
Unknown Object (File)
Fri, Dec 6, 4:00 AM
Unknown Object (File)
Thu, Dec 5, 12:01 AM
Unknown Object (File)
Wed, Nov 27, 2:04 PM
Unknown Object (File)
Fri, Nov 22, 10:14 PM
View All Files
Subscribers
epriestley
mbishopim3

Details

Reviewers
mbishopim3
fabe
btrahan
Maniphest Tasks
T6880: "Attempting to access attached data on PhabricatorRepositoryCommit (via getRepository())" error when creating or accessing diffs
Commits
Restricted Diffusion Commit
rPd4f992d2ed09: Continue after rejecting commits from a commit query
Summary

Fixes T6880. If matching commits have no visible/loadable repository, we shouldn't keep going forward in the loop.

Test Plan

Havne't built a repro locally yet so not 100% sure this fixes it.

Diff Detail

Repository
rP Phabricator
Branch
norepo
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3634
Build 3643: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 27005.Jan 6 2015, 3:36 PM
epriestley retitled this revision from to Continue after rejecting commits from a commit query.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: fabe, btrahan.
epriestley added a task: T6880: "Attempting to access attached data on PhabricatorRepositoryCommit (via getRepository())" error when creating or accessing diffs.
Herald added a subscriber: epriestley. · View Herald TranscriptJan 6 2015, 3:36 PM
mbishopim3 accepted this revision.Jan 6 2015, 3:51 PM
mbishopim3 added a reviewer: mbishopim3.
mbishopim3 added a subscriber: mbishopim3.

Confirmed working for our users.

This revision is now accepted and ready to land.Jan 6 2015, 3:51 PM
fabe accepted this revision.Jan 6 2015, 3:55 PM
fabe edited edge metadata.

I've reproduced it locally and this fixes it. Before the patch these were two separate foreaches.

We could still link the commit (like we do with a repo that is not accessible) and make it show "restricted" instead.
With this patch the commits would simply ne be linked at all.

Closed by commit rPd4f992d2ed09: Continue after rejecting commits from a commit query (authored by epriestley, committed by epriestley). · Explain WhyJan 6 2015, 4:02 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
query/
1 line

Diff 27005

View Options

src/applications/diffusion/query/DiffusionCommitQuery.php

Loading...
Phabricator · Built by Phacility