tdf#114488 Rank multiple images also for flat odf

Only the file extension was considered before
which is not available in flat odf.

Now both internal and external URLs are resolved to their respective mimetype.

Change-Id: Id7e290cd9a441a1a6c2ce5e8d764391c50495bea
Reviewed-on: https://gerrit.libreoffice.org/47824
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>

1 files changed, 28 insertions, 9 deletions

diff --git a/xmloff/source/core/xmlmultiimagehelper.cxx b/xmloff/source/core/xmlmultiimagehelper.cxx
index 3a5ad69d4c65..3d7725a3b24f 100644
--- a/xmloff/source/core/xmlmultiimagehelper.cxx
+++ b/xmloff/source/core/xmlmultiimagehelper.cxx
@@ -20,48 +20,67 @@
 #include <rtl/ustring.hxx>
 #include <xmlmultiimagehelper.hxx>
 
+#include <comphelper/graphicmimetype.hxx>
+
 using namespace ::com::sun::star;
 
 namespace
 {
     sal_uInt32 getQualityIndex(const OUString& rString)
     {
+        OUString sMimeType;
+        if (rString.startsWith("vnd.sun.star.GraphicObject"))
+        {
+            sMimeType = comphelper::GraphicMimeTypeHelper::GetMimeTypeForImageUrl(rString);
+        }
+        else if (rString.startsWith("vnd.sun.star.Package"))
+        {
+            sMimeType
+                = comphelper::GraphicMimeTypeHelper::GetMimeTypeForExtension(OUStringToOString(
+                    rString.copy(rString.lastIndexOf(".") + 1), RTL_TEXTENCODING_ASCII_US));
+        }
+        else
+        {
+            SAL_WARN("xmloff", "Unknown image source: " << rString);
+            return 0;
+        }
+
         // pixel formats first
-        if(rString.endsWith(".bmp"))
+        if(sMimeType == "image/bmp")
         {
             return 10;
         }
-        if(rString.endsWith(".gif"))
+        if(sMimeType == "image/gif")
         {
             return 20;
         }
-        if(rString.endsWith(".jpg"))
+        if(sMimeType == "image/jpeg")
         {
             return 30;
         }
-        if(rString.endsWith(".png"))
+        if(sMimeType == "image/png")
         {
             return 40;
         }
 
         // vector formats, prefer always
-        if(rString.endsWith(".svm"))
+        if(sMimeType == "image/x-svm")
         {
             return 1000;
         }
-        if(rString.endsWith(".wmf"))
+        if(sMimeType == "image/x-wmf")
         {
             return 1010;
         }
-        if(rString.endsWith(".emf"))
+        if(sMimeType == "image/x-emf")
         {
             return 1020;
         }
-        if(rString.endsWith(".pdf"))
+        if(sMimeType == "application/pdf")
         {
             return 1030;
         }
-        if(rString.endsWith(".svg"))
+        if(sMimeType == "image/svg+xml")
         {
             return 1040;
         }