sniko

[uSER=50433]ColdBlooded[/uSER] is 2006, too

Tagging a user to this post because they design like a god: [uSER=69001]Zettieee[/uSER]

  Sketchy. You don't have enough time to potentially make $1,000? It should also be noted that these sources most likely store the users passcode in plaintext or md5 (which is considered broken) and should not be used for passcodes. Also, don't pay $1,000 for a source like this in 2015/2016. http://pastebin.com/VHHucfjA I don't care if you call me for slander; $1,000 for insecure sources like this is a rip off. (Plus, it's only slander if it's not true; my claims are true.)

Sadly, I cannot. :( It's all gone. Everything. Gone.

  I played a lot when it first came out, then realised it's pretty much the same grind over and over and over and over again. I'm a CS:GO fan :)

Hmm, maybe Dave Macaulay made it so nobody can post in there anymore with this upgrade.

IIRC, only staff can reply in News and Announcements threads, so this is just displaying confusing text - as you're not staff.

What you're referring to is this thread, which relates to a job that was started the year before, month 10. In that thread, everything was marked as resolved, because it was. Anyway, here's the story... It was over a year ago, I charged you $200usd for a login system, which was completed, and just needed configuring for 3rd parties to use - it was a custom OAuth system. You then started scope creeping me, and in the end you got the authentication api, a portal GUI and login, and dashboard. I asked for more money to complete your requests but because you had unfortunately lost your job you couldn't but kept asking for more things to be done (included in the 200$)   Everything was uploaded to the gaming-social server and my private GitLab instance, but the gaming-social server has since been wiped, so I've no clue what you want. Please elaborate. I've opened Skype today and seen various messages from you. I don't use Skype anymore, please find me on Gitter.

This won't allow for multiple clocks on the same page. But nice!

See: http://makewebgames.io/showthread.php/45562-UOTS-engine-owner-switch?p=309648&viewfull=1#post309648 That thread details that IceCold doesn't own the engine anymore... [MENTION=65073]lucky3809[/MENTION] you might wish to read that to be updated :) Post that identifies [MENTION=69762]RCengineOwner[/MENTION] as IceColdCola: http://makewebgames.io/showthread.php/43304-Ruthless-city-engine-legal-notice?p=289623&viewfull=1#post289623

Below is a compiled list of constructive criticism.   The validation on line 16 is unnecessary as it will be handled within the switch cases. You're missing the $h->endpage() call to nicely end the HTML on line 5 and line 8 You don't need to bring this many variables into the local scope of your first function on line 86. You're only using $ir. Hardcoded values are a nuisance on line 209. The form on line 215 is wasted. See http://makewebgames.io/showthread.php/46088-In-Production-Brewery-Mod?p=312017&viewfull=1#post312017 You can make these queries into one on line 269. What is this if() statement for on line 273? Perhaps make use of max_energy (or whatever it's called) in the formula on line 290 so that you don't go over your max energy? It may be a nice idea to make the item id's a constant for more flexibility, for example on line 420 How about making the brew time (on line 479) to take into account a users skill, so it's not always constant (thus boring) See http://makewebgames.io/showthread.php/46088-In-Production-Brewery-Mod?p=311984&viewfull=1#post311984 Why not create another table instead of more columns in an already-crowded users table? Why BIGINT?   Overall, nice for your first. However, it could be refactored for easier flexibility, expandability, and less functions with hardcoded values.

That will result in a parse error, thus a white screen, thus nothing. Okay, I'm done. Starting to think you're a troll.

See: http://makewebgames.io/showthread.php/46077-Lottery-Mod?p=312094&viewfull=1&highlight=you%27re%20not%20supplying%20the%20correct%20GET%20value%20in%20the%20code%20field#post312094

Yep, nice spot. :D   Entering what? Do you have anything in the query string?

Then I would assume you're not supplying the correct GET value in the code field and it's just running up until exit;, thus no output. How are you manually running the cron?

