+- MyBB Community Forums (https://community.mybb.com)
+-- Forum: Development (https://community.mybb.com/forum-161.html)
+--- Forum: MyBB 1.8 Development (https://community.mybb.com/forum-165.html)
+---- Forum: 1.8 Bugs and Issues (https://community.mybb.com/forum-157.html)
+----- Forum: Pushed (https://community.mybb.com/forum-183.html)
+----- Thread: [Pushed] Wrong line endings in php files (Development Standard) (/thread-173279.html)
Wrong line endings in php files (Development Standard) - Evenprime - 2015-07-18
I noticed that some of the files of the project don't follow the http://docs.mybb.com/1.8/development/standards/
MyBB Development Standards Wrote:New Lines
All new lines should be denoted by the UNIX line feed (\n, also known as "LF") and not include the Windows carriage return (\r, also known as "CR"). CR and CRLF are not permitted; LF is the only permitted line ending.
The full list of the 1.8.5 release with wrong line endings (crlf instead of lf only):
admin\styles\default\style.php
inc\3rdparty\2fa\GoogleAuthenticator.php
inc\db_sqlite.php
install\resources\upgrade25.php
member.php
usercp2.php
It would be nice to see these, maybe with 1.8.6, corrected to follow the UNIX standard of lf only like the rest of the files. I know that such a change isn't easy to do (especially for the last two files), because it may result in difficulties for merges from other developers/forks in some cases.
RE: Wrong line endings in php files (Development Standard) - Euan T - 2015-07-18
Hi,
Thank you for your report. We have pushed this issue to our Github repository for further analysis where you can track our commits and progress with fixing this bug. Discussions regarding this bug may also take place there too.
Follow this link to visit the issue on Github: https://github.com/mybb/mybb/issues/2157
Thanks for contributing to MyBB!
Regards,
The MyBB Group
RE: Wrong line endings in php files (Development Standard) - Euan T - 2015-07-18
Note that we should also probably add a .gitattributes file if one doesn't already exist to force Unix line-endings. It does bug me having different mixes of line endings too.