e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
322 stars 214 forks source link

Q - weird behaviour with e_admin and word "page" in other plugin name #3696

Open Jimmi08 opened 5 years ago

Jimmi08 commented 5 years ago

I have the plugin that works with those types:

    switch ($type)
        {
        case "forum_thread" :    
        case "news_category" :
        case "download_category" :
    case "page" :
    case "news" :
    case "pcontent" :
    case "download" :

Now I noticed that field is displayed on plugin where it shouldn' be displayed. But this plugin has "page" in URL.

    protected $pluginName       = 'yandex_turbopages';
    protected $eventName        = "yandex_turbopages";

comment protected $eventName = "yandex_turbopages"; didn't help

but remove case "page" helped.

I thought it depends on the direct event name.

LaocheXe commented 5 years ago

I don't know if this is related or not - but a plugin that I am slowly working on had e_admin.php generated from plugin builder - after latest github update on 3-6-2019 I notice an issue in Admin Area -> News > List which is #3715