Thread Tools Display Modes
05-15-13, 01:30 PM   #1
AtsayaSargeras
A Kobold Labourer
Join Date: May 2013
Posts: 1
BloodyScreen, Renovation for MoP

Hello all!
New member to this forum, been on Curse for a while now and after a bunch of people complaining about this wonderful addon no longer being supported after 4.2... I attempted (key word there LOL) to go through the LUA's and Localization files to get this fantastic addon working again.
Many hours and frustrated keyboard smashes later, I am reaching out to see what I'm doing wrong!
Here we go;
The textures are still in .blp format so that shouldn't be a problem
There are no sound formats what-so ever to be replaced updated etc.
Upon going through the LUA files, I found the main problem was the arg's in "BS_Main"..
That being said, I spent a few days switching numbers around, trial and error, and eventually got some spatter on regular damage done. Seemed exciting, still not quite good enough. The addon is supposed to splatter blood on your screen when you critically strike.
I've hit a wall with progressive success on this addon, and my forehead can't take much more bashing into the keyboard! If anyone could take some time to look through the LUA's and point me in the right direction it would be very appreciated, and I know the 100k+ fans of this addon would be thrilled.
Thanks in advance for any advice or info,
Atsaya Sargeras Alliance US
  Reply With Quote
05-15-13, 03:37 PM   #2
ravagernl
Proceritate Corporis
Premium Member
AddOn Author - Click to view addons
Join Date: Feb 2006
Posts: 1,176
Without seeing any changes in any code whatsoever, nobody can help you.

What exactly did you try changing in the files? If it's large files, consider posting the changed lua/xml files in a zip file or post them on a site such as http://pastebin.com/ and then post the link over here.

EDIT: I'm assuming you're working off from the Bloody Screen (4.2 Fan Update)?
EDIT2: In case anyone knows what it looks like: http://www.youtube.com/watch?v=XtO3kdfkPdg

Last edited by ravagernl : 05-15-13 at 04:35 PM.
  Reply With Quote
05-15-13, 04:26 PM   #3
ravagernl
Proceritate Corporis
Premium Member
AddOn Author - Click to view addons
Join Date: Feb 2006
Posts: 1,176
Ok, my opinion is that it is not worth the time to update the existing addon.
Even if you fix the problem of COMBAT_LOG_EVENT_UNFILTERED (some extra event args were inserted at mop launch, and that is why the AddOn no longer works at all), there are so many things to consider that for me it would become unmanageable to update the AddOn for major future content patches(which is why probably nobody has bothered yet).

A lot of improvements can be made to reduce CPU stress, for example:
  1. Every localization string or configuration setting is initialized or saved as a global, while it is good practice to combine them into a dictionary or associative table and then store a local reference to this global table.
  2. The event handler, on the other hand, does make use of tables, but it does so in a way that a vararg table is created for every event (yes, for COMBAT_LOG_EVENT_UNFILTERED as well...):
    Code:
    --This function handles the used game events. 
    function BS_OnEvent(self, event, ...) 
    local args = {...}; 
    if (event == "ADDON_LOADED") then 
    if (args[1] == "BloodyScreen") then 
    BS_Init(); 
    end 
    end 
    
    if ((event == "UNIT_COMBO_POINTS") and (BS_BloodBehaviour == 1)) then 
    if (((BS_EnableOnPVP) and (UnitIsPlayer("target"))) or ((BS_EnableOnPVE) and (not UnitIsPlayer("target")))) then BS_AddSplatters(args); end 
    end 
    
    if ((event == "COMBAT_LOG_EVENT_UNFILTERED") and (BS_BloodBehaviour > 1)) then 
    if (((BS_EnableOnPVP) and (UnitIsPlayer("target"))) or ((BS_EnableOnPVE) and (not UnitIsPlayer("target")))) then BS_AddSplatters(args); end 
    end 
    end
  Reply With Quote
05-15-13, 08:30 PM   #4
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
If the rest of the code is even half as bad as that, the whole thing needs to be nuked from orbit and rewritten from scratch.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
05-16-13, 09:04 AM   #5
Resike
A Pyroguard Emberseer
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,290
Best line from the code:
Code:
spellname, _, _, _, _, _, _, _, _ = GetSpellInfo(BS_RangeCheckAction[class]);
It shouldn't take much time to update this addon, it's only like 300-400 lines.
  Reply With Quote
