Fixed extra
This commit is contained in:
parent
b83a6db697
commit
5d03a0ab35
|
@ -522,8 +522,6 @@ class Build extends Model
|
|||
*/
|
||||
public function setExtra($value)
|
||||
{
|
||||
$value = json_encode($value);
|
||||
|
||||
$this->validateString('Extra', $value);
|
||||
|
||||
if ($this->data['extra'] === $value) {
|
||||
|
|
|
@ -87,7 +87,7 @@ class BuildService
|
|||
}
|
||||
|
||||
if (!is_null($extra)) {
|
||||
$build->setExtra($extra);
|
||||
$build->setExtra(json_encode($extra));
|
||||
}
|
||||
|
||||
$build = $this->buildStore->save($build);
|
||||
|
|
|
@ -63,7 +63,7 @@ class BuildTest extends \PHPUnit_Framework_TestCase
|
|||
];
|
||||
|
||||
$build = new Build();
|
||||
$build->setExtra($info);
|
||||
$build->setExtra(json_encode($info));
|
||||
|
||||
$this->assertEquals('Item One', $build->getExtra('item1'));
|
||||
$this->assertEquals(2, $build->getExtra('item2'));
|
||||
|
|
|
@ -100,7 +100,7 @@ class BuildServiceTest extends \PHPUnit_Framework_TestCase
|
|||
$build->setFinished(new \DateTime());
|
||||
$build->setCommitMessage('test');
|
||||
$build->setCommitterEmail('test@example.com');
|
||||
$build->setExtra(['item1' => 1001]);
|
||||
$build->setExtra(json_encode(['item1' => 1001]));
|
||||
|
||||
$returnValue = $this->testedService->createDuplicateBuild($build);
|
||||
|
||||
|
|
Loading…
Reference in a new issue