switch to use QTemporaryFile

NEDJIMAbelgacem · wonder-sk · commit a297d8407755 · 2021-04-08T14:26:31.000-07:00
diff --git a/src/core/pointcloud/qgseptdecoder.cpp b/src/core/pointcloud/qgseptdecoder.cpp
@@ -27,6 +27,7 @@
 #include <iostream>
 #include <memory>
 #include <cstring>
+#include <QTemporaryFile>
 
 #include <zstd.h>
 
@@ -478,12 +479,16 @@ QgsPointCloudBlock *QgsEptDecoder::decompressLaz( const QString &filename,
          );
 }
 
+#include <QDebug>
+
 QgsPointCloudBlock *QgsEptDecoder::decompressLaz( const QByteArray &byteArrayData,
-    const QString &nodeStr,
     const QgsPointCloudAttributeCollection &attributes,
     const QgsPointCloudAttributeCollection &requestedAttributes )
 {
-  QString filename = QDir::tempPath() + QDir::separator() + QStringLiteral( "%1.laz" ).arg( nodeStr );
+  QTemporaryFile tempFile;
+  if ( !tempFile.open() )
+    return nullptr;
+  QString filename = tempFile.fileName();
   std::ofstream file( filename.toStdString(), std::ios::binary | std::ios::out );
   if ( file.is_open() )
   {
diff --git a/src/core/pointcloud/qgseptdecoder.h b/src/core/pointcloud/qgseptdecoder.h
@@ -36,7 +36,7 @@ namespace QgsEptDecoder
   QgsPointCloudBlock *decompressZStandard( const QString &filename, const QgsPointCloudAttributeCollection &attributes, const QgsPointCloudAttributeCollection &requestedAttributes );
   QgsPointCloudBlock *decompressZStandard( const QByteArray &data, const QgsPointCloudAttributeCollection &attributes, const QgsPointCloudAttributeCollection &requestedAttributes );
   QgsPointCloudBlock *decompressLaz( const QString &filename, const QgsPointCloudAttributeCollection &attributes, const QgsPointCloudAttributeCollection &requestedAttributes );
-  QgsPointCloudBlock *decompressLaz( const QByteArray &data, const QString &nodeStr, const QgsPointCloudAttributeCollection &attributes, const QgsPointCloudAttributeCollection &requestedAttributes );
+  QgsPointCloudBlock *decompressLaz( const QByteArray &data, const QgsPointCloudAttributeCollection &attributes, const QgsPointCloudAttributeCollection &requestedAttributes );
 };
 
 ///@endcond
diff --git a/src/core/pointcloud/qgspointcloudblockrequest.cpp b/src/core/pointcloud/qgspointcloudblockrequest.cpp
@@ -66,7 +66,7 @@ void QgsPointCloudBlockRequest::blockFinishedLoading()
       }
       else if ( mDataType == QLatin1String( "laszip" ) )
       {
-        mBlock = QgsEptDecoder::decompressLaz( mTileDownloadManagetReply->data(), mNode.toString(), mAttributes, mRequestedAttributes );
+        mBlock = QgsEptDecoder::decompressLaz( mTileDownloadManagetReply->data(), mAttributes, mRequestedAttributes );
       }
       else
       {

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ void QgsPointCloudBlockRequest::blockFinishedLoading()`
`66`	`66`	`}`
`67`	`67`	`else if ( mDataType == QLatin1String( "laszip" ) )`
`68`	`68`	`{`
`69`		`- mBlock = QgsEptDecoder::decompressLaz( mTileDownloadManagetReply->data(), mNode.toString(), mAttributes, mRequestedAttributes );`
	`69`	`+ mBlock = QgsEptDecoder::decompressLaz( mTileDownloadManagetReply->data(), mAttributes, mRequestedAttributes );`
`70`	`70`	`}`
`71`	`71`	`else`
`72`	`72`	`{`