possible fix for corner case paypal issue, where unconfirmed addresse…

…s would overwrite txn details with blank data
Fabrik · Nov 19, 2013 · a5869fb · a5869fb
1 parent c470ff8
commit a5869fb
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/plugins/fabrik_form/paypal/paypal.php b/plugins/fabrik_form/paypal/paypal.php
@@ -808,11 +808,15 @@ public function onIpn()
 									$set_array[] = "$set_field = $set_value";
 								}
 
-								$query->clear();
-								$query->update($table->db_table_name)
-								->set(implode(',', $set_array))
-								->where($table->db_primary_key . ' = ' . $db->quote($rowid));
-								$db->setQuery($query);
+								// Only update IPN storage if txn field and status set
+								if (!empty($ipn_txn_field) && !empty($ipn_status_field))
+								{
+									$query->clear();
+									$query->update($table->db_table_name)
+									->set(implode(',', $set_array))
+									->where($table->db_primary_key . ' = ' . $db->quote($rowid));
+									$db->setQuery($query);
+								}
 
 								if (!$db->execute())
 								{