From fab921f5f52435f3d22f54dd3c40fc18315e7920 Mon Sep 17 00:00:00 2001
From: lhwdev <lhwdev6@outlook.com>
Date: Mon, 31 Aug 2020 13:19:36 +0900
Subject: [PATCH] Release 1.2 - fixed bug that self test scheduling doesn't
 work in background when device is in Doze mode - automatically requires to
 turn off the battery optimization(will use another method later) - notice
 feature

---
 .idea/codeStyles/Project.xml                  |   3 +-
 app/build.gradle                              |   4 +-
 app/src/main/AndroidManifest.xml              |   1 +
 .../com/lhwdev/selfTestMacro/AlarmReceiver.kt |   2 +
 .../com/lhwdev/selfTestMacro/MainActivity.kt  |  95 ++++++++++++++
 .../java/com/lhwdev/selfTestMacro/utils.kt    | 118 +++++++++++++++---
 6 files changed, 202 insertions(+), 21 deletions(-)

diff --git a/.idea/codeStyles/Project.xml b/.idea/codeStyles/Project.xml
index d8b78a87..4149d560 100644
--- a/.idea/codeStyles/Project.xml
+++ b/.idea/codeStyles/Project.xml
@@ -24,7 +24,7 @@
         </value>
       </option>
       <option name="SPACE_BEFORE_WHEN_PARENTHESES" value="false" />
-      <option name="WRAP_EXPRESSION_BODY_FUNCTIONS" value="1" />
+      <option name="CODE_STYLE_DEFAULTS" value="KOTLIN_OFFICIAL" />
     </JetCodeStyleSettings>
     <codeStyleSettings language="CMake">
       <indentOptions>
@@ -192,6 +192,7 @@
       </arrangement>
     </codeStyleSettings>
     <codeStyleSettings language="kotlin">
+      <option name="CODE_STYLE_DEFAULTS" value="KOTLIN_OFFICIAL" />
       <option name="SPACE_BEFORE_IF_PARENTHESES" value="false" />
       <option name="SPACE_BEFORE_WHILE_PARENTHESES" value="false" />
       <option name="SPACE_BEFORE_FOR_PARENTHESES" value="false" />
diff --git a/app/build.gradle b/app/build.gradle
index 7295c07a..ac9ecd53 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -11,8 +11,8 @@ android {
 		applicationId "com.lhwdev.selfTestMacro"
 		minSdkVersion 19
 		targetSdkVersion 29
-		versionCode 1
-		versionName "1.0"
+		versionCode 4
+		versionName "1.2"
 		
 		testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
 	}
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 27c87c38..fc94e095 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -5,6 +5,7 @@
 	
 	<uses-permission android:name="android.permission.INTERNET" />
 	<uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
+	<uses-permission android:name="android.permission.REQUEST_IGNORE_BATTERY_OPTIMIZATIONS"/>
 	
 	<application
 		android:allowBackup="true"