05-16-13, 06:50 PM   #6
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
Okay, I couldn't help myself... I looked at the code. I think we have a strong contender for Worst Addon Code Ever Published. The original author apparently doesn't even know about "elseif" statements, and don't seem to realize that indices in Lua start at 1 instead of 0. Everything is a global, there are tons of "if X then if Y then Z end end" constructions that should be consolidated to "if X and Y then Z end", and basically it's just all bad.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
05-17-13, 02:47 AM   #7
Resike
A Pyroguard Emberseer
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,290
If i'm gonna have some time in the weekend i might be able to update it for 5.2.
  Reply With Quote
05-17-13, 12:41 PM   #8
ravagernl
Proceritate Corporis
Premium Member
AddOn Author - Click to view addons
Join Date: Feb 2006
Posts: 1,176
Originally Posted by Phanx View Post
Okay, I couldn't help myself... I looked at the code. I think we have a strong contender for Worst Addon Code Ever Published. The original author apparently doesn't even know about "elseif" statements, and don't seem to realize that indices in Lua start at 1 instead of 0. Everything is a global, there are tons of "if X then if Y then Z end end" constructions that should be consolidated to "if X and Y then Z end", and basically it's just all bad.
I was thinking the same thing.

And even worse, those globals are prefixed with BS: BS_GeneralSettings_Trigger_Criticals_Swipes. CamelCase variables mixed with underscores? You'd think BS actually stands for bull**** :P. And if variables can be prefixed, then they can be collected in an array.

Originally Posted by Resike View Post
If i'm gonna have some time in the weekend i might be able to update it for 5.2.
Have "fun"!

Last edited by ravagernl : 05-17-13 at 12:45 PM.
  Reply With Quote
05-17-13, 01:29 PM   #9
myrroddin
A Pyroguard Emberseer
 
myrroddin's Avatar
AddOn Author - Click to view addons
Join Date: Oct 2008
Posts: 1,240
Originally Posted by Phanx View Post
Okay, I couldn't help myself... I looked at the code. I think we have a strong contender for Worst Addon Code Ever Published.
I was curious, and now I'm down to 8 lives.

Phanx barely touched just how bad this is. And someone wants to fix it? Better to do a complete rewrite, keeping nothing but the textures.
  Reply With Quote
05-17-13, 04:03 PM   #10
Resike
A Pyroguard Emberseer
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,290
Originally Posted by myrroddin View Post
I was curious, and now I'm down to 8 lives.

Phanx barely touched just how bad this is. And someone wants to fix it? Better to do a complete rewrite, keeping nothing but the textures.
Nah even the textures are bad, should have been targa and RLE compressed imo.
  Reply With Quote
05-29-13, 06:55 PM   #11
ravagernl
Proceritate Corporis
Premium Member
AddOn Author - Click to view addons
Join Date: Feb 2006
Posts: 1,176
Originally Posted by Phanx View Post
If the rest of the code is even half as bad as that, the whole thing needs to be nuked from orbit and rewritten from scratch.
You know what, if anyone ever decides to do so, please use animation groups.
  Reply With Quote
05-29-13, 09:53 PM   #12
Dawn
A Molten Giant
 
Dawn's Avatar
AddOn Author - Click to view addons
Join Date: May 2006
Posts: 918
Don't care how badly written the code is, it's a very fun addon. Someone should rewrite it, for sure!

Even better would be using the bloody textures to indicate low player health! The lower the health the more blood, possibly pulsing screen corners and a heartbeat sound. Yeeeessss that would be it ... *evil genius grin*
__________________
Rock: "We're sub-standard DPS. Nerf Paper, Scissors are fine."
Paper: "OMG, WTF, Scissors!"
Scissors: "Rock is OP and Paper are QQers. We need PvP buffs."

"neeh the game wont be remembered as the game who made blizz the most money, it will be remembered as the game who had the most QQ'ers that just couldnt quit the game for some reason..."

  Reply With Quote
05-30-13, 11:24 AM   #13
reative_pl
A Murloc Raider
 
