--- lbbs/src/menu_proc.c	2025/10/13 07:13:39	1.34
+++ lbbs/src/menu_proc.c	2025/10/18 12:06:10	1.41
@@ -15,6 +15,7 @@
  ***************************************************************************/
 
 #include "article_cache.h"
+#include "article_favor_display.h"
 #include "article_view_log.h"
 #include "bbs.h"
 #include "bbs_cmd.h"
@@ -42,10 +43,15 @@ int list_section(void *param)
 	return REDRAW;
 }
 
+typedef union exec_handler_t{
+	void *p;
+	int (*handler)();
+} exec_handler;
+
 int exec_mbem(void *param)
 {
 	void *hdll;
-	int (*func)();
+	exec_handler func;
 	char *c, *s;
 	char buf[FILE_PATH_LEN];
 
@@ -68,8 +74,10 @@ int exec_mbem(void *param)
 		{
 			char *error;
 
-			if ((func = dlsym(hdll, c ? c : "mod_main")) != NULL)
-				func();
+			if ((func.p = dlsym(hdll, c ? c : "mod_main")) != NULL)
+			{
+				func.handler();
+			}
 			else if ((error = dlerror()) != NULL)
 			{
 				clearscr();
@@ -109,6 +117,13 @@ int copyright(void *param)
 	return REDRAW;
 }
 
+int version(void *param)
+{
+	display_file(DATA_VERSION, 1);
+
+	return REDRAW;
+}
+
 int reload_bbs_conf(void *param)
 {
 	clearscr();
@@ -211,7 +226,7 @@ int list_ex_section(void *param)
 {
 	SECTION_LIST *p_section;
 
-	p_section = section_list_find_by_name(param, NULL);
+	p_section = section_list_find_by_name(param);
 	if (p_section == NULL)
 	{
 		log_error("Section %s not found\n", (const char *)param);
@@ -228,8 +243,15 @@ int list_ex_section(void *param)
 
 int show_top10_menu(void *param)
 {
+	static int show_top10 = 0;
 	int ch = 0;
 
+	if (show_top10)
+	{
+		return NOREDRAW;
+	}
+	show_top10 = 1;
+
 	clearscr();
 	show_top("", BBS_name, "");
 	show_bottom("");
@@ -240,18 +262,27 @@ int show_top10_menu(void *param)
 		{
 			iflush();
 			ch = igetch(100);
+
+			if (ch != KEY_NULL && ch != KEY_TIMEOUT)
+			{
+				BBS_last_access_tm = time(NULL);
+			}
+
 			switch (ch)
 			{
 			case KEY_NULL: // broken pipe
+				log_error("KEY_NULL\n");
+				show_top10 = 0;
 				return 0;
 			case KEY_TIMEOUT:
 				if (time(NULL) - BBS_last_access_tm >= MAX_DELAY_TIME)
 				{
+					log_error("User input timeout\n");
+					show_top10 = 0;
 					return 0;
 				}
 				continue;
 			case CR:
-				igetch_reset();
 			default:
 				switch (menu_control(&top10_menu, ch))
 				{
@@ -270,15 +301,14 @@ int show_top10_menu(void *param)
 				}
 			}
 
-			BBS_last_access_tm = time(NULL);
-
 			if (ch == EXITMENU)
 			{
 				break;
 			}
 		}
 	}
-	
+
+	show_top10 = 0;
 	return REDRAW;
 }
 
@@ -306,9 +336,10 @@ int locate_article(void *param)
 
 int favor_topic(void *param)
 {
-	clearscr();
-	prints("此功能尚未实现");
-	press_any_key();
+	if (article_favor_display(&BBS_article_favor) < 0)
+	{
+		log_error("article_favor_display() error\n");
+	}
 
 	return REDRAW;
 }