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 · e3289f3 · e3289f3
1 parent 90df2b3
commit e3289f3
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/core/qgstemporalutils.cpp b/src/core/qgstemporalutils.cpp
@@ -116,7 +116,6 @@ bool QgsTemporalUtils::exportAnimation( const QgsMapSettings &mapSettings, const
       }
       feedback->setProgress( currentFrame / static_cast<double>( totalFrames ) * 100 );
     }
-    ++currentFrame;
 
     navigator.setCurrentFrameNumber( currentFrame );
 
@@ -155,6 +154,8 @@ bool QgsTemporalUtils::exportAnimation( const QgsMapSettings &mapSettings, const
     p.end();
 
     img.save( path );
+
+    ++currentFrame;
   }
 
   return true;