reative_pl's Avatar
AddOn Author - Click to view addons
Join Date: Jul 2009
Posts: 8
Code:
dKtgiaGEfL6LQQi7srX2eL2NIsMjbmBvCtcKFPOIVHe7uc2l1ULQ9RQQ(PiPHjQgNk1qfjAWQQ0WHshurvhtroNkzHIIBRKfdflNOhQQkpfSmO65sAIIeMkQAYsz6KUOiLtJ4YcxhLnkH(nKnJuBNG(OQkmnr8zu57kQ0ZKOhRQmAcz8IuDsvvu3IqDnL68iPxtG6VQYHvypzEddZB1q1W0mjZqzMPCdldndnZBi9uPPRH6IzNUW37ek3t43uYZVmT4KBdutLMUgQlkMsY(AAkzk5kj7uE2BtlozBOgh6uIiD1qMu55tvqcKwrEdQKWXfsZBOYWecJUcmAAs6gSAqgCeuNNk2WaNeXWAOZw8schxiRUWKbsZqMdc1(tsudh0OzEds0jmVHf7OeZB1Qvdr6yh9w0CgRg(mVlmzEdutLMUgQlENq5kPCAFtbp7(6k30It2gwOwCEyhQidqkttabG5K5GqnQPLo2dQsq9zw4gujHJlKM3aRgVXcX6QXyOIyYVG1jQbSNb1qAOXKdLG6gMmmwiwxnSZn8XQkA9erAHbLSIMbLk2Wqim6QHkdtim6kWOPjPB4LI5f08)indJfI11hcBihksAywgy14vrm5xymgy14HegsbtohJXaRgVpjrjFJJZyiegD1WFOfMH()VcG4mW6HWORguYkAgy14vrm5xW6e1zmWQXRfvL1jQXyOfvL1jQbDi5c1QbrbHtK6IB8Rn(oFpHNnFVZVnT4eCdh0OzEds0jmVHf7OeZB1QbwnEFOfMH6mwnidocQZtfByyHAXXWc1IZB2UaUbsZqMdc1mSqT48O6Iz3L54txzPKKuU33BtlEJBGwowbOYHoCHedZL0urU4nE2YRYY8RYDjL7eCtlob3qNT4LeoUqwDHjdJwJOeuFCEQKWXfYQ5TA1fWnVbQPstxd1fVtOCLuoTVPGNDFDLBAXjBdlulopSdvKbiLPjGaWCYG6X7pQFhupE)8c74tNzHBqLeoUqAEdSA8gleRRgJHkIj)cwNOgWEgudPHgtoucQByYWyHyD1Wo3WhRQO1tePfguYkAguQyddHWORgQmmHWORaJMMKUHxkMxqZ)J0mmwiwxFiSHCOiPHzzGvJxfXKFHXyGvJhsyifm5CmgdSA8(KeL8nooJHqy0vd)Hwyg6))kaIZaRhcJUAqjROzGvJxfXKFbRtuNXaRgVwuvwNOgJHwuvwNOg0HKluRgefeorQlUXV24789eE289o)20ItWnCqJM5nirNW8gwSJsmVvRwnidocQZtfByyHAXXWc1IZB2UGyCdKMHmheQ9)V4gwOwCEuDHPzW3gOLJvaQCOdxO0WCjnvKlEJNT8QSm)QCxs5ob30ItWnmAnIsq9X5PschxiRM3QHoBXljCCHS6ctwDHsZBi9uPPRH6IuUVN2xjzlXNYVZsb30I35gwOwCEyhQidqkttabG5KIyOI(PoHtK(YCqOgOsIGJzw4gujHJlKM3aRgVXcX6QXyOIyYVG1jQbSNb1qAOXKdLG6gMmmwiwxnSZn8XQkA9erAHbLSIMbLk2Wqim6QHkdtim6kWOPjPB4LI5f08)indJfI11hcBihksAywgy14vrm5xymgy14HegsbtohJXaRgVpjrjFJJZyiegD1WFOfMH()VcG4mW6HWORguYkAgy14vrm5xW6e1zmWQXRfvL1jQXyOfvL1jQbDi5c1QbrbHtK6IB8Rn(oFpHNnFVZVnT4eCdh0OzEds0jmVHf7OeZB1QvdYGJG68uXggwOwCmSqT48MTlig3aPziZbHA))BIHfQfNhvxmBz(oV86kNY90obpX0I3LgOLJvaQCOdxO0WCjnvKlEJNT8QSm)QCxs5ob30ItWnmAnIsq9X5PschxiRM3QHoBXljCCHS6ctgOMknDnuxmHY9oNs5vzE5EtHsUPfNuA1QbHUWusEUvBa
This is blood screen for Weak Aura. When you make crit you get blood animation on screen.
Just to show that you only need Weak Aura and no others mods.

