Filter only populated form properties

mailjet · Dec 13, 2018 · 940a755 · 940a755
1 parent 1491475
commit 940a755
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/widget/WP_Mailjet_Subscribe_Widget.php b/src/widget/WP_Mailjet_Subscribe_Widget.php
@@ -136,6 +136,9 @@ private function sendSubscriptionEmail($subscriptionOptionsSettings, $instance)
                             case "float":
                                 $propertyNameCopy = $propertyName;
                                 $fProperty = (float) $propertyNameCopy;
+                                if(!is_float($fProperty) || $fProperty == 0) {
+                                    return $incorectTypeValue;
+                                }
                                 if ($fProperty == 0 && $propertyName !== "0") {
                                     return $incorectTypeValue;
                                 }
@@ -222,7 +225,7 @@ private function activateConfirmSubscriptionUrl()
             $mailjetContactProperties = $this->getMailjetContactProperties();
             if (!empty($mailjetContactProperties)) {
                 foreach ($mailjetContactProperties as $property) {
-                    if (isset($properties[$property['ID']]) && $properties[$property['ID']] == '' ) {
+                    if (isset($properties[$property['ID']]) && $properties[$property['ID']] != '' ) {
                         $dataType = $property['Datatype'];
                         switch ($dataType) {
                             case "datetime":