是bug还是？

//系统频道
if($_SGET['action'] != 'index') {
        if(empty($channels['menus'][$_SGET['action']]['upnameid']) && $channels['menus'][$_SGET['action']]['upnameid'] != 'news') {
                $scriptfile = S_ROOT.'./'.$_SGET['action'].'.php';
        } else {
                $scriptfile = S_ROOT.'./news.php';
        }
在/index.php文件中，100行，大家看加亮部分，是不是有点说不通呢？
是康盛团队故意这么写的还是怎么回事，反正我没看明白。

说得通啊，先判断是否有首页，如果是首页就不再加载其他的文件了。
empty($channels['menus'][$_SGET['action']]['upnameid'])判断父频道是否为空，不为空，则有可能是资讯频道。
$channels['menus'][$_SGET['action']]['upnameid'] != 'news'判断是否是资讯频道，不是的话，就加载应有的页面。

首先判断$channels['menus'][$_SGET['action']]['upnameid']为空，然后判断$channels['menus'][$_SGET['action']]['upnameid']不等于news。如果第一条件成立，即为空，那么第二条件不等于news就肯定成立，整个if语句为真；如果第一个条件不成立，那么第二个条件就不用判断了，整个if语句条件为假，因此这个if语句完全可以只判断第一个条件，第二个条件还需要判断吗？。if(！empty($channels['menus'][$_SGET['action']]['upnameid']) && $channels['menus'][$_SGET['action']]['upnameid'] != 'news')我觉得应该是这样，楼上的说呢？

你的见识正确，第一个判断是为了区分是否是复制的文章频道，已经可以将结果区分出来，第二个判断可以去掉。
第二个判断，所有的系统内置的非文章频道的upnameid都是空，永远都不会等于news，所以说它永远成立，去掉也不影响程序的判断。

谢谢楼上的解答，谢谢。