diff --git a/modules/Emails/models/Record.php b/modules/Emails/models/Record.php
index a4da3a8d4c9026d1a760193c580b9bc7147637d4..000502348ca58aaa145f360cdc5fc6929e226b5e 100644
--- a/modules/Emails/models/Record.php
+++ b/modules/Emails/models/Record.php
@@ -123,7 +123,7 @@ class Emails_Record_Model extends Vtiger_Record_Model {
 			}
 
 			if (strpos($description, '$logo$')) {
-				$description = str_replace('$logo$',"<img src='cid:logo' />", $description);
+				$description = str_replace('$logo$',"<img src='cid:companyLogo' />", $description);
 				$logo = true;
 			}
 
@@ -161,8 +161,10 @@ class Emails_Record_Model extends Vtiger_Record_Model {
 					}
 				}
 				if ($logo) {
+					$companyDetails = Vtiger_CompanyDetails_Model::getInstanceById();
+					$companyLogoDetails = $companyDetails->getLogo();
 					//While sending email template and which has '$logo$' then it should replace with company logo
-					$mailer->AddEmbeddedImage(dirname(__FILE__).'/../../../layouts/v7/skins/images/logo_mail.jpg', 'logo', 'logo.jpg', 'base64', 'image/jpg');
+					$mailer->AddEmbeddedImage($companyLogoDetails->get('imagepath'), 'companyLogo', 'attachment', 'base64', 'image/jpg');
 				}
 
 				$ccs = array_filter(explode(',',$this->get('ccmail')));
diff --git a/packages/vtiger/optional/EmailTemplates.zip b/packages/vtiger/optional/EmailTemplates.zip
index ca8d54b0e7006ebcc13d712a1ff5b4ac00b7a88a..7067bee08933e07fc57493687e2390933deb1422 100644
Binary files a/packages/vtiger/optional/EmailTemplates.zip and b/packages/vtiger/optional/EmailTemplates.zip differ
diff --git a/pkg/vtiger/modules/EmailTemplates/modules/EmailTemplates/models/Module.php b/pkg/vtiger/modules/EmailTemplates/modules/EmailTemplates/models/Module.php
index 672e3d1d796384b1f6f1db52ad70fceee931e075..6e3e1dd3c16adcc747735429e810963f3653efb6 100644
--- a/pkg/vtiger/modules/EmailTemplates/modules/EmailTemplates/models/Module.php
+++ b/pkg/vtiger/modules/EmailTemplates/modules/EmailTemplates/models/Module.php
@@ -46,7 +46,7 @@ class EmailTemplates_Module_Model extends Vtiger_Module_Model {
 			if($systemtemplate) {
 				$sql = "UPDATE vtiger_emailtemplates SET templatename=?, description=?, module=?, body=?, deleted=?, systemtemplate=? WHERE templateid = ?";
 			} else {
-				$sql = "UPDATE vtiger_emailtemplates SET templatename=?, subject=?,  description=?, module=?, body=?, deleted=?, systemtemplate=?  WHERE templateid = ?";
+				$sql = "UPDATE vtiger_emailtemplates SET templatename=?, subject=?, description=?, module=?, body=?, deleted=?, systemtemplate=? WHERE templateid = ?";
 			}
 		}
 		if(!empty($recordId) && $systemtemplate) {
@@ -138,11 +138,11 @@ class EmailTemplates_Module_Model extends Vtiger_Module_Model {
 		$logoPath = $site_URL . '/' . $companyModuleModel->getLogoPath(); 
 		foreach ($basicFields as $columnName => $value) { 
 			//For column logo we need place logo in content 
-			if($columnName == 'logo'){ 
-					$allFields[] = array($moduleName.':'. vtranslate($columnName, $qualifiedModule),"<img src='".$logoPath."'>"); 
-			} else { 
-					$allFields[] = array($moduleName.':'. vtranslate($columnName, $qualifiedModule),"$".strtolower("companydetails")."-".$columnName."$"); 
-			} 
+			if($columnName == 'logo'){
+				$allFields[] = array($moduleName.':'. vtranslate($columnName, $qualifiedModule),"$$columnName$");
+			} else {
+				$allFields[] = array($moduleName.':'. vtranslate($columnName, $qualifiedModule),"$".strtolower("companydetails")."-".$columnName."$");
+			}
 		} 
 		// Social links will be having hyperlink redirected to URL mentioned 
 		foreach($socialFields as $columnName => $value){ 
@@ -188,8 +188,8 @@ class EmailTemplates_Module_Model extends Vtiger_Module_Model {
 		$moduleFields = $meta->getModuleFields();
 		$db = PearDatabase::getInstance();
 		//adding record id merge tag option 
-		$fieldInfo = array('columnname' => 'id','fieldname' => 'id','fieldlabel' =>vtranslate('LBL_RECORD_ID',  $this->getName()));
-		$recordIdField = WebserviceField::fromArray($db, $fieldInfo); 
+		$fieldInfo = array('columnname' => 'id','fieldname' => 'id','fieldlabel' =>vtranslate('LBL_RECORD_ID', $this->getName()));
+		$recordIdField = WebserviceField::fromArray($db, $fieldInfo);
 		$moduleFields[$recordIdField->getFieldName()] = $recordIdField;
 
 		$returnData = array();
@@ -233,7 +233,7 @@ class EmailTemplates_Module_Model extends Vtiger_Module_Model {
 				continue;
 			}
 			if($relModule == 'Users') {
-								if(in_array($relModuleField->getFieldDataType(),array('string','phone','email','text'))) {
+				if(in_array($relModuleField->getFieldDataType(),array('string','phone','email','text'))) {
 					$skipFields = array(98,115,116,31,32);
 					if(!in_array($relModuleField->getUIType(), $skipFields) && $relModuleField->getFieldName() != 'asterisk_extension'){
 						$relModuleFieldList[] = array('module' => $relModule, 'fieldname' => $relModuleField->getFieldName(), 'columnname' => $relModuleField->getColumnName(), 'fieldlabel' => $relModuleField->getFieldLabelKey());
@@ -254,8 +254,8 @@ class EmailTemplates_Module_Model extends Vtiger_Module_Model {
 		$db = PearDatabase::getInstance();
 		// Get modules names only those are active
 		$query = 'SELECT DISTINCT(name) AS modulename FROM vtiger_tab 
-				  LEFT JOIN vtiger_field ON vtiger_field.tabid = vtiger_tab.tabid
-				  WHERE (vtiger_field.uitype = ? AND vtiger_tab.presence = ?) ';
+					LEFT JOIN vtiger_field ON vtiger_field.tabid = vtiger_tab.tabid
+					WHERE (vtiger_field.uitype = ? AND vtiger_tab.presence = ?) ';
 		$params = array('13',0);
 		// Check whether calendar module is active or not.
 		if(vtlib_isModuleActive("Calendar")){