From 07845198da29eb6b63e044d9c9cfc3fa858148ee Mon Sep 17 00:00:00 2001 From: Paul Holden Date: Fri, 2 Aug 2024 10:06:03 +0100 Subject: [PATCH] MDL-83152 mod_glossary: fix chronological sorting of pending entries. --- mod/glossary/lib.php | 33 ++++++++++++++++++--------------- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/mod/glossary/lib.php b/mod/glossary/lib.php index de95bb8a08a..5991cef8ef2 100644 --- a/mod/glossary/lib.php +++ b/mod/glossary/lib.php @@ -1834,7 +1834,7 @@ function glossary_print_approval_menu($cm, $glossary,$mode, $hook, $sortkey = '' glossary_print_all_links($cm, $glossary, $mode, $hook); - glossary_print_sorting_links($cm, $mode, 'CREATION', 'asc'); + glossary_print_sorting_links($cm, $mode, $sortkey, $sortorder); } /** * @param object $cm @@ -2056,9 +2056,14 @@ function glossary_print_sorting_links($cm, $mode, $sortkey = '',$sortorder = '') $bopen = ''; $bclose = ''; - $neworder = ''; - $currentorder = ''; - $currentsort = ''; + $neworder = ''; + $currentorder = ''; + $currentsort = ''; + + if ($sortkey === '') { + $sortkey = 'CREATION'; + } + if ( $sortorder ) { if ( $sortorder == 'asc' ) { $currentorder = $asc; @@ -2081,20 +2086,18 @@ function glossary_print_sorting_links($cm, $mode, $sortkey = '',$sortorder = '') $icon = " " . $OUTPUT->pix_icon('asc', $newordertitle, 'glossary'); } } - $ficon = ''; - $fneworder = ''; - $fbtag = ''; - $fendbtag = ''; - $sicon = ''; - $sneworder = ''; + $ficon = ''; + $fneworder = ''; - $sbtag = ''; - $fbtag = ''; - $fendbtag = ''; - $sendbtag = ''; + $sicon = ''; + $sneworder = ''; - $sendbtag = ''; + $sbtag = ''; + $fbtag = ''; + $fendbtag = ''; + + $sendbtag = ''; if ( $sortkey == 'CREATION' or $sortkey == 'FIRSTNAME' ) { $ficon = $icon;