group manager buttons

[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.

Code Select Expand

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.

[Andi]

Hi

UDB_INTEGRATION is ever defined.

I think this line:

Code Select Expand

if (defined('UDB_INTEGRATION')) {

must be:

Code Select Expand

if (UDB_INTEGRATION != 'coppermine') {

[Casper]

Hi Andi,

that certainly works on an unbridged install.  if someone can confirm it does not affect bridged installs I will commit.

[omniscientdeveloper]

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

Code Select Expand


if (UDB_INTEGRATION != 'coppermine')


instead of "defined('UDB_INTEGRATION')"

[omniscientdeveloper]

This change has been committed.

[Tranz]

Thank you. The buttons are back.