Got some time and implemented a simple yet efficient fix for this, which is now live on MyBBoost. Will be rolled out with beta 3. If you want to apply the patch to beta 2, open inc/plugins/symposium.php, find the function
symposium_private_send_start and add under
$uid = (int) $mybb->get_input('uid'); the following piece of code:
// Redirect to appropriate conversation if existing
$convid = get_conversation_id($mybb->user['uid'], $uid);
$query = $db->simple_select('symposium_conversations', 'uid', "uid = {$mybb->user['uid']} AND convid = '" . $db->escape_string($convid) . "'");
if ($db->fetch_field($query, 'uid')) {
header('Location: private.php?action=read&convid=' . $convid);
exit;
}
In order to test this feature, click on the
Message button on the new hovercards on usernames.
Oops my bad, it is working anyway (but I thought the first time sending the message wasn't working o.O)
This is a noted bug that Shade said will be fixed in the next Beta...I noticed the same thing on my forum when I tested it.
This should be actually working in beta 2 as long as select2's option "multiple" is set to true; it's not here due to styling reasons. A definitive fix (which implies "multiple" to be always true) will be carried out in beta 3 which will support multiple participants in a single chat.
This should be actually working in beta 2 as long as select2's option "multiple" is set to true; it's not here due to styling reasons. A definitive fix (which implies "multiple" to be always true) will be carried out in beta 3 which will support multiple participants in a single chat.
Any ETA for Beta 3?
Also on this, when you click Send PM on profiles/posts it should take you to an existing conversation if you indeed have one? Maybe this would be too complicated but it's just a suggestion instead of giving people the new conversation screen over and over.
There is no ETA, as university commitments are quite unpredictable. Till end of February all development is halted, then I will see what I can do.
Got some time and implemented a simple yet efficient fix for this, which is now live on MyBBoost. Will be rolled out with beta 3. If you want to apply the patch to beta 2, open inc/plugins/symposium.php, find the function
symposium_private_send_start and add under
$uid = (int) $mybb->get_input('uid'); the following piece of code:
// Redirect to appropriate conversation if existing
$convid = get_conversation_id($mybb->user['uid'], $uid);
$query = $db->simple_select('symposium_conversations', 'uid', "uid = {$mybb->user['uid']} AND convid = '" . $db->escape_string($convid) . "'");
if ($db->fetch_field($query, 'uid')) {
header('Location: private.php?action=read&convid=' . $convid);
exit;
}
In order to test this feature, click on the
Message button on the new hovercards on usernames.
Hi Shade
I've tried this patch a couple times and it does not work for me. Any idea's?
I will release the new beta in the upcoming days, I've just finished my last exam and I have a couple days of rest.
Got some time and implemented a simple yet efficient fix for this, which is now live on MyBBoost. Will be rolled out with beta 3. If you want to apply the patch to beta 2, open inc/plugins/symposium.php, find the function symposium_private_send_start and add under $uid = (int) $mybb->get_input('uid'); the following piece of code:
// Redirect to appropriate conversation if existing
$convid = get_conversation_id($mybb->user['uid'], $uid);
$query = $db->simple_select('symposium_conversations', 'uid', "uid = {$mybb->user['uid']} AND convid = '" . $db->escape_string($convid) . "'");
if ($db->fetch_field($query, 'uid')) {
header('Location: private.php?action=read&convid=' . $convid);
exit;
}
In order to test this feature, click on the Message button on the new hovercards on usernames.
So this is only supposed to be moving you to the conversation if it exists, but still showing undefined for non existent conversations.