ober

Umm... yeah... none of those sites, except for us, are using PHP. Trust me on that.

That's about enough. Fenway was simply stating that if you're so eager to hand over your details, you probably didn't build it yourself. It's a fair enough statement without knowing more about you. Attacking a global mod and using language to stoop to an even lower level will get you nowhere around here.

6/10 1) Too many colors. Lose the green or lose the purple. 2) The background image repeats horribly. Find something different, and maybe don't let it repeat. 3) The header graphic is grainy and very poorly done. www.sxc.hu (find a better background) 4) Padding is your friend... use it in the content area. Text should not bump into the graphics that flow in the content. 5) The text changes too much... bold, large, italic, small... it's all over the place. Tone it down. 6) If you're going to have something without flowing text around it (videos on a few of the pages), then center the video.

The only way to do this is with something like AJAX. PHP will complete everything in the script before sending it to the client.

I'm not clear on what you want to replace it with or why? Are you sure you're not after the nl2br() function?

That would definitely be cleaner than my idea.

You need to surround all of those inputs with single quotes... they're all strings. The only time you can leave off the single quotes is if the value is strictly an integer or a float. Also, always use " or die (mysql_error()). That will be much more descriptive. mysql_query("INSERT INTO intowar1 (ipaddress, thedate, name, thename) VALUES('$ipaddress', '$thedate','" . mysql_real_escape_string($_GET['name']) . "', '$thetime')") or die('Theres an error. Please try again.#');

http://us2.php.net/asort Set the indexes of your array to the file name, then sort the array using the above function. Since your array values are simply integers, the biggest number will be the most recent file changed. Once it's sorted, I believe your last array element should be most recently modified file (and just grab the key of that one to get the file name).

Your checkboxes should all be named the same: "box1234" or something similar. When you submit the form, you search for anything in the $_POST array that begins with "box" (use substr) and get the number portion after it. The number portion would be the ID of the ad in your database. Then you simply build a query out of the string of IDs you just gathered and throw that in a new query: "SELECT X, Y, Z FROM table WHERE ad_id IN ($string_of_ids)".

The only other option I see is to setup a small shopping cart with a "donate" item that they "purchase". You could use multiple "items" with varying costs to cover different donation amounts.

I give it a 5/10. 1) Some of the icons are very fuzzy. 2) You have a "show links" tab or something once the user logs in that looks AWFUL. 3) It doesn't play well at smaller resolutions (even 1024x768). 4) There are places where you have boxes side by side and they're different sizes. It just doesn't look polished.

I don't see a huge difference. It looks cartoonish now and the black background isn't helping. The main content area still looks the same. Try again.

A quick google search turned up this: http://www.andymoore.info/php-to-detect-mobile-phones/

Search google for mod_rewrite. And you cannot have spaces in a URL.

PHP does not affect the browser. This is something running on the client on your friend's computer that is doing it (browser plugin or 3rd party configuration). I would look at your client-side scripting (any flash/JS, etc. that you're using).

We got hacked constantly.

Not bad... I'd reduce the height of the header image a little bit. And I'd lighten the colors of the links in the category area. Otherwise it's pretty good for a dark site.

http://collecta-mania.com.au/shipping_info.php -> Holy mother of God. What are you trying to do, blind the users? There is too much white space. I'd rather see you use a background color/image for the non content area. I'd drop the image above the content after the index page. I'd add a captcha to the contact page.

The design is awful... there's really no point in reviewing it. Start over. And what's the point with that small of space/bandwidth. There are many free hosting accounts you can get that give you more than that!

IPB... that's what we used to use here. It's a security nightmare (or at least it was). I'd steer clear.

Interesting... that looks very nice.

Well... we use SMF here (obviously) and it's rather reliable and completely customizable if you need it to be. All of my clients that run boards also run SMF. I'm not aware of a forum software and CMS in one.

SMF for the forum... not sure about the CMS. I roll my own.

Do a search... this gets asked many times/day.

Just add a counter variable to the while loop: $i = 1; while($row = mysql_fetch_array($result)) { echo $i . " " . $blah; $i++; }