Redex

I apologise about all the errors caused. Thanks.

Try removing this } from line 45 and try adding back line 47.

Thanks for that Danny. Here is one thing you may/ may not have known, i like the longer version ;). I'm not mocking you buy that way, thought i would get my point across, whilst playing with a bit of comedy. I apologise, if it has caused you any inconvenience.

If you want i don't mind, as i said anything you post to help the members is fine with me. Dave, if i knew how to do that, i would be adding lots of features ;)

Thanks buddy.

I would make my own modification, if i had the most simplest of ideas, and knew how to code ;) Just to clarify, no i'm not one of them people who did not attempt to learn, i have been through tutorials on xhtml,html,php,sql so yup i do know the basics, yet when it comes to making a modification i just don't know what to do ;)

Thanks Jordan, hopefully people replies will give me motivation in securing some other files.

Thanks Mcfarlin, a slight edit has been made to add more security. If anyone has copied this code, before this post please copy it again. Thanks.

Hello fellow members of the Makewebgames community. I'm sure everyone will be surprised, to see me posting in the free plug-ins section for once. I decided, to secure bank.php with the help of a friend ( Zeddicus ) and i'm pretty happy with the job me and him have achieved. Since this is my first ever, free plug in, i am excited to the replies i will be receiving. All criticism is also accepted, but please try to make me feel happy ;)   <?php include(DIRNAME(__FILE__).'/globals.php'); print "<h3>Bank</h3>"; if($ir['bankmoney']>-1) { $_GET['action'] = (isset($_GET['action']) && is_string($_GET['action'])) ? htmlentities($_GET['action'], ENT_QUOTES) : FALSE; if(!in_array($_GET['action'], array('deposit','withdraw','index',''))) { echo ('blah'); } switch($_GET['action']) { case "deposit": deposit(); break; case "withdraw": withdraw(); break; default: index(); break; } } else { $_GET['buy']=abs(intval($_GET['buy'])); if($_GET['buy'] != is_string($_GET['buy'])) { echo ('error'); } $_GET['buy'])) if(isset($_GET['buy'])) { if($ir['money']>49999) { print "Congratulations, you bought a bank account for \$50,000! [url='bank.php']Start using my account[/url]"; $db->query("UPDATE users SET money=money-50000,bankmoney=0 WHERE userid=$userid"); } else { print "You do not have enough money to open an account. [url='explore.php']Back to town...[/url]"; } } else { print "Open a bank account today, just \$50,000! [url='bank.php?buy']> Yes, sign me up![/url]"; } } function index() { global $db, $ir,$c,$userid,$h; print "\n[b]You currently have \${$ir['bankmoney']} in the bank.[/b] At the end of each day, your bank balance will go up by 2%. <table width='75%' cellspacing=1 class='table'> <tr> <td width='50%'>[b]Deposit Money[/b] It will cost you 15% of the money you deposit, rounded up. The maximum fee is \$3,000.<form action='bank.php?action=deposit' method='post'> Amount: <input type='text' name='deposit' value='{$ir['money']}' /> <input type='submit' value='Deposit' /></form></td> <td> [b]Withdraw Money[/b] There is no fee on withdrawals.<form action='bank.php?action=withdraw' method='post'> Amount: <input type='text' name='withdraw' value='{$ir['bankmoney']}' /> <input type='submit' value='Withdraw' /></form></td> </tr> </table>"; } function deposit() { global $db,$ir,$c,$userid,$h; if($_POST['deposit'] =! is_numeric($_POST['deposit'])) { echo ('some error here'); $h->endpage(); exit(); } $_POST['deposit']=abs(intval($_POST['deposit'])); if($_POST['deposit'] > $ir['money']) { print "You do not have enough money to deposit this amount."; } else { $fee=ceil($_POST['deposit']*15/100); if($fee > 3000) { $fee=3000; } $gain=$_POST['deposit']-$fee; $ir['bankmoney']+=$gain; $db->query("UPDATE users SET bankmoney=bankmoney+$gain, money=money-{$_POST['deposit']} where userid=$userid"); print "You hand over \${$_POST['deposit']} to be deposited, after the fee is taken (\$$fee), \$$gain is added to your account. [b]You now have \${$ir['bankmoney']} in the bank.[/b] [url='bank.php']> Back[/url]"; } } function withdraw() { global $db,$ir,$c,$userid,$h; if($_POST['withdraw'] =! is_numeric($_POST['withdraw'])) { echo ('some error here'); $h->endpage(); exit(); } $_POST['withdraw']=abs(@intval($_POST['withdraw'])); if($_POST['withdraw'] > $ir['bankmoney']) { print "You do not have enough banked money to withdraw this amount."; } else { $gain=$_POST['withdraw']; $ir['bankmoney']-=$gain; $db->query("UPDATE users SET bankmoney=bankmoney-$gain, money=money+$gain where userid=$userid"); print "You ask to withdraw $gain, the banking lady grudgingly hands it over. [b]You now have \${$ir['bankmoney']} in the bank.[/b] [url='bank.php']> Back[/url]"; } } $h->endpage(); ?>     Best Regards: Redex.

