#102 Add alternative for php pathinfo function is discouraged #123
Conversation
There was a problem hiding this comment.
Could you add also all functions from https://github.com/magento/magento2/blob/2.3/lib/internal/Magento/Framework/Filesystem/Io/File.php?
For instance mkdir, rmdir, unlink, etc.
| @ihor-sviziev all requested changes are now done. |
| @buskamuza could you please have a look at these recommendations? It looks like we need to suggest something different from |
There was a problem hiding this comment.
I'd suggest referencing \Magento\Framework\Filesystem\DriverInterface interface instead of concrete driver. But I don't insist.
Io should only be used as a last resort. Looks like there is only one method that exists in Io and doesn't exist in Filesystem: pathinfo.
| Thanks, @lenaorobei . The proposal is not ready yet, but at least we should avoid recommending |
| Thanks @buskamuza @lenaorobei I'll update this PR today |
| @buskamuza requested changes are done, please have a look when you get a chance |
| '^chdir$' => null, | ||
| '^chgrp$' => null, | ||
| '^chmod$' => null, | ||
| '^chmod$' => '\Magento\Framework\Filesystem\DriverInterface::changePermissions() or \Magento\Framework\Filesystem\DriverInterface::changePermissionsRecursively()', |
There was a problem hiding this comment.
Need to fix Line exceeds maximum limit of 120 characters; contains 171 characters here.
m2-community-project bot added Progress: review and removed Progress: changes requested labels 
…magento-coding-standard-334 [Imported] Fix unit tests
5 participants
#102