diff --git a/modules/CustomView/CustomView.php b/modules/CustomView/CustomView.php index 0fc173362e6ff8ffaf96872de62d10c34920f6ff..27e5df9a6ec1c442db7619a51eff2e57f865cd2a 100644 --- a/modules/CustomView/CustomView.php +++ b/modules/CustomView/CustomView.php @@ -916,6 +916,8 @@ class CustomView extends CRMEntity { $advft_criteria = Vtiger_Cache::get('advftCriteria',$cvid); if(!$advft_criteria){ + $advft_criteria = array(); + // Not a good approach to get all the fields if not required(May leads to Performance issue) $sql = 'SELECT groupid,group_condition FROM vtiger_cvadvfilter_grouping WHERE cvid = ? ORDER BY groupid'; $groupsresult = $adb->pquery($sql, array($cvid)); diff --git a/modules/Install/views/Index.php b/modules/Install/views/Index.php index 8fb55655fe83a0de134e7785cdbd42cd3b43aef1..cc8cc56d055cadfcd6be17349a9bcebd61d35ea2 100644 --- a/modules/Install/views/Index.php +++ b/modules/Install/views/Index.php @@ -28,9 +28,12 @@ class Install_Index_view extends Vtiger_View_Controller { } protected function applyInstallFriendlyEnv() { - // config.inc.php - will not be ready to control this yet. - // version_compare(PHP_VERSION, '5.5.0') <= 0 ? error_reporting(E_ERROR & ~E_NOTICE & ~E_DEPRECATED) : error_reporting(E_ERROR & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT); // Production - // version_compare(PHP_VERSION, '7.0.0') >= 0 ? error_reporting(E_WARNING & ~E_NOTICE) : error_reporting(E_WARNING & ~E_NOTICE & ~E_DEPRECATED & E_ERROR & ~E_STRICT); // Debug + // config.inc.php - will not be ready to control this yet + // if overriden in dev mode skip changes. + if (error_reporting() != E_ALL) { + version_compare(PHP_VERSION, '5.5.0') <= 0 ? error_reporting(E_ERROR & ~E_NOTICE & ~E_DEPRECATED) : error_reporting(E_ERROR & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT); // Production + // version_compare(PHP_VERSION, '7.0.0') >= 0 ? error_reporting(E_WARNING & ~E_NOTICE) : error_reporting(E_WARNING & ~E_NOTICE & ~E_DEPRECATED & E_ERROR & ~E_STRICT); // Debug + } set_time_limit(0); // override limits on execution time to allow install to finish } diff --git a/modules/Leads/models/Module.php b/modules/Leads/models/Module.php index 91754b926895cc27ed0b187d0cac9e210a3dc730..31a2e4d3592dd89745619842c18fad9c7b197071 100644 --- a/modules/Leads/models/Module.php +++ b/modules/Leads/models/Module.php @@ -233,6 +233,8 @@ class Leads_Module_Model extends Vtiger_Module_Model { } $params = array(); + + $dateFilterSql = ''; if(!empty($dateFilter)) { $dateFilterSql = ' AND createdtime BETWEEN ? AND ? '; //appended time frame and converted to db time zone in showwidget.php diff --git a/modules/Migration/views/Index.php b/modules/Migration/views/Index.php index 23d0e09036b55a637500a5ad7fb215042a6626ec..6d83cf687e6461600c4263d18b5c8eb5b4cb0568 100644 --- a/modules/Migration/views/Index.php +++ b/modules/Migration/views/Index.php @@ -28,7 +28,10 @@ class Migration_Index_View extends Vtiger_View_Controller { public function process(Vtiger_Request $request) { // Override error reporting to production mode - // version_compare(PHP_VERSION, '5.5.0') <= 0 ? error_reporting(E_WARNING & ~E_NOTICE & ~E_DEPRECATED) : error_reporting(E_WARNING & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT); + // if not set to strict development. + if (error_reporting() != E_ALL) { + version_compare(PHP_VERSION, '5.5.0') <= 0 ? error_reporting(E_WARNING & ~E_NOTICE & ~E_DEPRECATED) : error_reporting(E_WARNING & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT); + } // Migration could be heavy at-times. set_time_limit(0); diff --git a/modules/Potentials/models/Module.php b/modules/Potentials/models/Module.php index cc90a4899688ce31c7bac35925c3c5cfd84a7231..8692527678d8bcbf9c3ef5018723c22185fdd3c9 100644 --- a/modules/Potentials/models/Module.php +++ b/modules/Potentials/models/Module.php @@ -56,6 +56,8 @@ class Potentials_Module_Model extends Vtiger_Module_Model { $ownerSql = ' AND smownerid = ? '; $params[] = $owner; } + + $dateFilterSql = ''; if(!empty($dateFilter)) { $dateFilterSql = ' AND closingdate BETWEEN ? AND ? '; $params[] = $dateFilter['start']; @@ -141,6 +143,8 @@ class Potentials_Module_Model extends Vtiger_Module_Model { 'INNER JOIN vtiger_sales_stage ON vtiger_potential.sales_stage = vtiger_sales_stage.sales_stage WHERE vtiger_potential.sales_stage IN ('.generateQuestionMarks($picklistvaluesmap).') GROUP BY smownerid, sales_stage ORDER BY vtiger_sales_stage.sortorderid', $params); + + $data = array(); for($i=0; $i<$db->num_rows($result); $i++) { $row = $db->query_result_rowdata($result, $i); $row['link'] = decode_html($row['sales_stage']); @@ -161,6 +165,8 @@ class Potentials_Module_Model extends Vtiger_Module_Model { //TODO need to handle security $params = array(); $params[] = 'Closed Won'; + + $dateFilterSql = ''; if(!empty($dateFilter)) { $dateFilterSql = ' AND createdtime BETWEEN ? AND ? '; //appended time frame and converted to db time zone in showwidget.php diff --git a/modules/Vtiger/dashboards/History.php b/modules/Vtiger/dashboards/History.php index b336e51851cb2456e952b35f450f8991124b5d36..94cc32ed0744e4c7feb78a181a17ab5cc6f56e6a 100644 --- a/modules/Vtiger/dashboards/History.php +++ b/modules/Vtiger/dashboards/History.php @@ -27,6 +27,7 @@ class Vtiger_History_Dashboard extends Vtiger_IndexAjax_View { $modifiedTime = $request->get('modifiedtime'); //Date conversion from user to database format + $dates = array(); if(!empty($modifiedTime)) { $startDate = Vtiger_Date_UIType::getDBInsertedValue($modifiedTime['start']); $dates['start'] = getValidDBInsertDateTimeValue($startDate . ' 00:00:00'); diff --git a/modules/Vtiger/helpers/Util.php b/modules/Vtiger/helpers/Util.php index 98207671bf02ded55deb76d23942c6adce25ae6f..f793b582a8da26151e4f1b4b239943921d711d4d 100644 --- a/modules/Vtiger/helpers/Util.php +++ b/modules/Vtiger/helpers/Util.php @@ -269,6 +269,7 @@ class Vtiger_Util_Helper { $dateTimeInUserFormat = Vtiger_Datetime_UIType::getDisplayDateTimeValue($dateTime); } + if (substr_count($dateTimeInUserFormat,' ') == 1) $dateTimeInUserFormat .= ' '; // init for meridiem if not present list($dateInUserFormat, $timeInUserFormat, $meridiem) = explode(' ', $dateTimeInUserFormat); if($meridiem && $currentUser->get('hour_format') === '12' ){ $displayTime = $timeInUserFormat.' '.$meridiem; diff --git a/modules/Vtiger/models/DashBoard.php b/modules/Vtiger/models/DashBoard.php index 9073eede05d68b25f4aeca99c0d36cd0e000d831..40acffb223f3f763b070cd06192db9de38c71550 100644 --- a/modules/Vtiger/models/DashBoard.php +++ b/modules/Vtiger/models/DashBoard.php @@ -118,7 +118,7 @@ class Vtiger_DashBoard_Model extends Vtiger_Base_Model { for($i=0, $len=$db->num_rows($result); $i<$len; $i++) { $row = $db->query_result_rowdata($result, $i); $data = json_decode(decode_html($row['data']), true); - $sourceModule = $data['module']; + $sourceModule = $data && isset($data['module']) ? $data['module'] : ""; if(!empty($sourceModule) && !vtlib_isModuleActive($sourceModule)) { continue; }