Clean-up documentation for get_filesystem_method()

[DrewAPicture]

Hi, thanks for the patch. Some feedback:

• The list in the summary should use "or" instead of "and", e.g. "reading, writing, modifying, or deleting"
• The "valid values for these ..." sentence should be moved up follow the sentence describing the various transports, otherwise, "these" doesn't making any sense in that context.
• All three parameters are optionsl so they should be marked as such in the @param descriptions, along with listing accepted values and defaults for each. With that done, we can remove the @todo line in the DocBlock.

[DrewAPicture]

See comment:1 for feedback on 31267.diff​.

[HarishChaudhari]

Updated PHP documentation as suggested by @DrewAPicture

[HarishChaudhari]

Updated PHP documentation as suggested by @DrewAPicture, removed unnecessary space

[HarishChaudhari]

I have updated PHP documentation as suggested by @DrewAPicture. Made few tweaks using earlier patch, added link for transports as well.

Let me know if this looks good or any suggestions.

Please consider 31267.3.patch

[valendesigns]

Patch from root

[valendesigns]

The previous patch did not apply from the root directory. I've refreshed the patch and fixed some spacing issues in the @param section. Otherwise, everything looks pretty good. I leave it to you @DrewAPicture.

[DrewAPicture]

In 31884:

Clean up DocBlock information and syntax for get_filesystem_method().

Props ericlewis, HarishChaudhari, valendesigns.
Fixes #31267.