Bug #21769
closedHardcoded BE path in typo3/sysext/install/updates/class.tx_coreupdates_installnewsysexts.php
0%
Description
In typo3/sysext/install/updates/class.tx_coreupdates_installnewsysexts.php there are two places where the BE path is hardcoded.
(issue imported from #M12881)
Files
Updated by Xavier Perseguers almost 13 years ago
- Assignee deleted (
Steffen Kamper) - Target version deleted (
0)
Updated by Alexander Opitz over 11 years ago
- Status changed from Accepted to Needs Feedback
The issue is very old, does this issue exists in newer versions of TYPO3 CMS (4.5 or 6.1)?
Updated by Riccardo De Contardi over 11 years ago
as far as I can see:
The lines reported in the attached .diff file are still present in TYPO3 4.5.20 and 4.7.7;
in TYPO3 6.1:
- the file typo3/sysext/install/updates/class.tx_coreupdates_installnewsysexts.php does not exist
- I've found this line in /typo3/sysext/install/Classes/CoreUpdates/InstallSysExtsUpdate.php:
/** * @var string */ protected $extEmConfPath = 'typo3/sysext/@extensionKey/ext_emconf.php';
Is it related?
BTW in the same file there are some hardcoded labels in english ;)
Updated by Alexander Opitz over 11 years ago
- Status changed from Needs Feedback to New
- Assignee set to Alexander Opitz
Updated by Christian Kuhn over 11 years ago
- Status changed from New to Rejected
Mhh ... typo3/ path segment is hardcoded at lots of places. I also see no real point in changing that and using the constant instead. With the path put as default in the protected property in 6.2, it is also not possible to use the constant there (php does not allow this). Only solution would be to add a __construct compiling the path, which actually only makes the code more complex and harder to understand.
I'd vote for staying with this "hardcoded" path segment at that point ...
I'll close the issue with won't fix. Please re-open if this is really important for you for whatever reason.