[Вопрос]

Продолжаю переделывать админку...

Всем известны неудобства в плане кастомайзинга селектов.
В этой связи, я избавляюсь (где это возможно) от селектов.

Переделываю селекты на простые линки во всплывашках.
Пока использую для этого стандарные vbmenu.

PHP Code:

Возьмем для примера: adminfunctions_template.php => function print_style

<select name=\"styleEdit_$printstyleid\" id=\"menu_$styleid\" onchange=\"Sdo...>
    <optgroup label=\"" . $vbphrase['template_options'] . "\">
        <option value=\"template_templates\">" . $vbphrase['edit_templates'] . "</option>

ну и так далее... Переделал этот кусок на линки с vbmenu. 


PHP Code:

Убил кусок яваскрипта в: vbulletin_templatemgr.js

var win=window;var n=0;function Sdo(B,A)...
и так далее... 


В итоге:

• сохранил компактность
• сохранил полную работоспособность
• получил удобный кастомайзинг
• уменьшил объем кода

Все пучком, все работает, но мучает паранойя.
Почему разрабы использовали именно селекты? Что я упускаю?

Originally Posted by syn

Почему разрабы использовали именно селекты?

потому что это проще в разы, чем парить мозх еще и яваскриптами

так вот например в этом случае совсем наоборот.
тут как раз используется дополнительно (лишний?) яваскрипт.

PHP Code:

function Sdo(B, A) {
    B = B.split("_");
    if (B[0] == "template") {
        switch (B[1])
        {
            case "templates": goURL = "modify&expandset="; break;
            case "addtemplate": goURL = "add&dostyleid="; break;
            case "editstyle": goURL = "editstyle&dostyleid="; break;
            case "addstyle": goURL = "addstyle&parentid="; break;
            case "delete": goURL = "deletestyle&dostyleid="; break;
            case "download": goURL = "files&dostyleid="; break;
            case "revertall": goURL = "revertall&dostyleid="; break
        }
        if (goURL) {
            window.location = "template.php?s=" + SESSIONHASH + "&group=" + document.forms.tform.group.value + "&do=" + goURL + A
        }
    } else {
        if (B[0] == "css") {
            window.location = "css.php?s=" + SESSIONHASH + "&do=edit&dowhat=" + B[1] + "&group=" + document.forms.tform.group.value + "&dostyleid=" + A
        }
    }
} 


syn добавил 25.03.2015 в 21:08
а я просто взял линки и запихнул их в (уже имеющийся) vbmenu.

Originally Posted by syn

Почему разрабы использовали именно селекты? Что я упускаю?

Упускаете.)) Вырубите в браузере джаваскрипт и попробуйте теперь сделать что-нибудь в админке.)) Если бы вы более глубоко изучили устройство движка, то поняли бы, что весь его интерфейс продублирован для работы без джаваскрипта. Что, собственно, грамотно и правильно.

На счет этого я как раз таки в курсе.
Более того, часть дублирующего функционала я давно уже вырезал из движка.

На счет яваскриптов, это тема для отдельной дискуссии.
Но у меня именно такая политика - если пользователь решил вырубить яваскрипт, это уже его проблемы.

но все равно, сенкс за отзывы.

syn добавил 27.03.2015 в 04:42
Как раз в тему про выключенные яваскрипты попалось.

Наткнулся на очередной селект в админка/forum.php
И как эта конструкция будет работать с выключенным яваскриптом?

PHP Code:

'<select ... onchange="js_forum_jump(' . $forum['forumid'] . ')">' . construct_select_options($forumoptions) . '</select>'; 


И к нему соответственно: Это

PHP Code:

$forumoptions = array(
    'edit'        => $vbphrase['edit_forum'],
    'view'        => $vbphrase['view_forum'],
    'remove'    => $vbphrase['delete_forum'],
    'add'        => $vbphrase['add_child_forum'],
    'addmod'    => $vbphrase['add_moderator'],
    'annc'        => $vbphrase['add_announcement'],
    'perms'    => $vbphrase['view_permissions'],
    'podcast'    => $vbphrase['podcast_settings']
); 


И это:

PHP Code:

