Создание плагина для Joomla. Нужен совет - Модификации расширений для Joomla

Решил сделать доступ к загрузкам сайта только, когда чего-нибудь пожертвуют (вымогательство) через PayPal.
Не стал разбираться с их API, использовал простую кнопку.
При удачной оплате PayPal отсылает на одну и ту же страницу:
Спасибо за пожертвование, благодаря таким людям, как вы, сайт живет и развивается.
Помимо этой фигни, там стоит заглушка {donate}.

Сделал content plugin. Может есть лучший вариант? PHP только учусь, не судите строго.
Плагин ищет текст {donate} в материале, если есть пишет в базу jdownloads_donate пользователя, IP, дату, проверяя на уникальность. Пишет в cookie был здесь.


Код:
<?php

defined('_JEXEC') or die;
class plgContentDonate extends JPlugin
{
   public function __construct(& $subject, $config)
   {
      parent::__construct($subject, $config);
      $this->loadLanguage();
   }

   public function onContentBeforeDisplay($context, &$row, &$params, $page)
   {
   if(strstr($row->text, '{donate}')){    //Чтоб не выполнялось во всех материалах, а там где {donate} Может есть другой вариант?
   $user = &JFactory::getUser();
   if($user->guest){
      $username = '';
      $usernamesql = '';
      }
      else{
      $username = $user->username;
      $usernamesql = ' OR username = "'.$username.'"';
      }   
   $ip=$_SERVER['REMOTE_ADDR'];   
   $database = &JFactory::getDBO();
   $database->setQuery('SELECT username, IP FROM #__jdownloads_donate WHERE IP = "'.$ip.'"'.$usernamesql);
   $washere = $database->loadObjectList();   
   
   if(!$washere){
   $database->setQuery('INSERT INTO #__jdownloads_donate (id, username, IP, date)
                                                 VALUES (NULL, "'.$username.'", "'.$ip.'", "'.date("Y-m-d H:i:s").'")');
   $database->query();
   }
   $cookie = setcookie("donate", "1", 0x7FFFFFFF);
   
        $html=''; // Выводить вместо {donate} нечего, а заглушку {donate} убрать надо.
   $row->text  = str_replace('{donate}', $html, $row->text);
   }   
      return ;
   }
       
}


Вроде все работает. Но может есть лучшее решение?

Во первых перестаньте юзать & перед JFactory::getDBO(); и JFactory::getUser();он там не нужен!

а про $context проверить на принадлежность к тому, что обрабатывать собираемся

Istaan, мелочи, но исправил.
Fedor Vlasenko, Лишнее. Усложнять не охота и так работает.
Про логику плагина спросить хотел. Правильно ли это:  if(strstr($row->text, '{donate}')){
или у Joomla есть готовый вариант? (Штоб не делалось во всех материалах, а там где только {donate}

Про логику плагина спросить хотел. Правильно ли это:  if(strstr($row->text, '{donate}')){
или у Joomla есть готовый вариант? (Штоб не делалось во всех материалах, а там где только {donate}

Ну можно так, можно сделать что бы в параметрах появлялось новое поле (типа так:
и уже с этих полей устанавливать для доната эта страница или нет!

ну и ещё это:
можно сократить до:

и это:
до:

и зачем вы выносите $user->username в переменную $username? просто используйте $user->username и в запросах! если юзер будет гость там и будет пустота!

Fedor Vlasenko, Лишнее. Усложнять не охота и так работает

onContentBeforeDisplay обрабатывает не только com_content, но и все остальные
Потому чтобы не обрабатывать все остальное. Есть готовый пример плагин группы контент pagebreak. Начните с него.
/plugins/content/pagebreak

Istaan, спасибо за оптимизацию кода. Обычно делаю, стандартно, как у всех.
Fedor Vlasenko, $context нужна для проверки, чтоб исполнялось в конкретном компоненте (com_content) и нигде больше. Я правильно понял?

P.S. И парни вы молодцы! На таких людях, как Вы, Земля держится. Спасибо Вам за помощь!

Fedor Vlasenko, $context нужна для проверки, чтоб исполнялось в конкретном компоненте (com_content) и нигде больше. Я правильно понял?

увы нет, $context передается обычно из вьюхи.. и это банально строка $dispatcher->trigger('onContentPrepare', array ('com_content.article', &$item, &$this->params, $offset));

тоесть я могу в своем компоненте тоже вызывать событие onContentPrepare и передать $context=com_content.article и никто меня не остановит )))

Лучше проверять компонент, вид ну и слой...

увы нет, $context передается обычно из вьюхи.. и это банально строка $dispatcher->trigger('onContentPrepare', array ('com_content.article', &$item, &$this->params, $offset));

тоесть я могу в своем компоненте тоже вызывать событие onContentPrepare и передать $context=com_content.article и никто меня не остановит )))

Лучше проверять компонент, вид ну и слой...

Код: php

$input = JFactory::getApplication()->input;
if ( $input->getCmd( 'option', '' )!== 'com_content' && $input->getCmd( 'view', '' )!== 'article' && $nput->getCmd( 'layout', '' )!== 'default' ) {return;}

Да ну... Никто такой ерундой не занимается. Достаточно контекст com_content.article проверить.

Да ну... Никто такой ерундой не занимается. Достаточно контекст com_content.article проверить.

Дима если мне нужно только что бы в материале выводилось, как определить что это материал?
ибо даже модули передают com_content.article! Joomla 3 /modules/mod_articles_news/helper.php строка 96: $results = $app->triggerEvent('onContentAfterDisplay', array('com_content.article', &$item, &$params, 1));

Ну тогда достаточно $input->getCmd('view', '')!== 'article'

А зачем так всё усложнять? $context так критично или уязвимо, чтоб его обозначать где делать? Делал по примеру /plugins/content/vote/vote.php нет там ничего такого.

Ну тогда достаточно $input->getCmd('view', '')!== 'article'

а если какая то редиско в своем компоненте создаст вид article и заюзает контент плагины?)

а если какая то редиско в своем компоненте создаст вид article и заюзает контент плагины?)

и  никак не тянет этот способ использовать
http://joomlaforum.ru/index.php/topic,279892.msg
или привык уже к апи
cartgen мы не усложняем, наоборот хотим, чтобы плагин не работал там где ему не надо

и  никак не тянет этот способ использовать

Федь ну твой способ несомненно гуд ) просто пока дело не в этом )

Всем привет и спасибо за советы.

if($context != 'com_content.article') return; принял, как аксиому.

От монетизации подобным образом отказался. С 3000 пользователей за два дня: 10$ (все ищут халяву, а может надо подключать больше платежных систем). CTR adsense взлетел до 6%. Такими темпами лишат рекламы, что крайне не желательно. На рекламу больше надежды.

Зато приобрел опыт в создании первого плагина.
Еще раз спасибо и удачи вам!