? Pending

User tests: Successful: Unsuccessful:

avatar joeforjoomla
joeforjoomla
23 Oct 2018

Pull Request for Issue #22768 .

Summary of Changes

dirname(..) is not referring to the correct file/path anymore, that is causing the loading of modules language files failing.

avatar joeforjoomla joeforjoomla - open - 23 Oct 2018
avatar joeforjoomla joeforjoomla - change - 23 Oct 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Oct 2018
Category Libraries
avatar infograf768
infograf768 - comment - 24 Oct 2018

Why do we use dirname ?

If I use
$extensionLanguageDirectory = JPATH_BASE . '/modules/' . $this->module->module;

it works fine.
@laoneo ?

avatar laoneo
laoneo - comment - 24 Oct 2018

I think too that the extra call of dirname can be removed.

avatar infograf768
infograf768 - comment - 24 Oct 2018

@joeforjoomla
can you modify your PR ?

avatar joeforjoomla joeforjoomla - change - 24 Oct 2018
Labels Added: ?
avatar joeforjoomla
joeforjoomla - comment - 24 Oct 2018

I modified the PR, basically removing the extra dirname works the same

avatar infograf768
infograf768 - comment - 24 Oct 2018

I have tested this item successfully on a6f4746

Can now be merged on review.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22786.

avatar infograf768 infograf768 - test_item - 24 Oct 2018 - Tested successfully
avatar laoneo
laoneo - comment - 24 Oct 2018

Thanks!

avatar laoneo laoneo - change - 24 Oct 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-10-24 10:05:23
Closed_By laoneo
avatar laoneo laoneo - close - 24 Oct 2018
avatar laoneo laoneo - merge - 24 Oct 2018

Add a Comment

Login with GitHub to post a comment