ConstructorOnlyInitializesOrCallOtherConstructors rule should ignore lambdas

[krzyk]

ConstructorOnlyInitializesOrCallOtherConstructors and OnlyOneConstructorShouldDoInitialization rule fail for:

     public HtWire(final URI uri) {
         this(uri, Socket::new);
     }
 
     public HtWire(final String addr) {
         // @checkstyle MagicNumber (1 line)
         this(addr, new Constant<>(80), Socket::new);
     }
 
     public HtWire(final String addr, final int tcp) {
         this(addr, new Constant<>(tcp), Socket::new);
     }
 
     HtWire(final URI uri, final BiFunc<String, Integer, Socket> spplier) {
         this(
             uri.getHost(),
             () -> {
                 final int prt;
                 if (uri.getPort() == -1) {
                     prt = uri.toURL().getDefaultPort();
                 } else {
                     prt = uri.getPort();
                 }
                 return prt;
             },
             spplier
         );
}

When I remove the code in lambda, or replace it with a simpler one:

    HtWire(final URI uri, final BiFunc<String, Integer, Socket> spplier) {
        this(
            uri.getHost(),
            new Ternary<>(
                () -> uri.getPort() == -1,
                () -> uri.toURL().getDefaultPort(),
                uri::getPort
            ),
            spplier
        );
}

It doesn't fail.

It should ignore any code inside lambdas.

[krzyk]

@0crat in

[0crat]

@0crat in (here)

@krzyk Job #887 is now in scope, role is DEV

[0crat]

Bug was reported, see §29: +15 point(s) just awarded to @krzyk/z

[0crat]

@krzyk/z everybody who has role DEV are banned at this job; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV are banned at this job; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV are banned at this job; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV are banned at #887; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV are banned at #887; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV is banned at #887; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV is banned at #887; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV is banned at #887; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[0crat]

@krzyk/z everybody who has role DEV is banned at #887; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

[paulodamaso]

@0crat wait for #983 review

[0crat]

@0crat wait for #983 review (here)

@paulodamaso The impediment for #887 was registered successfully by @paulodamaso/z

[paulodamaso]

@krzyk Merged, please close.

[krzyk]

@paulodamaso thanks

[0crat]

@ypshenychka/z please review this job completed by @paulodamaso/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

[0crat]

The job #887 is now out of scope

[ypshenychka]

@0crat quailty good

[ypshenychka]

@0crat quality good

[0crat]

@0crat quailty good (here)

@ypshenychka Can't understand "quailty", try one of these:

• assign Assign a performer to this issue
• boost Set a boost for the job
• continue Remove a job's impediment
• hello Just say hello
• in Register this issue in scope (WBS)
• out Close the order and remove this job from scope
• pay Pay a user some extra cash
• quality Review a task
• resign Resign from this issue
• status Check the status of the job
• version Print current version of the bot
• wait Register an impediment for a job

[0crat]

@0crat quality good (here)

@ypshenychka The project doesn't have enough funds, can't make a payment

[0crat]

Order was finished, quality is "good": +35 point(s) just awarded to @paulodamaso/z

[0crat]

Quality review completed: +8 point(s) just awarded to @ypshenychka/z