Fix: Iteration::open() should access files

Author: franzpoeschel

include/openPMD/RecordComponent.hpp

@@ -81,16 +81,6 @@ namespace internal
          * Ignored otherwise.
          */
         Attribute m_constantValue{-1};
-        /**
-         * The same std::string that the parent class would pass as parameter to
-         * RecordComponent::flush().
-         * This is stored only upon RecordComponent::flush() if
-         * AbstractIOHandler::flushLevel is set to FlushLevel::SkeletonOnly
-         * (for use by the Span<T>-based overload of
-         * RecordComponent::storeChunk()).
-         * @todo Merge functionality with ownKeyInParent?
-         */
-        std::string m_name;
         /**
          * True if this component is an empty dataset, i.e. its extent is zero
          * in at least one dimension.
@@ -109,7 +99,6 @@ namespace internal
             BaseRecordComponentData::reset();
             m_chunks = std::queue<IOTask>();
             m_constantValue = -1;
-            m_name = std::string();
             m_isEmpty = false;
             m_hasBeenExtended = false;
         }

include/openPMD/RecordComponent.tpp

@@ -29,6 +29,7 @@
 #include "openPMD/auxiliary/ShareRawInternal.hpp"
 #include "openPMD/auxiliary/TypeTraits.hpp"
 #include "openPMD/auxiliary/UniquePtr.hpp"
+#include "openPMD/backend/Attributable.hpp"
 
 #include <memory>
 #include <type_traits>
@@ -111,7 +112,7 @@ RecordComponent::storeChunk(Offset o, Extent e, F &&createBuffer)
                 "using storeChunk() (see RecordComponent::resetDataset()).");
         }
         Parameter<Operation::CREATE_DATASET> dCreate(rc.m_dataset.value());
-        dCreate.name = rc.m_name;
+        dCreate.name = Attributable::get().m_writable.ownKeyWithinParent;
         IOHandler()->enqueue(IOTask(this, dCreate));
     }
 

include/openPMD/backend/Writable.hpp

@@ -89,6 +89,7 @@ class Writable final
     friend class ParticleSpecies;
     friend class Series;
     friend class Record;
+    friend class RecordComponent;
     friend class AbstractIOHandlerImpl;
     friend class ADIOS2IOHandlerImpl;
     friend class detail::ADIOS2File;

src/Iteration.cpp

@@ -161,12 +161,21 @@ Iteration &Iteration::open()
         it.m_closed = CloseStatus::Open;
         runDeferredParseAccess();
     }
-    if (getStepStatus() == StepStatus::OutOfStep)
+    switch (getStepStatus())
     {
+    case StepStatus::OutOfStep:
         beginStep(/* reread = */ false);
         setStepStatus(StepStatus::DuringStep);
+        break;
+    case StepStatus::DuringStep:
+    case StepStatus::NoStep: {
+        auto end = begin;
+        ++end;
+        s.flush_impl(begin, end, {FlushLevel::CreateOrOpenFiles});
     }
-    IOHandler()->flush(internal::defaultFlushParams);
+    break;
+    }
+    // IOHandler()->flush(internal::defaultFlushParams);
     return *this;
 }
 

src/RecordComponent.cpp

@@ -262,7 +262,6 @@ void RecordComponent::flush(
     auto &rc = get();
     if (flushParams.flushLevel == FlushLevel::SkeletonOnly)
     {
-        rc.m_name = name;
         return;
     }
     if (access::readOnly(IOHandler()->m_frontendAccess))