Hello I'm Redex. Replying to your post, i would like to clarify, making a template is not a quick job. As far as my knowledge goes, it requires you to have knowledge of languages such as Css ( cascading style sheets ) Php, HTML. Once you have the knowledge them, you will have too familiarize yourself with how to integrate that header with McCodes. Well, as you can see to make a header.php for yourself, you will have to spend time learning these languages and learning them well. Please, if you decide to take my advice, do not try to rush it so you can implement it into your game, take time and go through each language and make you sure your clearly understand the main bits. After that, try search for some of the things you don't understand in Manuals. Example : http://www.php.net/ is the manual for php, after that you can Google, for other manuals. I would advise you, if you decide to learn the 3 languages posted above then, postpone your current project for at least a year and maybe more becuase after that, if you're familiar with the three languages you have a better chance, in progressing at a faster pace with your game. Sites http://www.tizag.com/ http://www.w3schools.com/ http://devzone.zend.com/article/627 These are some of the sites, which i would think would be useful, for a beginner like you, and they will give you a head start before attempting to read the manual. The reason, why i say this is becuase the manual explains everything in a more complex way in my opinion, so after you understand more then you can go read the manual. Thanks, i hope my post helps you. If you have any more queries, about which sites to learn from or just need some motivation, then you can contact me via email or on Messenger which is posted below: [email protected] Best regards: Redex.

That's Ben for you ;)

Ben, nice first modification try. Next time, try run over the code a copoule more times, and look at certain bits, then see if you could improve it + secure it + update it. Thanks Best Regards: Redex.

I'm assuming Gucci Mane want's a detailed modification. I've seen his skills, and Ben since your still a learner, and he started the learning process earlier than you i think he would have better skills than you, so he will be able to create the simple modification better than you. I'm not trying to put you down, in learning coding I'm just trying to make my point, obviously he can make the simpler version what he wants is detailed versions of both his ideas. If you look at the post, he has specified that the modification he wants is wanting to be like Torn.com and if something is meant to be similar to Torn.com's modifications they have to be good and detailed and secured properly, some of which i don't think you have the capabilities to achieve. Note : Please don't take this as a offensive post, think of it more like a informative post Best regards: Redex.

One more edit i'm thinking about is make it compatible with other modifications availaible on McCodes. Example if you make this and edit it into the code of Crystal temple for McCodes, then people can swap there tokens for anything in the crystal temple. This would also, be a posotive point for the game owner, because then donators will be buying more tokens, and will increase the role of tokens in the game. Donator feature - please let me explain in more detail. Along with the other donator packs, on your game you can add a pack called Tokens to the donator page in your game. One advantage of this donator feature could be, the Tokens purchased from the Donator section of the game, have better value. They are worth for example two normal tokens and this would increase the demand for the donator tokens in the game. Donators with less money and crystals, in the game could also take this as a trading opportunity, because they can fix a deal with a in-game member saying they will trade this donator token, for a set amount of money or crystals. Tokens, in donator feature, could also be implemented into the crystal temple for McCodes, because that way, you will be able to take some of the tokens from the game back, concluding to the sales of more donator tokens again. The way I'm thinking for this to work with crystal temple, is you have to exchange this donator token for a set amount of money in the crystal temple. One more addition, you have to do, if you implement the above into the modification is you have to have a section where it displays normal tokens and donator tokens so the members, are aware of how much they got of each. This would increase, the circulation of money,crystals and tokens in the game whilst making it more interesting for the users. Thanks Best regards: Redex.

