Is the vB4 thread structure broken

[tommythejoat]

Originally Posted by MarkFL

All I can tell you for sure is that with that code in place, when a reply is made without replying to or quoting a specific post, then the "parentid" for that post will be set to zero.

Actually, the fix needed a slight tweak on our Message Board. Apparently the 'p' index does not test false unless you use isset to test it.

I was a bit confused when it did not work on the first try, but I decided it could not possibly harm to make it.

PHP Code:

if (!isset($_REQUEST['p'])) {
$this->post['parentid'] = 0;
} 


The only effect except eliminating the thread reference (that we find troubling) is that in hybrid or threaded view, the link graphics show a broken line for each new reference to the thread. It seems to me that is a benefit rather than a problem.

--------------- Added [DATE]1488068167[/DATE] at [TIME]1488068167[/TIME] ---------------

I do not think it is possible to make this "feature" retroactive.

In my opinion, the current vB4 behavior is broken. Clearly the reply to thread should set previous post to the starting thread post . There is no way to know what the poster intended or which button was used to create a new post since both point to the most recent post.

This fix does eliminate the problem. If one wanted the feature to actually reference the first post in the thread, that could be done or, retroactively in the future a query could be executed to set all the 0 parent postid values to the value of the first postid in the thread.

--------------- Added [DATE]1488080917[/DATE] at [TIME]1488080917[/TIME] ---------------

I thought I had this fixed but when I went to install it on the live site, it did not actually work and when I checked again on the test site the result was the same.

I hacked the plugin to insert a print_r on the $_REQUEST global and got this result for both reply from the border and reply with the button.

PHP Code:

capture vBulletin $_REQUEST for post thread at postdata_presave hook

Array ([do] => postreply [t] => 139871 [message_backup] => one more [message] => one more 
[wysiwyg] => 0 [signature] => 1 [fromquickreply] => 1 [s] => [securitytoken] => nnnnnnnnn
[p] => 1087469 [specifiedpost] => 1 [parseurl] => 1 [loggedinuser] => 2 [posthash] => nnnnnnnnn
[poststarttime] => 1488076181 [ajaxqrfailed] => 1 [threadid] => 139871 [postid] => 1087469 [styleid] => 
[langid] => [api] => [refererid] => [a] => [nojs] => [forumid] => [pollid] => [dofbredirect] => 
[pagenumber] => [noquote]=>) [quoteall] =>) 


'p' and 'postid' are both set to the same value and not missing or 0, etc.

Is there a different hook that will work, or is one of my other mods busting this. I notice that there is a jquery flag that looks like it may be bad.

[chloe101]

Have you found a fix for this yet? I'm running VB4.2.4 and just noticed the same exacerbating problem. I'm trying to find a fix for this and if not, I'll just remove that button because it's misleading & useless as is.