HomePhabricator

Fix a bug with DiffusionMergedCommitsQueryConduitAPIMethod

Description

Fix a bug with DiffusionMergedCommitsQueryConduitAPIMethod

Summary: We were adding "+1" to PHP_INT_MAX, which could cause us to wrap and run git log -n -9223372036854775808 .... Git, unable to give us negative nine trillion zillion results, would then exit without producing results.

Test Plan: Got results out of this method.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D12726

Details

Provenance
epriestleyAuthored on
epriestleyPushed on May 5 2015, 10:44 PM
Reviewer
btrahan
Differential Revision
D12726: Fix a bug with DiffusionMergedCommitsQueryConduitAPIMethod
Parents
rP7e4d5f76c046: Refactor logic for getting next/prev month and day on month and day views…
Branches
Unknown
Tags
Unknown