detekt

Author: manodnyab

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/toolwindow/OuterAmazonQPanel.kt

@@ -8,6 +8,7 @@ import com.intellij.openapi.components.service
 import com.intellij.openapi.project.Project
 import com.intellij.ui.components.panels.Wrapper
 import com.intellij.util.ui.components.BorderLayoutPanel
+import software.aws.toolkits.core.utils.error
 import software.aws.toolkits.core.utils.getLogger
 import javax.swing.JComponent
 
@@ -23,7 +24,7 @@ class OuterAmazonQPanel : BorderLayoutPanel() {
         try {
             wrapper.setContent(content)
         } catch (e: Exception) {
-            getLogger<OuterAmazonQPanel>().error("Error while creating window")
+            getLogger<OuterAmazonQPanel>().error { "Error while creating window" }
         }
     }
 

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/notifications/CustomizeNotificationsUi.kt

@@ -62,6 +62,7 @@ object NotificationManager {
                         AllIcons.General.Error
                     )
                 } else {
+                    val link = url ?: AwsToolkit.GITHUB_URL
                     val openLink = Messages.showYesNoDialog(
                         project,
                         message,
@@ -71,7 +72,7 @@ object NotificationManager {
                         AllIcons.General.Error
                     )
                     if (openLink == 0) {
-                        BrowserUtil.browse(url!!)
+                        BrowserUtil.browse(link)
                     }
                 }
             }

plugins/core/jetbrains-community/tst/software/aws/toolkits/jetbrains/core/notifications/NotificationManagerTest.kt

@@ -30,7 +30,7 @@ class NotificationManagerTest {
             "UpdateExtension",
             NotificationFollowupActionsContent(NotificationActionDescription("title", null))
         )
-        val sut = NotificationManager.createActions(projectRule.project,listOf(followupActions), "Dummy Test Action", "Dummy title")
+        val sut = NotificationManager.createActions(projectRule.project, listOf(followupActions), "Dummy Test Action", "Dummy title")
         assertThat(sut).isNotNull
         assertThat(sut).hasSize(2)
         assertThat(sut.first().title).isEqualTo("Update")

plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/core/explorer/webview/OuterToolkitPanel.kt

@@ -8,6 +8,7 @@ import com.intellij.openapi.components.service
 import com.intellij.openapi.project.Project
 import com.intellij.ui.components.panels.Wrapper
 import com.intellij.util.ui.components.BorderLayoutPanel
+import software.aws.toolkits.core.utils.error
 import software.aws.toolkits.core.utils.getLogger
 import javax.swing.JComponent
 
@@ -23,7 +24,7 @@ class OuterToolkitPanel : BorderLayoutPanel() {
         try {
             wrapper.setContent(content)
         } catch (e: Exception) {
-            getLogger<OuterToolkitPanel>().error("Error while creating window")
+            getLogger<OuterToolkitPanel>().error { "Error while creating window" }
         }
     }