-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello,

the logic in the systems history in qt-gui is broken, the buttons for walking 
forwards and backwards through the history are not associated with the 
correct functions correctly.

If you view a users history you can walk through it by pressing the Prev and 
Forward buttons.
This does not work if you view the systems history (System functions -> 
History) Click on Prev, nothing happens. Click on Save, it walks backwards. 
Now click on Prev again, it walks forwards. 

It's like this:
Save    --> walks back
Prev    --> walks forward
Forward --> walks forward

This is fixed by my patch.

Bye,
Thomas

- -- 
She is not refined.  She is not unrefined.  She keeps a parrot.
                -- Mark Twain
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE9y/tO+83LmoKU5MARAgElAJ9vs36iCQRoCUt2IQhfLfCb2vQv3wCeKXDs
1JHZ5vWlY3/Qx1R8pZ1yPAg=
=wTgP
-----END PGP SIGNATURE-----
Index: licq/plugins/qt-gui/src/userinfodlg.cpp
===================================================================
RCS file: /cvsroot/licq/qt-gui/src/userinfodlg.cpp,v
retrieving revision 1.58
diff -u -3 -p -r1.58 userinfodlg.cpp
--- licq/plugins/qt-gui/src/userinfodlg.cpp	5 Nov 2002 06:26:02 -0000	1.58
+++ licq/plugins/qt-gui/src/userinfodlg.cpp	8 Nov 2002 17:54:20 -0000
@@ -1224,8 +1224,10 @@ void UserInfoDlg::updateTab(const QStrin
     currentTab = GeneralInfo;
     btnMain3->setText(tr("&Update"));
     btnMain2->setText(m_bOwner ? tr("Retrieve") : tr("&Save"));
+		btnMain1->setText(m_bOwner ? tr("&Save") : tr("&Menu"));
     btnMain3->setEnabled(true);
     btnMain2->setEnabled(true);
+		btnMain1->setEnabled(true);
     if (!tabList[GeneralInfo].loaded)
       SetGeneralInfo(NULL);
   }
@@ -1233,8 +1235,10 @@ void UserInfoDlg::updateTab(const QStrin
   {
     btnMain3->setText(tr("&Update"));
     btnMain2->setText(m_bOwner ? tr("Retrieve") : tr("&Save"));
+		btnMain1->setText(m_bOwner ? tr("&Save") : tr("&Menu"));
     btnMain3->setEnabled(true);
     btnMain2->setEnabled(true);
+		btnMain1->setEnabled(true);
     currentTab = MoreInfo;
     if (!tabList[MoreInfo].loaded)
       SetMoreInfo(NULL);
@@ -1243,8 +1247,10 @@ void UserInfoDlg::updateTab(const QStrin
   {
     btnMain3->setText(tr("&Update"));
     btnMain2->setText(m_bOwner ? tr("Retrieve") : tr("&Save"));
+		btnMain1->setText(m_bOwner ? tr("&Save") : tr("&Menu"));
     btnMain3->setEnabled(true);
     btnMain2->setEnabled(true);
+		btnMain1->setEnabled(true);
     currentTab = WorkInfo;
     if (!tabList[WorkInfo].loaded)
       SetWorkInfo(NULL);
@@ -1253,8 +1259,10 @@ void UserInfoDlg::updateTab(const QStrin
   {
     btnMain3->setText(tr("&Update"));
     btnMain2->setText(m_bOwner ? tr("Retrieve") : tr("&Save"));
+		btnMain1->setText(m_bOwner ? tr("&Save") : tr("&Menu"));
     btnMain3->setEnabled(true);
     btnMain2->setEnabled(true);
+		btnMain1->setEnabled(true);
     currentTab = AboutInfo;
     if (!tabList[AboutInfo].loaded)
       SetAbout(NULL);
@@ -1263,8 +1271,10 @@ void UserInfoDlg::updateTab(const QStrin
   {
     btnMain3->setText(tr("Nex&t"));
     btnMain2->setText(tr("P&rev"));
+		if (m_bOwner) btnMain1->setText("");
     btnMain3->setEnabled(true);
     btnMain2->setEnabled(true);
+		btnMain1->setEnabled(m_bOwner ? false : true);
     currentTab = HistoryInfo;
     if (!tabList[HistoryInfo].loaded)
       SetupHistory();
@@ -1275,8 +1285,10 @@ void UserInfoDlg::updateTab(const QStrin
     currentTab = LastCountersInfo;
     btnMain3->setText("");
     btnMain2->setText("");
+		btnMain1->setText(m_bOwner ? tr("&Save") : tr("&Menu"));
     btnMain3->setEnabled(false);
     btnMain2->setEnabled(false);
+		btnMain1->setEnabled(true);
     if (!tabList[LastCountersInfo].loaded)
       SetLastCountersInfo(NULL);
   }
@@ -1302,7 +1314,6 @@ void UserInfoDlg::SaveSettings()
     SaveAbout();
     break;
   case HistoryInfo:
-    ShowHistoryPrev();
     break;
   case LastCountersInfo:
     break;
@@ -1329,7 +1340,7 @@ void UserInfoDlg::slotRetrieve()
   }
   else
   {
-    ShowHistoryNext();
+    ShowHistoryPrev();
     return;
   }
 

Reply via email to