Bug 149164

Summary: Case sensistive search option gets reset
Product: [Unmaintained] kpdf Reporter: James Shaw <js102>
Component: generalAssignee: Albert Astals Cid <aacid>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: 0.5.6   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description James Shaw 2007-08-24 12:57:40 UTC 
Version:           0.5.6 (using KDE KDE 3.5.6)
Installed from:    Ubuntu Packages
OS:                Linux

Open the search dialog, check "Case sensitive" search, and perform the search.  When the dialog box is opened again, "Case sensitive" has been unchecked.
Comment 1 Pino Toscano 2007-08-24 13:01:18 UTC 
Well, more than "reset", that option is not saved... :)
Comment 2 Pino Toscano 2007-08-26 20:22:37 UTC 
SVN commit 704964 by pino:

Save the case sensitivity of a search across multiple search sessions each time a document is open.

BUG: 149164


 M  +7 -1      part.cpp  


--- branches/KDE/3.5/kdegraphics/kpdf/part.cpp #704963:704964
@@ -768,6 +768,7 @@
 
 void Part::slotFind()
 {
+    static bool savedCaseSensitive = false;
     KFindDialog dlg( widget() );
     dlg.setHasCursor( false );
     if ( !m_searchHistory.empty() )
@@ -777,12 +778,17 @@
     dlg.setSupportsWholeWordsFind( false );
     dlg.setSupportsRegularExpressionFind( false );
 #endif
+    if ( savedCaseSensitive )
+    {
+        dlg.setOptions( dlg.options() | KFindDialog::CaseSensitive );
+    }
     if ( dlg.exec() == QDialog::Accepted )
     {
+        savedCaseSensitive = dlg.options() & KFindDialog::CaseSensitive;
         m_searchHistory = dlg.findHistory();
         m_searchStarted = true;
         m_document->resetSearch( PART_SEARCH_ID );
-        m_document->searchText( PART_SEARCH_ID, dlg.pattern(), false, dlg.options() & KFindDialog::CaseSensitive,
+        m_document->searchText( PART_SEARCH_ID, dlg.pattern(), false, savedCaseSensitive,
                                 KPDFDocument::NextMatch, true, qRgb( 255, 255, 64 ) );
     }
 }