Willman, thanks for your report. It sounds similar to the problem reported as MDLSITE-1061. I've added Sam as a watcher in case he wishes to comment.

The redirection does create the problem!While new discussions are created it gets redirected to another forum but it works good for reply messages!

New posts don't have this problem, because they unset the return URL, knowing that the correct one is computed lower.

A new one is also computed for new discussions, but it basically ignored because of the structure of the the code.

My patch mimics new posts and edits, unsetting $SESSION->fromdiscussion.

Ran all @mod_forum behat tests and unit tests, passed. I don't think I can make tests for this specific problem, as it requires multiple browser tabs.

Hi Eric,

Thanks a lot looks good.

Can I suggest that you add a comment around this line (and in fact duplicate version of it that you've seen) because its not quite clear why we're unsetting this, so it'd be good to explain it for other developers coming along.

I'm submitting this for integration.

I added a brief comment to the 3 places where the unsetting happens.

Hi Eric,

I notice you've also changed the spacing on } else if (!empty($delete)) { // User is deleting a post, but you haven't fixed the comment. Probably i'd prefer it if you leave it alone (but if you are fixing the spacing, may as well fix the missing period too.

Yeah, didn't see that. I just backed out the change per your preference.

The OCD in me sees something like the forums, and I want to go in a clean up all the style errors.

We have done those kinda cleanups wholesale (see MDL-40847), so if you are feeling brave we would probably accept it (but it can then cause problems merging between branches, which is one reason to not go around doing it it)

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

Thanks Eric, this was integrated in 2.4, 2.5 and master

Passing.
I also tested posting in both (1 then B), and making sure that the bug addresses the same concerns on the confirmation page.

Yes, it's happening (somewhere in the French Polynesia, right now). And you did it, raising Moodle to new excellency levels.

Or, if you prefer, yes, you fixed that boring issue.

Thanks anyway! Ciao