<script type="text/javascript">
function js_forum_jump(foruminfo)
{
    //var cp_collapse_forums = <?php echo intval($vbulletin->options['cp_collapse_forums']); ?>;
    if (foruminfo == 0)
    {
        alert('<?php echo addslashes_js($vbphrase['please_select_forum']); ?>');
        return;
    }
    else if (typeof(document.cpform.forumid) != 'undefined')
    {
        action = document.cpform.controls.options[document.cpform.controls.selectedIndex].value;
    }
    else
    {
        action = eval("document.cpform.f" + foruminfo + ".options[document.cpform.f" + foruminfo + ".selectedIndex].value");
    }
    if (action != '')
    {
        switch (action)
        {
            //case 'perms': page = "forumpermission.php?do=modify&f="; break; //cp_collapse_forums
            case 'edit': page = "forum.php?do=edit&f="; break;
            case 'remove': page = "forum.php?do=remove&f="; break;
            case 'add': page = "forum.php?do=add&parentid="; break;
            case 'addmod': page = "moderator.php?do=add&f="; break;
            case 'annc': page = "announcement.php?do=add&f="; break;
            case 'view': page = "/forumdisplay.php?f="; break;
            case 'perms': page = "forumpermission.php?do=modify&devnull="; break;
            case 'podcast': page = "forum.php?do=podcast&f="; break;
            case 'empty': page = "forum.php?do=empty&f="; break;
        }
        document.cpform.reset();
        jumptopage = page + foruminfo + "&s=<?php echo $vbulletin->session->vars['sessionhash']; ?>";
        if (action == 'perms')
        {
            window.location = jumptopage + '#forum' + foruminfo;
        }
        else
        {
            window.location = jumptopage;
        }
    }
    else
    {
        alert('<?php echo addslashes_js($vbphrase['invalid_action_specified']); ?>');
    }
}

Originally Posted by syn

как эта конструкция будет работать с выключенным яваскриптом?

помимо "джампа" на событии онченж, там есть кнопка: выбрал в менюхе опцию => нажал на кнопку =)

ну значение из массива принимается же где-то? верно?
во возьмем например это: передается из формы. так?

PHP Code:

'annc' => $vbphrase['add_announcement'], 


соответственно попробуем поискать где он принимается.
а нету такого. и кто его (этот параметр) примет?
и вполне ожидаемо. ведь нам нужен не annc, а add

Originally Posted by syn

ну значение из массива принимается же где-то? верно?

Нет, не верно.

Originally Posted by syn

это: передается из формы. так?

Нет.

Я не знаю, какую версию двига вы калупаете, но это в сущности не важно. Форма, в которой расположен ваш селект и, так же, кнопка Submit, имеет атрибут action, в котором указывается скрипт, который будет принимать форму и параметры, которые ему будут переданы. Там где вы ищите, есть несколько мест с вызовом функции print_form_header. Например:

PHP Code:

print_form_header('forum', 'doorder'); 


Первый аргумент - имя скрипта, второй - параметры. Посмотрим в код print_form_header, чтобы понять, что будет в итоге:

PHP Code:

echo "<form action=\"$phpscript.php?do=$do\"" . iif($uploadform, ' enctype="multipart/form-data"') . " method=\"$method\"" . iif($target, " target=\"$target\"") . " name=\"$clean_name\" id=\"$name\">\n"; 


Соответственно, форму будет принимать forum.php, которому передается: do=doorder. И если теперь посмотреть файл forum.php, то несложно найти код, обрабатывающий этот запрос:

PHP Code:

if ($_POST['do'] == 'doorder')
{ 


все верно, все так и есть. я как раз об этом и говорю.

как раз из той формы, что вы привели и будет передан annc
а принимать его должен forum.php => ($_POST['do'] == 'doorder')

и будет принят, вот только обработан не будет.


syn добавил 27.03.2015 в 18:14
может я не совсем понятно излагаю свои мысли.
попробую перефразировать.

при условии, что яваскрипт выключен, что сделает скрипт, когда получит в запросе annc
ничего не сделает, поскольку принимающему forum.php => ($_POST['do'] == 'doorder') ничего не известно об элементе массива annc.

проверил
кнопка "Вперед" не работает вообще на странице редактирования разделов (напротив раздела) при отключенном яваскрипте
не работает ни одна из опций выбранных в менюшке селект