revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-22 23:02:42 +01:00
Code Issues Releases Wiki Activity

Fix null test plan database error

Browse source 
Summary:
Some Differential fields are not nullable; when Test Plan is switched to non-required mode we can end up trying to save a null value to a non-nullable column (see D2193).

(I should probably just alter the schema to make these fields nullable, but that might have farther-reaching effects.)

Test Plan: Reproduced error, applied patch, no more error.

Reviewers: btrahan, vrana, jungejason

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D2200
This commit is contained in:
epriestley 2012-04-10 16:54:05 -07:00
parent 28dfeeb5d5
commit 2360504462
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/applications/differential/field/specification/testplan/DifferentialTestPlanFieldSpecification.php
View file

@ -19,7 +19,7 @@
final class DifferentialTestPlanFieldSpecification
extends DifferentialFieldSpecification {
private $plan;
private $plan = '';
// NOTE: This means "uninitialized".
private $error = false;
@ -29,7 +29,7 @@ final class DifferentialTestPlanFieldSpecification
}
protected function didSetRevision() {
$this->plan = $this->getRevision()->getTestPlan();
$this->plan = (string)$this->getRevision()->getTestPlan();
}
public function setValueFromRequest(AphrontRequest $request) {
@ -77,7 +77,7 @@ final class DifferentialTestPlanFieldSpecification
}
public function setValueFromParsedCommitMessage($value) {
$this->plan = $value;
$this->plan = (string)$value;
return $this;
}