Flag whitespace within methods/functions

[dextermb]

Hi all

We're interested in seeing if two rules exist:

1. Flag whitespace within methods/functions
2. Flag if no whitespace within functions/methods

Here's an example:

protected function setup(): void { parent::setup(); /** @var MockInterface */ $this->mock = \Mockery::mock(BookingRepository::class); $this->app->instance(BookingRepository::class, $this->mock); $this->bookingA = new Booking([ 'StartDateTime' => '2021-07-19 09:00:00.000', 'EndDateTime' => '2021-07-19 13:00:00.000', 'PractitionerId' => '2209', 'SessionId' => '344' ]); $this->bookingB = new Booking([ 'StartDateTime' => '2021-07-22 09:00:00.000', 'EndDateTime' => '2021-07-22 09:45:00.000', 'PractitionerId' => null, 'SessionId' => '121' ]); }

This would get flagged for rule 2.

Whereas...

protected function setup(): void { parent::setup(); /** @var MockInterface */ $this->mock = \Mockery::mock(BookingRepository::class); $this->app->instance(BookingRepository::class, $this->mock); $this->bookingA = new Booking([ 'StartDateTime' => '2021-07-19 09:00:00.000', 'EndDateTime' => '2021-07-19 13:00:00.000', 'PractitionerId' => '2209', 'SessionId' => '344' ]); $this->bookingB = new Booking([ 'StartDateTime' => '2021-07-22 09:00:00.000', 'EndDateTime' => '2021-07-22 09:45:00.000', 'PractitionerId' => null, 'SessionId' => '121' ]); }

This would get flagged for rule 1.

Any ideas what this would look like in terms of rulesets?

Thanks in advance.

[gsherwood]

There are no included sniffs that do either of those things. Whitespace is generally excluded from standards (like in the case of the PSRs) and most others I've seen say something to the effect of "whitespace can be used to increase readability", so such as sniff as never been requested. I'd be very unlikely to include it in the core release and I can't see a broad requirement for it.