Blocks Admin [config]

Eric Barr <[email protected]>
Newsgroups gmane.comp.web.envolution.devel
Message-ID <[email protected]>
I noticed that in the admin section for blocks that the collapseable menu
configuration isn't working.. some of the variables aren't in the new
language file and the advblock.php file is missing the if{} statement for
the collapseable menu.

the admin.php file in the language section for Blocks needs to have:

if (!defined('_CONFIGURE')) {
    define('_CONFIGURE','Configuration');
}
if (!defined('_UPDATE')) {
    define('_UPDATE', 'Update');
}
define('_BLOCKSCONFIG','Disable collapseable menu icons?');

added to it,

and the advblock.php needs to have

    if(pnUserLoggedIn()) {
        if(checkuserblock($row)=='1') {
            if (!empty($row['title'])) {
                $row['title'] .=" <a
href=\"modules.php?op=modload&name=Blocks&file=index&req=ChangeStatus&bid=$row[bid]&amp;authid=".pnSecGenAuthKey()."\"><img
src=\"images/global/upb.gif\" border=\"0\" alt=\"\"></a>";
            }
        } else {
            $row['content'] ='';
            if (!empty($row['title'])) {
                $row['title'] .=" <a
href=\"modules.php?op=modload&name=Blocks&file=index&req=ChangeStatus&bid=$row[bid]&amp;authid=".pnSecGenAuthKey()."\"><img
src=\"images/global/downb.gif\" border=\"0\"
alt=\"\"></a>";
            }
        }
    }

replaced with:

    // check for collapseable menus being enabled.
    if (pnModGetVar('Blocks', 'collapseable') == 1) {
	if (pnUserLoggedIn()) {
	    if (checkuserblock($row)=='1') {
		if (!empty($row['title'])) {
		    $row['title'] .=" <a
href=\"modules.php?op=modload&name=Blocks&file=index&req=ChangeStatus&bid=$row[bid]&amp;authid=".pnSecGenAuthKey()."\"><img
src=\"images/global/upb.gif\" border=\"0\" alt=\"\"></a>";
		}
	    } else {
		$row['content'] ='';
		if (!empty($row['title'])) {
		    $row['title'] .=" <a
href=\"modules.php?op=modload&name=Blocks&file=index&req=ChangeStatus&bid=$row[bid]&amp;authid=".pnSecGenAuthKey()."\"><img
src=\"images/global/downb.gif\" border=\"0\" alt=\"\"></a>";
		}
	    }
	}
    }

if this has already been fixed.. please disregard! thanks

-------------------
ApathyBoy
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.