Модуль Donate - Amoney для J1.5 - J2.5 и J3.1 - страница 2 - Модули и плагины

Насколько модуль безопасен? Есть ли вероятность, что сайт могли взломать через этот модуль?
Недавно мой сайт был заражен. Касперский отловил пару троянов и еще кучу зараженных файлов JS и PHP вычислил Айболитом. Произошло это сразу после установки модуля. Может так совпало? В логах нашел внешние запросы, обращающиеся к папке модуля. Ни у кого таких проблем не было?

Надёжно, только там, где нет интернета и нет ни какой связи.
Не занимаюсь вирусами и троянами, есть сомнение проверяйте архив антивирусной.

Онлайн-сканеры Dr.Web

В логах нашел внешние запросы, обращающиеся к папке модуля.

Вы не путаете, может хост пытался создать кэш

- Кстати хотел бы заметить, что данный модуль отработан с Joomla 1.0 до Joomla 3.0.
Если интересны сплетни в гугле полно инфо о заражение js-скриптов, взлом Joomla 2.5...   
 

leo78, спасибо за ответ!

Даже мысли не было, что в архиве содержится вирус! Я и писать то не хотел, чтобы не бросить тень на автора и модуль, тем более, что спец по безопасности из меня еще тот! Просто подумал - вдруг факты вторжения через скрипт модуля уже имели место и последуют советы по защите/обновлению или мои наблюдения окажутся полезными. Сейчас попробую поковырять логи за прошлый месяц. Если что нарою - сообщу. Просто не очень я это умею! 

Пробежал логи. Чего-то откровенно небезопасного не нашел. То, что меня насторожило выглядит так
Смутило, что внешний вызов, вызывающей ссылки нет. Обращение, правда, к картинке...

Честно, не та проблема должна вас интересовать, вот к примеру в скрипте сохранилось:
для Joomla 1.5 это нормально

а в Joomla 2.5 это должно быть уже

http://www.joomla25upgrade.com/joomla-25-tutorials/38-global-$mainframe-gone-now-its-jfactory-getapplication.html
http://docs.joomla.org/Application_(CMS)
http://stackoverflow.com/questions/11505574/steps-to-update-joomla-1-5-component-to-joomla-2-5-component

Кстати мод не ставился на 3.1.1 дело было в том что:
для Joomla 1.5
для Joomla 3.0

http://techjoomla.com/joomla-development/joomla-30-conversion-changes-needed-when-converting-a-common-installable-package-for-joomla-15x-a-25x-to-joomla-30.html

Вот, что надо действительно решать

Сейчас мод подгоняю под HTML5! - J3.1.1

По LiqPay.com, произошли изменения теперь не phone, а  card, эти новаторы переделали прием платежей и принимают банковские карты, но чтобы сайт принимал деньги, всего два банка чьи банковские карты принимают или расчётный счёт, короче, сбербанка карты и сберкнижки не принимают!

 
   

Архив обновил будьте внимательны
mod_amoney_j25-J30-J31v103b.zip   - для Joomla 3.1 версия 1.0.3b
mod_amoney_j15-J16-J17v103b.zip   - для Joomla 1.5 версия 1.0.3b
- присутствуют исправления

