Прокоментерию статью с моей точки зрения и так

Вот не понимаю зачем вот етот кусок кода в функции blocks

if ($view == 0) {
render_blocks($side, $value);
} elseif ($view == 1 AND is_user($user) || is_admin($admin)) {
render_blocks($side, $value);
} elseif ($view == 2 AND is_admin($admin)) {
render_blocks($side, $value);
} elseif ($view == 3 AND !is_user($user) || is_admin($admin)) {
render_blocks($side, $value);
}

если можно зделать так
if (is_admin($admin)) {
$view = "";
} elseif (is_user($user)) {
$view = "AND ((view = '0') OR (view = '1'))";
} else {
$view = "AND ((view = '0') OR (view = '3'))";
}
$result = $db->sql_query("SELECT bid, bkey, title, content, url, bposition, weight, blockfile, view, expire, action, subscription, refresh, time FROM ".$prefix."_blocks WHERE active='1' $view $querylang ORDER BY bposition, weight ASC");

И вместо етого куска кода
if ($view == 0) {
render_blocks($side, $value);
} elseif ($view == 1 AND is_user($user) || is_admin($admin)) {
render_blocks($side, $value);
} elseif ($view == 2 AND is_admin($admin)) {
render_blocks($side, $value);
} elseif ($view == 3 AND !is_user($user) || is_admin($admin)) {
render_blocks($side, $value);
}

оставим токо
render_blocks($side, $value);