Skip to content

Commit 418f6c1

Browse files
author
Chris Bellew
committed
Show title for select client/server dialogs.
1 parent ed10a38 commit 418f6c1

File tree

3 files changed

+23
-52
lines changed

3 files changed

+23
-52
lines changed

mobile/src/main/java/com/atomjack/vcfp/activities/MainActivity.java

Lines changed: 8 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -420,59 +420,27 @@ public void onItemClick(AdapterView<?> adapter, View view, int position,
420420
return;
421421
}
422422

423-
searchDialog = new Dialog(MainActivity.this);
423+
LayoutInflater inflater = getLayoutInflater();
424+
View layout = inflater.inflate(R.layout.search_popup, null);
424425

425-
searchDialog.setContentView(R.layout.search_popup);
426-
searchDialog.setTitle(getResources().getString(R.string.searching_for_plex_servers));
426+
AlertDialog.Builder builder = new AlertDialog.Builder(MainActivity.this);
427+
428+
builder.setView(layout);
429+
builder.setTitle(getResources().getString(R.string.searching_for_plex_servers));
427430

428-
searchDialog.setOnCancelListener(new DialogInterface.OnCancelListener() {
431+
builder.setOnCancelListener(new DialogInterface.OnCancelListener() {
429432
@Override
430433
public void onCancel(DialogInterface dialogInterface) {
431434
serverScanCanceled = true;
432-
// RemoteScan.cancelScan();
433435
}
434436
});
437+
searchDialog = builder.create();
435438
searchDialog.show();
436439
Intent scannerIntent = new Intent(MainActivity.this, PlexScannerService.class);
437440
scannerIntent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
438441
scannerIntent.putExtra(PlexScannerService.CLASS, MainActivity.class);
439442
scannerIntent.setAction(PlexScannerService.ACTION_SCAN_SERVERS);
440443
startService(scannerIntent);
441-
442-
/*
443-
if(authToken != null) {
444-
445-
446-
RemoteScan.refreshResources(authToken, new RemoteScan.RefreshResourcesResponseHandler() {
447-
@Override
448-
public void onSuccess() {
449-
// Finished getting servers from plex.tv. If no wifi connection is detected, just show the servers we found.
450-
if(currentNetworkState.equals(NetworkState.MOBILE)) {
451-
searchDialog.dismiss();
452-
localScan.showPlexServers();
453-
} else
454-
localScan.searchForPlexServers();
455-
}
456-
457-
@Override
458-
public void onFailure(int statusCode) {
459-
if(statusCode == 401) {
460-
authToken = null;
461-
VoiceControlForPlexApplication.getInstance().prefs.remove(Preferences.AUTHENTICATION_TOKEN);
462-
feedback.e(R.string.login_unauthorized);
463-
switchLogin();
464-
} else
465-
feedback.e(R.string.remote_scan_error);
466-
}
467-
});
468-
} else {
469-
Logger.d("not logged in");
470-
if(currentNetworkState.equals(NetworkState.MOBILE)) {
471-
feedback.e(R.string.mobile_connection_login_required);
472-
} else
473-
localScan.searchForPlexServers();
474-
}
475-
*/
476444
} else if (holder.tag.equals(holder.TAG_FEEDBACK)) {
477445
selectFeedback();
478446
} else if (holder.tag.equals(holder.TAG_ERRORS)) {

mobile/src/main/java/com/atomjack/vcfp/activities/VCFPActivity.java

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -860,15 +860,11 @@ public void showPlexServers(ConcurrentHashMap<String, PlexServer> servers, final
860860
cancelScan = false;
861861
return;
862862
}
863-
// if(searchDialog != null)
864-
// searchDialog.dismiss();
865-
if(deviceSelectDialog == null) {
866-
deviceSelectDialog = new Dialog(this);
867-
}
868-
deviceSelectDialog.setContentView(R.layout.server_select);
869-
deviceSelectDialog.setTitle("Select a Plex Server");
863+
864+
deviceSelectDialog = getDeviceSelectDialog(getResources().getString(R.string.select_plex_server));
870865
deviceSelectDialog.show();
871866

867+
872868
final ListView serverListView = (ListView) deviceSelectDialog.findViewById(R.id.serverListView);
873869
if(servers == null)
874870
servers = new ConcurrentHashMap<String, PlexServer>(VoiceControlForPlexApplication.servers);
@@ -887,6 +883,15 @@ public void onItemClick(AdapterView<?> parentAdapter, View view, int position, l
887883
});
888884
}
889885

886+
public Dialog getDeviceSelectDialog(String title) {
887+
AlertDialog.Builder builder = new AlertDialog.Builder(this);
888+
LayoutInflater inflater = getLayoutInflater();
889+
View layout = inflater.inflate(R.layout.server_select, null);
890+
builder.setView(layout);
891+
builder.setTitle(title);
892+
return builder.create();
893+
}
894+
890895
public void showPlexClients() {
891896
showPlexClients(false, null);
892897
}
@@ -901,11 +906,8 @@ public void showPlexClients(boolean showResume, final ScanHandler onFinish) {
901906
cancelScan = false;
902907
return;
903908
}
904-
if (deviceSelectDialog == null) {
905-
deviceSelectDialog = new Dialog(this);
906-
}
907-
deviceSelectDialog.setContentView(R.layout.server_select);
908-
deviceSelectDialog.setTitle(R.string.select_plex_client);
909+
deviceSelectDialog = getDeviceSelectDialog(getString(R.string.select_plex_client));
910+
909911
deviceSelectDialog.setOnCancelListener(new DialogInterface.OnCancelListener() {
910912
@Override
911913
public void onCancel(DialogInterface dialogInterface) {

mobile/src/main/res/values/strings.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,4 +152,5 @@
152152
<string name="login_incorrect">Sorry, but your username or password is incorrect.</string>
153153
<string name="email_device_logs_write_storage_denied">Permission to write to the device\'s external storage is required in order to email device logs.</string>
154154
<string name="video_attempted_on_audio_only_device">Sorry, but video playback is not supported on %s.</string>
155+
<string name="select_plex_server">Select a Plex Server</string>
155156
</resources>

0 commit comments

Comments
 (0)