From 2c34c21ed30babc89ce473545a0befbe43211360 Mon Sep 17 00:00:00 2001
From: Richard McGirr <rmcgirr83@gmail.com>
Date: Fri, 3 Jul 2020 06:56:19 -0500
Subject: [PATCH] Fix index page load

---
 core/nationalflags.php | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/core/nationalflags.php b/core/nationalflags.php
index 379c629..dd64fe5 100644
--- a/core/nationalflags.php
+++ b/core/nationalflags.php
@@ -197,17 +197,16 @@ public function top_flags()
 			return false;
 		}
 		// grab all the flags
-		$sql_array = array(
-			'SELECT'	=> 'u.user_flag, COUNT(u.user_flag) AS fnum',
-			'FROM'		=> array(USERS_TABLE => 'u'),
-			'WHERE'		=> $this->db->sql_in_set('u.user_type', array(USER_NORMAL, USER_FOUNDER)) . ' AND u.user_flag > 0',
-			'GROUP_BY'	=> 'u.user_flag',
-			'ORDER_BY'	=> 'fnum DESC, u.user_flag ASC',
-		);
+		$sql = 'SELECT f.*, COUNT(u.user_flag) as fnum
+			FROM ' . $this->flags_table . ' f
+				LEFT JOIN ' . USERS_TABLE . ' u on f.flag_id = u.user_flag
+			WHERE ' . $this->db->sql_in_set('u.user_type', array(USER_NORMAL, USER_FOUNDER)) . ' AND u.user_flag > 0
+			GROUP BY f.flag_id
+			ORDER BY fnum DESC, f.flag_name ASC';
 
 		// we limit the number of flags to display to the number set in the ACP settings
 		// and cache the query based on setting in ACP
-		$result = $this->db->sql_query_limit($this->db->sql_build_query('SELECT', $sql_array), $this->config['flags_num_display'], 0, $this->config['flags_cachetime']);
+		$result = $this->db->sql_query_limit($sql, $this->config['flags_num_display'], 0, $this->config['flags_cachetime']);
 
 		$count = 0;
 		$flags = $this->get_flag_cache();
@@ -216,9 +215,9 @@ public function top_flags()
 		{
 			++$count;
 			$this->template->assign_block_vars('flag', array(
-				'FLAG' 			=> $this->get_user_flag($row['user_flag']),
+				'FLAG' 			=> $this->get_user_flag($row['flag_id']),
 				'FLAG_USERS'	=> $this->user->lang('FLAG_USERS', (int) $row['fnum']),
-				'U_FLAG'		=> $this->helper->route('rmcgirr83_nationalflags_getflags', array('flag_id' => $flags[$row['user_flag']]['flag_id'])),
+				'U_FLAG'		=> $this->helper->route('rmcgirr83_nationalflags_getflags', array('flag_id' => $flags[$row['flag_id']]['flag_id'])),
 			));
 		}
 		$this->db->sql_freeresult($result);