Вопросы, советы, возможные модификации и вообще все про модуль статистики кланов

Модератор: Exinaus, shw

#7123 Exinaus 29 авг 2012, 00:52
shw писал(а):Admin тут посмотрел код модуля, предлагаю внести следующие правки :

Внесенные изменения:
header.php
Добавлен логотип клана, подгружающийся с сайта WG


Столкнулся с этой правкой на модуле некоторых кланов.
Во первых в мозилле этот логотип сдвигает данные с таблицами вправо. Вот так например: http://awesomescreenshot.com/0a5ef4o3d

Во вторых проверка логотипа в корне не верная. Перед выводом логотипа проверяется доступность сайта ВГ, но при этом не проверяется установлен ли сам логотип или нет. Плюс к этому у некоторых людей is_valid_url($config['td']) == true не срабатывает корректно, после того как функция была изменена под новые требования php в недавнем патче. Так что ее использование нежелательно. В самом модуле от нее отказались.
Я бы предложил заменить код проверки на например такой:
Код: Выделить всё$filename = "https://worldoftanks.ru/dcont/clans/emblems/{$config['clan']}/emblem_64x64.png";
$headers = get_headers($filename);
if(substr($headers[0], 9, 3) != '404') {
  echo 'File exist';
}

Вот так одновременно проверяется и доступность сайта, и наличие клан эмблемы.
#7124 shw 29 авг 2012, 06:25
Exinaus писал(а):Во вторых проверка логотипа в корне не верная.
Я бы предложил заменить код проверки на например такой:
Код: Выделить всё$filename = "https://worldoftanks.ru/dcont/clans/emblems/{$config['clan']}/emblem_64x64.png";
$headers = get_headers($filename);
if(substr($headers[0], 9, 3) != '404') {
  echo 'File exist';
}

Вот так одновременно проверяется и доступность сайта, и наличие клан эмблемы.

Согласен, что код некорректен, но я работал с тем что есть. Т.к. нет доступа к последним наработкам, которые планируются во внедрение.
А Ваш код я бы все-таки переправил на:
Код: Выделить всё<?php    $filename = $config['td'].'/dcont/clans/emblems/'.$config['clan'].'/emblem_64x64.png';
         $headers = get_headers($filename);
         if(substr($headers[0], 9, 3) != '404') {
            echo 'Выводим тут'}; ?>



Exinaus писал(а):Во первых в мозилле этот логотип сдвигает данные с таблицами вправо. Вот так например: http://awesomescreenshot.com/0a5ef4o3d

На лисице не тестировал. Согласен, глюк налицо.
Возможно ли протестировать обновленный файл?
(2.46 КБ) Скачиваний: 138


+ выкладываю для тестирования вкладку "Активность награды".
(2.27 КБ) Скачиваний: 142

изменения:

 Читать дальше

#7146 friper 30 авг 2012, 00:07
А можно подробней узнать, куда прописывать дополнительные функции, что бы отображалось, кто недавно пришол в клан, и кто ушёл?
#7147 shw 30 авг 2012, 06:48
friper писал(а):А можно подробней узнать, куда прописывать дополнительные функции, что бы отображалось, кто недавно пришол в клан, и кто ушёл?

Уже включены в стандартный модуль.
подробнее в первом и третьем посте темы viewtopic.php?f=30&t=2553
#7178 ivanovohost 31 авг 2012, 13:45
Вот что пишет крон в файл:
Код: Выделить всёNotice:  Undefined index:  pass in /****/********/*****/**********/www/clan.37ip.ru/cron.php on line 35
Вам нужно авторизироваться, что бы начать процесс


Может и вовсе стоит убрать авторизацию в обработчике?

Команда крона:


Пароль правильный! Через браузер выполняется.

Хостинг для статистики WOT - Что-то типа этого - http://clan.37ip.ru Домен второго уровня, база, cron. Все вопросы в асю: 4289765
#7180 shw 31 авг 2012, 15:41
ivanovohost писал(а):Вот что пишет крон в файл:
Код: Выделить всёNotice:  Undefined index:  pass in /****/********/*****/**********/www/clan.37ip.ru/cron.php on line 35
Вам нужно авторизироваться, что бы начать процесс

Даю сообщение-дубль
Проверить cron.php, строчка 30:
Если в вашем файле перед вторым isset нет "!" symbol, надо его добавить. Будет выглядеть так:
Код: Выделить всёif(!isset($_GET['user']) && !isset($_GET['pass'])){


Та же при использовании пароля необходимо использовать только цифры и маленькие буквы латинского алфавита.
#7181 ivanovohost 31 авг 2012, 15:50
Да я это знаю, все там нормально, вот 30я строка:
if(!isset($_GET['user']) && !isset($_GET['pass'])){


Пароль состоит из латиницы нижнего регистра!
Права на cron.php - 644

Хостинг для статистики WOT - Что-то типа этого - http://clan.37ip.ru Домен второго уровня, база, cron. Все вопросы в асю: 4289765
#7182 shw 31 авг 2012, 15:57
ivanovohost писал(а):Да я это знаю, все там нормально.
Пароль состоит из латиницы нижнего регистра!
Права на cron.php - 644

wget -q от кого выполняется?

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 1

Реклама | Adv