From e6d753e2a0f4673647d6275b5e4223390822c8bd Mon Sep 17 00:00:00 2001
From: Chuck Scott <cscott@gaslightmedia.com>
Date: Mon, 2 Nov 2015 13:50:31 -0500
Subject: [PATCH] Changed member records so the user can't edit the member slug
 anymore.

---
 classes/data/dataMembers.php  | 33 +++++++++++++++++++++++++++++++--
 models/admin/member/index.php |  6 +++++-
 views/admin/member/index.html | 11 +++++------
 3 files changed, 41 insertions(+), 9 deletions(-)

diff --git a/classes/data/dataMembers.php b/classes/data/dataMembers.php
index d1e7739b..5f941655 100644
--- a/classes/data/dataMembers.php
+++ b/classes/data/dataMembers.php
@@ -150,8 +150,8 @@ class GlmDataMembers extends GlmDataAbstract {
 		        'member_slug' => array(
 		                'field' => 'member_slug',
 		                'type' => 'text',
-		                'required' => true,
-		                'use' => 'a'
+		                'required' => false,
+		                'use' => 'gl'
 		        ),
 
 		        // Date created
@@ -207,6 +207,35 @@ class GlmDataMembers extends GlmDataAbstract {
 
 	}
 
+	/*
+	 * Update member slug - should be called after a member record is added or updated
+	 *
+	 * @param integer id ID of member that needs the slug updated
+	 * @access public
+	 */
+	public function updateSlug($id = false)
+	{
+
+	    if ($id == false) {
+	        return false;
+	    }
+
+	    $m = $this->getEntry($id);
+
+	    $slug = sanitize_title($m['name']);
+
+	    // Update the city selected for this memberInfo record
+	    $sql = "
+            UPDATE ".GLM_MEMBERS_PLUGIN_DB_PREFIX."members
+                    SET member_slug = '$slug'
+                    WHERE id = $id
+                ;";
+	    $this->wpdb->query($sql);
+
+	    return $slug;
+
+	}
+
 	/*
 	 * Get a simple members list - Name and ID only
 	 *
diff --git a/models/admin/member/index.php b/models/admin/member/index.php
index 30b7c881..7b3855b3 100644
--- a/models/admin/member/index.php
+++ b/models/admin/member/index.php
@@ -147,6 +147,9 @@ class GlmMembersAdmin_member_index extends GlmDataMembers
                     $memberID = $memberData['fieldData']['id'];
                     $haveMember = true;
 
+                    // Update the member slug then get member data again
+                    $this->updateSlug($memberID);
+
                     // Go directly to the member information entry page
                     return array(
                         'status' => true,
@@ -165,7 +168,6 @@ class GlmMembersAdmin_member_index extends GlmDataMembers
                     glmMembersAdmin::addNotice($memberData, 'DataBlock', 'Member Data');
                 }
 
-
             // Otherwise setup to get new mmember data
             } else {
 
@@ -194,6 +196,8 @@ class GlmMembersAdmin_member_index extends GlmDataMembers
             // If update was successful then use editEntry() to setup for the edit again.
             if ($memberData['status']) {
 
+                $slug = $this->updateSlug($memberID);
+
                 $memberData = $this->editEntry($memberID);
                 $updated = true;
 
diff --git a/views/admin/member/index.html b/views/admin/member/index.html
index acdd2055..c9ff480d 100644
--- a/views/admin/member/index.html
+++ b/views/admin/member/index.html
@@ -35,15 +35,14 @@
                 </td>
             </tr>
             
-            
+      {if $haveMember}
             <tr>
-                <th {if $member.fieldRequired.member_slug}class="glm-required"{/if}>Name for URLs:</th>
-                <td {if $member.fieldFail.member_slug}class="glm-form-bad-input"{/if}>
-                    <input type="text" name="member_slug" value="{$member.fieldData.member_slug}" class="glm-form-text-input" placeholder="Slug of member name used for URLs.">
-                    {if $member.fieldFail.member_slug}<p>{$member.fieldFail.member_slug}</p>{/if}
+                <th>Name for URLs:</th>
+                <td>
+                    {$member.fieldData.member_slug}
                 </td>
             </tr>
-            
+      {/if}
             
             <tr>
                 <th {if $member.fieldRequired.access}class="glm-required"{/if}>Member Display & Access:</th>
-- 
2.17.1