Extension talk:DiscussionThreading

See also --Danbrice 04:47, 29 July 2008 (UTC)
Extension_talk:DiscussionThread_Article also has similar discussions about this extension. Should the two pages be combined?

great idea --Lleoliveirabr 16:37, 5 April 2007 (UTC)
Very good extension, i always waited for something like this, but i need something with Portuguese BR language. This can be done? The way to install the extension is not good, have to do many modification to MediaWiki core files. Thinking in another way? Hope the development with this extension not be stopped soon. I didn't install this yet, but be sure I'll install when you tell me how to add Portuguese BR language.

re: great idea -- Jpond 03:14, 9 April 2007 (UTC)
Internationalised it this afternoon. You now need to also add the following array after the [en] array into the internationalisation file. I'd also appreciate it if you'd add the modifications on-line to the internationalisation file DiscussionThreading.i18n.php for others to use.

The file you need to change is DiscussionThreading.18n.php (also loaded into your extension directory). You can use this as an example (after changing the text of course).

$wgDiscussionThreadMessages['lang1'] = array(	'replysection' => 'lang1 - tag',	'replysectionhint' => "lang1 - hint",	'threadnewsection' => 'lang1 - new',	'threadnewsectionhint' => "'lang1 - Start a new thread" );

If you could tell me the BR words and phrases, I'd be glad to add them.

Competitive simultaneous edits - last person to save, wins
Peter Blaise asks: Is it subject to the MediaWiki anomaly of, when 2 people have a page open for editing at the same time, the "last person to save, wins"? I think the MediaWiki "last person to save, wins" single page orientation is a significant limit that must be overcome, not just for true discussion thread capability, but to respect the contribution of threshold contributors. The real challenge is for MediaWiki to grow to incorporate email-type real list-serve management for discussions, and perhaps for articles, also. I know many people who would not re-post nor revisit if their typing were "lost" due to bad timing during competitive simultaneous edits, as happened at the demo site for this beta test. However, I honor your pursuit. Perhaps the "first person to save"'s typing is immediately dropped one notch down in the page history and is not lost after all? I've yet to comprehend how history works, so I have a steep learning curve.

Note, however, that I will try this extension in a closed environment of a dozen editors of legal policy, and I hope it will survive for public release of the wiki project. Thank you very much.

Re: Competitive simultaneous edits - last person to save, wins -- Jpond 16:15, 9 April 2007 (UTC)
This is something you should take up with the implementers in the BugZilla space. In the words of the core developers, the standard answer is "don't try and turn MW into something it's not intended to do."

Remember, this functionality is intended to enhance and make usable - not necessarily to supercede exiting functions such as a listserv. Note that the MW developers use listserv's as their primary form of communication and documentation.

As devil's advocate to this line of thinking, my program managers basically went passive agressive on me and stopped using the collaborative discussion function until I added the automated tagging. Too many complaints about no one knew who was talking and who they were discussing with.

Installation Questions
1) it appears that there are 2 modifications to linker.php but they both appear to be the same???

The edit was to two different functions, the line number in the second one should have been 1062, changed --Jpond 12:29, 21 April 2007 (UTC) 2) the require_once line followed by the $wgSectionThreadingOn line would both be in the local setting file

3) the first major section of code would be save to DiscussionThreading.php in extensions directory

4) the second major section of code would be saved to DiscussionThreading.i18n.php in the extensions directory

At this point I am not getting the thread. I must have missed something in the installation. any ideas would be of great help Thanks --Dtsig 04:29, 21 April 2007 (UTC) I also had to do a weird manual flush of the php caching (see new instructions) --Jpond 12:29, 21 April 2007 (UTC)

Added Installation Instructions --Jpond 12:09, 21 April 2007 (UTC)
Sorry - agree the instructions were missing, have been added at:

Extension:DiscussionThread_Article

