Don't check for non-existing property -> fix address book

program/steps/addressbook/save.inc

@@ -28,7 +28,7 @@ if ($CONTACTS->readonly)
 }
 
 // check input
-if ((!get_input_value('_name', RCUBE_INPUT_POST) || !get_input_value('_email', RCUBE_INPUT_POST)) && $OUTPUT->action)
+if ((!get_input_value('_name', RCUBE_INPUT_POST) || !get_input_value('_email', RCUBE_INPUT_POST)))
 {
   $OUTPUT->show_message('formincomplete', 'warning');
   rcmail_overwrite_action(empty($_POST['_cid']) ? 'add' : 'show');
@@ -53,8 +53,6 @@ foreach ($a_save_cols as $col)
 if (!empty($cid))
 {
   if ($CONTACTS->update($cid, $a_record))
-  {
-    if ($OUTPUT->action)
   {
     // define list of cols to be displayed
     $a_js_cols = array();
@@ -65,7 +63,6 @@ if (!empty($cid))
 
     // update the changed col in list
     $OUTPUT->command('parent.update_contact_row', $cid, $a_js_cols);
-    }
       
     // show confirmation
     $OUTPUT->show_message('successfullysaved', 'confirmation');    
@@ -95,8 +92,6 @@ else
 
   // insert record and send response
   if ($insert_id = $CONTACTS->insert($a_record))
-  {
-    if ($OUTPUT->action)
   {
     // add contact row or jump to the page where it should appear
     $CONTACTS->reset();
@@ -108,7 +103,6 @@ else
     // update record count display
     $CONTACTS->reset();
     $OUTPUT->command('parent.set_rowcount', rcmail_get_rowcount_text());
-    }
 
     // show confirmation
     $OUTPUT->show_message('successfullysaved', 'confirmation');

program/steps/settings/save_identity.inc

@@ -76,7 +76,7 @@ if ($_POST['_iid'])
   }
 }
 
-// insert a new contact
+// insert a new identity record
 else
 {
   if ($insert_id = $USER->insert_identity($save_data))
@@ -87,12 +87,6 @@ else
 
     if (!empty($_POST['_standard']))
       $default_id = $insert_id;
-
-    if ($_POST['_framed'])
-    {
-      // add contact row or jump to the page where it should appear
-      // ....
-    }
   }
   else
   {
@@ -109,6 +103,6 @@ if ($default_id)
   $USER->set_default($default_id);
 
 // go to next step
-rcmail_overwrite_action($OUTPUT->action ? 'edit-identity' : 'identities');
+rcmail_overwrite_action('identities');
 
 ?>