[Kde-imaging] [Bug 135237] Filenames with multiple periods in them do not show up in the file listing (incorrect extension identification)

Gilles Caulier caulier.gilles at free.fr
Wed Oct 11 13:31:33 CEST 2006


------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=135237         




------- Additional Comments From caulier.gilles free fr  2006-10-11 13:31 -------
SVN commit 594457 by cgilles:

kipi-plugins from trunk : Raw converter and JPEGLossLess plugin don't check properly the filenale extension.
CCBUGS: 135237


 M  +1 -1      jpeglossless/utils.cpp  
 M  +2 -2      rawconverter/dcrawiface.cpp  
 M  +1 -1      rawconverter/plugin_rawconverter.cpp  


--- trunk/extragear/libs/kipi-plugins/jpeglossless/utils.cpp #594456:594457
 @ -61,7 +61,7  @
     QString rawFilesExt(kipi_raw_file_extentions);
 
     QFileInfo fileInfo(file);
-    if (rawFilesExt.upper().contains( fileInfo.extension().upper() ))
+    if (rawFilesExt.upper().contains( fileInfo.extension(false).upper() ))
         return true;
 
     return false;
--- trunk/extragear/libs/kipi-plugins/rawconverter/dcrawiface.cpp #594456:594457
 @ -141,7 +141,7  @
     QFileInfo fileInfo(path);
     QString   rawFilesExt(kipi_raw_file_extentions);
 
-    if (!fileInfo.exists() || !rawFilesExt.upper().contains( fileInfo.extension().upper() ))
+    if (!fileInfo.exists() || !rawFilesExt.upper().contains( fileInfo.extension(false).upper() ))
         return false;
 
     // Try to extract embedded thumbnail using dcraw with options:
 @ -256,7 +256,7  @
         return false;
     }
 
-    if (!rawFilesExt.upper().contains( fileInfo.extension().upper() ))
+    if (!rawFilesExt.upper().contains( fileInfo.extension(false).upper() ))
     {
         identify = i18n("Not a RAW file");
         return false;
--- trunk/extragear/libs/kipi-plugins/rawconverter/plugin_rawconverter.cpp #594456:594457
 @ -116,7 +116,7  @
     QString rawFilesExt(kipi_raw_file_extentions);
 
     QFileInfo fileInfo(filePath);
-    if (rawFilesExt.upper().contains( fileInfo.extension().upper() ))
+    if (rawFilesExt.upper().contains( fileInfo.extension(false).upper() ))
         return true;
 
     return false;


More information about the Kde-imaging mailing list