Mercurial > cgi-bin > hgweb.cgi > SimpleResizer

annotate app/src/main/java/com/bartsent/simpleresizer/EditImage.kt @ 43:9cb9bb5da247

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
At long last it auto-deletes old cruft files.
author David Barts <n5jrn@me.com>
date Sat, 10 Apr 2021 17:29:08 -0700
parents 45e4df5226c0
children 2b91619da650
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
1 package com.bartsent.simpleresizer
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
2
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
3 import android.Manifest
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
4 import android.content.ContentUris
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
5 import android.content.ContentValues
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
6 import android.content.Intent
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
7 import android.content.pm.PackageManager
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
8 import android.graphics.Bitmap
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
9 import android.graphics.BitmapFactory
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
10 import android.graphics.Canvas
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
11 import android.graphics.Matrix
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
12 import android.net.Uri
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
13 import android.os.Bundle
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
14 import android.os.Environment
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
15 import android.provider.MediaStore
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
16 import android.provider.OpenableColumns
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
17 import android.util.Log
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
18 import android.view.Menu
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
19 import android.view.MenuItem
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
20 import android.view.View
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
21 import android.widget.EditText
10
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
22 import android.widget.ProgressBar
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
23 import android.widget.Toast
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
24 import androidx.appcompat.app.AlertDialog
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
25 import androidx.appcompat.app.AppCompatActivity
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
26 import androidx.appcompat.widget.PopupMenu
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
27 import androidx.lifecycle.ViewModel
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
28 import androidx.lifecycle.ViewModelProvider
22
c29f941d09cd Add landscape mode.
David Barts <n5jrn@me.com>
parents: 21
diff changeset
29 import androidx.preference.PreferenceManager
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
30 import com.bartsent.simpleresizer.databinding.ActivityEditImageBinding
22
c29f941d09cd Add landscape mode.
David Barts <n5jrn@me.com>
parents: 21
diff changeset
31 import com.bartsent.simpleresizer.lib.ThreadPools
6
e8059b166de1 Lanczos works, but is painfully slow.
David Barts <n5jrn@me.com>
parents: 5
diff changeset
32 import com.bartsent.simpleresizer.lib.getScaledInstance
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
33 import com.google.android.material.floatingactionbutton.FloatingActionButton
5
247e03baf77c Fix rotation.
David Barts <n5jrn@me.com>
parents: 4
diff changeset
34 import java.io.File
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
35 import java.io.IOException
33
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
36 import java.util.concurrent.Callable
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
37 import java.util.concurrent.Future
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
38
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
39 class EditImage : AppCompatActivity() {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
40 class State: ViewModel() {
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
41 var uri: Uri? = null
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
42 var bitmap: Bitmap? = null
33
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
43 var reader: Future<Unit>? = null
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
44 var permissionsCallback: (() -> Unit)? = null
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
45 var sharable: Boolean = false
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
46 }
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
47 private lateinit var viewModel: State
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
48
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
49 private val STDDIMS = arrayOf<Int>(1600, 1280, 1024, 800, 640, 512, 400, 320).apply {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
50 sort()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
51 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
52
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
53 private val IMAGE_TO_SEND = "simple_resizer_sent_image.jpg"
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
54
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
55 private lateinit var binding: ActivityEditImageBinding
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
56
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
57 override fun onCreate(savedInstanceState: Bundle?) {
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
58 super.onCreate(savedInstanceState)
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
59 binding = ActivityEditImageBinding.inflate(layoutInflater)
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
60 setContentView(binding.root)
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
61 PreferenceManager.setDefaultValues(applicationContext, R.xml.root_preferences, false)
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
62 viewModel = ViewModelProvider(this).get(State::class.java)
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
63 }
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
64
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
65 override fun onCreateOptionsMenu(menu: Menu?): Boolean {
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
66 menuInflater.inflate(R.menu.menu_edit, menu)
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
67 return super.onCreateOptionsMenu(menu)
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
68 }
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
69
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
70 override fun onOptionsItemSelected(item: MenuItem): Boolean {
40
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
71 when (item.itemId) {
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
72 R.id.settings_item -> {
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
73 startActivity(
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
74 Intent(Intent.ACTION_APPLICATION_PREFERENCES, null, this,
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
75 SettingsActivity::class.java))
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
76 return true
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
77 }
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
78 R.id.about_item -> {
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
79 startActivity(Intent(this, About::class.java ))
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
80 return true
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
81 }
cfb19d4ccf78 About screen (but it has broken Edit screen).
David Barts <n5jrn@me.com>
parents: 37
diff changeset
82 else -> return false
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
83 }
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
84 }
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
85
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
86 // Cribbed from: https://stackoverflow.com/questions/5568874/how-to-extract-the-file-name-from-uri-returned-from-intent-action-get-content
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
87 private fun getFileName(uri: Uri): String? {
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
88 var result: String? = null
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
89 if (uri.scheme == "content") {
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
90 contentResolver.query(uri, null, null, null, null).use { cursor ->
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
91 if (cursor != null && cursor.moveToFirst())
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
92 result = cursor.getString(cursor.getColumnIndex(OpenableColumns.DISPLAY_NAME))
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
93 }
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
94 }
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
95 if (result == null) {
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
96 val uriPath = uri.path
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
97 result = uriPath?.substring(uriPath.lastIndexOf('/') + 1)
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
98 }
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
99 return result
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
100 }
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
101
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
102 private fun showFatalError(message: String): Unit {
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
103 AlertDialog.Builder(this).also {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
104 it.setMessage(message)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
105 it.setNeutralButton(R.string.ok_text) { dialog, _ ->
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
106 dialog.dismiss()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
107 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
108 it.setOnDismissListener {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
109 finish()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
110 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
111 it.create()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
112 }.show()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
113 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
114
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
115 fun showError(message: String): Unit {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
116 AlertDialog.Builder(this).also {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
117 it.setMessage(message)
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
118 it.setNeutralButton(R.string.ok_text) { dialog, _ ->
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
119 dialog.dismiss()
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
120 }
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
121 it.create()
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
122 }.show()
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
123 }
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
124
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
125 override fun onResume() {
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
126 super.onResume()
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
127
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
128 // Read the URI, die if we can't.
29
eec88eba58e9 Fix receiving from another app.
David Barts <n5jrn@me.com>
parents: 25
diff changeset
129 val imageUri = intent?.data ?: intent?.extras?.get(Intent.EXTRA_STREAM) as? Uri
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
130 if (imageUri == null) {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
131 if (viewModel.bitmap == null)
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
132 showFatalError(getString(R.string.error_no_uri))
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
133 else
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
134 setImage(viewModel.bitmap!!)
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
135 return
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
136 }
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
137
21
7e7e71724770 No longer breaks when phone rotated.
David Barts <n5jrn@me.com>
parents: 18
diff changeset
138 // User has opened a new image.
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
139 if (imageUri != viewModel.uri) {
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
140 viewModel.uri = imageUri
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
141 makeMundane()
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
142 binding.progressBar.visibility = ProgressBar.VISIBLE
33
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
143 viewModel.reader = ThreadPools.WORKERS.submit(Callable<Unit> {
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
144 val newBitmap = contentResolver.openInputStream(imageUri).use {
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
145 BitmapFactory.decodeStream(it)
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
146 }
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
147 runOnUiThread {
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
148 binding.progressBar.visibility = ProgressBar.INVISIBLE
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
149 if (newBitmap == null)
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
150 showFatalError(getString(R.string.error_bad_image))
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
151 else
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
152 setImage(newBitmap)
33
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
153 viewModel.reader = null
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
154 }
33
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
155 })
21
7e7e71724770 No longer breaks when phone rotated.
David Barts <n5jrn@me.com>
parents: 18
diff changeset
156 return
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
157 }
21
7e7e71724770 No longer breaks when phone rotated.
David Barts <n5jrn@me.com>
parents: 18
diff changeset
158
7e7e71724770 No longer breaks when phone rotated.
David Barts <n5jrn@me.com>
parents: 18
diff changeset
159 // Rotation (of the phone).
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
160 val oldBitmap = viewModel.bitmap
21
7e7e71724770 No longer breaks when phone rotated.
David Barts <n5jrn@me.com>
parents: 18
diff changeset
161 if (oldBitmap != null)
7e7e71724770 No longer breaks when phone rotated.
David Barts <n5jrn@me.com>
parents: 18
diff changeset
162 setImage(oldBitmap)
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
163 binding.fabulous.visibility = if (viewModel.sharable) FloatingActionButton.VISIBLE else FloatingActionButton.GONE
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
164 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
165
33
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
166 override fun onDestroy() {
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
167 // Read tasks may get badly constipated, since the image may well be on
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
168 // cloud server like Google Pictures. So be sure to terminate any active
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
169 // read task with extreme prejudice.
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
170 val reader = viewModel.reader
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
171 if (reader != null) {
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
172 reader.cancel(true)
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
173 viewModel.reader = null
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
174 }
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
175 super.onDestroy()
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
176 }
bead5d7e8c69 Deal with constipated read tasks.
David Barts <n5jrn@me.com>
parents: 32
diff changeset
177
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
178 private fun setImage(image: Bitmap): Unit {
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
179 binding.imageSize.text = getString(R.string.image_size_text, image.width, image.height)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
180 binding.image.setImageBitmap(image)
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
181 viewModel.bitmap = image
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
182 binding.root.invalidate()
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
183 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
184
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
185 private fun unsetImage(): Unit {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
186 viewModel.uri = null
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
187 viewModel.bitmap = null
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
188 }
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
189
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
190 private val CUSTOM = 999998
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
191 private val CANCEL = 999999
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
192
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
193 fun scaleClicked(view: View): Unit {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
194 val (maxSize, horizontal) = viewModel.bitmap!!.run {
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
195 if (width > height) Pair(width, true) else Pair(height, false)
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
196 }
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
197 PopupMenu(this, view).apply {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
198 menu.run {
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
199 STDDIMS.filter { it < maxSize }.forEach { major ->
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
200 val minor = major * 3 / 4
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
201 add(Menu.NONE, major, Menu.NONE,
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
202 if (horizontal) "$major ✕ $minor" else "$minor ✕ $major")
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
203 }
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
204 add(Menu.NONE, CUSTOM, Menu.NONE, R.string.custom_text)
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
205 add(Menu.NONE, CANCEL, Menu.NONE, R.string.cancel_text)
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
206 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
207 setOnMenuItemClickListener(::scaleMenuItemClicked)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
208 show()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
209 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
210 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
211
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
212 private fun scaleMenuItemClicked(item: MenuItem) : Boolean =
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
213 when (item.itemId) {
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
214 CUSTOM -> { showCustomScaleDialog(); true }
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
215 CANCEL -> true
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
216 in STDDIMS -> { doScale(item.itemId); true }
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
217 else -> false
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
218 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
219
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
220 private fun copyColorSpace(old: Bitmap, new: Bitmap): Unit {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
221 if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.O) {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
222 val oldColorSpace = old.colorSpace
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
223 if (oldColorSpace != null)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
224 new.setColorSpace(oldColorSpace)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
225 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
226 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
227
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
228 private fun doScale(newMax: Int): Unit {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
229 val oldBitmap = viewModel.bitmap!!
6
e8059b166de1 Lanczos works, but is painfully slow.
David Barts <n5jrn@me.com>
parents: 5
diff changeset
230 val factor = newMax.toDouble() / maxOf(oldBitmap.width, oldBitmap.height).toDouble()
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
231 if (factor >= 1.0) {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
232 throw IllegalArgumentException("can only scale down")
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
233 }
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
234 val scaleType = PreferenceManager.getDefaultSharedPreferences(applicationContext).getString(
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
235 "scale_type", "speed" )
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
236 // Log.d("EditImage", "scaling, scale_type = $scaleType")
10
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
237 binding.progressBar.visibility = ProgressBar.VISIBLE
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
238 ThreadPools.WORKERS.execute {
15
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
239 val newWidth = (oldBitmap.width.toDouble() * factor + 0.5).toInt()
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
240 val newHeight = (oldBitmap.height.toDouble() * factor + 0.5).toInt()
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
241 val newBitmap = if (scaleType == "quality")
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
242 oldBitmap.getScaledInstance(newWidth, newHeight)
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
243 else
20da616dcda0 Add preferences.
David Barts <n5jrn@me.com>
parents: 11
diff changeset
244 Bitmap.createScaledBitmap(oldBitmap, newWidth, newHeight, true)
10
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
245 runOnUiThread {
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
246 binding.progressBar.visibility = ProgressBar.INVISIBLE
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
247 setImage(newBitmap)
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
248 makeFabulous()
10
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
249 }
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
250 }
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
251 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
252
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
253 private fun showCustomScaleDialog(): Unit {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
254 val image = viewModel.bitmap!!
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
255 val curMaxDim = maxOf(image.width, image.height)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
256 val dialogView = layoutInflater.inflate(R.layout.dialog_custom_scale, null)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
257 AlertDialog.Builder(this).also {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
258 it.setPositiveButton(R.string.ok_text) { dialog, _ ->
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
259 val maxDim = dialogView.findViewById<EditText>(R.id.custom_scale)?.text.toString().toIntOrNull()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
260 dialog.dismiss()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
261 if (maxDim == null || maxDim < 8 || maxDim >= curMaxDim) {
16
3ed74dc0e34a Fix error messages, make scaling menu more user-friendly, fix return from settings.
David Barts <n5jrn@me.com>
parents: 15
diff changeset
262 Toast.makeText(applicationContext, R.string.bad_scale, Toast.LENGTH_LONG).show()
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
263 } else {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
264 doScale(maxDim)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
265 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
266 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
267 it.setNegativeButton(R.string.cancel_text) { dialog, _ ->
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
268 dialog.dismiss()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
269 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
270 it.setView(dialogView)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
271 it.create()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
272 }.show()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
273 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
274
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
275 fun rotateClicked(view: View): Unit {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
276 PopupMenu(this, view).apply {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
277 setOnMenuItemClickListener {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
278 when(it.itemId) {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
279 R.id.r_90_cw -> {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
280 doRotate(90)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
281 true
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
282 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
283 R.id.r_180 -> {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
284 doRotate(180)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
285 true
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
286 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
287 R.id.r_90_ccw -> {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
288 doRotate(270)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
289 true
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
290 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
291 R.id.r_cancel -> true
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
292 else -> false
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
293 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
294 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
295 inflate(R.menu.rotate)
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
296 show()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
297 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
298 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
299
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
300 private fun doRotate(deg: Int): Unit {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
301 val oldBitmap = viewModel.bitmap!!
5
247e03baf77c Fix rotation.
David Barts <n5jrn@me.com>
parents: 4
diff changeset
302 if (deg % 90 != 0) {
247e03baf77c Fix rotation.
David Barts <n5jrn@me.com>
parents: 4
diff changeset
303 throw IllegalArgumentException("$deg not a multiple of 90")
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
304 }
5
247e03baf77c Fix rotation.
David Barts <n5jrn@me.com>
parents: 4
diff changeset
305 val (w, h) = if (deg % 180 == 0) Pair(oldBitmap.width, oldBitmap.height) else Pair(oldBitmap.height, oldBitmap.width)
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
306 val rotater = Matrix().apply {
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
307 setRotate(deg.toFloat(), oldBitmap.width.toFloat()/2.0f, oldBitmap.height.toFloat()/2.0f)
5
247e03baf77c Fix rotation.
David Barts <n5jrn@me.com>
parents: 4
diff changeset
308 postTranslate((w - oldBitmap.width).toFloat()/2.0f, (h - oldBitmap.height).toFloat()/2.0f)
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
309 }
10
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
310 binding.progressBar.visibility = ProgressBar.VISIBLE
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
311 ThreadPools.WORKERS.execute {
11
678adef4774f Move more stuff into worker threads.
David Barts <n5jrn@me.com>
parents: 10
diff changeset
312 val newBitmap = Bitmap.createBitmap(w, h, oldBitmap.config)
678adef4774f Move more stuff into worker threads.
David Barts <n5jrn@me.com>
parents: 10
diff changeset
313 copyColorSpace(oldBitmap, newBitmap)
10
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
314 Canvas(newBitmap).run {
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
315 drawBitmap(oldBitmap, rotater, null)
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
316 }
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
317 runOnUiThread {
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
318 binding.progressBar.visibility = ProgressBar.INVISIBLE
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
319 setImage(newBitmap)
5626557ac542 Add progress bar and a worker thread. Still way slow.
David Barts <n5jrn@me.com>
parents: 6
diff changeset
320 }
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
321 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
322 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
323
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
324 fun cancelClicked(view: View): Unit {
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
325 unsetImage()
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
326 makeMundane()
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
327 finish()
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
328 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
329
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
330 private fun makeFabulous(): Unit {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
331 binding.fabulous.visibility = FloatingActionButton.VISIBLE
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
332 viewModel.sharable = true
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
333 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
334
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
335 private fun makeMundane(): Unit {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
336 binding.fabulous.visibility = FloatingActionButton.GONE
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
337 viewModel.sharable = false
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
338 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
339
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
340 private val REQUEST_WRITE_EXTERNAL = 42
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
341
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
342 override fun onRequestPermissionsResult(requestCode: Int, permissions: Array<String>, grantResults: IntArray): Unit {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
343 if (requestCode != REQUEST_WRITE_EXTERNAL) {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
344 Log.e("EditImage", "unexpected request code in onRequestPermissionsResult!")
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
345 return
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
346 }
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
347 if ((grantResults.isNotEmpty() && grantResults[0] == PackageManager.PERMISSION_GRANTED)) {
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
348 val cb = viewModel.permissionsCallback
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
349 if (cb != null) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
350 viewModel.permissionsCallback = null
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
351 cb()
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
352 }
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
353 } else {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
354 showError(getString(R.string.error_unable_no_permissions))
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
355 }
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
356 }
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
357
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
358 private fun requestWritePermission(): Unit {
37
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
359 requestPermissions(arrayOf(Manifest.permission.WRITE_EXTERNAL_STORAGE), REQUEST_WRITE_EXTERNAL)
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
360 }
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
361
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
362 private fun needsWritePermission(callback: () -> Unit): Boolean {
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
363 if (android.os.Build.VERSION.SDK_INT < android.os.Build.VERSION_CODES.Q) {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
364 if (checkSelfPermission(Manifest.permission.WRITE_EXTERNAL_STORAGE) == PackageManager.PERMISSION_DENIED) {
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
365 viewModel.permissionsCallback = callback
37
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
366 if (shouldShowRequestPermissionRationale(Manifest.permission.WRITE_EXTERNAL_STORAGE))
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
367 AlertDialog.Builder(this).also {
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
368 it.setMessage(
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
369 getString(
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
370 R.string.permission_needed,
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
371 getString(R.string.app_name)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
372 )
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
373 )
37
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
374 it.setNeutralButton(R.string.ok_text) { dialog, _ ->
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
375 dialog.dismiss()
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
376 requestWritePermission()
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
377 }
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
378 it.create()
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
379 }.show()
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
380 else
0dbd924cb5e8 Explain why permission needed, per guidelines.
David Barts <n5jrn@me.com>
parents: 36
diff changeset
381 requestWritePermission()
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
382 return true
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
383 }
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
384 }
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
385 return false
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
386 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
387
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
388 fun shareClicked(view: View): Unit {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
389 // If we need WRITE_EXTERNAL_STORAGE, request it and bail. We will be called again
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
390 // (with the permission) if it is granted.
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
391 if (needsWritePermission({ shareClicked(view) }))
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
392 return
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
393
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
394 // If we get here, we have permission to save (if we need it).
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
395 val contentValues = makeContentValues(IMAGE_TO_SEND)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
396
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
397 // Delete any old file(s)
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
398 val cols = arrayOf<String>(MediaStore.MediaColumns.DISPLAY_NAME, MediaStore.MediaColumns.MIME_TYPE)
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
399 val query = StringBuilder().run {
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
400 for (col in cols) {
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
401 if (isNotEmpty())
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
402 append(" and ")
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
403 append(col)
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
404 append(" = ?")
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
405 }
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
406 toString()
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
407 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
408 try {
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
409 val cursor = contentResolver.query(
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
410 MediaStore.Images.Media.EXTERNAL_CONTENT_URI,
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
411 arrayOf<String>(MediaStore.MediaColumns._ID),
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
412 query,
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
413 cols.map { contentValues.getAsString(it) }.toTypedArray(),
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
414 null)
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
415 var deleted = 0
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
416 cursor?.use {
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
417 // Log.d("EditImage", "${it.count} entries matched")
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
418 while (it.moveToNext()) {
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
419 val uri =
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
420 ContentUris.withAppendedId(MediaStore.Images.Media.EXTERNAL_CONTENT_URI, it.getLong(0))
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
421 deleted += contentResolver.delete(uri, null, null)
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
422 }
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
423 }
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
424 // Log.d("EditImage", "$deleted entries deleted")
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
425 } catch (e: Exception) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
426 Log.e("EditImage", "unexpected exception when sharing!", e)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
427 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
428
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
429 // Save new file, use it to share data.
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
430 saveAs(contentValues) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
431 val shareIntent: Intent = Intent().apply {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
432 action = Intent.ACTION_SEND
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
433 putExtra(Intent.EXTRA_STREAM, it)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
434 type = "image/jpeg"
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
435 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
436 startActivity(Intent.createChooser(shareIntent, resources.getText(R.string.share_text)))
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
437 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
438 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
439
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
440 fun doneClicked(view: View): Unit {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
441 // If we need WRITE_EXTERNAL_STORAGE, request it and bail. We will be called again
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
442 // (with the permission) if it is granted.
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
443 if (needsWritePermission({ doneClicked(view) }))
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
444 return
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
445
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
446 // If we get here, we have permission to save (if we need it).
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
447 val image = viewModel.bitmap!!
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
448 var fileName = getFileName(viewModel.uri!!)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
449 if (fileName == null) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
450 val d = java.util.Date()
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
451 fileName = "IMG_%tY%tm%td_%tH%tM%tS".format(d, d, d, d, d, d)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
452 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
453 val dot = fileName.lastIndexOf('.')
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
454 if (dot != -1)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
455 fileName = fileName.substring(0, dot)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
456 fileName = "${fileName}_${image.width}x${image.height}.jpg"
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
457 saveAs(makeContentValues(fileName)) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
458 unsetImage()
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
459 makeMundane()
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
460 finish()
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
461 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
462 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
463
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
464 private fun makeContentValues(fileName: String): ContentValues = ContentValues().apply {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
465 put(MediaStore.MediaColumns.DISPLAY_NAME, fileName)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
466 put(MediaStore.MediaColumns.MIME_TYPE, "image/jpeg")
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
467 if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.Q) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
468 put(MediaStore.MediaColumns.RELATIVE_PATH,
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
469 File(Environment.DIRECTORY_PICTURES, getString(R.string.app_name)).path)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
470 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
471 }
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
472
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
473 private fun saveAs(contentValues: ContentValues, callback: (Uri) -> Unit) {
32
be08576794af Replace static object with ViewModel.
David Barts <n5jrn@me.com>
parents: 30
diff changeset
474 val image = viewModel.bitmap!!
25
648908f492c0 Fix output file names.
David Barts <n5jrn@me.com>
parents: 22
diff changeset
475 binding.progressBar.visibility = ProgressBar.VISIBLE
648908f492c0 Fix output file names.
David Barts <n5jrn@me.com>
parents: 22
diff changeset
476 ThreadPools.WORKERS.execute {
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
477 var errorMessage: String? = null
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
478 val myUri = try {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
479 contentResolver.insert(MediaStore.Images.Media.EXTERNAL_CONTENT_URI, contentValues)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
480 } catch(e: Exception) {
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
481 Log.e("EditImage", "unexpected exception when saving!", e)
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
482 null
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
483 }
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
484 try {
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
485 if (myUri == null) {
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
486 throw IOException(getString(R.string.error_create_mediastore))
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
487 }
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
488 val stream = contentResolver.openOutputStream(myUri)
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
489 if (stream == null) {
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
490 throw IOException(getString(R.string.error_get_output))
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
491 }
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
492 val quality = maxOf(0, minOf(100,
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
493 PreferenceManager.getDefaultSharedPreferences(applicationContext).getInt(
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
494 "jpeg_quality", 85)))
43
9cb9bb5da247 At long last it auto-deletes old cruft files.
David Barts <n5jrn@me.com>
parents: 42
diff changeset
495 // Log.d("EditImage", "saving, jpeg_quality = $quality")
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
496 stream.use {
25
648908f492c0 Fix output file names.
David Barts <n5jrn@me.com>
parents: 22
diff changeset
497 if (!image.compress(Bitmap.CompressFormat.JPEG, quality, it)) {
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
498 throw IOException(getString(R.string.error_save_bitmap))
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
499 }
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
500 }
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
501 } catch (ioe: IOException) {
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
502 errorMessage = ioe.message ?: getString(R.string.error_io)
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
503 } catch (se: SecurityException) {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
504 errorMessage = se.message ?: getString(R.string.error_security)
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
505 } catch (e: Exception) {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
506 Log.e("EditImage", "unexpected exception when saving!", e)
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
507 errorMessage = e.message ?: getString(R.string.error_unexpected, e::class.qualifiedName)
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
508 }
18
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
509 runOnUiThread {
eedf995462d9 Parallalized, but ConstraintLayout started hosing the edit window.
David Barts <n5jrn@me.com>
parents: 16
diff changeset
510 binding.progressBar.visibility = ProgressBar.INVISIBLE
30
aacf7a856b5f Bug fixes, incl for getting "stuck" when file chooser backed out of.
David Barts <n5jrn@me.com>
parents: 29
diff changeset
511 if (errorMessage == null) {
42
45e4df5226c0 Shares, but creates multiple versions of cruft file.
David Barts <n5jrn@me.com>
parents: 40
diff changeset
512 callback(myUri!!)
36
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
513 } else {
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
514 showError(errorMessage)
70f1d11d53ad Attempt to make it work on Android 9 and earlier.
David Barts <n5jrn@me.com>
parents: 33
diff changeset
515 }
2
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
516 }
06825e49f7aa Got it scaling and rotating, needs settings, etc.
David Barts <n5jrn@me.com>
parents: 1
diff changeset
517 }
0
13935000c2d9 Stuff Andriod Studio ignored for some reason.
David Barts <n5jrn@me.com>
parents:
diff changeset
518 }
1
f26f61a8a9ad Make the EditImage activity stateful.
David Barts <n5jrn@me.com>
parents: 0
diff changeset
519 }