G7470

Just going to note from comments above... Taking from experience with programming within a semi-corporate environment, the client never knows exactly what they want. Plain and simple. They may not even have the slightest idea in their head as to what it will look like and what it will do. Usability is one of the key elements to whether or not a project can be considered "successful". There are others, which include meeting the deadline and meeting the given budget. When building any kind of web application (or web pages in this case), the code should be very readable not only for you, but for the client too. I know that is something that many people will debate on whether or not it is important, but at least in my eyes it clearly is important. If the client can read and understand at least through your comments what the code is doing, there is a much better chance that you will have a return customer, which in terms of freelancing is one of the greatest feelings in the world to have because with a return customer you know that you are doing something right. I have had some successful and also some failed projects within the semi-corporate environment I work in as well as with the work I do for fun, but usability of the clients' users and knowledge transfer is one of the greatest assets you have as a programmer. Don't be afraid to share your knowledge with others, as who knows: maybe they will be making that £500k investment and will not be afraid to give you a nice chunk of that change just for your great help and advice to get them there. If not even for the money, just knowing that the knowledge base of development can continue to grow and expand into technologies that are not even in existence yet as well as making some great new friends in the process. :) ~G7470

*shrugs* I wasn't too interested on seeing it. The controversial topic + a comedy does make it seem more appealing, but eh, just wasn't my cup of tea. Hopefully it will be released on demand at some point. I have a feeling it won't be though. ~G7470

I agree with [MENTION=69670]Script47[/MENTION] on this. The main links should be in between latest news and the date. There's plenty of empty space there to put at least a few main links there so that those are much easier to find for the users. ~G7470

Nice, clean layout. Simplistic interface yet has its complexities. Very well done. :) Do you have any set idea of the bonuses/differences between the different races/classes? ~G7470

[MENTION=65073]lucky3809[/MENTION] if GRPG was in fact placed under the Open Source License, then yes, re-selling the original code itself is a violation; however, if that legacy code is different from back when it was supposedly licensed as open source until now, then it is 100% valid. In any case, I had quite some fun going through your previous posts for threads, and I could not find one post that validated your claim the GRPG was in fact licensed as such. Maybe a pirated copy was done this way because, as you probably know, publius was away from GRPG for quite some time, and during that time, a copy of the code may have floated around and placed on this license at that point in time. You really do not know as nor do I of these claims, which is why engine/code ownership/licensing is such a tricky and dicey task without having an official site or point of validation. Take MCCodes for example. They have a validation point, but who knows that every site that has MCCodes actually purchased their license? At this point in time, I would say that if you have a copy of the Open Source License, then leave it as such. The claims of which are going back-and-forth in regards to GRPG will really not allow us to grow as a community if we shoot down attempts at making our development skills and code base better. ~G7470

The forum troll at his finest. :) ~G7470

Wow is the post you referenced confusing, yet you say it so straight-forward like it all makes perfect sense. Regardless, in most cases with licenses, licenses can become null and void at any time. Terms of Use of said licenses normally has this clause in there in order to provide liability in cases like this. If I were you, if you are planning on selling this engine, make sure you have a specific Terms of Use for the new licenses so that this kind of thing does not happen again (like a change of ownership of the engine clause). Even with what was noted above, with many engines, the core of it is very similar, so it is extremely difficult to say what belongs to what anyways. Just recode it, make it better, and the rest of us will provide you with feedback. ~G7470

I wouldn't say that. I receive requests for mods from people that have MCCodes, GRPG, and other engines. The engines aren't "dead" really, it is what you do with what you have. ~G7470

It's slightly different haha. Looks like there was an added ) onto it from his post. Might have been just a typo of some sort. Those happen to me all the time. I got your PM, no worries. I responded. ~G7470

FYI, from the posts above, the specific code that you just now referenced was not there as written. Please make note to update your code appropriately so that we can provide more accurate assistance. :) ~G7470

Ok. In that case, it is the insert query itself. If data does in fact exist in the soldier table, verify that the auto_increment current value is set appropriately. So say your latest row in the table has a soldierid of 4. Then, your auto_increment value should be set to 5. In cPanel, that can be checked through the operations tab I believe in PhpMyAdmin once that table is selected. I say this because I have ran into this issue in the past, and the query itself seems to look fine to me. EDIT: Just to note, the auto_increment value only has to be greater than the largest value. It does not necessarily have to be the very next value, but that is more appropriate. I say this because the only real issue with auto_increment is making sure that the auto_increment value that the table is expecting cannot have already been used, as this would be a primary key violation. ~G7470

I agree with [MENTION=70347]NonStopCoding[/MENTION] I'm thinking it has something to do with that $_POST variable, but in a little different way. I would echo out the $soldiername before running the query to verify that there is a value going in there as well as the $user_id fields, like the following:   while ($row=mysqli_fetch_array($run)){ $user_id = $row[0]; } $soldiername = $_POST['soldiername']; echo $soldiername . " "; echo $user_id . " ";   One you have verified that those values are being filled in, you can remove the echo statements, but that is a good way to error check. If one of these values are not getting filled in, we can go from there. ~G7470

