[Blizzard_87]

Death From Above
Version 1.0.2
By Blizzard_87

ThreadStart
Description
This Plugin Allows The Player To Purchase An Ability Called 'Death From Above'
Once Purchased The Player Can Then Use His +jump Button To Leap High Into The Air
Then Once At Height Player Likes Then Presses +duck Button To Plummet Back To The Ground Thus
Creating A Explosion Dealing Damage/Killing Nearby Enemys.

Cvars
dfa_mode

• "0" = Plugin Set To Off
• "1" = Explosion Damage Dealth From Fall Dmg
• "2" = Explosion Damage Dealth From dfa_dmg_amount cvar amount

dfa_cost

• "1000" = Price In Money For Purchasing DFA

dfa_debug

• "0" = Sets Debug Messages Off
• "1" = Shows Amount Of Damage From Fall/Cvar Damge In Client_Print Msgs

dfa_acend_power

• "1000.0" = The Velocity Power Player Gets For Jumping

dfa_decend_power

• "2000.0" = The Velocity Power Player Is Plummeted To Ground ( Higher Number Faster Player Falls ) Higher Fall Velocity Higher Damage On dfa_mode "1"

dfa_dmg_amount

• "200" = the damage dealt within the explosion if dfa_mode "2" set

Client Commands
say /dfa = Buys Death From Above Ability.

Credits
Blizzard - Author & Idea
hornet - got bitwise from him off another plugin i did with his help and on ham enable / disable forwards

ChangeLog
---------
October 20/10/2013 - v1.0.2

• bug if pressing crouch multi times whilst on ground would kill player

October 19/10/2013 - v1.0.1

• Fixed Flying Glitch Being Able To Multi Jump

October 18/10/2013 - v1.0.0

• Original Release.

Plugin Download Count
30+ Current

ThreadEnd
.

[Napoleon_be]

seems nice for fun servers that are using rtd & stuff
good job

[Jhob94]

Is that Deidara's Jutsu?

[nikolov]

Probably you shoud to make timer after using JUMP, because you can easily fly. Smth like: "You have 5 seconds to press your DUCK key or it will be pressed automatically..."

[Blizzard_87]

Originally Posted by Napoleon_be seems nice for fun servers that are using rtd & stuff good job

thanks

Originally Posted by Jhob94 Is that Deidara's Jutsu?

Huh?

Originally Posted by LordOfNothing Make a video

i dont have a public server to be able to... if someone wants to they can ?

Originally Posted by nikolov Probably you shoud to make timer after using JUMP, because you can easily fly. Smth like: "You have 5 seconds to press your DUCK key or it will be pressed automatically..."

i could or i could just make it so after you use JUMP, it doesnt let you jump again while in the air... which would stop the glitch of flying


EDIT:

Plugin Updated Version 1.0.1

Fixed Flying Glitch.

[Blizzard_87]

Updated Plugin v1.0.2

[ FIXED ]
bug if pressing crouch multi times whilst on ground would kill player

[HamletEagle]

1.cm is kinda pointless, amxx already provide charsmax(), which is esentially the same thing. But for consistency you should use charsmax.
2.

PHP Code:

if( is_user_alive( id ) && get_pcvar_num( g_pMode ) ) 


Checking again get_pcvar_num it's pointless, you will reach here only if it's not 0(you do a check above this line).
3.

PHP Code:

set_task( 0.9, "ShowHud", id, _, _, "b" ); 


Don't create a task for each player, make a global one from a place like init and loop all players inside it. You could use directly 1.0. Also set the hudmessage hold time the same as the task time.But, after all, I am not sure such hud is needed, you can likely let only the message when player buy the power, this should be enough.
4.Remove the stock keyword from your functions.
5.What you do with float_to_num is silly, look for floatround.
6.AMXX 1.8.3 has a native for colored messages, so you could check the version and decide if you will use your stock or the native.
7.Any specific reasons for defining FALL_DMG given that we already have a DMG_FALL defined in amxx ? So, remove FALL_DMG and use DMG_FALL. Also bitsDamageType == FALL_DMG should be bitsDamageType & FALL_DMG.
8.cs_get_user_team should be retrieved inside no_team_dmg, or ExecuteExplosion(but only one time, so cache this value).
9.In Reduce you call again cs_get_user_money, but this value is cached inside CmdDeathFromAbove.
10.Add ML, it's easy and make plugin more user friendly.
11.g_pMode cvar check should be done inside CBasePlayer_TakeDamage and passing the right damage value to ExecuteExplosion. Checking inside ExecuteExplosion and only using the value if the cvar is 1 is weird. This way it will make more sense.
12.You could likely remove the debug messages and the cvar, it's of no use for the end user. Do you have a specific reason for putting them in a released plugin ?

A bunch with which you can start.

[Blizzard_87]

When I get time I'll have a look into this. As I made this year's ago. I'll probably just re write it.

[HamletEagle]

15 days have passed and I don't see any changes from the author. Since section needs to be cleaned I'll unapprove this for now, but don't worry, your plugin can be reviewed at any time. PM a plugin approver after you made the changes.

[Blizzard_87]

Yeah no worries. Just been real busy. Once I get to this I'll pm you.