From: Steve Sutton <steve@gaslightmedia.com>
Date: Wed, 14 Dec 2016 16:10:44 +0000 (-0500)
Subject: Taking care of errors.
X-Git-Tag: v1.1.1^2~9
X-Git-Url: http://cvs2.gaslightmedia.com/gitweb/?a=commitdiff_plain;h=e0e2380faafeb06077f3f1716e531140e801a701;p=WP-Plugins%2Fglm-member-db-leads.git

Taking care of errors.

php error for undefined variables.
---

diff --git a/models/admin/management/leads.php b/models/admin/management/leads.php
index cf2150b..6cbf262 100644
--- a/models/admin/management/leads.php
+++ b/models/admin/management/leads.php
@@ -174,7 +174,7 @@ class GlmMembersAdmin_management_leads // extends GlmDataLeadsManagement
                  * WordPress option. Using update_option here in case the option
                  * already exists.
                  */
-                if ( $_REQUEST['import_fields'] ) {
+                if ( isset( $_REQUEST['import_fields'] ) && $_REQUEST['import_fields'] ) {
                     update_option(
                         $option_name,
                         serialize( $_REQUEST['import_fields'] )
@@ -585,18 +585,18 @@ class GlmMembersAdmin_management_leads // extends GlmDataLeadsManagement
             'source_id'       => $source_id,
             'lead_id'         => $lead_id,
             'gf_entry_id'     => $entry['id'],
-            'fname'           => ( ( $lead['fname'] )   ? $lead['fname']   : ''),
-            'lname'           => ( ( $lead['lname'] )   ? $lead['lname']   : ''),
-            'org'             => ( ( $lead['org'] )     ? $lead['org']     : ''),
-            'addr1'           => ( ( $lead['addr1'] )   ? $lead['addr1']   : ''),
-            'addr2'           => ( ( $lead['addr2'] )   ? $lead['addr2']   : ''),
-            'city'            => ( ( $lead['city'] )    ? $lead['city']    : ''),
-            'state'           => ( ( $lead['state'] )   ? $lead['state']   : ''),
-            'zip'             => ( ( $lead['zip'] )     ? $lead['zip']     : ''),
-            'country'         => ( ( $lead['country'] ) ? $lead['country'] : ''),
-            'phone'           => ( ( $lead['phone'] )   ? $lead['phone']   : ''),
-            'phone2'          => ( ( $lead['phone2'] )  ? $lead['phone2']  : ''),
-            'fax'             => ( ( $lead['fax'] )     ? $lead['fax']     : ''),
+            'fname'           => ( isset( $lead['fname'] ) && ( $lead['fname'] )   ? $lead['fname']   : ''),
+            'lname'           => ( isset( $lead['lname'] ) &&( $lead['lname'] )   ? $lead['lname']   : ''),
+            'org'             => ( isset( $lead['org'] ) &&( $lead['org'] )     ? $lead['org']     : ''),
+            'addr1'           => ( isset( $lead['addr1'] ) &&( $lead['addr1'] )   ? $lead['addr1']   : ''),
+            'addr2'           => ( isset( $lead['addr2'] ) &&( $lead['addr2'] )   ? $lead['addr2']   : ''),
+            'city'            => ( isset( $lead['city'] ) &&( $lead['city'] )    ? $lead['city']    : ''),
+            'state'           => ( isset( $lead['state'] ) &&( $lead['state'] )   ? $lead['state']   : ''),
+            'zip'             => ( isset( $lead['zip'] ) &&( $lead['zip'] )     ? $lead['zip']     : ''),
+            'country'         => ( isset( $lead['country'] ) &&( $lead['country'] ) ? $lead['country'] : ''),
+            'phone'           => ( isset( $lead['phone'] ) &&( $lead['phone'] )   ? $lead['phone']   : ''),
+            'phone2'          => ( isset( $lead['phone2'] ) &&( $lead['phone2'] )  ? $lead['phone2']  : ''),
+            'fax'             => ( isset( $lead['fax'] ) &&( $lead['fax'] )     ? $lead['fax']     : ''),
             'date_submitted'  => $entry['date_created'],
             'user_trace_info' => $entry['ip'],
         );
diff --git a/views/admin/management/leads.html b/views/admin/management/leads.html
index ee107fc..62a17d1 100644
--- a/views/admin/management/leads.html
+++ b/views/admin/management/leads.html
@@ -51,6 +51,7 @@
         </td>
     </tr>
     {/if}
+    {debug}
     {if $form && !$importing}
     <tr>
         <td>
@@ -70,13 +71,13 @@
                         <td>{$field.id}</td>
                         <td>{$field.type}</td>
                         <td>{$field.label}</td>
-                        <td>{if !$fields.inputs.0.id}<input type="text" name="import_fields[{$field.id}]" value="{if $import_fields[$field.id]}{$import_fields[$field.id]}{/if}" />{else}&nbsp;{/if}</td>
+                        <td>{if $field.inputs && !$field.inputs.0.id}<input type="text" name="import_fields[{$field.id}]" value="{if $import_fields[$field.id]}{$import_fields[$field.id]}{/if}" />{else}&nbsp;{/if}</td>
                         <td>
                             {if $field.inputs}
                                 <table>
                                     <tbody>
                                         {foreach $field.inputs as $input}
-                                            {if !$input.isHidden}
+                                            {if !isset($input.isHidden)}
                                             <tr>
                                                 <td>
                                                     <label>{$input.label}</label><br>