Adam

$text_ad = stripslashes($row['text_ad']); You need slashes to escape the quotes, try removing the "stripslashes()" function.. or use "addslashes()" in the query .. Adam

Can't say how they will have done it without seeing anything but, you'd be best looking into PHP security.. http://www.sitepoint.com/article/php-security-blunders/ - should protect from any further problems! Adam

His printed the sQL out: you're trying to get the ID form $_POST, when you don't have it in your form: echo "<form name=\"newad\" method=\"post\" enctype=\"multipart/form-data\" action=\"\">"; echo "<input type=\"file\" name=\"cons_image\" >"; echo "<input name=\"Submit\" type=\"submit\" id=\"image1\" value=\"Upload image\" />"; echo "</form>"; You need to add an input named ID to be able to get the ID from $_POST.

$id = (int) $_POST['id']; echo $id; Try using that instead of: $id = $_POST['id']; echo "$id";

Could you post the code where $consname2 is set? Something isn't going right there!

We already know what we need, as revraz said, $consname2 is empty. Where is $consname2 given a value?

$sql="UPDATE general SET image= '$consname2' WHERE id= $id"; $query = mysql_query($sql)or die(mysql_error()); echo 'SQL: ' . $sql . '<br />Affected rows: ' . mysql_affected_rows($query); try that .. tell us what it says..

Have you changed it to: $query = mysql_query($sql)or die(mysql_error()); (Note $query =) ??

No. The "where" determines which records get updated - whether it be 1 or 1000. $query = mysql_query($sql)or die(mysql_error());

I'd probably try looping through every folder images are in > saving the path and file name in an array > sorting the array by the file name > selecting top 3.. first idea that came to mind. I don't think people will just create this little app for you, try working on an idea then coming back when you have some code and get stuck or run into problems... Adam

What is happening? change this line to: $sql="UPDATE general SET image= '$consname2' WHERE id= '$id'"; Adam

They're just HTML entities.. http://www.w3schools.com/tags/ref_entities.asp

Or could just use auto_increment in the database and set the starting number to say 79000000 .. ? To do that in PHPMyAdmin just click on the "operations" tab on the table structure view page, should be a field for "auto_increment" .. Or as an SQL query: ALTER TABLE `yourTable` AUTO_INCREMENT = 79000000 Adam

Can tidy your code up a lot- by the way using foreach () to loop through an array is much easier! I don't quite know what you were trying to do with the first for (), perhaps just loop once as there's only one entry in the array? Remember a loop is for repeating code, could have just used: $y2 = explode(",", $x2[0]);.. But I've changed the second for () as an example to show you.. for ($i = 0; $i < 1; $i++) { // $y2 is the array that I posted above. The elements 0, Happiness, and Stone are in this array. $y2 = explode(",", $x2[$i]); // 3 elements in the array; we loop 3 times and display the results. "Basic Form" gets displayed three times, when it should only be displayed once foreach ($y2 as $val) { echo $dex->calcEvolutionMethod($val); } } Try changing this line to: echo $val . ': ' . $dex->calcEvolutionMethod($val); So you know it's definitely getting the right values passed.. Adam

VN21Nov2008-1.pdf VN21Nov2008-2.pdf VN21Nov2008-3.pdf VN21Nov2008-4.pdf VN21Nov2008-5.pdf Are they all strictly in this format to start with? Will be there any like: VN1Nov2008-10.pdf ?? I am asking because to use substr() you'd need to know the positioning of each section. Adam

Try playing with... $file = basename(htmlentities($SERVER['PHP_SELF']), ".php");

Your site just isn't optimized for search engines... For example... http://www.fatalinjury.org/list.php?dir=nature Not search engine friendly at all .. Try giving the images meaningful names and preferably a short description. They'll be more likely to end up on google and other image search engines - which will boost traffic a hell of a lot! Adam

It's better but as everybody else has said, seems to lack something! The nav definitely needs work! Looks like you spent all your time working on the rounded corners then when you came to design the nav, was fed up / bored. The main text is also a bit bland, looks tiring to read.. there's no headers, images or anything to catch your eye and keep you interested. The logo doesn't really fit the style you're trying to achieve, or at least the "SPIGN" text doesn't.. and the 2 blues don't go well together. Adam

Doubt anyone here's going to do that for you for free to be honest .. http://www.phpfreaks.com/forums/index.php/board,8.0.html

"Looks to me like mID is not numeric." Think corbin was barking up the same tree.. Echo out the value of $mID to double check it has a valid value... Adam

Well why not just send it in the URL as a $_GET param? The purpose of #anchorName is to jump to different parts of the page without reloading, so it's not going to make another request to the server anyway. Adam

Can we see some code? How are you converting to the text to an image? What does the text originally look like? "0xDA" looks like hexadecimal .. http://en.wikipedia.org/wiki/Hexadecimal Adam

Is it simply a file content search? If so of course it's possible, what do you think google does to about 1,000,000,000,000 sites!? If you're wanting to search a database from each site, that could get more complicated, but still quite possible... ? Adam

None come to mind.. Might be worth posting in the freelance section, seems a bit specific.. Adam

Yeah use: $text = str_replace("\n", "<br />", $text);