-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
if(type == SummaryType.DETAILED_EXPORT_SUMMARY){ | ||
setDetailedExportSummaryFields(document, copy); | ||
} else { | ||
copyField(document, copy, _Fields.ID); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be moved to a verbosly named function
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just a
listOfFields.stream()
.forEach(renderedField -> copyField(document, copy, renderedField));
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did the first one, but not the second.
try { | ||
projects = projectClient.getProjectsById(ids, user); | ||
} catch (TException e) { | ||
log.error("Error fetching linked projects.", e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above: this should fail hard
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
Done with refactoring from my side. Is it enough? |
Yes, code looks good. |
* one line per project - linked release combination * only directly linked releases * more fields exported (almost all)
* vendor, ClearingInfo, COTSDetails in separate columns * possibility to select whether linked releases are exported with the project * names instead of couchDB ids for linked projects and linked releases * excelExport for project respects filter
* selectable with or without releases * respects filters (but not keyword search) * more component fields exported
* prevents differences between excel export and view * button is automatically enabled as soon as page is refreshed by "Apply Filters" or "enter" on an input
ec17300
to
1c1f30e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested on vagrant based build and with projects import from mock and manually created releases to have multi release component. Also tested filtering with components and projects and both report types. All good.
For projects and components: closes #223