Тестирование компонента Погода для ICMS1

ЕСТЬ РЕШЕНИЕ ЗАКРЫТО
#1 15 сентября 2015 в 22:02
Всем привет!
Порядка 1200 дней назад был опубликован Компонент POGODA.
Компонент был сыроват, чтобы что-то менять, приходилось править файлы. Кроме того, периодически был недоступен провайдер погоды www.weather.ua. Поэтому уже давно назревал вопрос о дальнейшей доработке. Ну вот, не прошло и 3-х лет 😊, компонент готов. Но нужна помощь в тестировании. Совершенно нет времени тестировать. А проверить нужно досконально. Помогите пожалуйста, кому не лень.
Кратко об изменениях с версии 1.0.0:
1. Изменен провайдер погоды на openweathermap.org ;
2. Все настройки компонента теперь в админке. Для каждой страницы(текущая, 3, 5, 7, 10 дней, 2 недели) можно указать свой тайтл, h1, ключевики, описание, текстовые блоки до прогноза и после...
3. Добавлен модуль информера текущей погоды.

Забрать и тестировать .
Инструкция по установке в файле README.md. Не стоит пока устанавливать на рабочие сайты. Если устанавливаете поверх предыдущей версии компонента, все файлы и таблицы в БД перезапишутся. Не забудьте сохранить резервные копии.

Найденные недочеты выкладывайте здесь. Появится время, скопом всё поправлую и пустим компонент в свободное плавание.
Спасибо!
#2 15 сентября 2015 в 23:21
Приветствую.1.10.6. Код для города в ссылке не показывает,
  1. http://openweathermap.org/find?q=pskov
Вот так выглядет на странице
#3 16 сентября 2015 в 06:01

Приветствую.1.10.6. Код для города в ссылке не показывает,
Код PHP:
openweathermap.org/find?q=pskov

Lora
Это страница для результатов поиска. Далее нужно перейти на страницу города, т.е. перейти по ссылке найденного города. И там уже смотреть в адресной строке код. Для Пскова код будет 504341 .
Другой вопрос, у вас без кода страница компонента грузится? Вроде бы должна быть страница 404. Спасибо, проверю.
#4 16 сентября 2015 в 09:08

Вроде бы должна быть страница 404

Марат
Так и есть. smileКод я взял из файла.Но вот что то не кажет ничего. Да и со стилями что-то.
#5 16 сентября 2015 в 09:17

Но вот что то не кажет ничего.

Lora
Итак, порядок запуска. Установили компонент, внесли в админке код города. Сохранили. Если сразу же хотите увидеть результат, тогда нужно в админке вручную запустить задачи CRON компонента. Нет данных в БД, поэтому и не показывается ничего.

Да и со стилями что-то.

Lora
Стили подточены под фиксированную ширину без сайдбара. Всё же лучше, наверно, задать меньшую ширину. Посмотрю потом. Пока можете поправить как вам нужно, стили вынесены в отдельный файл для удобства.
#6 16 сентября 2015 в 13:16
Сори, с CRON тормознул. Запустил, главная вроде работает.
, но вот остальное не хочет. ссылка в модуле "Подробный прогноз погоды" ни куда не видёт. Так же страница погодаы на 3,5,… дней ничего не выводит.
#7 16 сентября 2015 в 13:41
Lora, ничего не пойму из Ваших объяснений. Как понять:

ссылка в модуле "Подробный прогноз погоды" ни куда не видёт

Lora
?
Если в шаблоне он ведет на страницу "/pogoda".

Так же страница погодаы на 3,5,… дней ничего не выводит.

Lora
У компонента 3 задачи для CRON. Вы все три запускали?
#8 16 сентября 2015 в 15:46

У компонента 3 задачи для CRON. Вы все три запускали?

Марат
Конечно нет laugh
Запустил.Всё фунциклирует. До 5-ти дней показывает почасовую, затем по дням.
#9 16 сентября 2015 в 15:49
Единственно реальный был компонент погоды. Марат поддерживает создание автоматом нужных мест для указания погоды?
#10 16 сентября 2015 в 16:15
Ага по тестируем. Спасибо Марат.
Что пока заметил: модуль не имеет настроек, но при сохранении "настроек" с морды модуля перекидывает на 404 страницу:
сайт/admin/index.php?view=modules&do=save_auto_config&id=94&ajax=1&title_only=1

Ерунда конечно, но

Найденные недочеты выкладывайте здесь.

Марат

В настройках компонента:
Текст до блока погоды, Текст после блока погоды, сохраняет только текст, ссылки в текст, изображения, видео, и пр. форматирование текста не сохраняются
#11 16 сентября 2015 в 18:00
зачем плодить лишние запросы к базе?
#12 16 сентября 2015 в 20:02
Василий Сергеевич, спасибо за дельные замечания. Поправлю.

Марат поддерживает создание автоматом нужных мест для указания погоды?

letsgo
Компонент только для одного города. Какой город в админке задан, для такого и выведет.

зачем плодить лишние запросы к базе?

я другое дерево
Поконкретнее пожалуйста.
#13 16 сентября 2015 в 22:18

Что пока заметил: модуль не имеет настроек, но при сохранении "настроек" с морды модуля перекидывает на 404 страницу: сайт/admin/index.php?view=modules&do=save_auto_config&id=94&ajax=1&title_only=1

Василий Сергеевич
К сожалению, это не ошибка модуля. Чтобы исправить, нужно открыть cистемный файл /templates/_default_/admin/modconfig.php и перед строкой 37:
  1. <input type="submit" name="save" value="<?php echo $_LANG['SAVE']; ?>" />
нужно вставить:
  1. <input type="hidden" name="csrf_token" value="<?php echo cmsUser::getCsrfToken(); ?>" />
Ошибка из-за того, что в модуле без конфига не предусмотрена отправка ксрф токена, а валидация присутствует.
#14 16 сентября 2015 в 22:33

Поконкрет

Марат
сколько запросов к базе делает при выводе ваш компонент?
#15 16 сентября 2015 в 22:59

сколько запросов к базе делает при выводе ваш компонент?

я другое дерево
2 простых запроса. Это много?
Используя этот сайт, вы соглашаетесь с тем, что мы используем файлы cookie.