diff options
Diffstat (limited to 'app/src/main/java/github/daneren2005/dsub/fragments/AdminFragment.java')
-rw-r--r-- | app/src/main/java/github/daneren2005/dsub/fragments/AdminFragment.java | 25 |
1 files changed, 9 insertions, 16 deletions
diff --git a/app/src/main/java/github/daneren2005/dsub/fragments/AdminFragment.java b/app/src/main/java/github/daneren2005/dsub/fragments/AdminFragment.java index 552712f7..dcd3d16a 100644 --- a/app/src/main/java/github/daneren2005/dsub/fragments/AdminFragment.java +++ b/app/src/main/java/github/daneren2005/dsub/fragments/AdminFragment.java @@ -37,7 +37,6 @@ import github.daneren2005.dsub.adapter.UserAdapter; import github.daneren2005.dsub.view.UpdateView; public class AdminFragment extends SelectRecyclerFragment<User> { - private static String TAG = AdminFragment.class.getSimpleName(); @Override public boolean onOptionsItemSelected(MenuItem item) { @@ -45,10 +44,8 @@ public class AdminFragment extends SelectRecyclerFragment<User> { return true; } - switch (item.getItemId()) { - case R.id.menu_add_user: - UserUtil.addNewUser(context, this, (objects.size() > 0) ? objects.get(0) : null); - break; + if (item.getItemId() == R.id.menu_add_user) { + UserUtil.addNewUser(context, this, (objects.size() > 0) ? objects.get(0) : null); } return false; @@ -65,18 +62,14 @@ public class AdminFragment extends SelectRecyclerFragment<User> { @Override public boolean onContextItemSelected(MenuItem menuItem, UpdateView<User> updateView, User user) { - switch(menuItem.getItemId()) { - case R.id.admin_change_email: - UserUtil.changeEmail(context, user); - break; - case R.id.admin_change_password: - UserUtil.changePassword(context, user); - break; - case R.id.admin_delete_user: - UserUtil.deleteUser(context, user, adapter); - break; + int id = menuItem.getItemId(); + if (id == R.id.admin_change_email) { + UserUtil.changeEmail(context, user); + } else if (id == R.id.admin_change_password) { + UserUtil.changePassword(context, user); + } else if (id == R.id.admin_delete_user) { + UserUtil.deleteUser(context, user, adapter); } - return true; } |