Apply suggestions from code review

Co-authored-by: Claudio Cambra <[email protected]> Signed-off-by: Joshua Noeske <[email protected]>
nextcloud · Jan 5, 2025 · 42cd0f9 · 42cd0f9
1 parent e14655f
commit 42cd0f9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/gui/ocsjob.cpp b/src/gui/ocsjob.cpp
@@ -139,8 +139,7 @@ bool OcsJob::finished()
 
     } else {
         // save new ETag value
-        auto etagHeader = reply()->header(QNetworkRequest::ETagHeader);
-        if (etagHeader.isValid()) {
+        if (const auto etagHeader = reply()->header(QNetworkRequest::ETagHeader); etagHeader.isValid()) {
             emit etagResponseHeaderReceived(etagHeader.toByteArray(), statusCode);
         }
 

diff --git a/src/libsync/networkjobs.cpp b/src/libsync/networkjobs.cpp
@@ -999,8 +999,7 @@ bool JsonApiJob::finished()
     }
 
     // save new ETag value
-    auto etagHeader = reply()->header(QNetworkRequest::ETagHeader);
-    if (etagHeader.isValid()) {
+    if (const auto etagHeader = reply()->header(QNetworkRequest::ETagHeader); etagHeader.isValid()) {
         emit etagResponseHeaderReceived(etagHeader.toByteArray(), statusCode);
     }