Installation Success and 3 ideas, --Dtsig 17:30, 25 April 2007 (UTC)

 * Bravo .. worked through first time (well actually I forgot the < in the php tag when defining the international php page and got a bunch of text at the top :-).
 * Here are 3 changes I am thinking of:
 * add tab NEW next to the edit/reply tabs. This would allow the creation of a new section automatically.  Now the user has to EDIT the page
 * Same can be done by pressing the + tab. Agree this is confusing, do you think the new link would resolve? --Jpond 15:59, 25 April 2007 (UTC)
 * Yeah. One of the things we are trying to do is remove as much of the markup as possible (not really possible but :).  So in this case the New tab would create a new == Comment Title == level item.  Similar to that mentioned below about creating a new discussion.--Dtsig 17:30, 25 April 2007 (UTC)
 * a popup box for entering text. might make the users a bit happier not editing in all that goopy html/MW stuff
 * Interesting, but a lot of work. --Jpond 15:59, 25 April 2007 (UTC)
 * If threading is on, instead of a blank page create a first header. edit/reply never show until there is a 1st level item (or so it seems).
 * The way I read this is the first time someone hits the 'discussion' tab, it would automagically pop into the create section (e.g., same as + tab).--Jpond 15:59, 25 April 2007 (UTC)
 * Yes, with the top level already there. Something like == Comment Name == This makes sure that the top level is there.  As I mentioned, the edit/reply tabs don't show up unless there is at least header tag. Possibly something like this?

$wgHooks['EditFormPreloadText'][] = 'newPage'; function newPage(&$text) {	$wnew = $_GET['new'] if ($wnew==1) { $temp1 ="== Comment Title Here =="; $text=$temp1; }	return true; } --Dtsig 17:58, 25 April 2007 (UTC)

Looking at this --Jpond 18:07, 25 April 2007 (UTC)
Summary: I'm going to be looking at this to see if possible:
 * 1) First Click on Discussion to go into '+' (add section) mode
 * 2) Modify header bar to include 'new' link
 * 3) Will not use pop-ups (too many browser variables), but will investigate using the '+' (add section) form for replies as well as new.  The downside here is that the text to which the responder is refering will not be visible.  No guarantees on this one.

No schedule on this, I'm really tied up on some other things for the next couple of weeks - will not be before the release of 1.10.

The above will require additional patches - which really isn't desireable.
 * Cool .. i might look at putting the text into the page automatically (at least on a new page). Will pass on any results --Dtsig 18:53, 25 April 2007 (UTC)

Requested Enhancements Implemented --Jpond 03:33, 15 May 2007 (UTC)

 * 1) First Edit of Discussion now goes into add comment mode (heading + text)
 * 2) Header bars now have [new][edit][reply], the 'new' link goes into add comment mode
 * 3) I'm not going to try and implement pop-ups - just too much effort
 * 4) modifications to EditPage.php no longer required - implemented in hooks.  Linker.php still needs minor patching

Conflict with another extension --Dtsig 15:39, 11 May 2007 (UTC)
We have been having problems getting Extension:SelectCategory extension to work when it finally dawned on me that there must be a conflict somewhere. So I started turning off all extensions and trying one at a time with SelectCategory. This is the only one that caused a conflict. Would it be possible for you to look into this? I have noted on the other extension the same information and would appreciate it if the two of you could play nicely . Let me know of any info, setup or testing needed. Thanks

