Adam

From a design point of view, it all just looks a little dull to be honest. The colour scheme's plain and bland, the text is very standard, the logo has some slight pixelation on the stars, etc. As you say it does give off the 'used car salesman' image. Also 'four' stars.. Why not five? Makes your service sound less than perfect. The layout isn't bad, and with some improvements could make the site ten times better, but you just need to liven it up. Move away from safe, neutral colours and add in some lively colours. Colour scheme generators like Adobe Kuler can help there. From a code point of view; there's a lot of inline styles, constant use of deprecated tags and attributes (font, center, bgcolor, etc.), no use of headers (h1, h2, etc.).. It was clear the moment I opened the source it had been created with 'instant web site builder' type software. I've never used "Intuit SiteBuilder by Homestead", but I'd recommend paying someone to go in and improve it for you; the most obvious benefit being your site will be better indexed by search engines.

Variables and array indexes are case-sensitive, functions and class names aren't.

'rsUser' != 'RSUSER'

Can you explain how you have the data stored a little clearer? 'Files' and 'entries' is a little vague to give you any kind of code solution..

What about search engines? They need to be able to navigate around the site.

Okay. Well security is a wide subject, do you have a particular area or piece of code you'd like more help with?

No that's to protect you from session hijacking.. For a good thorough look into security I'd recommend reading the security tutorial on the main PHPFreaks site.

Have a read of this: http://talks.php.net/show/phpworks2004-php-session-security/0

When you say "pure PHP / Mysql", do you mean you're using custom database sessions, or standard PHP sessions?

strip_tags allows you to specify a string of tags to accept, if that helps?

There is no standard PHP function to do that. You can use a loop though: $seomthing = array(); foreach ($new_array as $key => $value) { $something['quantity'][$key] = $value['quantity']; } print_r($something);

Neither "hide the truth". You have to consider the type of users that will be visiting the site, and which browsers they're likely to be using. w3schools will obviously attract web developers, who are more likely to have the latest browsers better equipped for development (Firefox, with add-ons at least). Stat counter can provide less biased results as it's used on a whole variety of websites, aimed at different users. Anybody could really visit a website using stat counter so there's no one type of user. To gain the statistics they just parse the user agent string.

You mean w3schools? You really have to consider how biased the results are for that website though. It makes sense that a website aimed at web developers would have less people using a browser web developers generally despise. Also whichbrowserforme.org when it mentions IE6 is just using the text from the bringdownie6.com website, which has been around for a while now.

No because you're opening the string with a single quote, then closing with a double:

What did it show? Quite likely you're using up all the memory. Try increasing it with: ini_set('memory_limit', '150M'); Although I'll add; displaying 128mb worth of data in a browser isn't the best idea.

It's easier to use single quotes when echoing HTML, as you use double quote more: echo '<td><a href="' . $rows['url'] . '">..link text..</a></td>';

This wasn't you, was it..? http://www.metro.co.uk/news/843818-i-hate-ryanair-website-is-banned-well-almost

None of it works? I click log-in and I get a 404. Try to insert a.. rage post or whatever and it just says "could not insert post". I even tried log-out and got a 404. If I were someone genuinely mad, and for some reason felt like writing it all down and posting it on your website for no one to see except other mad people, I'd just be even more mad right now. From a design point of view, you said it yourself it's very basic. There's not much to critique. The blocky images and text don't do it for me.. the fonts can be hard to read.. and the colour scheme is dull. As hinted at before, for some reason you have both a login and out link? Also the background seems to drag on to about twice the size of the content? Looking at the source there's nothing correct about it. You have no HTML tags, no head or body tags, no doctype, etc. If you're going to create websites, the best advice I can give is to go back to stage 1 and learn HTML/CSS correctly.

htmlspecialchars - the order is important though, because if you apply htmlspecialchars() after nl2br() you'll convert the br tags to entities too: echo nl2br(htmlspecialchars($comment));

Ah right, sorry I think I misunderstood what it was you were asking. So you mean using the available flights, you need to plan a route (obviously with minimal time between flights) to get from one destination to another?

First you need to define where these places are, most likely by assigning each a longitude and latitude value. Then based on those values you'll need to use some advanced mathematics to work out the distance, position, etc. from one another. Then perhaps the GD library to create a visual display of the route, based on the figures you calculated. The Google Maps API may make things easier for you here though..

Would be nice. Unfortunately I don't think the parser would be able to look-ahead to the rest of the table and work out what the value should be?

My mistake, just forgot to escape the plus. Although can make it simpler, as you kind of did, by using block brackets for matching the plus/minus: var_dump(preg_match('/^[-+]?[0-9]+$/', '-123456')); // returns 1 The pattern you came up with though is just making it more complicated than it needs to be.

You can use: ^(-|+)?[0-9]+$

document.getElementById(id).style.backgroundColor = '#123456';