diff --git a/modules/Settings/Leads/models/Module.php b/modules/Settings/Leads/models/Module.php index 6a120d1f390dc5c491975e39035f7d768ed9eb93..3f65a55fa1ce1e94a4771198d91820a83d308ccf 100644 --- a/modules/Settings/Leads/models/Module.php +++ b/modules/Settings/Leads/models/Module.php @@ -33,7 +33,7 @@ class Settings_Leads_Module_Model extends Vtiger_Module_Model { * @return <Array> list of field ids */ public function getMappingSupportedFieldIdsList() { - if (!$this->supportedFieldIdsList) { + if (!property_exists($this,'supportedFieldIdsList') || !$this->supportedFieldIdsList) { $selectedTabidsList[] = getTabid($this->getName()); $restrictedFieldNames = array('campaignrelstatus'); $restrictedUitypes = $this->getRestrictedUitypes(); diff --git a/modules/Settings/Potentials/models/Mapping.php b/modules/Settings/Potentials/models/Mapping.php index 923dd6cef63756cd54cffe47c08ff6f5a0ad72e1..6b5edbac0e5ec6deec64bc7ada8071f1eebf3a1a 100644 --- a/modules/Settings/Potentials/models/Mapping.php +++ b/modules/Settings/Potentials/models/Mapping.php @@ -51,7 +51,7 @@ class Settings_Potentials_Mapping_Model extends Settings_Leads_Mapping_Model { * @return <Array> list of mapping details */ public function getMapping($editable = false) { - if (!$this->mapping) { + if (!property_exists($this,'mapping') || !$this->mapping) { $db = PearDatabase::getInstance(); $query = 'SELECT * FROM vtiger_convertpotentialmapping'; if ($editable) { @@ -95,7 +95,7 @@ class Settings_Potentials_Mapping_Model extends Settings_Leads_Mapping_Model { $db = PearDatabase::getInstance(); $deleteMappingsList = $updateMappingsList = $createMappingsList = array(); foreach ($mapping as $mappingDetails) { - $mappingId = $mappingDetails['mappingId']; + $mappingId = isset($mappingDetails['mappingId']) ? $mappingDetails['mappingId'] : ''; if ($mappingDetails['potential']) { if ($mappingId) { if ((array_key_exists('deletable', $mappingDetails)) || (!$mappingDetails['project'])) {