Opened 14 years ago

Closed 13 years ago

#119 closed task (wontfix)

PHPCodeSniffer report is different on Linux and Windows

Reported by: adube Owned by:
Priority: trivial Milestone: 1.2.0
Component: packaging/scripts/config Version: trunk
Keywords: phpcodesniffer Cc:

Description

When running PHPCodeSniffer on the core source, we get different results on Windows and Linux. Here are some test results made on r840 :

  • Linux 10.04 and 8.04, with either the system version (1.2.2) or the one included in GeoPrisma lib (1.2.0a1) by using the 'test' button on the website (locally) :
    • 6 errors
    • 2 warnings
  • on www.geoprisma.org :
    • 234 errors
    • 2 warnings

The only error that is repeated 228 times is the following : There must be a single space between the closing parenthesis and the openning brace of a multi-line IF statement, found 3 spaces on Sniff Borealis.ControlStructures.MultiLineConditionSniff

On Linux, there IS actually only 2 spaces, but on Windows ( tested with version 7, with Notepad++ ) there is an extra space.

Change History (3)

comment:1 by adube, 13 years ago

Milestone: Future

comment:2 by adube, 13 years ago

Milestone: Future1.2.0
Version: trunk

Shouldn't matter anymore with #356.

comment:3 by adube, 13 years ago

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.