BUG: READ_IF_PRESENT: return false if not read. Fixes #3193

In the original change (68c5d90a)

• processors that did not have headerOk() would still call readStream
• if no headerOk() would still return true

We assume that

• headerOk is synced amongst all processors
• readStream should be called on all processors, possibly with differing argument

Went through

• all changes in that commit
• all containers using readOptional() (read_if_present, lazy_read)

Tested a bit:

• original testcase from gitlab issue
• restarting from cumulativeContErr (uniformDimensionedField)
• restarting from GeometricField (e.g. phi)

and don't see any other problems.

If ok should be backported to master

@mark

assigned to @andy

added 1 commit

• eaee2bb9 - BUG: plenumPressure. Unitialised variables. Fixes #3282 (closed)

Compare with previous version

added 74 commits

• eaee2bb9...ab30b030 - 72 commits from branch develop
• a0af9673 - BUG: READ_IF_PRESENT: return false if not read. Fixes #3193 (closed)
• 309c2e02 - BUG: plenumPressure. Unitialised variables. Fixes #3282 (closed)

Compare with previous version

added 2 commits

• 18b27939 - BUG: READ_IF_PRESENT: return false if not read. Fixes #3193 (closed)
• 1b1b3ec8 - BUG: plenumPressure. Unitialised variables. Fixes #3282 (closed)

Compare with previous version

mentioned in commit 15059f78

merged