fixes #2700 (Scriver: при открытии окна через попап отображается лишь одно сообщение)

3 files changed, 38 insertions, 50 deletions

diff --git a/plugins/Scriver/src/msgdialog.cpp b/plugins/Scriver/src/msgdialog.cpp
index 43fd82a142..f0d55d784c 100644
--- a/plugins/Scriver/src/msgdialog.cpp
+++ b/plugins/Scriver/src/msgdialog.cpp
@@ -256,17 +256,17 @@ bool CMsgDialog::OnInitDialog()
 					notifyUnread = true;

 			}

 

+			DB::ECPTR pCursor(DB::EventsRev(m_hContact, m_hDbEventFirst));

+

 			DBEVENTINFO dbei = {};

 			MEVENT hPrevEvent;

 			switch (historyMode) {

 			case LOADHISTORY_COUNT:

 				for (int i = g_plugin.iLoadCount; i > 0; i--) {

-					if (m_hDbEventFirst == 0)

-						hPrevEvent = db_event_last(m_hContact);

-					else

-						hPrevEvent = db_event_prev(m_hContact, m_hDbEventFirst);

+					hPrevEvent = pCursor.FetchNext();

 					if (hPrevEvent == 0)

 						break;

+

 					dbei.cbBlob = 0;

 					m_hDbEventFirst = hPrevEvent;

 					db_event_get(m_hDbEventFirst, &dbei);

@@ -276,26 +276,23 @@ bool CMsgDialog::OnInitDialog()
 				break;

 

 			case LOADHISTORY_TIME:

-				if (m_hDbEventFirst == 0) {

+				if (m_hDbEventFirst == 0)

 					dbei.timestamp = time(0);

-					hPrevEvent = db_event_last(m_hContact);

-				}

-				else {

+				else

 					db_event_get(m_hDbEventFirst, &dbei);

-					hPrevEvent = db_event_prev(m_hContact, m_hDbEventFirst);

-				}

 

 				DWORD firstTime = dbei.timestamp - 60 * g_plugin.iLoadTime;

 				for (;;) {

+					hPrevEvent = pCursor.FetchNext();

 					if (hPrevEvent == 0)

 						break;

+

 					dbei.cbBlob = 0;

 					db_event_get(hPrevEvent, &dbei);

 					if (dbei.timestamp < firstTime)

 						break;

 					if (DbEventIsShown(dbei))

 						m_hDbEventFirst = hPrevEvent;

-					hPrevEvent = db_event_prev(m_hContact, hPrevEvent);

 				}

 				break;

 			}

@@ -303,16 +300,14 @@ bool CMsgDialog::OnInitDialog()
 

 		m_pParent->AddChild(this);

 

-		MEVENT hdbEvent = db_event_last(m_hContact);

-		if (hdbEvent) {

+		DB::ECPTR pCursor(DB::EventsRev(m_hContact));

+		while (MEVENT hdbEvent = pCursor.FetchNext()) {

 			DBEVENTINFO dbei = {};

-			do {

-				db_event_get(hdbEvent, &dbei);

-				if (dbei.eventType == EVENTTYPE_MESSAGE && !(dbei.flags & DBEF_SENT)) {

-					m_lastMessage = dbei.timestamp;

-					break;

-				}

-			} while ((hdbEvent = db_event_prev(m_hContact, hdbEvent)));

+			db_event_get(hdbEvent, &dbei);

+			if (dbei.eventType == EVENTTYPE_MESSAGE && !(dbei.flags & DBEF_SENT)) {

+				m_lastMessage = dbei.timestamp;

+				break;

+			}

 		}

 

 		SendMessage(m_hwnd, DM_OPTIONSAPPLIED, 0, 0);

diff --git a/plugins/TabSRMM/src/msgdialog.cpp b/plugins/TabSRMM/src/msgdialog.cpp
index dfb50c91b7..8a09bd4c2f 100644
--- a/plugins/TabSRMM/src/msgdialog.cpp
+++ b/plugins/TabSRMM/src/msgdialog.cpp
@@ -586,7 +586,8 @@ bool CMsgDialog::OnInitDialog()
 

 		DM_OptionsApplied();

 

-		for (MEVENT hdbEvent = db_event_last(m_hContact); hdbEvent; hdbEvent = db_event_prev(m_hContact, hdbEvent)) {

+		DB::ECPTR pCursor(DB::EventsRev(m_hContact));

+		while (MEVENT hdbEvent = pCursor.FetchNext()) {

 			DBEVENTINFO dbei = {};

 			db_event_get(hdbEvent, &dbei);

 			if (dbei.eventType == EVENTTYPE_MESSAGE && !(dbei.flags & DBEF_SENT)) {

diff --git a/plugins/TabSRMM/src/msgdlgother.cpp b/plugins/TabSRMM/src/msgdlgother.cpp
index 0dec572846..01e98df270 100644
--- a/plugins/TabSRMM/src/msgdlgother.cpp
+++ b/plugins/TabSRMM/src/msgdlgother.cpp
@@ -463,49 +463,41 @@ void CMsgDialog::FindFirstEvent()
 	if (m_bActualHistory)
 		historyMode = LOADHISTORY_COUNT;
 
+	DBEVENTINFO dbei = {};
+	DB::ECPTR pCursor(DB::EventsRev(m_hContact, m_hDbEventFirst));
+
 	switch (historyMode) {
 	case LOADHISTORY_COUNT:
 		int i;
-		MEVENT hPrevEvent;
-		{
-			DBEVENTINFO dbei = {};
-			// ability to load only current session's history
-			if (m_bActualHistory)
-				i = m_cache->getSessionMsgCount();
-			else
-				i = g_plugin.getWord(SRMSGSET_LOADCOUNT, SRMSGDEFSET_LOADCOUNT);
 
-			for (; i > 0; i--) {
-				if (m_hDbEventFirst == 0)
-					hPrevEvent = db_event_last(m_hContact);
-				else
-					hPrevEvent = db_event_prev(m_hContact, m_hDbEventFirst);
-				if (hPrevEvent == 0)
-					break;
-				dbei.cbBlob = 0;
-				m_hDbEventFirst = hPrevEvent;
-				db_event_get(m_hDbEventFirst, &dbei);
-				if (!DbEventIsShown(&dbei))
-					i++;
-			}
+		// ability to load only current session's history
+		if (m_bActualHistory)
+			i = m_cache->getSessionMsgCount();
+		else
+			i = g_plugin.getWord(SRMSGSET_LOADCOUNT, SRMSGDEFSET_LOADCOUNT);
+
+		for (; i > 0; i--) {
+			MEVENT hPrevEvent = pCursor.FetchNext();
+			if (hPrevEvent == 0)
+				break;
+
+			dbei.cbBlob = 0;
+			m_hDbEventFirst = hPrevEvent;
+			db_event_get(m_hDbEventFirst, &dbei);
+			if (!DbEventIsShown(&dbei))
+				i++;
 		}
 		break;
 
 	case LOADHISTORY_TIME:
-		DBEVENTINFO dbei = {};
 		if (m_hDbEventFirst == 0)
 			dbei.timestamp = time(0);
 		else
 			db_event_get(m_hDbEventFirst, &dbei);
 
 		DWORD firstTime = dbei.timestamp - 60 * g_plugin.getWord(SRMSGSET_LOADTIME, SRMSGDEFSET_LOADTIME);
-		for (;;) {
-			if (m_hDbEventFirst == 0)
-				hPrevEvent = db_event_last(m_hContact);
-			else
-				hPrevEvent = db_event_prev(m_hContact, m_hDbEventFirst);
-			if (hPrevEvent == 0)
-				break;
+
+		while (MEVENT hPrevEvent = pCursor.FetchNext()) {
 			dbei.cbBlob = 0;
 			db_event_get(hPrevEvent, &dbei);
 			if (dbei.timestamp < firstTime)