View Issue Details

IDProjectCategoryView StatusLast Update
0004834OXID eShop (all versions)2.3. Extensions (modules, themes)public2013-02-21 13:21
Reporterrimvydas_paskevicius 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.7.2 / 5.0.2 revision 53018 
Target VersionFixed in Version4.7.4 / 5.0.4 revision 57063 
Summary0004834: On module activation/deactivation for EE version in some cases user is logged out from admin
DescriptionWhen activating/deactivating module, in some cases user is forced to logout from admin. The reason is in admin/module_main.tpl file:

[{if $updatenav }]
    [{oxscript add="top.oxid.admin.reloadNavigation('`$oxid`');" priority=10}]
[{/if}]

This if statement should be removed, as it is only valid for saving/updating subshops settings for version EE.

And in controllers/admin/module_main.php

            if ( $oModule->deactivate() ) {
                $this->_aViewData["updatenav"] = "1";
            }

There is no need to set this parameter also. Should be only:

$oModule->deactivate()
TagsNo tags attached.
ThemeBoth
BrowserAll
PHP Versionany
MySQL Versionany

Relationships

related to 0004218 resolvedandrius.silgalis navigation/menu not refreshed after new module was activated 

Activities

FibreFoX

2013-01-09 17:17

reporter   ~0008243

In the case that the module has some menu-items the admin-interface HAS to be refreshed! But i agree that the user should still be logged in :)

mark

2013-02-13 15:37

reporter   ~0008409

Really annoying - launching oxid 5 in three weeks, please change for 5.0.4!!
Spent 1,5h before looking here. Please increase from minor

[{if $updatenav }]
    [{assign var="oConfig" value=$oView->getConfig()}]
    [{assign var="shp" value=$oConfig->getShopId()}]
    [{oxscript add="top.oxid.admin.reloadNavigation('`$shp`');" priority=10}]
[{/if}]