I’m configuring Spark to be used in our company. I noticed that ROAR pop up doesn’t show up when you broadcast the message to all users? It only shows when you message individual users directly. Is this normal behaviour or is there an issue?
I’ve noticed and filed this as SPARK-1534 a while ago. sadly the developer who has provided this plugin doesn’t work on Spark anymore, so i’m not sure when it will be fixed.
Spark is not firing *fireGlobalMessageReceievedListeners *when new broadcast message is received, my code is doing this:
check if Roar is installed
fire listener (Roar will notice this and shows popup windows)
play incomming message sound (if you have enabled it)
This is solution which I am using in our branded Spark. Another Roar bug is showing many new popups when you enter old conference room (with old messages) under certain circumstances. I can provide solution for this too if you are interested.
I’m using Windows systems, can you advise where I can find broadcast.plugin on Windows? I tried to find it but failed. Or is it named differently under windows systems?
Why do you want to find it? Do you want to edit it and compile on your own? Tomas was saying about the source code files, not the regular files in Spark installation. One has to edit the source file and the compile the new binaries and libraries (the ones used in Spark installation).
Sorry, I understood that this part of code could have been entered into the existing installation files. Would the update be available soon to fix this problem?
after this patch Roar should show popups for broadcast messages
sound for new brodacast message will be played (if you have enabled sound for incoming messages)
Next patch is for Roar plugin to avoid showing many popups when you join old groupchat room (groupchat with already written messages). I am not sure if this patch is correct because i commented many lines in original source code and made old groupchat check much more simpler. So anyone experienced should check these changes, but for me it works fine.
I have uploaded your patch to SPARK-1534 (closed SPARK-1467 as it was for sound in broadcasts) and filed SPARK-1565 and SPARK-1566 for your other patches. Will try to test them myself tomorrow and then will assign to some fairly active developer to commit to the trunk.
Ugh, the problem was not with the format i think, but that full path to java file was not in the patch I’m applying patches to the root of the source, so it couldn’t find a file to patch. Not a problem, but i have updated all your patches with the full path, just in case.
BroadcastPlugin patch tested. Works fine. Shows popups and plays sound for broadcast message.
RoarMessageListener - interesting, that this only happens when JTattoo skins are in use, and Substance skin doesn’t show Roar popups when joining a room with history. Done minimal testing with both skins and looks like issue is fixed. Though in both cases (with patch or without) Roar doesn’t show popups for new normal messages in the room… Probably another issue. Or maybe this is by design as it can be a non stopping flood of popups if a room is very active or if you have many rooms opened…
Last one doesn’t work for me. Popup still shows only the name of the sender of a broadcast message.
Will update tickets with newer patch versiond and assign first two to Mircea. He was active lately, so maybe he will apply them quick enough.
RoarMessageListener - I am using system skin in Spark (it shows like Windows in options) and Roar is working properly for messages in group chat rooms. So this is not by design and i don’t think i made any other changes in Roar source code. Can you try it with default system skin too?
BottomRight and TopRight patches - This is weird, i don’t see any reason why it should not work for you. This is only simple check if message is from broadcast and then adding some new text to popup. Can you debug Spark client and check why my code fails on your setup?
Weird. It is indeed is only working when Windows skin is selected. But the default for new installations is JTattoo Luna skin. Filed as SPARK-1568
How should i debug Spark to see where it fails? Btw, i’m sending broadcast from another Spark via Send broadcast menu, sending simple message to all Online users.
With Spark debugger i get this packet:
fgggggggggggggggggggggg
bp2gy1
It comes as a broadcast message (“i” icon in the tab and prefix Broadcast before the message in the chat window).
Nice thing this Smack Debug Window Use it first time, really helpful.
Important part is not message xml (i receive messages in same format as you) but message type. You must receive messages with type normal (you can see this in Smack Debug Window) and then should my patch work:
Hmm, i tried to change my skins to JTattoo and Substance. For both Roar shows new messages in group chats. It seems we need someone else to test it and post results.
About broadcast popup - I think it must be something else on your side, how did you apply my patches? You mentioned something about Netbeans. I am using Netbeans too and for plugins there is one catch. If you made some change in source codes of plugin you need to first rebuild plugin itself (this will copy new jar file into Spark build folder). After this you can build whole Spark and changes should be visible. It takes me some time until i figured this out
Hmm, i tried to change my skins to JTattoo and Substance. For both Roar shows new messages in group chats. It seems we need someone else to test it and post results.
My bad. I was testing with the virtual machine and covering the chat window with virtual machine window while sending a message, but the chat window itself stayed active, so no popups were shown. After i minimized the chat window and then sent the message, popup showed up
you need to first rebuild plugin itself
Right! I remember now that i had to do this when i was testing changes for another plugin. I have rebuilt Roar and now your changes work. Well, it is “(sender’s name) - Broadcast”, but i think the order doesn’t matter.
Tomas, your patches for showing popup and playing sound and adding name to the popup are now applied to the trunk. It looks like in the broadcastplugin patch you have also fixed another ticket of mine SPARK-1528. One can see these changes in the latest build (657):