This is really simple but feel free to make something amazing

Last edited by reative_pl : 05-30-13 at 11:39 AM.
  Reply With Quote
05-30-13, 03:43 PM   #14
reative_pl
A Murloc Raider
 
reative_pl's Avatar
AddOn Author - Click to view addons
Join Date: Jul 2009
Posts: 8
Originally Posted by Dawn View Post
Don't care how badly written the code is, it's a very fun addon. Someone should rewrite it, for sure!

Even better would be using the bloody textures to indicate low player health! The lower the health the more blood, possibly pulsing screen corners and a heartbeat sound. Yeeeessss that would be it ... *evil genius grin*
It's no problem to make this idea in Weak Aura, really easy... But... i don't like concept. You have low life and you don't see 50% screen because you see only some blood graphics? Some sound, some pulsing screen corners ?! ...and you die, and say WTF was that?! Some sh** addon! Then maybe I turn off this in pvp? ...and in raids... and instances... and... nah, that was sh** addon - most people say.

But i like concept with pulsing screen corners and a heartbeat sound. I do this. Some low pulsed corners and sound - nothing more. Will be nice addon ,to say: "Hey! You have low HEALTH!"
  Reply With Quote
05-30-13, 04:14 PM   #15
Resike
A Pyroguard Emberseer
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,290
Originally Posted by reative_pl View Post
It's no problem to make this idea in Weak Aura, really easy... But... i don't like concept. You have low life and you don't see 50% screen because you see only some blood graphics? Some sound, some pulsing screen corners ?! ...and you die, and say WTF was that?! Some sh** addon! Then maybe I turn off this in pvp? ...and in raids... and instances... and... nah, that was sh** addon - most people say.

But i like concept with pulsing screen corners and a heartbeat sound. I do this. Some low pulsed corners and sound - nothing more. Will be nice addon ,to say: "Hey! You have low HEALTH!"
Well you could set the fadeout time to lower values, so it didnt cover your screen all the time, also with opacity setting it woulnt really matter.

Edit: An awsome addon idea just hit in, Metro stlye gasmask effect with blood, and you have to use a key to wipe the blood from your screen.

Last edited by Resike : 05-30-13 at 04:16 PM.
  Reply With Quote
05-30-13, 05:32 PM   #16
Dawn
A Molten Giant
 
Dawn's Avatar
AddOn Author - Click to view addons
Join Date: May 2006
Posts: 918
Originally Posted by reative_pl View Post
It's no problem to make this idea in Weak Aura, really easy... But... i don't like concept. You have low life and you don't see 50% screen because you see only some blood graphics? Some sound, some pulsing screen corners ?! ...and you die, and say WTF was that?! Some sh** addon! Then maybe I turn off this in pvp? ...and in raids... and instances... and... nah, that was sh** addon - most people say.

But i like concept with pulsing screen corners and a heartbeat sound. I do this. Some low pulsed corners and sound - nothing more. Will be nice addon ,to say: "Hey! You have low HEALTH!"
Yeah, what I actually meant is something more appropriate. Like some pulsing bloody textures on the edges of the screen. Fading in some more layers/textures the lower you get. Since animating flowing blood is off the table this is probably as close at it gets to get the feeling of flooding your screen with red juice.

Just watched some Let's plays of some survival shooters (Metro First Light and Resident Evil 6) and it's pretty much exactly doing what I had in mind for such an addon.

TLDR; the way BloodyScreen fades in and out some textures is visually neat. Just the position and condition that triggers it is kinda off.

The splatter in the middle of the screen is definitely not good for a fun gameplay experience in the long run.
__________________
Rock: "We're sub-standard DPS. Nerf Paper, Scissors are fine."
Paper: "OMG, WTF, Scissors!"
Scissors: "Rock is OP and Paper are QQers. We need PvP buffs."

