Provide Drupal root path on file system admin form

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

This could use an issue summary update for the proposed solution. Seems there is still discussion going on so when the path forward is decided it should be added to the IS. Will help the committer.

Rerolled #21 in a MR. Added descriptions for items regarding absolute values if applicable.

Thanks!

Code wise I think it looks fine. Not sure if we will need a trigger_error for the new parameter or not.

This could use a simple change record though to announce the new parameter option.

Added CR https://www.drupal.org/node/3341696 →
Updated constructor to handle new parameter addition.

Thanks think this good for committer review now.

This is changing the user interface, adding tag. As such this should have before and after screenshot available from the issue summary. This is also adding text to the public and private file descriptions, so I think this should have a usability review.

I read the change record, well several times, and I am still not sure what 'related' means in "public, private, and temporary paths are related". Tagging for a change record update. This is not a complicated change but a screenshot could be useful in the change record.

I skimmed the patch and made some comments. I also note this was set to RTBC with an unresolved comment.

Changed the relative paths with absolute as this will show the actual location of specific stream wrappers.
Added check for path accessibility. The realpath() file_system method fails absolute path resolving if the directory not exists.
I have added screenshots before → , after → , and after with warning messages → .

Brought this up in #ux channel and it was briefly looked at but was mentioned

taken a look but it would be helpful if the issue summary would have been updated illustrating the changes that will be made. at the moment the ui changes section only states The content of the page File system will be updated. . even if i apply a patch i consider it helpful to get an overview which changes were made.

I agree that IS needs to be updated, but please check the screenshots from #35.
The latest changes in MR convert paths to absolute if they are relative. And if the destination does not exist, the path value is printed as is (relative or absolute) with an appropriate warning message that reuses existing sentences and translations.
In case such behavior is acceptable I'll update IS as I can.

rerolled
assets filesystem wrapper added since then
if #35 comparison approach is good to go with, I'm ready to create updated screenshots and attach them to the IS.

Rebased the MR and added screenshots and user interface changes to the issue summary. I'm not sure the change record is really necessary.

The one minor concern I have is that it's now unclear if you were to move your Drupal installation to another location, would the file system paths remain in their original locations? That would depend on what's in your settings.php file, and you can no longer determine the answer from the config page.