Re:Conflict with another extension Extension:SelectCategory --Jpond 16:42, 11 May 2007 (UTC)
This pointed out two problems, there was not a 'return (true)' statement when the discussion threading was disabled (though I don't think this was causing the problem. We think the problem may actually be a conflict somewhere in the 'wfDebug' function used for debugging MediaWiki.  In reality, this isn't needed in production, so both lines should have been removed anyway - and this seemed to fix the problem.  On another note, this function is very commonly used and may show up somewhere completely independently of DiscussionThreading.

Re:Conflict with another extension Extension:SelectCategory --Dtsig 16:50, 11 May 2007 (UTC)
After testing as we suggested it does appear the the problem comes from the wfDebug Thanks again for the quick response to the problem. DTSig

Re:Conflict with another extension Extension:DynamicPageList --Erinmr 07:21, 8 June 2007 (UTC)
We have also found a conflict with the Discussion Threading extension and the dynamic page list. Undefined variable wgSectionThreadingOn Line 68, see wiki page. Any help on this would be greatly appreciated. Erin

Re:Re:Conflict with another extension Extension:DynamicPageList --jdpond 13:20, 8 June 2007 (UTC)
Erin, this is the second time this has happened, see Dave's comments Extension_talk:DiscussionThread_Article. I'm looking at this now to see what happened. Please contact me via email and let's work this out to either document or modify to avoid problems for other users.

Re:Re:Conflict with another extension Extension:DynamicPageList --Dtsig 13:25, 8 June 2007 (UTC)
Erin .. did you put $wgSectionThreadingOn = true; in your localsettings? I see that var showing in the error message

Re:Re:Conflict with another extension Extension:DynamicPageList --jdpond 13:30, 8 June 2007 (UTC)
Erin, my bad. add this line right before line 68 in discussion threading for a temp fix, will fix release now: function efDiscussionLink4other ($callobj, $title, $section, $url , &$result) { +		global $wgSectionThreadingOn; if($wgSectionThreadingOn && $title->isTalkPage ) { $commenturl = '&section='.$section.'&replyto=yes'; $curl = $callobj->makeKnownLinkObj( $title, wfMsg('replysection'), 'action=edit'.$commenturl ); $newthreadurl = '&section=new'; $nurl = $callobj->makeKnownLinkObj( $nt, wfMsg('threadnewsection'), 'action=edit'.$newthreadurl ); $result = "[".$nurl."][".$url."][".$curl."]"; }		return (true); }

Re:Re:Conflict with another extension Extension:DynamicPageList --jdpond 15:51, 11 June 2007 (UTC)
Version 1.1 fixes this and makes forward compatible with EditSectionLink enhancement which will not require patches in future release.

Re:Re:ReConflict with another extension Extension:DynamicPageList --Erinmr 14:09, 13 June 2007 (UTC)
Thanks John, this seems to work now.

No New/Reply buttons --Erinmr15:26, 13 June 2007 (UTC)
They have disappeared since the latest install. Am I doing something wrong? Thanks! Example

Re: No New/Reply buttons --Erinmr15:26, 13 June 2007 (UTC)

 * Jack, I first tried your new version of the Discussion threading extension and there it seemed like the new/reply buttons disappeared. I then went back to the old version and tried the fix and that works for me.

Re:Re: No New/Reply buttons --jdpond 19:49, 13 June 2007 (UTC)

 * Erin, I went back and checked - if you are still having problems, please notify so I can fix. The TAR had an extra directory (1_10) that should have been deleted (the zip was fine).  If you were using the patched Linker.php, you need to use the 1_10_0 version.  The next version (1_11_0) and trunk eliminate the need for patching.  I had to slightly modify the DiscussionThreading.php file to accomodate.  Remember, you have to update all three files for DiscussionThreading version 1.1:


 * includes/Linker.php
 * extensions/DiscussionThreading/DiscussionThreading.php
 * extensions/DiscussionThreading/DiscussionThreading.i18n.php

order of the comments
I like your extension very much. But is it possible to change your extension that I can see the newest Article at the top of the Discussion page. You understand what I mean? I wanna chance the order of the inserts.

re: Order of the Comments --jdpond 14:41, 26 September 2007 (UTC)

 * Great idea. Right now the extension uses the exact same code as the "+" tab, which adds to the bottom.  I'd have to investigate to see if there's a hook I could use to reorder.  I'm assuming this is only for new threads.  For consistency sake, would we want to consider inverted order for sub thread too?  That may be too much work.

re: Order of the Comments

 * Just finished looking into this one. I'll look for a better way, but as a temporary measure, you can get this by patching includes/Article.php in function replaceSection:

Problem with MW 1.11
Nice extension. But while using with MW 1.11, if the extension is activated, the section edit links are disappeared from articles. if i roll back to the original linker.php, then edit links are there, but not the "new" and "reply" links in talk pages. any solutions?

DiscussionThread Article
Why is there this extension, and DiscussionThread Article? What is the difference? Are they two different extensions? --Rovo 12:13, 1 October 2007 (UTC)

Bug : two signatures
Hi,

Thanks for this great extension !

I have a probleme there are two signatures if :


 * I create a new comment
 * I write Subject/headline but not text in comment textarea
 * Click on Save Page
 * I have an error message : Please enter a comment below but I have wiki signature inserted in the Subject/headline
 * I write comment in textarea
 * Click on Save Page
 * There are 2 signatures in discussion page

I have added french translation in Extension:DiscussionThread_Article

CSS tip to make reply body match title? - LNick 19:04, 26 February 2008 (UTC)
I used the CSS tip at the bottom of the installation article to indent the headings of replies, and I like it very much! However the body of the reply isn't indented. Does anyone have a good CSS trick to do that as well? Or perhaps this could be an enhancement for the extension.

Re: CSS tip to make reply body match title? --Danbrice 05:02, 29 July 2008 (UTC)

 * I've come up with a patch that does this, not using CSS, but instead just the correct number of old colons (based on the number of equal signs) to indent the body. I've also added some "replace me" text so that the user knows where to edit (which wasn't obvious to me when I first used this extension), and some javascript to select that "replace me" text in the text area too.


 * In the file extensions/DiscussionThreading.php, Add this:

// Add an appropriate number of colons to indent the body. // Include replace me text, so the user knows where to reply $replaceMeText = " Replace this text with your reply"; $text .= "\n\n".str_repeat(":",strlen($matches[1])-1).$replaceMeText; // Insert javascript hook that will select the replace me text global $wgOut; $wgOut->addScript("         function efDiscussionThread{            var ctrl = document.editform.wpTextbox1;            if (ctrl.setSelectionRange) {              ctrl.focus;              var end = ctrl.value.length;              ctrl.setSelectionRange(end-".strlen($replaceMeText).",end-1);              ctrl.scrollTop = ctrl.scrollHeight;                                           } else if (ctrl.createTextRange) {              var range = ctrl.createTextRange;              range.collapse(false);              range.moveStart('character', -".strlen($replaceMeText).");              range.select;            }          }          addOnloadHook(efDiscussionThread);          ");


 * Just before the two lines:

$efform->replyadded = true; $efform->textbox1 = $text;


 * I've tested it on IE 6.0 and Firefox 2.0, would be interested to know if the "replace me" text selection works on other browsers too. I let me know if you have a better way to write the above.


 * Update: I've updated the above code to also work better with Safari (3.1.2), and to scroll to the bottom of a large thread for both Safari and Firefox (IE already did that). --Danbrice 14:26, 29 July 2008 (UTC)

GREAT Modification --jdpond 06:02, 29 July 2008 (UTC)
Unless someone comes up with a problem, I'd like to incorporate into the code (would not retrofit prior to 1.10). Any way we can get someone to commit this as an extension in SVN so it could be collaboratively maintained? I took a shot at it when first developed, but it went nowhere.

CSS Tip Reproduced -- LNick 12:32, 29 July 2008 (UTC)
Thanks Danbrice! This is exactly what I was looking for! I was so dissatisfied I had removed the CSS trick to indent headers, and then had a lot of trouble finding it again! I've reproduced it here to help others. I don't know if you (or anyone else) has any ideas on how to limit the heading indents to comment pages?

Indenting Headers If you are really serious about threading, you can indent headers by modifying the corresponding CSS file (eg. skins/monobook/main.css). To modify all headers (talk or regular article), you can add a 'padding-left: xem;' into each of the headers 3-6. Eg.:

h3 { font-size: 132%; padding-left: 1em;} h3 .editsection { font-size: 76%; font-weight: normal; } h4 { font-size: 116%; padding-left: 2em;} h4 .editsection { font-size: 86%; font-weight: normal; } h5 { font-size: 100%; padding-left: 3em;} h5 .editsection { font-weight: normal; } h6 { font-size: 80%; padding-left: 4em; } h6 .editsection { font-size: 125%; font-weight: normal; }

Does not work with FCKEditor
The extention does not work with FCKEditor. There is a problem with reply. Anyone is aware of this problem ? Thanks.

Suggestion to make the identing work --Joris 15:30, 28 August 2008 (UTC)
$wgHooks['ParserAfterStrip'][] = 'efDiscussionParser'; function efDiscussionParser(&$parser, &$text, &$strip_state) {	if($parser->getTitle->isTalkPage) { $lines = explode("\n", $text); $level = 0; foreach($lines as &$line) { if(preg_match('/^(={3,})(.*)(={3,})$/', $line, $matches)) { $l = strlen($matches[1]); if($l > $level) { $line = "\n". $line; } else if ($l < $level){ $line = str_repeat(" ",$level - $l). "\n\n". $line; }				$level = $l; }		}		$text = implode("\n", $lines). str_repeat("\n ",$level - $l); }	return true; }

In combination with /* Threaded Discussion */ .reply { margin-left: 15px; }

.reply h1, .reply h2, .reply h3, .reply h4, .reply h5, .reply h6 { font-size: 14px; background-color: #EEE; }

.reply .editsection { font-size: 86%; background-color: #EEE; }

Will make it look like this:



Small notice about code
strlen function works well only with US ASCII chars (single byte). If i write my "replace me text" message in Russian or other language and save DiscussionThreading.php file in UTF-8. So i get wrong strlen value and function with "select replace me text" doesnt work well. You should replace it with mb_string (multibyte string) for better result i suggest. mbstring in PHP But mbstring is a non-default extension in PHP. And words "replace me" should be in i8n file. [mailto:kovshovdenis@gmail.com contact]