[Kde-imaging] [Bug 97298] avi files not properly put into gallery

Renchi Raju renchi at pooh.tam.uiuc.edu
Tue Jan 18 18:59:07 CET 2005


------- 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=97298         
renchi pooh tam uiuc edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From renchi pooh tam uiuc edu  2005-01-18 18:59 -------
CVS commit by pahlibar: 


enable uploading for movie (and other non-image files) to gallery
BUG: 97298


  M +10 -8     gallerytalker.cpp   1.6
  M +9 -4      gallerywindow.cpp   1.10


--- kdeextragear-libs-1/kipi-plugins/galleryexport/gallerytalker.cpp  #1.5:1.6
 @ -194,7 +194,8  @ bool GalleryTalker::addPhoto( const QStr
 
     QImage image(photoPath);
-    if (image.isNull())
-        return false;
 
+    if (!image.isNull())
+    {
+        // image file - see if we need to rescale it
     if (rescale && (image.width() > maxDim || image.height() > maxDim))
     {
 @ -205,4 +206,5  @ bool GalleryTalker::addPhoto( const QStr
                   << path << endl;
     }
+    }
     
     if (!form.addFile(path))

--- kdeextragear-libs-1/kipi-plugins/galleryexport/gallerywindow.cpp  #1.9:1.10
 @ -569,8 +569,13  @ void GalleryWindow::slotAddPhotoNext()
     m_uploadQueue.pop_front();
 
-    m_talker->addPhoto( m_lastSelectedAlbum, pathComments.first,
+    bool res = m_talker->addPhoto( m_lastSelectedAlbum, pathComments.first,
                         pathComments.second,
                         m_resizeCheckBox->isChecked(),
                         m_dimensionSpinBox->value() );
+    if (!res)
+    {
+        slotAddPhotoFailed( "" );
+        return;
+    }
     
     m_progressDlg->setLabelText( i18n("Uploading file %1 ")


More information about the Kde-imaging mailing list