SeToY
Well-known member
I created an issue for them in their very own bug tracker for this: https://github.com/Audentio/xf2addon-issues/issues/2049
Seems like they don't care, though.
[TH] Auto Merge Double Posts [Deleted]
- Thread starter Lukas W.
- Start date
Seems like they don't care, though.
Lukas W.
Well-known member
That's the point of merging the double posts, yes.
Obviously. Such an essential bug and it hasn't been fixed yet.
We'll get to it with the next update wave, as usual.
SeToY
Well-known member
Na, for some people it's not. For some people it's more to keep the threads neat and tidy or to not increase post counts on double posts. That's why this suggestion was brought up:
Can you guys add an option to Mark threads which contain merged double posts as new? In my opinion it’s kinda confusing the way it works right now. Maybe you could add it as an optional setting for the next update?
Different users, different use-cases.
Na, for some people it's not. For some people it's more to keep the threads neat and tidy or to not increase post counts on double posts. That's why this suggestion was brought up:
Can you guys add an option to Mark threads which contain merged double posts as new? In my opinion it’s kinda confusing the way it works right now. Maybe you could add it as an optional setting for the next update?
Different users, different use-cases.
It's no secret that TH products are great for specific use-cases when it fits on release, but lack in update philisophy tremendously. It might not look like an "essential bug" to you (as you so cynically put it), but getting left alone again and again does not feel good.Such an essential bug and it hasn't been fixed yet.
Last edited:
That's the point of merging the double posts, yes.
Obviously. Such an essential bug and it hasn't been fixed yet.
That's sad! Because the users already knew it that way! This add-on for XF1 did it. Show a thread as unread if a double post was created. if the thread has been read in the meantime
[WMTech] Double Post Merge
Description: This add-on merges consecutive posts created by the same user within a specified time frame. Merged post content is optionally divided by a configurable text tag. Features: Based on user group permissions. Time limit within which...
xenforo.com
Maybe you can add it as an option?
Lukas W.
Well-known member
That's sad! Because the users already knew it that way! This add-on for XF1 did it. Show a thread as unread if a double post was created. if the thread has been read in the meantime
[WMTech] Double Post Merge
Description: This add-on merges consecutive posts created by the same user within a specified time frame. Merged post content is optionally divided by a configurable text tag. Features: Based on user group permissions. Time limit within which...
Maybe you can add it as an option?
We have no plans to add it at this time, sorry. We have all of our free addons on public GitHub repositories, so anyone who wants to implement this, can open a pull request to the repo and we'll review it.
No, and we don't plan to look into that. If tapatalk doesn't follow the normal XenForo controllers, there's nothing we can do from here.
ThemeHouse
Well-known member
ThemeHouse updated [TH] Auto Merge Double Posts with a new update entry:
Version 1.1.1 Patch Level 1 released!
Read the rest of this update entry...
Version 1.1.1 Patch Level 1 released!
Read the rest of this update entry...
hasanakyuz
Active member
ThemeHouse-AutoMergeDoublePost-1.1.1 Patch Level 1
error after writing a message
error after writing a message
SeToY
Well-known member
hasanakyuz
Active member
files are in the right place but still giving error
SeToY
Well-known member
Yes, I think just moving the files is not enough since there are still occurances of the old string. For example:
\XF\Pub\Controller\Thread.php
But the namespace is:
\Repository\MergeTime.php
My guess is that we'll need to wait for a new version (preferably a fixed and upgradable one). How that could have worked while testing after development is beyond me.
\XF\Pub\Controller\Thread.php
Code:
$mergeRepo = $this->repository('KL\AutoMergeDoublePost:MergeTime');But the namespace is:
\Repository\MergeTime.php
Code:
namespace ThemeHouse\AutoMergeDoublePost\Repository;My guess is that we'll need to wait for a new version (preferably a fixed and upgradable one). How that could have worked while testing after development is beyond me.
hasanakyuz
Active member
can you share the previous version file
ThemeHouse
Well-known member
ThemeHouse updated [TH] Auto Merge Double Posts with a new update entry:
Version 1.1.1 Patch Level 2 released!
Read the rest of this update entry...
Version 1.1.1 Patch Level 2 released!
Read the rest of this update entry...
zastavra
Active member
A regular upgrade will fix the problem.
SeToY
Well-known member
You were able to upgrade the add-on directly? Because I now have two of them:
And as I understand when uninstalling the old add-on, it deletes attached permissions, phrases and options. Hence installing the new one would result in default options and I'd have to redo every permission.
Lukas W.
Well-known member
Just copy the settings and phrases over. We changed the namespace of the add-on, and miscommunicated internally that that already happened, so it got pushed earlier than planned. There isn't any migration planned however, seeing that there's only four options and two permissions.
