From 699007c53b3b29bf798088254abd107ce6bae549 Mon Sep 17 00:00:00 2001 From: Harsh Shandilya Date: Fri, 7 Jun 2019 12:13:14 +0530 Subject: PasswordStore: Clear some lint warnings and deprecated API Signed-off-by: Harsh Shandilya --- .../main/java/com/zeapo/pwdstore/PasswordStore.java | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) (limited to 'app/src/main') diff --git a/app/src/main/java/com/zeapo/pwdstore/PasswordStore.java b/app/src/main/java/com/zeapo/pwdstore/PasswordStore.java index 6bf52d2f..44182602 100644 --- a/app/src/main/java/com/zeapo/pwdstore/PasswordStore.java +++ b/app/src/main/java/com/zeapo/pwdstore/PasswordStore.java @@ -175,7 +175,7 @@ public class PasswordStore extends AppCompatActivity { // Inflate the menu; this adds items to the action bar if it is present. getMenuInflater().inflate(R.menu.main_menu, menu); searchItem = menu.findItem(R.id.action_search); - searchView = (SearchView) MenuItemCompat.getActionView(searchItem); + searchView = (SearchView) searchItem.getActionView(); searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() { @Override @@ -304,16 +304,21 @@ public class PasswordStore extends AppCompatActivity { PasswordRepository.initialize(this); } - File localDir = PasswordRepository.getRepositoryDirectory(getApplicationContext()); - - localDir.mkdir(); + final File localDir = PasswordRepository.getRepositoryDirectory(getApplicationContext()); try { + if (!localDir.mkdir()) + throw new IllegalStateException("Failed to create directory!"); PasswordRepository.createRepository(localDir); - new File(localDir.getAbsolutePath() + "/.gpg-id").createNewFile(); - settings.edit().putBoolean("repository_initialized", true).apply(); + if (new File(localDir.getAbsolutePath() + "/.gpg-id").createNewFile()) { + settings.edit().putBoolean("repository_initialized", true).apply(); + } else { + throw new IllegalStateException("Failed to initialize repository state."); + } } catch (Exception e) { e.printStackTrace(); - localDir.delete(); + if (!localDir.delete()) { + Log.d(TAG, "Failed to delete local repository"); + } return; } checkLocalRepository(); -- cgit v1.2.3