Minor fixes to 2 modules (GevondenCC_NL & Abandon_lookup) (FreePBX#294)

* Updated source-GevondenCC_NL.module - Changed parameter in debugprint to prevent error - Added additional check on result * Updated source-Abandon_lookup.module - Added check for parameter Use_CID
udo1toni · Mar 30, 2017 · d6d6800 · d6d6800
1 parent 3493cd1
commit d6d6800
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/sources/source-Abandon_lookup.module b/sources/source-Abandon_lookup.module
@@ -29,7 +29,7 @@ class Abandon_lookup extends superfecta_base {
 		if ($run_param['Artificial_CNAM'] == NULL) {
 			$run_param['Artificial_CNAM'] = $this->source_param['Artificial_CNAM']['default'];
 		}
-		if ($run_param['Use_CID'] == 'on') {
+		if (isset($run_param['Use_CID']) && $run_param['Use_CID'] == 'on') {
 			$run_param['Artificial_CNAM'] = $thenumber;
 		}
 

diff --git a/sources/source-GevondenCC_NL.module b/sources/source-GevondenCC_NL.module
@@ -16,7 +16,7 @@ class GevondenCC_NL extends superfecta_base {
 		$caller_id = null;
 		$name = "";
 
-		$this->DebugPrint("Searching gevonden.cc - {$this->thenumber} ... ");
+		$this->DebugPrint("Searching gevonden.cc - {$thenumber} ... ");
 
 		// Test for NL
 		$loc_number = $thenumber;
@@ -57,7 +57,7 @@ class GevondenCC_NL extends superfecta_base {
 		}
 
 		// If we found a match, return it
-		if (strlen($name) > 1)
+		if (is_string($name) && strlen($name) > 1)
 		{
 			$caller_id = $name;
 		} else {