When this query runs, check the data in the soldier table. Are all of the fields of the soldier table filled in correctly? If not, then that could be where your problem is. If all of the data in the soldier table is in fact correct, then the issue is not directly related to the insert; rather, it is the display. There must be some kind of condition or other issue that is causing the "soldier field" to be empty. Do you have to maybe refresh the page to get the soldier to display? Not sure what else I could advise you to do without actually seeing this in action or doing a full code trace. ~G7470

[MENTION=69409]The Phantom[/MENTION] my advice to you is if you don't understand the code that was posted, then don't use it. Yes, the number of lines was reduced, but at what cost? If you as the developer do not understand your code, then how can you fix any bugs that arise? You can't. So keep that in mind. ~G7470

Well "secure enough" is something that is really your opinion on whether it is or not as you can go crazy with securing your code. The biggest thing you want to do though is to validate the user input (such as the $_GET variable(s)) as this is the most common security problem. You're just about there, but you're not validating whether $_GET['st'] is a number or not in order to run line 15 appropriately. So what I would do is replace lines 11-14 with this:   if(!$_GET['st'] || !is_numeric($_GET['st'])) { $_GET['st'] = 0; }   For reference on the is_numeric function: http://php.net/manual/en/function.is-numeric.php

Ahhh, that guy. His name is very familiar to me. He asked me to fix up something for him with $$ in exchange. I supplied the fix and haven't seen any $$ from him. ~G7470

Be careful when doing stuff like this because in most cases, a user is spamming like that without the owner's knowledge. I've had that happen to me several times before, and when I was an owner, I explicitly stated to not spam. Vigilante justice can be useful, but only on those of which who are encouraging as well as actively participating in such behaviors. With that in mind, where's your proof that this guy is the owner or that the owner encouraged this spam? ~G7470

For a simple game this may be harmless, but in a corporate environment, this is just asking for trouble. Sending passwords as raw text creates numerous security holes in the application,which can cause anywhere from a white hat to the game being completely compromised. Anyways, back on topic, I agree with many others' posts here. Make the login and register much more appealing to users, and you may see a much higher influx in the number of registered users in the game. ~G7470

I have done something similar to this on a gRPG-like engine (but it was custom). Took a long time to do in order to iron out all of the kinks, but it is not terrible. Just make sure you give yourself plenty of time to test out a change like this before running it on a live game environment, as changing something like this will change A LOT about how your game runs. I unfortunately am fairly busy with other projects at the moment, but I can help with some guidance to do it if you would like that. If so, feel free to send me a PM or email me directly (email is under my visitor message). ~G7470

I did a marathon of making a windows app using Construct. Very interesting piece of software. The app is not live at all, so unfortunately I cannot really give you a preview. :( ~G7470

What you will want to do first is run a query (if not already done) to see what users are in their clan:   $res = $db->query("SELECT * FROM `users` WHERE `gang` = '".$ir['gang']."'");   OR, if you do not have the current user's clan saved in a variable or in a class:   $res = $db->query("SELECT * FROM `users` WHERE `gang` = (SELECT `gang` FROM `users` WHERE `id` = '".$ir['userid']."')");   Then, once you have that information, you can then create a loop like this:   echo "<select name='vpdropdown'>"; echo "<option value='0'>None Selected</option>"; while ($qres = $db->fetch_row($res)) { echo "<option value='".$qres['userid']."'>".$qres['username']."</option>"; } echo "</select>"; Then, once you have that, all you would have to do to find the value is to do is get the value of the "vpdropdown" using $_GET or $_POST as well as check to make sure that the value is not zero. I would also run a check (if not done already) to verify that the ID of the user given is in fact in the clan. Hopefully all of that makes sense haha. If you have any questions, let me know or post here so others may answer. ~G7470

If you don't plan on making money off of this, then my suggestion would be to use a free hosting platform such as 000webhost or x10hosting. They will provide you with a URL and hosting for free as long as you follow their terms and guidelines. I like to use these kind of free hosting sites for development sites, but they can work on a production site as well. Just keep in mind that with free hosting comes a bigger probability of outages. ~G7470

I would suggest putting in your own text instead of leaving the dummy text on your web pages. That would help attract some more attention, as many people looking for hosting these days would see that and walk the other direction. Regardless, I wish you the best of luck. As what Peter said, there is a lot of competition out there. ~G7470

An account will not necessarily help with much (if anything at all) in terms of this issue, but I have an account. However, PM me here if you need help. ~G7470

Looks like there are some things wrong with this picture...   <script> is the incorrect tag for css. You should use <style> instead to indicate css. So, your css should look like this now: <style> img { position: absolute; background-image: url("{$ir['display_pic']}"); width: 60px; hight: 60px; left: 0px; top: 0px; z-index: -1; } </style> Not sure why you're including PHP code in the css. This really should not be necessary. Also, not sure what was originally in the "background-image" property (if any), but this should not be changed. So, with this in mind, your css should now look like this: <style> img { position: absolute; background-image: (what was originally here. if none, delete this property) width: 60px; hight: 60px; left: 0px; top: 0px; z-index: -1; } </style> Verify that the path to the avatar images is correct (if uploaded). If the value of $ir['display_pic']} is only the file name for an uploaded image, this will not work with how it is currently coded. Do realize that by adding css to the <img> tag (which you are doing with that css) that this will change the properties for ALL images that are coded in such a fashion regardless of what they are. If you have taken this into account, then that's good!   Feel free to respond to this post or PM me if you have any questions. :) ~G7470