From fbf01a8959d8213038a660622efc2691b9578bc1 Mon Sep 17 00:00:00 2001
From: danielbui78 <danielbui78@gmail.com>
Date: Sat, 21 Oct 2023 14:40:36 -0400
Subject: [PATCH] Force JPEG compression option

---
 include/DzBridgeAction.h |  1 +
 src/DzBridgeAction.cpp   | 17 ++++++++++++++---
 2 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/include/DzBridgeAction.h b/include/DzBridgeAction.h
index 3274530..5fb00b6 100644
--- a/include/DzBridgeAction.h
+++ b/include/DzBridgeAction.h
@@ -202,6 +202,7 @@ namespace DzBridgeNameSpace
 
 		// Texture Settings
 		bool m_bConvertToPng = false;
+		bool m_bConvertToJpg = false; // m_bConvertToJpg will override m_bConvertToPng
 		bool m_bExportAllTextures = false;
 		bool m_bCombineDiffuseAndAlphaMaps = false;
 		bool m_bResizeTextures = false;
diff --git a/src/DzBridgeAction.cpp b/src/DzBridgeAction.cpp
index 560d148..6b683f5 100644
--- a/src/DzBridgeAction.cpp
+++ b/src/DzBridgeAction.cpp
@@ -2412,6 +2412,8 @@ QString DzBridgeAction::exportAssetWithDtu(QString sFilename, QString sAssetMate
 
 }
 
+// TODO: This method will fail because uncompressed textures of the same dimension
+// will have the same file size.  Instead, must use a file content hash function.
 QString DzBridgeAction::makeUniqueFilename(QString sFilename)
 {
 	if (QFileInfo(sFilename).exists() != true)
@@ -2675,9 +2677,10 @@ void DzBridgeAction::writeMaterialProperty(DzNode* Node, DzJsonWriter& Writer, Q
 	if (TextureName != "")
 	{
 		// DB 2023-Oct-5: Save to PNG, Export all Textures
-		if (m_bConvertToPng)
+		if (m_bConvertToPng || m_bConvertToJpg)
 		{
-			if (TextureName.endsWith(".png", Qt::CaseInsensitive) == false && 
+			if ( m_bConvertToJpg || 
+				TextureName.endsWith(".png", Qt::CaseInsensitive) == false && 
 				TextureName.endsWith(".jpg", Qt::CaseInsensitive) == false &&
 				TextureName.endsWith(".jpeg", Qt::CaseInsensitive) == false)
 			{
@@ -2687,7 +2690,15 @@ void DzBridgeAction::writeMaterialProperty(DzNode* Node, DzJsonWriter& Writer, Q
 				QString cleanedTempPath = dzApp->getTempPath().toLower().replace("\\", "/");
 				QString filestem = QFileInfo(TextureName).fileName();
 				QString pngFilename = cleanedTempPath + "/" + filestem + ".png";
-				imageMgr->saveImage(pngFilename, image);
+				if (m_bConvertToJpg)
+				{
+					pngFilename = cleanedTempPath + "/" + filestem + ".jpg";
+					image.save(pngFilename, "jpg", 95);
+				}
+				else
+				{
+					imageMgr->saveImage(pngFilename, image);
+				}
 				dtuTextureName = TextureName = pngFilename;
 			}
 		}