? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
7 Jan 2018

Pull Request for Issue # .

Summary of Changes

This PR did two things:

  1. Store namespace for module when modules are installed/updated

  2. Update xml files to have correct namespace tag for namespaced modules

This PR is created so that @wilsonge can finish his PR #18703

Testing Instructions

Can be merged by code review

avatar joomdonation joomdonation - open - 7 Jan 2018
avatar joomdonation joomdonation - change - 7 Jan 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jan 2018
Category Modules Administration Libraries Front End
avatar brianteeman
brianteeman - comment - 12 Jan 2018

i am a bit confused as the changes in the adapter are only for installed modules and none of these modules are installed that way

avatar joomdonation
joomdonation - comment - 12 Jan 2018

As I mentioned, this PR was created so that @wilsonge can finish his PR #18703. That PR creates namespace maps for modules and components but our current code doesn't store namespace of modules during install it (we store it for components already). Hope it answers your question

avatar brianteeman
brianteeman - comment - 12 Jan 2018

our current code doesn't store namespace of modules during install

The modules you have updated are not "intalled" by the installer so i dont see how updating the xml of these modules will have any effect

avatar joomdonation
joomdonation - comment - 12 Jan 2018

When someone wants to develop a new module which supports namespace, he can looks at the XML file of one of core modules to know how to set namespace for the module (so that the module classes will be uploadable). We do that for our core components, too

avatar brianteeman
brianteeman - comment - 22 Jan 2018

I have tested this item successfully on 606a6f3


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

avatar brianteeman brianteeman - test_item - 22 Jan 2018 - Tested successfully
avatar wilsonge wilsonge - change - 22 Jan 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-01-22 21:44:08
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 22 Jan 2018
avatar wilsonge wilsonge - merge - 22 Jan 2018

Add a Comment

Login with GitHub to post a comment