TextConfiguration: Fix bug that would cause ftag blocks to be mis-configured...

TextConfiguration: Fix bug that would cause ftag blocks to be mis-configured when 'postfix' was not set.

When creating and configuring an instance of the FTag config block, if 'postfix' was left as the empty string (i.e. ''), the block would not be configured properly. To prevent this, the config sequence returned from 'makeConfig' is configured before being added to the global config sequence that accumulates configured blocks.

A small function was also added that will print out all config blocks in a config sequence and their configuration to help debugging similar issues in the future.

assigned to @jolamber

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig

This merge request affects 1 file:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/ConfigText.py

Adding @krumnack ,@tadej ,@pagessin as watchers

added Analysis CPAlgorithms analysis-review-required main review-pending-level-1 labels

Looks good, approving from the analysis release side.

Nils [Analysis RC]

added analysis-review-approved label and removed analysis-review-required label

The system selected 18 tests to probe the Athena changeset (out of 64 available tests). Link to tests selection rules

CI Result SUCCESS (hash 882114c3)

	Athena	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 79269]

The changes look okay and there are no issues with the CI. Approving from L1.

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit a52d0661

added sweep:ignore label