Diffusion Phabricator c0bf08058b77

Check for empty output from git ls-tree
Auditedc0bf08058b77
Actions

Description

Check for empty output from git ls-tree

Summary: Fixes T10155

Test Plan: View an empty repository in diffusion, check for the exception.
See T10155 for steps to reproduce

Reviewers: epriestley

Subscribers:

Details

Auditors

epriestley

Group Auditors

Cowboy Commits

Provenance

20after4	Authored on Sep 9 2016, 2:04 AM
20after4	Pushed on Sep 10 2016, 11:14 AM

Differential Revision

D16525: Check for empty output from git ls-tree

Parents

rP05f5653091ee: Lots of favicons

Branches

Unknown

Tags

Unknown

Tasks

T10155: Error when displaying a repository with only one empty commit

Build Status

Buildable 13674
Build 17634: Run Core Tests

Event Timeline

20after4 added a task: T10155: Error when displaying a repository with only one empty commit.Sep 10 2016, 11:14 AM

20after4 committed rPc0bf08058b77: Check for empty output from git ls-tree (authored by 20after4).Sep 10 2016, 11:02 AM

Herald added auditors: Cowboy Commits. · View Herald TranscriptSep 10 2016, 11:14 AM

Harbormaster completed building B13674: rPc0bf08058b77: Check for empty output from git ls-tree.Sep 10 2016, 11:15 AM

dfumagalli mentioned this in T11613: New install exception is wrong if preflight setup checks fail.Sep 10 2016, 7:12 PM

20after4 added an edge: D16525: Check for empty output from git ls-tree.Oct 13 2016, 6:22 PM

epriestley accepted this commit.Jan 31 2017, 9:46 PM

All concerns with this commit have now been addressed.Jan 31 2017, 9:46 PM

Changes (1)

Path

Size

src/

applications/

diffusion/

conduit/

DiffusionBrowseQueryConduitAPIMethod.php

rPc0bf08058b77

View Options

src/applications/diffusion/conduit/DiffusionBrowseQueryConduitAPIMethod.php