diff --git a/app/src/main/java/com/lhwdev/selfTestMacro/AlarmReceiver.kt b/app/src/main/java/com/lhwdev/selfTestMacro/AlarmReceiver.kt
index e74e3800..69599fa2 100644
--- a/app/src/main/java/com/lhwdev/selfTestMacro/AlarmReceiver.kt
+++ b/app/src/main/java/com/lhwdev/selfTestMacro/AlarmReceiver.kt
@@ -13,8 +13,10 @@ class AlarmReceiver : BroadcastReceiver() {
 	}
 	
 	override fun onReceive(context: Context, intent: Intent) {
+		val result = goAsync()
 		GlobalScope.launch { // TODO: is this okay?
 			context.submitSuspend()
+			result.finish()
 		}
 		
 		val pref = PreferenceState(context.prefMain())
diff --git a/app/src/main/java/com/lhwdev/selfTestMacro/MainActivity.kt b/app/src/main/java/com/lhwdev/selfTestMacro/MainActivity.kt
index 47681cb6..4231e2b0 100644
--- a/app/src/main/java/com/lhwdev/selfTestMacro/MainActivity.kt
+++ b/app/src/main/java/com/lhwdev/selfTestMacro/MainActivity.kt
@@ -1,22 +1,45 @@
 package com.lhwdev.selfTestMacro
 
+import android.annotation.SuppressLint
 import android.app.TimePickerDialog
+import android.content.Context
+import android.content.DialogInterface
 import android.content.Intent
+import android.net.Uri
+import android.os.Build
 import android.os.Bundle
+import android.os.PowerManager
+import android.provider.Settings
+import android.text.Html
+import android.text.SpannableStringBuilder
+import android.text.method.LinkMovementMethod
+import android.text.util.Linkify
 import android.view.Menu
 import android.view.MenuItem
+import android.widget.TextView
 import android.widget.Toast
+import androidx.appcompat.app.AlertDialog
 import androidx.appcompat.app.AppCompatActivity
+import androidx.core.text.HtmlCompat
 import androidx.lifecycle.lifecycleScope
 import kotlinx.android.synthetic.main.activity_main.toolbar
 import kotlinx.android.synthetic.main.content_main.submit
 import kotlinx.android.synthetic.main.content_main.time
+import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
+import org.json.JSONObject
+import java.net.URL
+
+
+const val IGNORE_BATTERY_OPTIMIZATION_REQUEST = 1001
 
 
 @Suppress("SpellCheckingInspection")
 class MainActivity : AppCompatActivity() {
 	
+	private var batteryOptimizationPromptShown = false
+	
 	override fun onCreate(savedInstanceState: Bundle?) {
 		super.onCreate(savedInstanceState)
 		
@@ -37,6 +60,8 @@ class MainActivity : AppCompatActivity() {
 		setContentView(R.layout.activity_main)
 		setSupportActionBar(toolbar)
 		
+		checkNotice()
+		
 		title = "자가진단: ${pref.studentInfo}"
 		
 		val intent = createIntent()
@@ -62,6 +87,8 @@ class MainActivity : AppCompatActivity() {
 		
 		time.setOnClickListener {
 			TimePickerDialog(this, { _, newHour, newMin ->
+				checkBatteryOptimizationPermission()
+				
 				pref.hour = newHour
 				pref.min = newMin
 				updateTime()
@@ -82,6 +109,74 @@ class MainActivity : AppCompatActivity() {
 		}
 	}
 	
+	private fun checkNotice() = lifecycleScope.launch(Dispatchers.IO) {
+		val versions =
+			JSONObject(URL("https://raw.githubusercontent.com/wiki/lhwdev/covid-selftest-macro/_notice.md").readText())
+		
+		/*
+		 * Spec:
+		 * {"$version | all": {"id": "$id", "priority": "once | every", "title": "$title", "message": "$message"}}
+		 *
+		 * Version spec: 1.0 1.3..2.1 ..1.5
+		 */
+		
+		val thisVersion = Version(BuildConfig.VERSION_NAME)
+		
+		for(key in versions.keys()) if(key == "all" || thisVersion in VersionSpec(key)) {
+			val notice = Notice(versions.getJSONObject(key))
+			val show = when(notice.priority) {
+				Notice.Priority.once -> notice.id !in preferenceState.shownNotices
+				Notice.Priority.every -> true
+			}
+			
+			if(show) withContext(Dispatchers.Main) {
+				AlertDialog.Builder(this@MainActivity).apply {
+					setTitle(notice.title)
+					val message = HtmlCompat.fromHtml(notice.message, 0)
+					setMessage(message)
+					setPositiveButton("확인", null)
+				}.show().apply {
+					findViewById<TextView>(android.R.id.message)!!.movementMethod =
+						LinkMovementMethod.getInstance()
+				}
+				
+				preferenceState.shownNotices += notice.id
+			}
+		}
+	}
+	
+	/*
+	 * > Task automation app
+	 *   App's core function is scheduling automated actions, such as for instant messaging, voice calling, or new photo management
+	 *   Acceptable
+	 *
+	 * I won't upload it on Play Store (so far), but it's always good to follow guidelines
+	 * In case I upload, I won't use this api, instead will use foreground service.
+	 * https://developer.android.com/training/monitoring-device-state/doze-standby#exemption-cases
+	 */
+	private fun checkBatteryOptimizationPermission() {
+		val pwrm = applicationContext.getSystemService(Context.POWER_SERVICE) as PowerManager
+		val name = applicationContext.packageName
+		
+		if(!batteryOptimizationPromptShown
+			&& android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.M
+			&& !pwrm.isIgnoringBatteryOptimizations(name)) {
+			AlertDialog.Builder(this).apply {
+				setTitle("베터리 최적화 설정을 꺼야 알림 기능이 정상적으로 작동됩니다.")
+				setPositiveButton("설정") { _, _ ->
+					@SuppressLint("BatteryLife")
+					val intent = Intent(Settings.ACTION_REQUEST_IGNORE_BATTERY_OPTIMIZATIONS)
+					intent.data = Uri.parse("package:$packageName")
+					startActivityForResult(intent, IGNORE_BATTERY_OPTIMIZATION_REQUEST)
+				}
+				
+				setNegativeButton("취소", null)
+			}
+			
+			batteryOptimizationPromptShown = true
+		}
+	}
+	
 	override fun onCreateOptionsMenu(menu: Menu): Boolean {
 		// Inflate the menu; this adds items to the action bar if it is present.
 		menuInflater.inflate(R.menu.menu_main, menu)
diff --git a/app/src/main/java/com/lhwdev/selfTestMacro/utils.kt b/app/src/main/java/com/lhwdev/selfTestMacro/utils.kt
index 208ed8dc..6399749f 100644
--- a/app/src/main/java/com/lhwdev/selfTestMacro/utils.kt
+++ b/app/src/main/java/com/lhwdev/selfTestMacro/utils.kt
@@ -27,6 +27,48 @@ import kotlin.properties.ReadWriteProperty
 import kotlin.reflect.KProperty
 
 
+// our project is too small to use form like 'major.minor.bugFixes'
+data class Version(val major: Int, val minor: Int) : Comparable<Version> {
+	override fun compareTo(other: Version) =
+		if(major == other.major) minor - other.minor
+		else major - other.major
+}
+
+// {"id": "$id", "priority": "once | every", "title": "$title", "message": "$message"}
+class Notice(obj: JSONObject) {
+	enum class Priority { once, every }
+	
+	val id: String = obj.getString("id")
+	
+	val priority = when(obj.getString("priority")) {
+		"once" -> Priority.once
+		"every" -> Priority.every
+		else -> error("wow")
+	}
+	
+	val title: String = obj.getString("title")
+	val message: String = obj.getString("message")
+}
+
+
+fun Version(string: String): Version {
+	val split = string.split('.').map { it.toInt() }
+	require(split.size == 2)
+	return Version(split[0], split[1])
+}
+
+data class VersionSpec(val from: Version, val to: Version) {
+	operator fun contains(version: Version) = version in from..to
+}
+
+fun VersionSpec(string: String): VersionSpec {
+	val index = string.indexOf("..")
+	if(index == -1) return Version(string).let { VersionSpec(it, it) }
+	val from = string.take(index)
+	val to = string.drop(index + 2)
+	return VersionSpec(Version(from), Version(to))
+}
+
 class PreferenceState(val pref: SharedPreferences) {
 	var firstState by pref.preferenceInt("first", 0)
 	var hour by pref.preferenceInt("hour", -1)
@@ -41,11 +83,20 @@ class PreferenceState(val pref: SharedPreferences) {
 	var cert by pref.preferenceString("cert")
 	
 	var studentInfo
-		get() = StudentInfo(schoolName = pref.getString("schoolName", "?")!!, studentName = pref.getString("studentName", "?")!!)
+		get() = StudentInfo(
+			schoolName = pref.getString("schoolName", "?")!!,
+			studentName = pref.getString("studentName", "?")!!
+		)
 		set(value) = pref.edit {
 			putString("schoolName", value.schoolName)
 			putString("studentName", value.studentName)
 		}
+	
+	var shownNotices: Set<String>
+		get() = pref.getStringSet("shownNotices", setOf())!!
+		set(value) = pref.edit {
+			putStringSet("shownNotices", value)
+		}
 }
 
 lateinit var preferenceState: PreferenceState
@@ -80,7 +131,8 @@ fun Context.prefMain() = getSharedPreferences("main", AppCompatActivity.MODE_PRI
 
 fun Context.createIntent() = PendingIntent.getBroadcast(
 	this, AlarmReceiver.REQUEST_CODE, Intent(this, AlarmReceiver::class.java),
-	PendingIntent.FLAG_UPDATE_CURRENT)
+	PendingIntent.FLAG_UPDATE_CURRENT
+)
 
 fun Context.updateTime(intent: PendingIntent) {
 	val alarmManager = getSystemService(Context.ALARM_SERVICE) as AlarmManager
@@ -90,18 +142,27 @@ fun Context.updateTime(intent: PendingIntent) {
 }
 
 @SuppressLint("NewApi")
-fun Context.scheduleNextAlarm(intent: PendingIntent, hour: Int, min: Int, nextDay: Boolean = false) {
-	(getSystemService(Context.ALARM_SERVICE) as AlarmManager).setExact(AlarmManager.RTC_WAKEUP, Calendar.getInstance().run {
-		val new = clone() as Calendar
-		new[Calendar.HOUR_OF_DAY] = hour
-		new[Calendar.MINUTE] = min
-		new[Calendar.SECOND] = 0
-		new[Calendar.MILLISECOND] = 0
-		if(nextDay || new <= this) new.add(Calendar.DAY_OF_YEAR, 1)
-		Log.e("HOI", "$this")
-		Log.e("HOI", "$new")
-		new.timeInMillis
-	}, intent)
+fun Context.scheduleNextAlarm(
+	intent: PendingIntent,
+	hour: Int,
+	min: Int,
+	nextDay: Boolean = false
+) {
+	(getSystemService(Context.ALARM_SERVICE) as AlarmManager).setExact(
+		AlarmManager.RTC_WAKEUP,
+		Calendar.getInstance().run {
+			val new = clone() as Calendar
+			new[Calendar.HOUR_OF_DAY] = hour
+			new[Calendar.MINUTE] = min
+			new[Calendar.SECOND] = 0
+			new[Calendar.MILLISECOND] = 0
+			if(nextDay || new <= this) new.add(Calendar.DAY_OF_YEAR, 1)
+			Log.e("HOI", "$this")
+			Log.e("HOI", "$new")
+			new.timeInMillis
+		},
+		intent
+	)
 }
 
 @Suppress("NOTHING_TO_INLINE")
@@ -124,7 +185,14 @@ suspend fun checkStudentInfoSuspend(site: URL, cert: String) = withContext(Dispa
 		.openConnection() as HttpURLConnection
 	checkRequest.requestMethod = "POST"
 	DataOutputStream(checkRequest.outputStream).use {
-		it.writeBytes("rtnRsltCode=SUCCESS&qstnCrtfcNoEncpt=${URLEncoder.encode(cert, "utf-8")}&schulNm=&stdntName=&rspns01=1&rspns02=1&rspns07=0&rspns08=0&rspns09=0")
+		it.writeBytes(
+			"rtnRsltCode=SUCCESS&qstnCrtfcNoEncpt=${
+				URLEncoder.encode(
+					cert,
+					"utf-8"
+				)
+			}&schulNm=&stdntName=&rspns01=1&rspns02=1&rspns07=0&rspns08=0&rspns09=0"
+		)
 		it.flush()
 	}
 	
@@ -159,10 +227,20 @@ suspend fun Context.submitSuspend() = withContext(Dispatchers.IO) {
 		.openConnection() as HttpURLConnection
 	request.requestMethod = "POST"
 	request.setRequestProperty("Content-Type", "application/x-www-form-urlencoded")
-	request.setRequestProperty("User-Agent", "Mozilla/5.0 (Windows; U; Windows NT 6.1; en-GB; rv:1.9.2.8) Gecko/20100722 Firefox/3.6.8")
+	request.setRequestProperty(
+		"User-Agent",
+		"Mozilla/5.0 (Windows; U; Windows NT 6.1; en-GB; rv:1.9.2.8) Gecko/20100722 Firefox/3.6.8"
+	)
 	HttpURLConnection.setFollowRedirects(true)
 	DataOutputStream(request.outputStream).use {
-		it.writeBytes("rtnRsltCode=SUCCESS&qstnCrtfcNoEncpt=${URLEncoder.encode(cert, "utf-8")}&schulNm=&stdntName=&rspns01=1&rspns01=2&rspns02=1&rspns03=1&rspns05=1&rspns13=1&rspns14=1&rspns15=1&rspns04=1&rspns11=1&rspns07=0&rspns07=1&rspns08=0&rspns08=1&rspns09=0&rspns09=1")
+		it.writeBytes(
+			"rtnRsltCode=SUCCESS&qstnCrtfcNoEncpt=${
+				URLEncoder.encode(
+					cert,
+					"utf-8"
+				)
+			}&schulNm=&stdntName=&rspns01=1&rspns01=2&rspns02=1&rspns03=1&rspns05=1&rspns13=1&rspns14=1&rspns15=1&rspns04=1&rspns11=1&rspns07=0&rspns07=1&rspns08=0&rspns08=1&rspns09=0&rspns09=1"
+		)
 		it.flush()
 	}
 	Log.i("HOI", "${request.responseMessage} / ${request.inputStream.reader().readText()}")
@@ -171,5 +249,9 @@ suspend fun Context.submitSuspend() = withContext(Dispatchers.IO) {
 	
 	// log
 	File(getExternalFilesDir(null)!!, "log.txt")
-		.appendText("self-tested at ${DateFormat.getDateTimeInstance().format(Date())} ${request.responseMessage}\n")
+		.appendText(
+			"self-tested at ${
+				DateFormat.getDateTimeInstance().format(Date())
+			} ${request.responseMessage}\n"
+		)
 }