Autoloading for classes

[liayn]

Autoloading for classes

[liayn]

Forget my point b) again. Overlooked the definition.
The vendor name should still be changed though.

But why do you need this?
require_once(ExtensionManagementUtility::extPath('generic_gallery').'Classes/Domain/Model/Dto/EmConfiguration.php');

[liayn]

Another question: why do you exclude PHP 7.1?

[fnagel]

Hey there!

use TYPO3 as vendor in the namespace

Afaik only TYPO3\CMS is reserved but I know the current NS is not optimal. I plan to move ALL my extensions to a custom vendor NS but still undecided which one ;-)

But why do you need this?
require_once(ExtensionManagementUtility::extPath('generic_gallery').'Classes/Domain/Model/Dto/EmConfiguration.php');

Changed this lately (see 9abd1ff) as GeneralUtility::requireOnce has been deprecated. To be honest, I took the idea from this straight from EXT:news and it seems it's changed there too.

why do you exclude PHP 7.1?

Only because it's untested and I don't like to say its ready if I'm not sure about it. Are you able to confirm everything works fine?

[liayn]

Vendor= FNAGEL?

Changed this lately (see 9abd1ff) as GeneralUtility::requireOnce has been deprecated. To be honest, I took the idea from this straight from EXT:news and it seems it's changed there too.

Why do you need to require the file at all? We have autoloading in place, so it will find the file anyways.

I didn't test with PHP7.1 but there is really low chance that something really breaks with it, since there are no real breaking changes.

[fnagel]

Vendor= FNAGEL?

or FNA (which might be taken already) or FelixNagel or FelixNagelCom (upper camel case is better, right?). Anyway, I need some time to change this for all my extensions...

Why do you need to require the file at all?

Good question, actually. Doesn't seem to make sense. As said I took this straight from EXT:news: https://github.com/georgringer/news/blob/master/Classes/Utility/EmConfiguration.php#L29

Anyway, seems worth trying to remove it.

I didn't test with PHP7.1 but there is really low chance that something really breaks with it, since there are no real breaking changes.

I don't have any experience with 7.1 yet, so I can't speak from experience but I guess you're right.

[liayn]

FNAGEL or FNA

It simply does not matter. And I doubt that even if it is taken that the extensions will have the same name too. So that's unique enough.

require once in ext:news

I would say this is legacy and a leftover. This thing is in there since at least 2015. We can ask Georg for details.

PHP 7.1

We run live sites on 7.1 without any issues.
(Except if you use ext:yag, which really is not 7.1 compatible)

[fnagel]

Closed with b177dcd