Веб [DLE] Каталог скинов

@Genocide,Ответь в скайпе, а то я до тебя никак достучаться не могу

 

Спасибо, сегодня буду делать для WP, выложу сюда.

 

Обновил скрипт "Каталог скинов".
Теперь не требуется вводить данные для соединения к базе данных, нагрузка на базу данных минимизирована. Соединение с базой данных обеспечивает встроенные DLE функции.

 

Уже получше)
Но все же с логикой, а точнее с вложенными if лучше поиграться в сторону линейности кода...
Например:

Спойлер

Код:

//Сейчас так:
$upload = 0;
$imageinfo = getimagesize ($_FILES['filename']['tmp_name']);
if($_FILES['filename']['type'] = "image/png")  {
    if($imageinfo['mime'] = 'image/png') {
        if($imageinfo["0"] == '64') {
            if($imageinfo["1"] == '32') {
                if (preg_match("/\.(png)$/i", $_FILES['filename']['name'])) {
                    if(is_uploaded_file($_FILES["filename"]["tmp_name"])) {
                        $upload = 1;
                    }
                }
            }
        }
    }
}
if($upload == 0) {
    echo "<div class='warningbad'>Загруженный скин не подходит по каким-либо параметрам.</div>";
} else {
    $db->query("INSERT INTO s_skins VALUES('NULL', '$skinname')");
    $skinid = $db->insert_id();
    echo "<div class='warninggood'>Скин \"$skinname\" успешно добавлен в каталог скинов.</div>";
    move_uploaded_file($_FILES["filename"]["tmp_name"], "$dir_skins/skin_$skinid.png");
}
 
 
//Лучше так:
 
$error = false;
$imageinfo = getimagesize($_FILES['filename']['tmp_name']);
if( $_FILES['filesname']['type'] != 'image/png') $error = true;
if( ($error === false) && ($imageinfo['mime'] != 'image/png']) ) $error = true;
if( ($error === false) && ($imageinfo['0'] != 64) ) $error = true;
if( ($error === false) && ($imageinfo['1'] != 32) ) $error = true;
if( ($error === false) && !(preg_match("/\.(png)$/i", $_FILES['filename']['name'])) ) $error = true;
if( $error === true ) {
    echo "<div class='warningbad'>Загруженный скин не подходит по каким-либо параметрам.</div>";
} else {
    $db->query("INSERT INTO s_skins VALUES('NULL', '$skinname')");
    $skinid = $db->insert_id();
    echo "<div class='warninggood'>Скин \"$skinname\" успешно добавлен в каталог скинов.</div>";
    move_uploaded_file($_FILES["filename"]["tmp_name"], "$dir_skins/skin_$skinid.png");
}

Кстати, если сделать $error не логической, а целочисленной и присваивать каждой ошибке свой код, то можно потом через switch-case отловить и сообщить пользователю, какая именно ошибка произошла...
И полностью интегрировать как модуль, а не через статические страницы...

 

Спасибо большое. Вы явно благоприятно влияете на меня

 

Не не не, не "вы", а "ты", до "вы" мне еще расти и расти, вот эдак лет через 20, будет мне 40, вот тогда можно и на "вы"
Жду новую версию, дабы дать новые советы

 

помогите нубу....
Warning: rename(/home/djtrue/lifez-project.ru/engine/modules/skins/lib/djtrue.png,/home/djtrue/lauch/skins/djtrue.png) [function.rename]: No such file or directory in /home/djtrue/lifez-project.ru/engine/modules/skins/install.php on line 15

Warning: Cannot modify header information - headers already sent by (output started at /home/djtrue/lifez-project.ru/engine/modules/skins/install.php:15) in /home/djtrue/lifez-project.ru/engine/modules/skins/install.php on line 38

 

Неправильно директорию указал. Вот которую нужно: /home/djtrue/lifez-project.ru/engine/modules/skins/

 

Денвер?
А разница?
Укажите относительный путь и подключите как модуль, а не инклудом, и будет вам счастье.

 

еще 1 проблемы, когда всё установил, пишет Невозможно загрузить шаблон: /templates/Default/content.tpl

 

Блин, вам всем готовенькое только подавай....
Читайте документацию.

 

Еще бы плащи сделать с загрузкой только для специальных групп) PEX Iconomy и префиксы. Цены бы не было

 

Смотри index.php в корне сайта

 

Файлик индекс перевёл в utf-8, всё корректно, кроме отображения картинки. Все файлы на местах, подобно инструкции. В чём может быть суть?

 

Чувак, дрочу на твои скрипты. Делай ЛК и продавай его, сразу же куплю!

 

ЛК в разработке как модуль для DLE, будет распространяться бесплатно.

 

БЕСПЛАТНО?!
пойду дрочить...

 

@Geroll, я кончил

 

а /var пере /www/ обязательно ставить?

 

У каждого свой абсолютный путь.