Thanks for proposing this new feature but my approach would be different.

To achieve this I would better implement something like this :

• in getDataSecondStepAction(), check If $GLOBALS['TYPO3_CONF_VARS']['BE']['groupHomePath'] is defined and well configured and if so, add a variable to the view groupHomePath = true
• in the GetDataSecondStep.html view if groupHomePath = true then add a checkbox with the property property="groupHomePath" and of course the setter and getter in SiteGeneratorDto
• if this checkbox is checked then the form field 'Base folder name' must be disabled with some JS
• then in Classes/Wizard/StateCreateFolder.php, you can set the right storageUid and baseFolderName according to the flag in DTO
• could be interesting to have a checkbox in extension configuration telling to the view if the groupHomePath should be checked by default or not

@Oktopuce I'll try to implement that.

@Oktopuce Just pushed some changes. Have a look at it. I didn't implement your last suggestion yet though.

As I needed this feature and could not wait, I decided to include it in our extension extending Site Generator.
Of course there will be no option to choose between the to methods, but I don't think of it as an issue. At least not for us.
So it is up to you if we should abandon this feature from Site Generator.

Sorry but there were some development in your PR that I would like to be different but haven't got the time to review them.
And yes this is not an issue but a feature.