Skip to content

Feature/contextual variables report#514

Merged
Hajarel-moukh merged 9 commits into
mainfrom
feature/contextual-variables-report
Jul 8, 2026
Merged

Feature/contextual variables report#514
Hajarel-moukh merged 9 commits into
mainfrom
feature/contextual-variables-report

Conversation

@Hajarel-moukh

Copy link
Copy Markdown
Contributor

No description provided.

@Hajarel-moukh
Hajarel-moukh force-pushed the feature/contextual-variables-report branch from cca1514 to 8747864 Compare July 3, 2026 14:00
) throws GenesisException {
List<ContextualVariableFileReportDto> files = new ArrayList<>();

for (Mode mode : Mode.values()) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can avoid code duplication by modifiying saveContextualVariableFiles :

@OverRide
public int saveContextualVariableFiles(String collectionInstrumentId, FileUtils fileUtils, String contextualFolderPath) throws GenesisException {
return saveContextualVariableFilesWithReport(collectionInstrumentId, fileUtils, contextualFolderPath).processedFiles();
}

@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

@Hajarel-moukh
Hajarel-moukh merged commit 4809549 into main Jul 8, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants