можно ещё проще
код стремимся делать оптимальным и красивым.
Вообще надо стремиться не создавать лишних переменных, это расход памяти и вычислительных мощностей.
строки
PHP:
$cat_id = $forum_data['cat_id'];
$cat_name = $forums['cat_title_html'][$forum_data['cat_id']];
вовсе лишние.
Вместо
PHP:
'CAT_ID' => $cat_id,
'CAT_NAME' => htmlCHR($cat_name),
так
PHP:
'CAT_ID' => $forum_data['cat_id'],
'CAT_NAME' => $forums['cat_title_html'][$forum_data['cat_id']],
причём htmlCHR накладывать тут не надо, т.к. cat_title_html уже говорит о том что на текст уже наложена эта функция. Где это делается можно посмотреть в includes/datastore/build_cat_forums.php в 43-й строке.
Ну и вместо
$view_cat_url = "index.php?c=$cat_id";
соответственно
$view_cat_url = "index.php?c=". $forum_data['cat_id'];