![]() |
If I understand you right, post 24 (https://vborg.vbsupport.ru/showpost....5&postcount=24) has a tip that lets you define a comment for double posts. Also note the correction in post #25 as well. :)
|
@Xenon
It seems like this hack does provide some kind of "backdoor": It allows the user to add more images to his post then he would normally be allowed to have. Furthermore, it seems like this adds a query whenever a new reply is posted. What about adding $threadinfo['lastposter'] == $bbuserinfo['username'] to the condition before $doublepost = ...? As this evaluates to false if there is no double post (and thus the whole term will be false) it's not necessary to continue evaluating the query, or am I wrong? |
@Kirby: hmm, right, the backdoor is possible, will have to take a look at it...
As for the second part. yes, you can avoid the query in the if clause, by that, but you will still need the query when you update the post later (because of deleting the postindex, updating the pagetext... Also as we have had some problems sometimes, when the lastposter cache isn't correctly or something like, then the string compare would resolve incorrectly. I think it doesn't matter that much, but i'll consider it when i have the time to look through all of my hacks soon :) |
Hmm, i was just thinking: Sometimes, when users make doublepost it is, because they just can post a special ammount of images per post.
So, the question is, should i really fix that 'backdoor'? |
Quote:
|
Quote:
|
nice to get two totally different answers ^^
well, i'll think about it ^^ |
Well, actually I got a couple of users who explicitly use this got get more images into their posts :(
|
Ok, i have uploaded Version 1.4.
It now checks if the new post is valid regarding the settings (max images/ max characters...) If it is, then it will merge the post, if it's not, then the new post will be a true doublepost. I did it that way, because the limit will appear when doing an edit on a merged post otherwise. Also i have removed a bit overhead of the code. My test worked well, but i consider it as RC, until some of you can confirm they don't have problems (there shouldn't be, but you cannot be too sure ;)) |
Really nice and needed hack thnx Xenon
my question and forgive me if it was mentioned before, what if a user clicks the Quick Reply icon to replay to the first post and clicked it again to replay lets say to the 3rd post in there and clicked again to reply to the 6th reply in there, would this hack merge all 3 posts as you see there seperate replies Thnx |
All times are GMT. The time now is 04:58 AM. |
Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information | |
---|---|
|
|
![]() |
|
Template Usage:
Phrase Groups Available:
|
Included Files:
Hooks Called:
|