From df103a8cb2db47a5478c884643549b62e682fb30 Mon Sep 17 00:00:00 2001 From: DBPreston Date: Wed, 11 Apr 2012 18:32:27 +0200 Subject: [PATCH 1/4] Added code at line 237 to shorten reply title to a manageable length --- Discussions.module | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Discussions.module b/Discussions.module index 765d458..37fc644 100644 --- a/Discussions.module +++ b/Discussions.module @@ -234,7 +234,7 @@ class Discussions extends WireData implements Module { $newPost->name = $sanitizer->pageName($post->discussions_title, true); } else { $newPost->template = wire('templates')->get('discussions-reply'); - $newPost->title = $sanitizer->text($post->discussions_message); + $newPost->title = implode(' ',array_slice(explode(' ', $sanitizer->text($post->discussions_message)),0,8)); $newPost->name = md5(time() . $user->name); } From dd4b31bdf7ca845debf9c06794e83e28f0fa7db1 Mon Sep 17 00:00:00 2001 From: DBPreston Date: Wed, 11 Apr 2012 18:34:58 +0200 Subject: [PATCH 2/4] Line 353 change pageLabelField to 'title' for useability. --- Discussions.module | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Discussions.module b/Discussions.module index 37fc644..97fa4d4 100644 --- a/Discussions.module +++ b/Discussions.module @@ -350,7 +350,7 @@ class Discussions extends WireData implements Module { $template_reply->parentTemplates = array($template_topic->id); $template_reply->slashUrls = 1; $template_reply->noChildren = 1; - $template_reply->pageLabelField = 'discussions_message'; + $template_reply->pageLabelField = 'title'; $template_reply->save(); $template_forum->childTemplates = array($template_topic->id); From 222ee70ab77e9f89203939bc95bc579a885efe07 Mon Sep 17 00:00:00 2001 From: DBPreston Date: Thu, 12 Apr 2012 11:09:27 +0200 Subject: [PATCH 3/4] Added code to prevent duplicate post names/titles and fixed redirect to new page broken as a result. --- Discussions.module | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/Discussions.module b/Discussions.module index 97fa4d4..e3645e3 100644 --- a/Discussions.module +++ b/Discussions.module @@ -232,6 +232,17 @@ class Discussions extends WireData implements Module { $newPost->template = wire('templates')->get('discussions-topic'); $newPost->title = $sanitizer->text($post->discussions_title); $newPost->name = $sanitizer->pageName($post->discussions_title, true); + // Make sure post name is unique (after http://processwire.com/talk/topic/18-how-do-i-import-lots-of-data-into-pages/page__view__findpost__p__36) + $name = $newPost->name; + $title = $newPost->title; + $n = 0; + while(count($page->children("name=$name")) > 0) { + $n++; + $name = $newPost->name . '-' . $n; // i.e. sears-tower-1, sears-tower-2, etc. + $title = $newPost->title . ' ' .$n; + } + $newPost->name = $name; + $newPost->title = $title; } else { $newPost->template = wire('templates')->get('discussions-reply'); $newPost->title = implode(' ',array_slice(explode(' ', $sanitizer->text($post->discussions_message)),0,8)); @@ -248,7 +259,7 @@ class Discussions extends WireData implements Module { // After saving new reply we redirect to prevent double posts if ($post->discussions_title) { - $session->redirect("./" . $sanitizer->pageName($post->discussions_title, true)); + $session->redirect("./" . $name); } else { // Redirect to last page there is $pageNum = $this->_countLastPage($page); From d72289dda3ccf23716bb9145b0003badaf2e7d5a Mon Sep 17 00:00:00 2001 From: David Preston Date: Wed, 20 Sep 2017 12:14:17 +0100 Subject: [PATCH 4/4] Update Discussions.module Added namespace - let's get this thing up to date! --- Discussions.module | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Discussions.module b/Discussions.module index e3645e3..3924520 100644 --- a/Discussions.module +++ b/Discussions.module @@ -1,4 +1,4 @@ -