Then run the file manually and check the output. There will be output because of your or die(mysql_error());.   Have you forgot to run the SQL statements to create the tables/columns? Have you misspelled a table/column name? Do you have an open connection to your database?

  Not running at all. The queries or the cron? Please be more specific when describing a problem.   Seriously change the way you code. Your editor (or you) is adding stars, which will result in a parse error, which will result in a FATAL error, which will result in the file not running.   Tip:Some simple debugging will help you. Look at your error logs, put verbose error reporting on (error_reporting(E_ALL); ini_set('display_errors', 1);) and work from there. Don't work blind and rely on people always helping you because you cannot be bothered/don't know how to debug. Note:When describing a problem, please be more descriptive! A report like "Not running at all." is ambiguous (what isn't running? the file? the cron? the query?) and definitely doesn't help us debug it. Obligatory:http://blog.teamtreehouse.com/how-to-debug-in-php (Please read.)

What error/issue are you encountering with the cron?   Cron doesn't run at all Cron doesn't run at the correct time Query fails What fails about it? Have you debugged it? How did you debug it?    

No problem. Glad I could be of assistance. - Jedi, out!

  Aaaaaand to be that guy, you can't have "-" in variable names or constants.

/reiterating

In the quoted posts in your replies (that you have your main reply to) can you reformat your replies, ie:   It'll make it easier for other people to read and realise there is an actual reply in the quote.   [noparse] A 3 days old domain, sounds fishy to say the least, speccially if you come here asking for donations. I wasn't necessarily asking for donations as stated. This is optional. [/noparse]

Errrm. Although they've got better, I'd still avoid them.   http://www.w3fools.com/ http://www.phptherightway.com/ http://php.net/

You're looping through your results and re-adding the item to the user, which is correct. However, the part which is wrong is your SELECT statement to fetch listings by a user and your DELETE statement. I'm going to assume that imID is the primary autoincrement key, which means change your value from {$_GET['ID']} to {$ra['imID']} You're using while ($ra = $db->num_rows($q)>0){ which should become while ($ra = $db->fetch_array($q)){. (You can put the num_rows() check above the while.)   function remove_all() { global $db, $ir, $c, $userid, $h; $q = $db->query("SELECT im.*,i.* FROM itemmarket im LEFT JOIN items i ON im.imITEM=i.itmid WHERE imADDER=$userid"); if (!$db->num_rows($q)) { print "<div id=text3>Error, either this item does not exist, or you are not the owner. <a href='itemmarket.php'><strong>Back</strong></a>"; $h->endpage(); exit; } while ($ra = $db->fetch_row($q)){ item_add($userid, $ra['imITEM'], $ra['imQTY']); $i = ($db->insert_id()) ? $db->insert_id() : 99999; $db->query("DELETE FROM itemmarket WHERE imID={$ra['imID']}"); $db->query("INSERT INTO imremovelogs VALUES (NULL, {$ra['imITEM']}, {$ra['imADDER']}, $userid, {$ra['imID']}, $i, unix_timestamp(), '{$ir['username']} removed a ".$db->escape($ra['itmname'])." from the item market.')"); } print "All item removed from market! <a href='itemmarket.php'><strong>Back</strong></a></div>"; }

Put this function in your global_func.php file so you can use it elsewhere.   /** * getUserSpouse * Fetches a users spouse if they have one. If they don't, return "Not married". * *@param int $intUserid The userid you want their spouse *@param bool $blReturnLink If TRUE then return link to user profile, else just return their name */ function getUserSpouse($intUserid = $_SESSION['userid'], $blReturnLink = TRUE) { global $db; //Ugh, really hate to global, but.... $strDbQuery = "SELECT `username`, `userid` FROM `users` WHERE `userid` IN(SELECT `married` FROM `users` WHERE `userid` = ". $intUserid .")"; $objGet = $db->query($strDbQuery); if($db->num_rows($objGet ) == 0) { return "Not married"; } $arrResult = $db->fetch_array($objGet); return $blReturnLink ? '<a href=\'viewuser.php?u='. $arrResult['userid'] .'\'>'. $arrResult['username'] .'</a>' : $arrResult['username']; }   So, on your viewuser.php code, you can call it by; echo '<strong>Spouse:</strong> '. getUserSpouse($_GET['u']);   Tip: It should be noted that using number_format() on your gang id and userid on your links will cause problems when you have id's over 1,000. https://eval.in/379222 (assuming you use the overly used abs( (int) $var ). Note: I'm assuming you've done the validation on $_GET['u'] to ensure it's an integer.