From c0e83fa65765f111ec26b4c7bf2a2a7cce6118b6 Mon Sep 17 00:00:00 2001 From: Cameron Date: Mon, 16 Jan 2017 12:09:31 -0800 Subject: [PATCH] SEF UURL Warning added to NewsPost --- e107_admin/newspost.php | 24 +++++++++++++++++++ e107_languages/English/admin/lan_newspost.php | 5 +++- 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/e107_admin/newspost.php b/e107_admin/newspost.php index 614d8bab2..5115b8aeb 100644 --- a/e107_admin/newspost.php +++ b/e107_admin/newspost.php @@ -509,6 +509,9 @@ class news_admin_ui extends e_admin_ui $new_data['news_sef'] = empty($new_data['news_sef']) ? eHelper::title2sef($new_data['news_title']) : eHelper::secureSef($new_data['news_sef']); + $this->checkSEFSimilarity($new_data); + + $tmp = explode(chr(35), $new_data['news_author']); $new_data['news_author'] = intval($tmp[0]); @@ -566,6 +569,9 @@ class news_admin_ui extends e_admin_ui $new_data['news_sef'] = eHelper::title2sef($new_data['news_title']); } + + $this->checkSEFSimilarity($new_data); + if(!empty($new_data['news_author'])) { $tmp = explode(chr(35), $new_data['news_author']); @@ -581,6 +587,24 @@ class news_admin_ui extends e_admin_ui } + /** + * Display a warning if there is a mismatch with the SEF Url. + * @param $new_data + */ + private function checkSEFSimilarity($new_data) + { + $expectedSEF = eHelper::title2sef($new_data['news_title']); + similar_text($expectedSEF,$new_data['news_sef'],$percSimilar); + + if($percSimilar < 60) + { + e107::getMessage()->addWarning(LAN_NEWS_108); // The SEF URL is unlike the title of your news item. + } + + + } + + public function afterCreate($new_data, $old_data, $id) { diff --git a/e107_languages/English/admin/lan_newspost.php b/e107_languages/English/admin/lan_newspost.php index 3ee443d04..61df9c647 100644 --- a/e107_languages/English/admin/lan_newspost.php +++ b/e107_languages/English/admin/lan_newspost.php @@ -232,4 +232,7 @@ define("LAN_NEWS_105", "Email notification triggered!"); define("LAN_NEWS_106", "News item visibility must include 'everyone' for email notifications to work."); define("LAN_NEWS_107", "Checking for Ping Status"); -?> +//v2.1.4 +define("LAN_NEWS_108", "The SEF URL is unlike the title of your news item."); + +?> \ No newline at end of file