Check all possible occurences of classes to display warnings
This will not be any help before or during migration, but afterwards. It will just add nonfixable warnings and check in a more stupid way like all strings.
We have to check how this will work, but as we might have some circumstances we can't fix ...
No child items are currently assigned. Use child items to break down this issue into smaller parts.
Link issues together to show that they're related. Learn more.
When this merge request is accepted, this issue will be closed automatically.
Activity
- Author Owner
Also we should provide them as fixable, if we find a mapping as usual.
Examples are
$defaultViewObjectName
properties in classes. They reference, at least in the example project, legacy class names as strings.Also they are often part of a larger String, e.g. in hook configuration and stuff like that. See
$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['tslib/class.tslib_fe.php']['configArrayPostProc']
for an example in our current project. Or$TYPO3_CONF_VARS['SC_OPTIONS']['extbase']['commandControllers']
.We have to be carefull, as not always replacing is the right option, e.g. hooks contain old class names:
tslib/class.tslib_fe.php
.Also see #45 (closed)
- Daniel Siepmann added status/Backlog label
added status/Backlog label
- Daniel Siepmann removed milestone
removed milestone
- Daniel Siepmann changed milestone to %0.1.0 alpha
changed milestone to %0.1.0 alpha
- Daniel Siepmann added component/PHP label
added component/PHP label
- Daniel Siepmann added priority/High label
added priority/High label
- Daniel Siepmann added status/SelectedForDevelopment and removed status/Backlog labels
added status/SelectedForDevelopment and removed status/Backlog labels
- Daniel Siepmann added status/Working and removed status/SelectedForDevelopment labels
added status/Working and removed status/SelectedForDevelopment labels
- Daniel Siepmann assigned to @DanielSiepmann
assigned to @DanielSiepmann
- Daniel Siepmann mentioned in commit afd6f781
mentioned in commit afd6f781
- Daniel Siepmann mentioned in merge request !84 (merged)
mentioned in merge request !84 (merged)
- Daniel Siepmann added status/MergeRequest and removed status/Working labels
added status/MergeRequest and removed status/Working labels
- Author Owner
As most occurrences are already handled, see #72 (closed), we added Strings as possible occurrence. This should cover all configuration directives. We split white spaces and some special characters that are known, like
&
.As classes in strings are not always classes, but sometimes table names, they are only added as warning, not fixable.
- Daniel Siepmann mentioned in commit f3448b08
mentioned in commit f3448b08
- Daniel Siepmann added status/Working and removed status/MergeRequest labels
added status/Working and removed status/MergeRequest labels
- Daniel Siepmann mentioned in commit 7a16dac5
mentioned in commit 7a16dac5
- Daniel Siepmann added status/MergeRequest and removed status/Working labels
added status/MergeRequest and removed status/Working labels
- Daniel Hürtgen closed via merge request !84 (merged)
closed via merge request !84 (merged)
- Daniel Siepmann closed via commit afd6f781
closed via commit afd6f781
- Daniel Siepmann closed via commit 7a16dac5
closed via commit 7a16dac5
- Daniel Hürtgen closed via commit 0c91d7b2
closed via commit 0c91d7b2
- Daniel Hürtgen mentioned in commit 0c91d7b2
mentioned in commit 0c91d7b2