summaryrefslogtreecommitdiff
path: root/libopie2
authoreilers <eilers>2004-08-29 12:48:43 (UTC)
committer eilers <eilers>2004-08-29 12:48:43 (UTC)
commit45327ef3c0f093fc227682f79158632afc09e6d1 (patch) (unidiff)
treebdfe4e602abcef87117528d2c9c4166820488d11 /libopie2
parentdd159675e6e3c361bc20eaa6994265e73b6599ef (diff)
downloadopie-45327ef3c0f093fc227682f79158632afc09e6d1.zip
opie-45327ef3c0f093fc227682f79158632afc09e6d1.tar.gz
opie-45327ef3c0f093fc227682f79158632afc09e6d1.tar.bz2
Fixed some quirks..
Diffstat (limited to 'libopie2') (more/less context) (ignore whitespace changes)
-rw-r--r--libopie2/opiepim/backend/otodoaccesssql.cpp11
1 files changed, 8 insertions, 3 deletions
diff --git a/libopie2/opiepim/backend/otodoaccesssql.cpp b/libopie2/opiepim/backend/otodoaccesssql.cpp
index 6f65c48..12a8bea 100644
--- a/libopie2/opiepim/backend/otodoaccesssql.cpp
+++ b/libopie2/opiepim/backend/otodoaccesssql.cpp
@@ -454,46 +454,51 @@ QArray<int> OPimTodoAccessBackendSQL::queryByExample( const OPimTodo& , int, con
454} 454}
455OPimTodo OPimTodoAccessBackendSQL::find(int uid ) const{ 455OPimTodo OPimTodoAccessBackendSQL::find(int uid ) const{
456 FindQuery query( uid ); 456 FindQuery query( uid );
457 return todo( m_driver->query(&query) ); 457 return todo( m_driver->query(&query) );
458 458
459} 459}
460
461// Remember: uid is already in the list of uids, called ints !
460OPimTodo OPimTodoAccessBackendSQL::find( int uid, const QArray<int>& ints, 462OPimTodo OPimTodoAccessBackendSQL::find( int uid, const QArray<int>& ints,
461 uint cur, Frontend::CacheDirection dir ) const{ 463 uint cur, Frontend::CacheDirection dir ) const{
462 uint CACHE = readAhead(); 464 uint CACHE = readAhead();
463 odebug << "searching for " << uid << "" << oendl; 465 odebug << "searching for " << uid << "" << oendl;
464 QArray<int> search( CACHE ); 466 QArray<int> search( CACHE );
465 uint size =0; 467 uint size =0;
466 OPimTodo to; 468 OPimTodo to;
467 469
468 // we try to cache CACHE items 470 // we try to cache CACHE items
469 switch( dir ) { 471 switch( dir ) {
470 /* forward */ 472 /* forward */
471 case 0: // FIXME: Not a good style to use magic numbers here (eilers) 473 case Frontend::Forward:
472 for (uint i = cur; i < ints.count() && size < CACHE; i++ ) { 474 for (uint i = cur; i < ints.count() && size < CACHE; i++ ) {
473 odebug << "size " << size << " " << ints[i] << "" << oendl; 475 odebug << "size " << size << " " << ints[i] << "" << oendl;
474 search[size] = ints[i]; 476 search[size] = ints[i];
475 size++; 477 size++;
476 } 478 }
477 break; 479 break;
478 /* reverse */ 480 /* reverse */
479 case 1: // FIXME: Not a good style to use magic numbers here (eilers) 481 case Frontend::Reverse:
480 for (uint i = cur; i != 0 && size < CACHE; i-- ) { 482 for (uint i = cur; i != 0 && size < CACHE; i-- ) {
481 search[size] = ints[i]; 483 search[size] = ints[i];
482 size++; 484 size++;
483 } 485 }
484 break; 486 break;
485 } 487 }
488
486 search.resize( size ); 489 search.resize( size );
487 FindQuery query( search ); 490 FindQuery query( search );
488 OSQLResult res = m_driver->query( &query ); 491 OSQLResult res = m_driver->query( &query );
489 if ( res.state() != OSQLResult::Success ) 492 if ( res.state() != OSQLResult::Success )
490 return to; 493 return to;
491 494
492 return todo( res ); 495 todo( res ); //FIXME: Don't like polymorphism here. It makes the code hard to read here..(eilers)
496 return cacheFind( uid );
493} 497}
498
494void OPimTodoAccessBackendSQL::clear() { 499void OPimTodoAccessBackendSQL::clear() {
495 ClearQuery cle; 500 ClearQuery cle;
496 OSQLResult res = m_driver->query( &cle ); 501 OSQLResult res = m_driver->query( &cle );
497 CreateQuery qu; 502 CreateQuery qu;
498 res = m_driver->query(&qu); 503 res = m_driver->query(&qu);
499} 504}