Merge pull request #336 from wuseal/fix/#296

[feature] #296  Analytics Switch Support

build.gradle

@@ -1,5 +1,5 @@
 buildscript {
-    ext.kotlin_version = '1.3.61'
+    ext.kotlin_version = '1.4.20'
 
     repositories {
         mavenLocal()

gradle/wrapper/gradle-wrapper.properties

@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-5.5.1-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-6.5-all.zip

src/main/kotlin/extensions/ExtensionsCollector.kt

@@ -6,6 +6,7 @@ import extensions.ted.zeng.PropertyAnnotationLineSupport
 import extensions.wu.seal.*
 import extensions.xu.rui.PrimitiveTypeNonNullableSupport
 import extensions.nstd.ReplaceConstructorParametersByMemberVariablesSupport
+import wu.seal.jsontokotlin.interceptor.AnalyticsSwitchSupport
 
 /**
  * extension collect, all extensions will be hold by this class's extensions property
@@ -25,6 +26,7 @@ object ExtensionsCollector {
             PrimitiveTypeNonNullableSupport,
             ForceInitDefaultValueWithOriginJsonValueSupport,
             DisableDataClassSupport,
-            ReplaceConstructorParametersByMemberVariablesSupport
+            ReplaceConstructorParametersByMemberVariablesSupport,
+            AnalyticsSwitchSupport
     )
 }

src/main/kotlin/wu/seal/jsontokotlin/feedback/NetWork.kt

@@ -1,5 +1,6 @@
 package wu.seal.jsontokotlin.feedback
 
+import wu.seal.jsontokotlin.interceptor.AnalyticsSwitchSupport
 import java.net.HttpURLConnection
 import java.net.URL
 
@@ -48,6 +49,9 @@ fun sendHistoryActionInfo() {
 }
 
 fun sendData(url: String, log: String) {
+    if (AnalyticsSwitchSupport.enableAnalytics().not()) {
+       return
+    }
     Thread {
         try {
             with(URL(url).openConnection() as HttpURLConnection) {

src/main/kotlin/wu/seal/jsontokotlin/interceptor/AnalyticsSwitchSupport.kt

@@ -0,0 +1,32 @@
+package wu.seal.jsontokotlin.interceptor
+
+import extensions.Extension
+import wu.seal.jsontokotlin.model.classscodestruct.KotlinClass
+import wu.seal.jsontokotlin.ui.jCheckBox
+import wu.seal.jsontokotlin.ui.jHorizontalLinearLayout
+import javax.swing.JPanel
+
+
+/**
+ * Not used for Kotlin data class, only used for global config
+ */
+object AnalyticsSwitchSupport: Extension() {
+
+    const val configKey = "wu.seal.analytics_switch"
+
+    override fun createUI(): JPanel {
+        return jHorizontalLinearLayout {
+            jCheckBox("Enable anonymous analytic", getConfig(configKey).toBoolean(), { isSelected -> setConfig(
+                configKey, isSelected.toString()) })
+            fillSpace()
+        }
+    }
+
+    override fun intercept(kotlinClass: KotlinClass): KotlinClass {
+       return kotlinClass  //do nothing for kotlinClass
+    }
+
+    fun enableAnalytics(): Boolean {
+        return getConfig(configKey).toBoolean()
+    }
+}

src/test/kotlin/wu/seal/jsontokotlin/interceptor/AnalyticsSwitchSupportTest.kt

@@ -0,0 +1,20 @@
+package wu.seal.jsontokotlin.interceptor
+
+import com.winterbe.expekt.should
+import org.junit.Test
+import wu.seal.jsontokotlin.utils.BaseTest
+
+class AnalyticsSwitchSupportTest : BaseTest() {
+
+    @Test
+    fun enableAnalytics() {
+        val originalValue = AnalyticsSwitchSupport.enableAnalytics()
+        val expectedValue = true
+        AnalyticsSwitchSupport.getTestHelper()
+            .setConfig(AnalyticsSwitchSupport.configKey, expectedValue.toString())
+        val nowValue = AnalyticsSwitchSupport.enableAnalytics()
+        nowValue.should.be.equal(expectedValue)
+        AnalyticsSwitchSupport.getTestHelper()
+            .setConfig(AnalyticsSwitchSupport.configKey, originalValue.toString())
+    }
+}

src/test/kotlin/wu/seal/jsontokotlin/utils/BaseTest.kt

@@ -0,0 +1,11 @@
+package wu.seal.jsontokotlin.utils
+
+import org.junit.Before
+import wu.seal.jsontokotlin.test.TestConfig
+
+open class BaseTest {
+    @Before
+    fun beforeTest() {
+        TestConfig.setToTestInitState()
+    }
+}