commit 9f815574feb0ea9c53dfd6b2ee6ca85e6f8c01a4
author Isaac Katzenelson <isaack@android.com> Thu Aug 25 10:37:47 2011 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Thu Aug 25 10:37:47 2011 -0700
tree 6f1ebe9f7918bfb0bea544d12247c038e4df0e01
parent 2f05a8501e901ce0b8220a44b9db74f8caa9436d
parent e92a978f8d3b98364abae85559e547659de1bc58

Merge "Remove group editing for readonly groups"

src/com/android/contacts/group/GroupDetailFragment.java

diff --git a/src/com/android/contacts/group/GroupDetailFragment.java b/src/com/android/contacts/group/GroupDetailFragment.java
index 1164069..f54dbd6 100644
--- a/src/com/android/contacts/group/GroupDetailFragment.java
+++ b/src/com/android/contacts/group/GroupDetailFragment.java
@@ -403,10 +403,9 @@
         mOptionsMenuEditable = isGroupEditable() && isVisible();
         mOptionsMenuGroupPresent = isGroupPresent() && isVisible();
 
-        // Editing a group is always possible if a group is selected
-        // TODO: check for external group (member editable) buganizer #5049046
+        // Editing is not possible for read only groups
         final MenuItem editMenu = menu.findItem(R.id.menu_edit_group);
-        editMenu.setVisible(mOptionsMenuGroupPresent);
+        editMenu.setVisible(mOptionsMenuGroupPresent && mOptionsMenuEditable);
 
         final MenuItem deleteMenu = menu.findItem(R.id.menu_delete_group);
         deleteMenu.setVisible(mOptionsMenuEditable);