Move currentFrame increment to the end of the loop

This fixes a bug that causes the first frame in a temporal animation to be skipped since currentFrame is initialized to be zero, but then before the first frame is rendered frame number is incremented. This is a fix for #42932.
qgis · Apr 26, 2021 · ce2476c · ce2476c
1 parent 3e4acec
commit ce2476c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/core/qgstemporalutils.cpp b/src/core/qgstemporalutils.cpp
@@ -98,7 +98,6 @@ bool QgsTemporalUtils::exportAnimation( const QgsMapSettings &mapSettings, const
       }
       feedback->setProgress( currentFrame / static_cast<double>( totalFrames ) * 100 );
     }
-    ++currentFrame;
 
     navigator.setCurrentFrameNumber( currentFrame );
 
@@ -137,6 +136,8 @@ bool QgsTemporalUtils::exportAnimation( const QgsMapSettings &mapSettings, const
     p.end();
 
     img.save( path );
+
+    ++currentFrame;
   }
 
   return true;