FriendsOfFlarum Split, separates posts to a new discussion

Jasper

I am very grateful to the developers for their efforts. I have a question. Is the segmentation operation reversible?

luceos

Jasper No, there's no merge (or undo split) option yet. Having a merge extension makes sense though.

Jasper

luceos thank you _:: 😀

luceos

Tagged 0.3.2 which fixes the permission problem 😊 That it even existed is a huge embarrassment, my apologies.

lisk

luceos can you help me if it's not difficult?
Tell me where to add this part of the code to combine Split and Byobu
https://github.com/flagrow/byobu/blob/0.1.2/js/forum/dist/extension.js#L18-L20
I want to Split asked who to add to the discussion as Byobu

MikeJones

when splitting can i pick individual posts to send with it?

luceos

MikeJones no only a range for now

MikeJones

luceos Got it. Just installed it and it worked perfectly for my use case. But yea picking individual posts may be useful in the future!

luceos

MikeJones yes this is just the initial version based on a very pressing issue here on discuss.

luceos

Released 0.4 which fixes a bug that disables loading of any successive extension. The extension is moved to the Friends of Flarum team as well. Luckily this doesn't really have an impact because the permission wasn't prefixed and there are not database migrations; upgrading still requires you to delete flagrow/split first:

$ composer remove flagrow/split
$ composer require fof/split

matteocontrini

luceos what was it, specifically? I can't find the fix in recent commits 🤔

MikeJones

luceos just updated thanks!

luceos

Here's the PR by @Franz back then: https://github.com/FriendsOfFlarum/split/pull/19

luceos

Tagged 0.4.1 which fixes an issue when either of the discussions involved in a split action is having their slug (title) changed. Thanks @matteocontrini for reporting and confirming the fix.

MikeJones

luceos back to back updates, nice!

miczja94

Can you add function move many posts from disscussion?

matteocontrini

miczja94 it's already possible by using the "split to here" button

miczja94

matteocontrini How? I can move one post, not multi posts.

Edit: Ok, thanks i already know 🙂

tolgaaaltas

In future updates, can we split post to available discussion? I generally split posts a new one and merge to available one. It would be nice to be able to search like the Merge extension. Thank you to all FoF team for these beautiful extensions.

datitisev

tolgaaaltas Most likely not, as this is achievable by using both fof/split and fof/merge-discussions. There's not a lot of reasons to add the same functionality to split.

rob006

datitisev Most likely not, as this is achievable by using both fof/split and fof/merge-discussions. There's not a lot of reasons to add the same functionality to split.

Honestly, these extensions should be merged IMO. You almost always need both of them since they share the same purpose - moving posts between discussions. Requiring two steps for moving posts from one discussion to another is unnatural and looks more like a workaround than complete solution.

tolgaaaltas

I have got an error when i use this extension with merge.
[2019-09-24 20:07:39] production.ERROR: BadMethodCallException: Call to undefined method FoF\Split\Posts\DiscussionSplitPost::formatContent() in /home/zincirsiz/web/hello.net/public_html/vendor/illuminate/support/Traits/ForwardsCalls.php:50

When I split and merge last discussions doesn't load normally and discussion page give an error. If i only split or merge any discussion, not a problem.

datitisev Thanks for reply.

matteocontrini

datitisev Most likely not, as this is achievable by using both fof/split and fof/merge-discussions. There's not a lot of reasons to add the same functionality to split.

Achievable with an high probability of making mistakes and with a lot of noise.

I think that moving a post to another discussion is a very basic function that every forum platform has, and it shouldn't be that hard or require to install two extensions.

Pollux

datitisev Most likely not, as this is achievable by using both fof/split and fof/merge-discussions. There's not a lot of reasons to add the same functionality to split.

I understand your reasoning and for my usage I don't see too many cases where I want to split posts into an existing discussion. However... when splitting posts from an existing discussion it often happens that these are not consecutive. In this case I have to split into two or more separate discussions and then merge these again into one. It can be done but seems awkward and it is definetely error prone.

One more thing: It seems there is an upper limit to the number of posts than can be split at once even when they are consecutive. It used to be in the range of 20 posts in beta 7. Now it seems to be higher but it's still there (about 35 posts when I tested here at discuss.flarum.org).

« Previous Page Next Page »