Pull to refresh
7
0
Дионисий @Dionisiy

User

Send message
Тогда уже не «Null Оператор ??», а «Null coalescing оператор ??».
Правильно не Joomla®, а Joomla!®

В плагине для Joomla! 1.6-2.5 есть недоработки:
1. Cобытия onContentAfterSave и onContentChangeState должны проверять $context, эти события могут вызывается и не для статьи. Для статьи $context = 'com_content.article'.

2. Зачем в методе ping() это?
$plugin = &JPluginHelper::getPlugin('content', 'pinger');


3. Обновление параметров через «WHERE element=%s» некорректно, лучше через «WHERE element=%s AND folder='content' ». Могут быть другие плагины с element = 'pinger' и тупо перепишутся все параметры.

4. Методы get_date() и ping() должны быть protected, иначе они регистрируются как события.

5. Зачем каждый раз импортить jimport('joomla.version');, если JVersion используется только при определенных условиях в методе ping()? Надо импортировать в самом ping().

4. Зачем в конце кода ?>?
Все переделал, встречаем бубновые тултипы!
Спасибо, добавил. Кроме того добавил -webkit-gradient под Android.
Ха, а в Андроиде просто нет символов ► и ◄ (треугольных стрелок влево и вправо), задача усложняется.
Вы не придираетесь, а совершенно правы! Переименовал статью.
Спасибо за перечисление недостатков технологии, они известны, поэтому я нигде и не отмечал, что этот метод отличается универсальностью и гибкостью. Целью было показать возможность использования.

Для определенных задач этот метод более чем достаточен, и я выбираю его.

Information

Rating
Does not participate
Location
Беларусь
Date of birth
Registered
Activity