[#36015] - ComponentAdapter Manifest Namespace-Tag not stored
- New
- Medium
- Build: 4.0-dev
- # 36015
only collecting some informations
The "public function refreshManifestCache()" is a database store only method
The following line can never be true
because of from :
joomla-cms/libraries/src/Installer/Installer.php
Line 2288 in a94decc
joomla-cms/libraries/src/Installer/Installer.php
Line 2319 in a94decc
Question is:
write namespace to database ? ( -1 )
then the $data['namespace'] must be set with path="" if exist
or
remove the
checking because it is never true
joomla-cms-bot
- | Labels |
Added:
No Code Attached Yet
|
||
Hackwar
- | Labels |
Added:
bug
|
||
@joomdonation is your comment still valid since #39011 (ignoring the comment that its for child templates)
@brianteeman Yes, it is still valid. These lines of code allow storing namespace of component to namespace column in #__extensions table. However, we dropped namespace column from #__extensions table few years ago, so that lines of code is useless. We could also remove these lines of code https://github.com/joomla/joomla-cms/blob/5.4-dev/libraries/src/Installer/Adapter/ComponentAdapter.php#L892-L893 , too.
@JProof We do not store namespace of component in database, so I would say removing the if block
joomla-cms/libraries/src/Installer/Adapter/ComponentAdapter.php
Lines 1479 to 1482 in a94decc