The OPAL Discussion Forum

["Ting Yau" <2100011354 AT alumni.pku.edu.cn>]

## Bug Report: `BCFFTZ` is silently ignored due to default value of deprecated
`BCFFTT`

### Description

The `BCFFTZ` parameter, intended to replace the deprecated `BCFFTT`, is
**completely ignored** in both `hasPeriodicZ()` and `initSolver()`.

The constructor initializes `BCFFTT` with a default value (`"OPEN"`). Since
`itsAttr[deprecated::BCFFTT]` is never empty, the code never falls back to
reading `BCFFTZ`.

### Code in Structure\FieldSolver.cpp

**`FieldSolver::hasPeriodicZ()`**

```cpp
if (itsAttr[deprecated::BCFFTT])
    return (Attributes::getString(itsAttr[deprecated::BCFFTT]) == "PERIODIC");
return (Attributes::getString(itsAttr[BCFFTZ]) == "PERIODIC");
```

**`FieldSolver::initSolver()`**

```cpp
std::string bcz = Attributes::getString(itsAttr[deprecated::BCFFTT]);
if (bcz.empty()) {
    bcz = Attributes::getString(itsAttr[BCFFTZ]);
}
```