diff --git a/CRM/Member/Form/MembershipConfig.php b/CRM/Member/Form/MembershipConfig.php
index 557023675bd4..a4d56edef52b 100644
--- a/CRM/Member/Form/MembershipConfig.php
+++ b/CRM/Member/Form/MembershipConfig.php
@@ -69,7 +69,6 @@ public function preProcess() {
    * Set default values for the form. MobileProvider that in edit/view mode
    * the default values are retrieved from the database
    *
-   *
    * @return array
    *   defaults
    */
diff --git a/CRM/Member/Form/MembershipType.php b/CRM/Member/Form/MembershipType.php
index 135ebb044941..2de58a9c4cd8 100644
--- a/CRM/Member/Form/MembershipType.php
+++ b/CRM/Member/Form/MembershipType.php
@@ -29,7 +29,6 @@
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2018
- * $Id$
  *
  */
 
@@ -61,8 +60,8 @@ public function preProcess() {
    * Set default values for the form. MobileProvider that in edit/view mode
    * the default values are retrieved from the database
    *
-   *
-   * @return void
+   * @return array
+   *   defaults
    */
   public function setDefaultValues() {
     $defaults = parent::setDefaultValues();
@@ -111,7 +110,6 @@ public function setDefaultValues() {
    * @return void
    * @throws \CRM_Core_Exception
    * @throws \CiviCRM_API3_Exception
-   * @throws \HTML_QuickForm_Error
    */
   public function buildQuickForm() {
     parent::buildQuickForm();
@@ -330,9 +328,9 @@ public function postProcess() {
         'max_related',
       );
 
-      $params = $ids = array();
+      $params = array();
       foreach ($fields as $fld) {
-        $params[$fld] = CRM_Utils_Array::value($fld, $submitted, 'NULL');
+        $params[$fld] = CRM_Utils_Array::value($fld, $submitted, 'null');
       }
 
       if ($params['minimum_fee']) {
@@ -360,7 +358,7 @@ public function postProcess() {
         }
       }
       if (!$hasRelTypeVal) {
-        $params['relationship_type_id'] = $params['relationship_direction'] = $params['max_related'] = 'NULL';
+        $params['relationship_type_id'] = $params['relationship_direction'] = $params['max_related'] = 'null';
       }
 
       if ($params['duration_unit'] == 'lifetime' &&
@@ -370,20 +368,20 @@ public function postProcess() {
       }
 
       $periods = array('fixed_period_start_day', 'fixed_period_rollover_day');
-      foreach ($periods as $per) {
-        if (!empty($params[$per]['M']) && !empty($params[$per]['d'])) {
-          $mon = $params[$per]['M'];
-          $dat = $params[$per]['d'];
+      foreach ($periods as $period) {
+        if (!empty($params[$period]['M']) && !empty($params[$period]['d'])) {
+          $mon = $params[$period]['M'];
+          $dat = $params[$period]['d'];
           $mon = ($mon < 10) ? '0' . $mon : $mon;
           $dat = ($dat < 10) ? '0' . $dat : $dat;
-          $params[$per] = $mon . $dat;
+          $params[$period] = $mon . $dat;
         }
-        elseif ($per == 'fixed_period_rollover_day' && !empty($params['month_fixed_period_rollover_day'])) {
+        elseif ($period == 'fixed_period_rollover_day' && !empty($params['month_fixed_period_rollover_day'])) {
           $params['fixed_period_rollover_day'] = $params['month_fixed_period_rollover_day']['d'];
           unset($params['month_fixed_period_rollover_day']);
         }
         else {
-          $params[$per] = 'NULL';
+          $params[$period] = 'null';
         }
       }
       $oldWeight = NULL;
@@ -398,10 +396,10 @@ public function postProcess() {
       );
 
       if ($this->_action & CRM_Core_Action::UPDATE) {
-        $ids['membershipType'] = $this->_id;
+        $params['id'] = $this->_id;
       }
 
-      $membershipType = CRM_Member_BAO_MembershipType::add($params, $ids);
+      $membershipType = CRM_Member_BAO_MembershipType::add($params);
 
       CRM_Core_Session::setStatus(ts('The membership type \'%1\' has been saved.',
         array(1 => $membershipType->name)