Fix periodic job display in builds page

A recent change to the builds page handled pre- and post-merge
jobs but generates an error on periodic jobs, which have no
change and no newrev.  Handle that case as well by using the
branch as the change link.

Change-Id: Idc5da35612aacbb4eaab6f417b7cf4467812e6d4
This commit is contained in:
James E. Blair 2018-10-17 08:10:15 -07:00
parent 849e26f5e7
commit 6259418add
1 changed files with 1 additions and 1 deletions

View File

@ -74,7 +74,7 @@ class BuildsPage extends TableFilters {
)
const linkChangeFormat = (value, rowdata) => (
<Table.Cell>
<a href={rowdata.rowData.ref_url}>{value ? rowdata.rowData.change+','+rowdata.rowData.patchset : rowdata.rowData.newrev.substring(7,1)}</a>
<a href={rowdata.rowData.ref_url}>{value ? rowdata.rowData.change+','+rowdata.rowData.patchset : rowdata.rowData.newrev ? rowdata.rowData.newrev.substring(7,1) : rowdata.rowData.branch}</a>
</Table.Cell>
)
this.columns = []