group manager buttons group manager buttons
 

News:

CPG Release 1.6.26
Correct PHP8.2 issues with user and language managers.
Additional fixes for PHP 8.2
Correct PHP8 error with SMF 2.0 bridge.
Correct IPTC supplimental category parsing.
Download and info HERE

Main Menu

group manager buttons

Started by Tranz, February 13, 2005, 02:20:12 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Tranz

I noticed that my group control panel was missing a couple of buttons. I found this was where the code was for showing the buttons.
if (defined('UDB_INTEGRATION')) {
    echo <<<EOT
        <tr>
            <td colspan="14" align="center" class="tablef">
                        <input type="submit" name="apply_modifs" value="{$lang_groupmgr_php['apply']}" class="button">&nbsp;&nbsp;&nbsp;
                </td>
        </form>
        </tr>

EOT;
} else {
    echo <<<EOT
        <tr>
            <td class="tablef"><input type="checkbox" name="checkAll2" onClick="selectAll(this,'delete_group');" class="checkbox" title="$lang_check_uncheck_all" /></td>
            <td colspan="13" align="center" class="tablef">
                        <input type="submit" name="apply_modifs" value="{$lang_groupmgr_php['apply']}" class="button">&nbsp;&nbsp;&nbsp;
                        <input type="submit" name="new_group" value="{$lang_groupmgr_php['create_new_group']}" class="button">&nbsp;&nbsp;&nbsp;
                        <input type="submit" name="del_sel" value="{$lang_groupmgr_php['del_groups']}" onClick="return confirmDel()" class="button">
                </td>
        </form>
        </tr>

EOT;


It makes sense but the thing is I do not have integration. So there is an error elsewhere that is causing the script to think I have udb enabled and only showing the one button.

Casper

Confirmed same on my unbridged install.
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here

Andi

Hi :)

UDB_INTEGRATION is ever defined.

I think this line:
if (defined('UDB_INTEGRATION')) {

must be:
if (UDB_INTEGRATION != 'coppermine') {
hope, I could help you... :)

Casper

Hi Andi,

that certainly works on an unbridged install.  if someone can confirm it does not affect bridged installs I will commit.
It has been a long time now since I did my little bit here, and have done no coding or any other such stuff since. I'm back to being a noob here

omniscientdeveloper

#4
I'll check it out when I get home this evening. It should say:


if (UDB_INTEGRATION != 'coppermine')


instead of "defined('UDB_INTEGRATION')"

omniscientdeveloper

This change has been committed.

Tranz

Thank you. The buttons are back.