Покритикую конструктивно, не злитесь
1. в точке входа require JModuleHelper::getLayoutPath(''mod_amoney', $params->get('layout', 'default'));
должен же я иметь возможность макет альтернативный создавать
2. $document =& JFactory::getDocument(); в версии php 5.3 $document = JFactory::getDocument();
и одной строки хватит
3. jQuery.noConflict() вы думаете вообще нормально использовать, данный вызов. Используется для того, чтобы убрать конфликт скриптов. Так зачем оно вам пишите красиво без конфликтов
4. Для картинок используйте спрайты
5. Js здесь закончу не хватает. Зачем выводить а пом, js скрывать. Назначьте в CSS display: none;

Fedor Vlasenko
хотите критикуйте, главное, чтобы я понял, какая строка Вам в скрипте не понравилась и что вы предлагаете.
- а так, зачем мне что?
В основном в модуле много всего, половина которого не нужны, но сделано было по просьбам просящих, хотите помочь пишите конкретней, я подумаю, если меня всё устроит внесу в архив.

3. jQuery.noConflict() вы думаете вообще нормально использовать, данный вызов. Используется для того, чтобы убрать конфликт скриптов. Так зачем оно вам пишите красиво без конфликтов

Какое переключение вас не устраивает:
Кстати из четырёх вариантов кода, только последний четвертый не работает 

3. jQuery.noConflict() вы думаете вообще нормально использовать, данный вызов. Используется для того, чтобы убрать конфликт скриптов. Так зачем оно вам пишите красиво без конфликтов

Код: javascript

(function($){
    /* Код, использующий $ */
})(jQuery);

- Такой вариант не работает

Архив обновил будьте внимательны
mod_amoney_j25-J30-J31v103v.zip  - для Joomla 3.1
- изменены не критическое, но в других модулях приводила к белому экрану.

Fedor Vlasenko, Ну почему он раскрывается каждый раз?
демо



 

Спасибо. Главное, чтобы вы совершенствовались. Я показал, что вместо ссылок <a> следовало сделать <span> сделать его блочным
display:block; или можно <div> взять он уже блочный. Далее сделать из всех картинок одну - спрайт. И выводить в CSS , бекграунд спрайтом

Спасибо. Главное, чтобы вы совершенствовались. Я показал, что вместо ссылок <a> следовало сделать <span> сделать его блочным
display:block; или можно <div> взять он уже блочный. Далее сделать из всех картинок одну - спрайт. И выводить в CSS , бекграунд спрайтом

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

плохо ссылки поисковики ходят, вес страницы уменьшатся
спрайт в вашем случае уменьшит количество запросов, оптимизирует скорость загрузки, а значит модуль будет работать быстрее
http://habrahabr.ru/post/159027/
и не думайте, что это все после проделанной работы опять глянем, что усовершенствовать, упростить нужно 

Тоже очень простой

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

не поверите, даже ещё не начинал думать, если ввод спрайтов не упрощение, тогда, что есть упрощение...

минимализм, понятливость кода, оптимизация при полной функциональности

идея понята, а вот в CSS прописать все картинки из одного файла, ещё не понял, буду пробовать.
но минимализм меня доконает (

В принципе код уже 50% готов
Было много возни кроме меня, над этим модулем работало пять или шесть человек, которые помогали его править и хотел их поблагодарить
Fedor Vlasenko, Shustry и с другого форума, SelenIT, red4pony, klierik, Daiver, hypnocolor.

Исходники обнови. А мы тебе опять напишем рекомендации. Совершенству нет предела 

Исходники обнови. А мы тебе опять напишем рекомендации. Совершенству нет предела 

Пока не всё готово но на днях сделаю точно, надо скрипт подчистить и файлы лишние стереть. Попробую на этой неделе всё сделать ну или на следующей.

Архив обновил будьте внимательны, сверяйтесь с архивом некоторых файлов уже нет 
mod_amoney_j15-J16-J17v104.zip - для Joomla 1.5 версия 1.0.4
mod_amoney_j25-J30-J31v104.zip - для Joomla 3.1 версия 1.0.4

Недавно мой сайт был заражен. Касперский отловил пару троянов и еще кучу зараженных файлов JS и PHP вычислил Айболитом.

jHackGuard (плагин для защиты сайта от взлома(защита от взлома сайта, SQL инъекций, XSS атак))

leo78
Спасибо за рекомендацию! Я поставил RSFirewall! Вот жду, когда что-нибудь серьезное случится. Пока только пароли пытаются подбирать...

Здравствуйте.

А как переделать модуль, чтоб при вставке в материал была только кнопка "оплатить"? Просто использоваться будет только вебмоней, только 1 кошелек, и сума (ето все можно указать в настройках данного модуля в админке).

Благодарен.

watar
Поля удалять нельзя, скрипт работать не будет, можно скрыть их для пользователей.
В админке отключаем ненужные системы и кошельки.
Далее в Теге form надо  скрытое поле тега input, с помощью hidden
Находим
Правим

А как бы добавить вместо Сбербанка - Газэнергобанк?