New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ticket/14804] Add core event to MCP after merging topics #4472
Conversation
Renamed $topic_data and defined $to_topic_data in order to avoid ambiguity and to stop overwriting a variable. PHPBB3-14804
* @event core.mcp_merge_after | ||
* @var array all_topic_data The data from all topics involved in the merge | ||
* @var int to_topic_id The ID of the topic into which the rest are merged | ||
* @since 3.1.10-RC1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's too late for 3.1.10-RC1 as it's already out. Should go to 3.1.11-RC1.
/** | ||
* Perform additional actions after merging topics. | ||
* | ||
* @event core.mcp_merge_after |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name could be more descriptive, like core.mcp_forum_merge_topics_after
or something like that.
…nges to 3.1 PHPBB3-14804
…ket/14804 PHPBB3-14804
I haven't worked with Travis very much, so I'm not sure if I'm able to manually trigger a build without committing or closing & reopening the PR. |
[ticket/14804] Add core event to MCP after merging topics
Thanks for the pull request. |
Checklist:
Tracker ticket (set the ticket ID to your ticket ID):
https://tracker.phpbb.com/browse/PHPBB3-14804
Renamed $topic_data and defined $to_topic_data in order to avoid ambiguity
and to stop overwriting a variable.
PHPBB3-14804