Problem/Motivation
The module has issues with coding standards.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | interdiff_2-4.txt | 4.43 KB | tmaiochi |
| #5 | 3259815-4.patch | 7.83 KB | tmaiochi |
| #2 | editablefields-fix_codding_standards-3259815-2.patch | 3.82 KB | yurii o. |
Issue fork editablefields-3259815
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
yurii o. commentedpls check it
Comment #3
tmaiochi commentedI'll review this.
Comment #4
gilmord@Yurii O. patches is the old way to resolve the issues
please follow a new forks & MRs guide and create a merge request
thanks!
Comment #5
tmaiochi commentedThe patch had some phpcs problems. I fixed some of them, but it has a warning about unused variable, however the variable is in using. So I left it at that and didn't change anything about this warning. I also sending the interdiff.
Comment #6
gilmord@tmaiochi please use the forks & MRs flow mentioned a few comments above, so I can merge it.
Comment #7
victoria-marina commentedI'll review it.
Comment #8
victoria-marina commentedThe #5 worked for me! RTBC
Comment #11
gilmordThanks all