mirror of
https://github.com/commons-app/apps-android-commons.git
synced 2025-10-26 20:33:53 +01:00
Fixed null ptr issue on changing modes
This commit is contained in:
parent
0051dcc9c0
commit
ca1bf88127
1 changed files with 67 additions and 41 deletions
|
|
@ -95,13 +95,20 @@ class UploadProgressActivity : BaseActivity() {
|
|||
}
|
||||
|
||||
fun updateMenuItems(currentPosition: Int) {
|
||||
if (menu != null) {
|
||||
menu!!.clear()
|
||||
if (currentPosition == 0) {
|
||||
if (isPendingIconsVisible){
|
||||
if (!isPaused){
|
||||
if (isPendingIconsVisible) {
|
||||
if (!isPaused) {
|
||||
if (menu!!.findItem(R.id.pause_icon) == null) {
|
||||
menu!!.add(Menu.NONE, R.id.pause_icon, Menu.NONE, getString(R.string.pause))
|
||||
.setIcon(android.R.drawable.ic_media_pause).setOnMenuItemClickListener {
|
||||
menu!!.add(
|
||||
Menu.NONE,
|
||||
R.id.pause_icon,
|
||||
Menu.NONE,
|
||||
getString(R.string.pause)
|
||||
)
|
||||
.setIcon(android.R.drawable.ic_media_pause)
|
||||
.setOnMenuItemClickListener {
|
||||
pendingUploadsFragment!!.pauseUploads()
|
||||
setPausedIcon(true)
|
||||
true
|
||||
|
|
@ -109,17 +116,29 @@ class UploadProgressActivity : BaseActivity() {
|
|||
.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
|
||||
}
|
||||
if (menu!!.findItem(R.id.cancel_icon) == null) {
|
||||
menu!!.add(Menu.NONE, R.id.cancel_icon, Menu.NONE, getString(R.string.cancel))
|
||||
.setIcon(android.R.drawable.ic_menu_close_clear_cancel).setOnMenuItemClickListener {
|
||||
menu!!.add(
|
||||
Menu.NONE,
|
||||
R.id.cancel_icon,
|
||||
Menu.NONE,
|
||||
getString(R.string.cancel)
|
||||
)
|
||||
.setIcon(android.R.drawable.ic_menu_close_clear_cancel)
|
||||
.setOnMenuItemClickListener {
|
||||
pendingUploadsFragment!!.deleteUploads()
|
||||
true
|
||||
}
|
||||
.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
|
||||
}
|
||||
}else{
|
||||
} else {
|
||||
if (menu!!.findItem(R.id.resume_icon) == null) {
|
||||
menu!!.add(Menu.NONE, R.id.resume_icon, Menu.NONE, getString(R.string.resume))
|
||||
.setIcon(android.R.drawable.ic_media_play).setOnMenuItemClickListener {
|
||||
menu!!.add(
|
||||
Menu.NONE,
|
||||
R.id.resume_icon,
|
||||
Menu.NONE,
|
||||
getString(R.string.resume)
|
||||
)
|
||||
.setIcon(android.R.drawable.ic_media_play)
|
||||
.setOnMenuItemClickListener {
|
||||
pendingUploadsFragment!!.restartUploads()
|
||||
setPausedIcon(false)
|
||||
true
|
||||
|
|
@ -129,7 +148,7 @@ class UploadProgressActivity : BaseActivity() {
|
|||
}
|
||||
}
|
||||
} else if (currentPosition == 1) {
|
||||
if (isErrorIconsVisisble){
|
||||
if (isErrorIconsVisisble) {
|
||||
if (menu!!.findItem(R.id.retry_icon) == null) {
|
||||
menu!!.add(Menu.NONE, R.id.retry_icon, Menu.NONE, getString(R.string.retry))
|
||||
.setIcon(R.drawable.ic_refresh_white_24dp).setOnMenuItemClickListener {
|
||||
|
|
@ -139,8 +158,14 @@ class UploadProgressActivity : BaseActivity() {
|
|||
.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
|
||||
}
|
||||
if (menu!!.findItem(R.id.cancel_icon) == null) {
|
||||
menu!!.add(Menu.NONE, R.id.cancel_icon, Menu.NONE, getString(R.string.cancel))
|
||||
.setIcon(android.R.drawable.ic_menu_close_clear_cancel).setOnMenuItemClickListener {
|
||||
menu!!.add(
|
||||
Menu.NONE,
|
||||
R.id.cancel_icon,
|
||||
Menu.NONE,
|
||||
getString(R.string.cancel)
|
||||
)
|
||||
.setIcon(android.R.drawable.ic_menu_close_clear_cancel)
|
||||
.setOnMenuItemClickListener {
|
||||
failedUploadsFragment!!.deleteUploads()
|
||||
true
|
||||
}
|
||||
|
|
@ -149,6 +174,7 @@ class UploadProgressActivity : BaseActivity() {
|
|||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
fun hidePendingIcons() {
|
||||
isPendingIconsVisible = false
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue