diff --git a/modules/Calendar/actions/DragDropAjax.php b/modules/Calendar/actions/DragDropAjax.php
index b42597544ad66bc6d1916db1a9d6a5a6d7a5549f..f266a5b9f23455e577d3d7024a5a6de1c81a3e1e 100755
--- a/modules/Calendar/actions/DragDropAjax.php
+++ b/modules/Calendar/actions/DragDropAjax.php
@@ -267,7 +267,7 @@ class Calendar_DragDropAjax_Action extends Calendar_SaveAjax_Action {
 					$resultDateTime = $this->changeDateTime($oldEndDateTime,$dayDelta,$minuteDelta,$secondsDelta);
 					$parts = explode(' ',$resultDateTime);
 					$record->set('due_date',$parts[0]);
-					if(activitytype != 'Task') {
+					if($activitytype != 'Task') {
 						$record->set('time_end',$parts[1]);
 					}
 
diff --git a/modules/Migration/models/Module.php b/modules/Migration/models/Module.php
index b781bfc0970ed1f379a9f15567d136300c959b2e..51f9d991258ac3437a1f0f5ec3aeaddf7878406c 100644
--- a/modules/Migration/models/Module.php
+++ b/modules/Migration/models/Module.php
@@ -25,7 +25,7 @@ class Migration_Module_Model extends Vtiger_Module_Model {
 	 * Static Function to get the instance of Vtiger Module Model for the given id or name
 	 * @param mixed id or name of the module
 	 */
-	public static function getInstance($value='') {
+	public static function getInstance($value=null) {
 		return new self($value);
 	}
 	
diff --git a/modules/Reports/models/Record.php b/modules/Reports/models/Record.php
index cf8b793399d136e86dae4e7f4a434dcdea4cafd1..5862a1a7dcba0db9657bad293dab9539bb53d1f2 100644
--- a/modules/Reports/models/Record.php
+++ b/modules/Reports/models/Record.php
@@ -131,7 +131,7 @@ class Reports_Record_Model extends Vtiger_Record_Model {
 	 * @param <String> $module
 	 * @return <Reports_Record_Model>
 	 */
-	public static function getInstanceById($recordId) {
+	public static function getInstanceById($recordId, $module=null) {
 		$db = PearDatabase::getInstance();
 
 		$self = new self();