"neeh the game wont be remembered as the game who made blizz the most money, it will be remembered as the game who had the most QQ'ers that just couldnt quit the game for some reason..."

  Reply With Quote
05-30-13, 05:35 PM   #17
Dawn
A Molten Giant
 
Dawn's Avatar
AddOn Author - Click to view addons
Join Date: May 2006
Posts: 918
Originally Posted by Resike View Post
Well you could set the fadeout time to lower values, so it didnt cover your screen all the time, also with opacity setting it woulnt really matter.

Edit: An awsome addon idea just hit in, Metro stlye gasmask effect with blood, and you have to use a key to wipe the blood from your screen.
Hehe, that's exactly what I was talking about. Posted at the same time.


E: Just to point out, the broken gasmask visual was kinda stupid. It's not like a broken glass doesn't impact the functionality of a gasmask, at all. Does it? Yet you can still use it and extend the duration by swapping filters. *facepalm*

The way they splattered your screen with blood (and dirt) and the ability to wipe it off was really fun though.
__________________
Rock: "We're sub-standard DPS. Nerf Paper, Scissors are fine."
Paper: "OMG, WTF, Scissors!"
Scissors: "Rock is OP and Paper are QQers. We need PvP buffs."

"neeh the game wont be remembered as the game who made blizz the most money, it will be remembered as the game who had the most QQ'ers that just couldnt quit the game for some reason..."


Last edited by Dawn : 05-30-13 at 05:42 PM.
  Reply With Quote
05-31-13, 12:15 AM   #18
Resike
A Pyroguard Emberseer
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,290
Originally Posted by Dawn View Post
Hehe, that's exactly what I was talking about. Posted at the same time.


E: Just to point out, the broken gasmask visual was kinda stupid. It's not like a broken glass doesn't impact the functionality of a gasmask, at all. Does it? Yet you can still use it and extend the duration by swapping filters. *facepalm*

The way they splattered your screen with blood (and dirt) and the ability to wipe it off was really fun though.
Well the gasmask's functionality not just to don't let the gas reach your lung, but to cover your eyes against other type of gas like tear gas.
So it could function perfectly even with totally cracked screen.

http://farm4.static.flickr.com/3080/...82388578_m.jpg

Last edited by Resike : 05-31-13 at 12:17 AM.
  Reply With Quote
05-31-13, 06:10 AM   #19
Resike
A Pyroguard Emberseer
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,290
Gas Mask

Okay i managed to rip pretty much every files from Metro, and made a small showcase about the gas mask effect.

1. Download the latest alpha Power Auras:

https://github.com/Resike/PowerAuras/zipball/master

2. Import this Set into a new Page (Else it will overwrite the current page):

https://gist.github.com/Resike/5684577

And it's pretty much done. It's been made for 1680 x 1050 resolution, so you might need to change on the "Size" and the "Deforamtion" sliders a bit on every aura.

Also you can change color, blendmode, strata, animation, pretty much everything.

If a Photoshop master could change the base yellowish color of the glass to white i would appreciate it much.

Last edited by Resike : 05-31-13 at 11:07 AM.
  Reply With Quote
05-31-13, 07:47 AM   #20
Dawn
A Molten Giant
 
Dawn's Avatar
AddOn Author - Click to view addons
Join Date: May 2006
Posts: 918
Are you missing a texture file link, or am I just keep not seeing it?

E: Nevermind it's part of your power auras ... will check the textures in PS.

E2: ... attached the edited textures. note: I did it the fast way, maybe the alpha needs some tweaking, or something like that. Just treat as untested and report back if it looks ... "unexpected".
Attached Files
File Type: zip Custom_mask_rev1.zip (5.70 MB, 488 views)
__________________
Rock: "We're sub-standard DPS. Nerf Paper, Scissors are fine."
Paper: "OMG, WTF, Scissors!"
Scissors: "Rock is OP and Paper are QQers. We need PvP buffs."

"neeh the game wont be remembered as the game who made blizz the most money, it will be remembered as the game who had the most QQ'ers that just couldnt quit the game for some reason..."


Last edited by Dawn : 05-31-13 at 08:03 AM.
  Reply With Quote

WoWInterface » Developer Discussions » General Authoring Discussion » BloodyScreen, Renovation for MoP

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off