PSR2.Namespaces.NamespaceDeclaration does not handle namespaces defined over multiple lines

[niconoe-]

Hi guys,

In the PSR-2 (and so the incomming PSR-12), it is said that it should have a blank line after the namespace declaration.

For very long namespaces (line will get over 120 chars), the Sniff bugs as it declares the following invalid while it actually is:

<?php

namespace My\Very\Ultra\SuperLongNamespace\That\Get\Over\OneHundredAndTwenty\Characters\EvenIf\This\Is\Actually\Not
    \Good;

use External\A;
use External\B;

//...

What is required by the sniff is to do this:

<?php

namespace My\Very\Ultra\SuperLongNamespace\That\Get\Over\OneHundredAndTwenty\Characters\EvenIf\This\Is\Actually\Not
    
    \Good;

use External\A;
use External\B;

//...

Giving a useless blank line between the keyword "namespace" and the final ";" ending the statement.

Is it possible to fix that?
And please, don't tell me to reconsider my architecture so my namespaces must be shorter, I know that, but I can't do it right now and it must not give style issue to have long names ;).

Thanks a lot.

[gsherwood]

Looks like a bug to me, presumably because the sniff never considered that a namespace would be split over multiple lines.

[gsherwood]

Thanks for the bug report. This has been fixed and will be released in 3.3.1

[niconoe-]

Wow, thank you so much for this very fast fix!
Have a nice day