[nikhilgupta345]

Description:
This plugin allows players to return their weapons back to the store for a refund.

Client Commands:
/return - Returns current weapon.

CVars:
rw_percent <percentage> <default:0.80> - How much money players should get back. MUST BE A DECIMAL.
rw_buyzone <0/1> <default:1> - Whether players must be in a buyzone to return their weapons.

Changelog:

July 30, 2011  - v1.0 - Plugin Release
                    - v1.1 - Few optimizations
                    - v1.2 - Added support for shield return

Credits:
nikhilgupta345 - Code
XxAvalanchexX - ham_strip_weapon stock
Exolent - cs_set_user_shield stock

Installation:
-Click on 'Get Plugin' below
-Place 'weapon_return.amxx' into your plugins folder
-Add the text 'weapon_return.amxx' to the bottom of plugins.ini
-Restart server or change map

[delayedCoder]

Your first '0' constant isn't really necessary.

Otherwise, good work and great idea!

[nikhilgupta345]

It's easier to be able to use the users weapon id as an index in the array.

[abdul-rehman]

You have included Hamsandwich just for ham_strip_weapons which is not a good idea, remove ham and include fun and then use strip_user_weapons, bcoz its more effecient and much faster.

[ConnorMcLeod]

You don't need to check for weapon_knife and weapon_c4.
Instead you can :
- check if iWeapon is CSW_C4 or CSW_KNIFE
or
- check weapon cost before you send ham_strip_weapon and then check if price is > 0

You don't need to clamp money, just use min(cs_get_user_money( id ) + iRefund, 16000).

You could add shield return.

Other than that, code is ok.

[bibu]

Does already exist...

http://forums.alliedmods.net/showthread.php?p=472306

[xPaw]

You also should check if user is alive.

[nikhilgupta345]

Originally Posted by abdul-rehman You have included Hamsandwich just for ham_strip_weapons which is not a good idea, remove ham and include fun and then use strip_user_weapons, bcoz its more effecient and much faster.

I only need to strip the weapon their holding, and strip_user_weapons strips all the weapons that a player has.

Originally Posted by ConnorMcLeod You don't need to check for weapon_knife and weapon_c4. Instead you can : - check if iWeapon is CSW_C4 or CSW_KNIFE or - check weapon cost before you send ham_strip_weapon and then check if price is > 0 You don't need to clamp money, just use min(cs_get_user_money( id ) + iRefund, 16000). You could add shield return. Other than that, code is ok.

Alright, I just couldn't find a way to strip a users shield. Got any ideas?

Originally Posted by bibu Does already exist... http://forums.alliedmods.net/showthread.php?p=472306

Didn't find that before. I guess the only thing that makes mine different is the fact that I have grenade support + will be adding shield support soon if I figure out how to do it xD

Also, my CVar allows you to put a percentage, rather than dividing by a certain amount. And since it must be an integer in the other plugin, you have less choices for prices.

Approvers can determine if the changes suffice to make this plugin approved.

Originally Posted by xPaw You also should check if user is alive.

Easy enough.



Updated to version 1.1

[Emp`]

Will approve under these conditions:

• If rw_percent is set to 0, it should disable the command/plugin.
• If iRefund is 0, you should not be able to return the weapon.
• Name your variables correctly: iPercent -> fPercent
• Have it print in chat how much money you recieved.

[nikhilgupta345]

Originally Posted by Emp` Will approve under these conditions: If rw_percent is set to 0, it should disable the command/plugin. If iRefund is 0, you should not be able to return the weapon. Name your variables correctly: iPercent -> fPercent Have it print in chat how much money you recieved.

Done.

Any ideas about the stripping of shields by the way?