Skip to content

Commit 669aa40

Browse files
authored
One more follow-up to template flow. (wikimedia#4547)
1 parent 76db33e commit 669aa40

File tree

4 files changed

+12
-28
lines changed

4 files changed

+12
-28
lines changed

app/src/main/java/org/wikipedia/edit/EditSectionActivity.kt

+1-11
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ import org.wikipedia.edit.insertmedia.InsertMediaViewModel
4949
import org.wikipedia.edit.preview.EditPreviewFragment
5050
import org.wikipedia.edit.richtext.SyntaxHighlighter
5151
import org.wikipedia.edit.summaries.EditSummaryFragment
52-
import org.wikipedia.edit.templates.TemplatesSearchActivity
5352
import org.wikipedia.extensions.parcelableExtra
5453
import org.wikipedia.history.HistoryEntry
5554
import org.wikipedia.login.LoginActivity
@@ -155,15 +154,6 @@ class EditSectionActivity : BaseActivity(), ThemeChooserDialog.Callback, EditPre
155154
}
156155
}
157156

158-
private val requestInsertTemplate = registerForActivityResult(ActivityResultContracts.StartActivityForResult()) {
159-
if (it.resultCode == TemplatesSearchActivity.RESULT_INSERT_TEMPLATE_SUCCESS) {
160-
it.data?.let { data ->
161-
val newWikiText = data.getStringExtra(TemplatesSearchActivity.RESULT_WIKI_TEXT)
162-
binding.editSectionText.inputConnection?.commitText(newWikiText, 1)
163-
}
164-
}
165-
}
166-
167157
private val editTokenThenSave: Unit
168158
get() {
169159
cancelCalls()
@@ -240,7 +230,7 @@ class EditSectionActivity : BaseActivity(), ThemeChooserDialog.Callback, EditPre
240230

241231
SyntaxHighlightViewAdapter(this, pageTitle, binding.root, binding.editSectionText,
242232
binding.editKeyboardOverlay, binding.editKeyboardOverlayFormatting, binding.editKeyboardOverlayHeadings,
243-
Constants.InvokeSource.EDIT_ACTIVITY, requestInsertMedia, requestInsertTemplate)
233+
Constants.InvokeSource.EDIT_ACTIVITY, requestInsertMedia)
244234

245235
binding.editSectionText.setOnClickListener { finishActionMode() }
246236
onEditingPrefsChanged()

app/src/main/java/org/wikipedia/edit/SyntaxHighlightViewAdapter.kt

+9-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ class SyntaxHighlightViewAdapter(
2828
private val wikiTextKeyboardHeadingsView: WikiTextKeyboardHeadingsView,
2929
private val invokeSource: Constants.InvokeSource,
3030
private val requestInsertMedia: ActivityResultLauncher<Intent>,
31-
private val requestInsertTemplate: ActivityResultLauncher<Intent>,
3231
showUserMention: Boolean = false
3332
) : WikiTextKeyboardView.Callback {
3433

@@ -63,6 +62,15 @@ class SyntaxHighlightViewAdapter(
6362
}
6463
}
6564

65+
private val requestInsertTemplate = activity.registerForActivityResult(ActivityResultContracts.StartActivityForResult()) {
66+
if (it.resultCode == TemplatesSearchActivity.RESULT_INSERT_TEMPLATE_SUCCESS) {
67+
it.data?.let { data ->
68+
val newWikiText = data.getStringExtra(TemplatesSearchActivity.RESULT_WIKI_TEXT)
69+
editText.inputConnection?.commitText(newWikiText, 1)
70+
}
71+
}
72+
}
73+
6674
override fun onPreviewLink(title: String) {
6775
val dialog = LinkPreviewDialog.newInstance(HistoryEntry(PageTitle(title, pageTitle.wikiSite), HistoryEntry.SOURCE_INTERNAL_LINK))
6876
ExclusiveBottomSheetPresenter.show(activity.supportFragmentManager, dialog)

app/src/main/java/org/wikipedia/talk/TalkReplyActivity.kt

+1-11
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ import org.wikipedia.edit.SyntaxHighlightViewAdapter
2828
import org.wikipedia.edit.insertmedia.InsertMediaActivity
2929
import org.wikipedia.edit.insertmedia.InsertMediaViewModel
3030
import org.wikipedia.edit.preview.EditPreviewFragment
31-
import org.wikipedia.edit.templates.TemplatesSearchActivity
3231
import org.wikipedia.extensions.parcelableExtra
3332
import org.wikipedia.history.HistoryEntry
3433
import org.wikipedia.login.LoginActivity
@@ -108,15 +107,6 @@ class TalkReplyActivity : BaseActivity(), UserMentionInputView.Listener, EditPre
108107
}
109108
}
110109

111-
private val requestInsertTemplate = registerForActivityResult(ActivityResultContracts.StartActivityForResult()) {
112-
if (it.resultCode == TemplatesSearchActivity.RESULT_INSERT_TEMPLATE_SUCCESS) {
113-
it.data?.let { data ->
114-
val newWikiText = data.getStringExtra(TemplatesSearchActivity.RESULT_WIKI_TEXT)
115-
binding.replyInputView.editText.inputConnection?.commitText(newWikiText, 1)
116-
}
117-
}
118-
}
119-
120110
public override fun onCreate(savedInstanceState: Bundle?) {
121111
super.onCreate(savedInstanceState)
122112
binding = ActivityTalkReplyBinding.inflate(layoutInflater)
@@ -190,7 +180,7 @@ class TalkReplyActivity : BaseActivity(), UserMentionInputView.Listener, EditPre
190180

191181
SyntaxHighlightViewAdapter(this, viewModel.pageTitle, binding.root, binding.replyInputView.editText,
192182
binding.editKeyboardOverlay, binding.editKeyboardOverlayFormatting, binding.editKeyboardOverlayHeadings,
193-
Constants.InvokeSource.TALK_REPLY_ACTIVITY, requestInsertMedia, requestInsertTemplate, true)
183+
Constants.InvokeSource.TALK_REPLY_ACTIVITY, requestInsertMedia, true)
194184

195185
messagePreviewFragment = supportFragmentManager.findFragmentById(R.id.message_preview_fragment) as EditPreviewFragment
196186

app/src/main/java/org/wikipedia/talk/template/AddTemplateActivity.kt

+1-5
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,6 @@ class AddTemplateActivity : BaseActivity(), UserMentionInputView.Listener {
6464
}
6565
}
6666

67-
private val requestInsertTemplate = registerForActivityResult(ActivityResultContracts.StartActivityForResult()) {
68-
// TODO: implement this
69-
}
70-
7167
public override fun onCreate(savedInstanceState: Bundle?) {
7268
super.onCreate(savedInstanceState)
7369
binding = ActivityAddTemplateBinding.inflate(layoutInflater)
@@ -93,7 +89,7 @@ class AddTemplateActivity : BaseActivity(), UserMentionInputView.Listener {
9389

9490
SyntaxHighlightViewAdapter(this, PageTitle("Main Page", wikiSite), binding.root, binding.addTemplateInputView.editText,
9591
binding.editKeyboardOverlay, binding.editKeyboardOverlayFormatting, binding.editKeyboardOverlayHeadings,
96-
Constants.InvokeSource.ADD_TEMPLATE_ACTIVITY, requestInsertMedia, requestInsertTemplate, true)
92+
Constants.InvokeSource.ADD_TEMPLATE_ACTIVITY, requestInsertMedia, true)
9793

9894
lifecycleScope.launch {
9995
repeatOnLifecycle(Lifecycle.State.CREATED) {

0 commit comments

Comments
 (0)