Excellent, upgrade. Good detail, in the explanation, to tell the customers exactly what they are getting for there money. One improvement, in the post could be list emails where you can be contacted like: Yahoo, Messenger, Google mail. In addition to that, you could also list your paypal email, for anyone wanting to send you money, if your not online, then you can contact them from there paypal address, and send them the modification. Overall, it's a nice upgrade from the basic McCodesv2 script and I'm pretty sure it is going to be a successful modification and used by many. The style, and graphics, of the slots caught my attention, so that was a positive point as well, because no-one want's a modification which looks dull. Talking about the target market, i think you will be appealing this mod to both genders. Loyalty, I'm sure many people agree with me here, and we think your a loyal Website Developer, so we assume that if the customer has any problems, whilst installing the modification then you will help them right? I'm sure you will, just conforming for other people though. Nice use of different languages, and i would recommend, text based game owners to buy this, as it's obviously more detailed from the original, but also well coded i would think because it's a modification from Floydian. In my opinion, since this modification is going to be appealing to both genders it's a good investment, because if you want to edit this slightly then you could make the Tokens, a donator item for people to actually buy with real money, if they want more than which is provided in-game. Talking about making this tokens, into also a donator item, Floydian, if you want to make a donator add-on for this then I'm sure many people would like that as well. Since you have made two versions, it's good for the customers, because they can decide what features they want in their Slot Machine, and then choose the version which fits the needs of their game. Thanks for providing screenshots, because sometimes, Website Developers forget screenshots, and that becomes a negative side to the modification they are selling but you got it spot on. Good luck with the sales. Thanks Best Regards: Redex

Hello Dayo, I'm sorry for the criticism without no good comment first but i thought it would be good to clear this up first. I didn't look through all the code, but i saw on the first line of the code, it says include "tglobals.php" I'm pretty sure you meant include "globals.php" if not I'm sorry i might have misunderstood.

Hello Gwink, firstly, i would like to say, nice unique game you got their, some nice features and presented in a different manner to other games nowadays. Secondly, for me the way you have mixed simplicity with the detailed background doesn't work, because it starts to feel a bit odd in my point of view. Example : The Grey simple background on log in and the detailed background on the sides of that - the one that takes the majority of the space up. I think, they do not blend in together well, so that was the downside for me. On the whole, it's a nice unique approach to the game, with some unique and entertaining ideas, and we can clearly see you have put lots of effort into making it, so good job. I wish this game is a success for you, and if i can help you with anything let me know ;) . Best Regards : Redex.

I always provide luck for everyone :P ;)

2 sales of this mod, and he will be enjoying a cheeseburger in a little cheeseburger shop :P

One piece of advice my friend, don't go lowering your price becuase people criticize your work, believe in your work and have the confidence to stand your ground and prove your right, well hope this helps.

He loves his cash though :P

Hey Gwink, i think I'll vote for option one please. The reason for my opinion, is becuase if you have a good unique game, however have not got all the exciting features becuase it is still new or for any other reason, then the target market you should be aiming at is inexperienced RPG players, this would be ideal becuase they have never seen another game to compare the bad aspects of your game to others and they would be discovering something new and exciting which will keep them attached to the game for longer and they would not get bored of the current default McCode feautures ( if your using that engine ) until you add more new exciting features. One more reason why option one would be good, is becuase for new players is that if they have not played a RPG game before this one, it is unlikely for them to have people on the Messenger or other social networking sites who have played a text based game before so your looking at more referrals as well, and then the cycle will keep repeating for the new players which will join so that would be a good advertising way for your game as well. Finally, the new players will not get bored of the game and are likely to enjoy and stay on the game for longer than other players so you might even be getting a few donations from that view becuase they would want to test out the features you would be getting as a donator becuase they would not have experienced that before. Hope this helps Redex

OK - update: downloaded Zip, anything else i need?

I know i cannot code, but hopefully i will be able to implement some of my ideas into this engine, and give some feedback towards it. I can work well with people and i have a bit of knowledge about php,html,xhtml so can i join?

Your very welcome.