diff options
| author |   jrh3 <jrh3@att.com> | 2019-06-14 10:31:40 -0400 |
|---|---|---|
| committer | jrh3 <jrh3@att.com> | 2019-06-14 10:36:34 -0400 |
| commit | 82cc7e0ccf799e0ea5d9afd033a3cff88468b6bf (patch) | |
| tree | eb10d3d9278656866b1323d7a50d650ce0e98b33 /main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java | |
| parent | a240d7a4020d0346040fe4d86682a6ab8fcd757a (diff) | |
Fix sonar issues in PAP
Replace string concatenation with "{}" place-holders in logging
statements.
Refactored PdpStatusMessageHandler and PdpGroupStateChangeProvider
to extract common/duplicate code block into PdpMessageGenerator.
Refactored selectPdpGroupsForRegistration() to reduce "cyclomatic complexity";
extracted selectPdpSubGroupsForRegistration from it.
Issue-ID: POLICY-1791
Signed-off-by: jrh3 <jrh3@att.com>
Change-Id: Ia0ebc1cf41cd2fa288130b4a58614a4983f47f8b
Signed-off-by: jrh3 <jrh3@att.com>
Diffstat (limited to 'main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java')
| -rw-r--r-- | main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java b/main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java index f19d7db8..99677fb8 100644 --- a/main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java +++ b/main/src/main/java/org/onap/policy/pap/main/comm/TimerManager.java @@ -285,7 +285,7 @@ public class TimerManager implements Runnable { return false; } - logger.debug("{} timer " + cancelMsg + " {}", TimerManager.this.name, this); + logger.debug("{} timer {} {}", TimerManager.this.name, cancelMsg, this); return true; } |