1 files changed, 3 insertions, 3 deletions

diff --git a/noncore/apps/checkbook/checkbook.cpp b/noncore/apps/checkbook/checkbook.cpp
index 77c1f57..ab25516 100644
--- a/noncore/apps/checkbook/checkbook.cpp
+++ b/noncore/apps/checkbook/checkbook.cpp
@@ -290,49 +290,49 @@ void Checkbook::loadCheckbook()
 	currBalance = balanceEdit->text().toFloat( &ok );
 	startBalance = currBalance;
 
 	// Load transactions
 	TranInfo *tran;
 	QString trandesc = "";
 	float amount;
 	QString stramount;
 	for ( int i = 1; trandesc != QString::null; i++ )
 	{
 		tran = new TranInfo( config, i );
 		trandesc = tran->desc();
 		if ( trandesc != QString::null )
 		{
 			currBalance -= tran->fee();
 			amount = tran->amount();
 			if ( tran->withdrawal() )
 			{
 				amount *= -1;
 			}
 			currBalance += amount;
 			stramount.sprintf( "%c%.2f", currencySymbol, amount );
 
 			// Add to transaction list
-			transactions.append( tran );
+			transactions.inSort( tran );
 
 			// Add to transaction table
 			( void ) new QListViewItem( tranTable, QString::number( i ), tran->datestr(),
 										trandesc, stramount );
 		}
 		else
 		{
 			delete tran;
 		}
 	}
 	balanceLabel->setText( tr( "Current balance: %1%2" ).arg( currencySymbol ).arg( currBalance, 0, 'f', 2 ) );
 
 	highTranNum = transactions.count();
 }
 
 void Checkbook::adjustBalance( float amount )
 {
 	currBalance += amount;
 	balanceLabel->setText( tr( "Current balance: %1%2" ).arg( currencySymbol ).arg( currBalance, 0, 'f', 2 ) );
 
 }
 
 TranInfo *Checkbook::findTranByID( int id )
 {
@@ -394,49 +394,49 @@ void Checkbook::slotStartingBalanceChanged( const QString &newbalance )
 	startBalance = newbalance.toFloat( &ok );
 	adjustBalance( startBalance );
 }
 
 void Checkbook::slotNewTran()
 {
 	highTranNum++;
 	TranInfo *traninfo = new TranInfo( highTranNum );
 
 	Transaction *currtran = new Transaction( this, name,
 											 traninfo,
 											 currencySymbol );
 	currtran->showMaximized();
 	if ( currtran->exec() == QDialog::Accepted )
 	{
 		float amount = traninfo->amount();
 		if ( traninfo->withdrawal() )
 		{
 			amount *= -1;
 		}
 		QString stramount;
 		stramount.sprintf( "%c%.2f", currencySymbol, amount );
 
 		// Add to transaction list
-		transactions.append( traninfo );
+		transactions.inSort( traninfo );
 
 		// Add to transaction table
 		( void ) new QListViewItem( tranTable, QString::number( highTranNum ),
 									traninfo->datestr(), traninfo->desc(), stramount );
 
 		adjustBalance( amount );
 	}
 	else
 	{
 		highTranNum--;
 		delete traninfo;
 	}
 }
 
 void Checkbook::slotEditTran()
 {
 	bool ok;
 	QListViewItem *curritem = tranTable->currentItem();
 	if ( !curritem )
 	{
 		return;
 	}
 
 	int tranid = curritem->text( 0 ).toInt( &ok );
@@ -541,49 +541,49 @@ void Checkbook::drawBalanceChart()
 		if ( tran->withdrawal() )
 		{
 			amount *= -1;
 		}
 		balance += amount;
 		if ( i == 1 || i == count / 2 || i == count )
 		{
 			label = tran->datestr();
 		}
 		else
 		{
 			label = "";
 		}
 		list->append( new DataPointInfo( label, balance ) );
 	}
 
 	graphInfo = new GraphInfo( GraphInfo::BarChart, list );
 }
 
 void Checkbook::drawCategoryChart( bool withdrawals )
 {
 	DataPointList *list = new DataPointList();
 
 	TranInfo *tran = transactions.first();
-	if ( tran->withdrawal() == withdrawals )
+	if ( tran && tran->withdrawal() == withdrawals )
 	{
 		list->append( new DataPointInfo( tran->category(), tran->amount() ) );
 	}
 	tran = transactions.next();
 
 	DataPointInfo *cat;
 	for ( ; tran; tran = transactions.next() )
 	{
 		if ( tran->withdrawal() == withdrawals )
 		{
 			// Find category in list
 			for ( cat = list->first(); cat; cat = list->next() )
 			{
 				if ( cat->label() == tran->category() )
 				{
 					break;
 				}
 			}
 			if ( cat && cat->label() == tran->category() )
 			{ // Found category, add to transaction to category total
 				cat->addToValue( tran->amount() );
 			}
 			else
 			{ // Didn't find category, add category to list