You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We used to have an installer that could be use to set up your environment file via a browser form.
We got rid of that some time ago, but there's bits of logic left in Content Controller to handle that. At first glance, these bits look mildly scary because they are printing out credentials, pinging back some silverstripe domain and deleting files. Glancing at the fine print of the code, I don't think there's any risk. But it's certainly not code we should be keeping around.
I don't think there's anything useful that can be done with this code anyway, so we can probably removed it in CMS 5.
NOTE Don't merge these until the CMS 5 PRs have been merged in and up - this is to ensure we don't get deprecation notices popping up in weird places due to bad mergeups.
We used to have an installer that could be use to set up your environment file via a browser form.
We got rid of that some time ago, but there's bits of logic left in Content Controller to handle that. At first glance, these bits look mildly scary because they are printing out credentials, pinging back some silverstripe domain and deleting files. Glancing at the fine print of the code, I don't think there's any risk. But it's certainly not code we should be keeping around.
I don't think there's anything useful that can be done with this code anyway, so we can probably removed it in CMS 5.
silverstripe-cms/code/Controllers/ContentController.php
Lines 442 to 515 in 56646af
Acceptance criteria
successfullyinstalled
anddeleteinstallfiles
are removed fromContentController
in CMS6 and deprecated in CMS 5.CMS 5 PRs
CMS 6 PRs
NOTE Don't merge these until the CMS 5 PRs have been merged in and up - this is to ensure we don't get deprecation notices popping up in weird places due to bad mergeups.
The text was updated successfully, but these errors were encountered: