fatal error

KevinJ

I get an error

Fatal error: Cannot redeclare list_categories() (previously declared in /home/darkness/public_html/forums/modules/functions.php:4) in /home/darkness/public_html/forums/modules/functions.php on line 3

Here is the first part since its on line 3. If you need the whole code please just ask.

<?php

include"connect.php";
include"functions.php";

list_categories();

switch($_GET['cat']) {

case"$row[id]";
if(!isset($_POST['$row[id]'])) {
if($_GET['id']) {
$id = (int)$_GET['id'];
$query3 = mysql_query("SELECT * FROM tutorials WHERE valid = '1' AND cat_id = '$id' ORDER BY id DESC");
while($row = mysql_fetch_array($query3)) {
echo"
<table width='100%'>
    <tr>

NogDog

The message is telling you that you're trying to define a function called list_categories() which has already been defined. Take a look through both of your include files and look for more than one place where you define such a function (e.g.: search for "function list_categories"). Then determine of one of the extra calls is redundant, or needs to be renamed to something else.

Weedpacket

Incidentally, the error message refers to lines 3 and 4 of "functions.php". Since the code you posted contains the line 'include "functions.php"', I doubt that what you posted is from functions.php. If it is then there's something very wrong with what you've written🙂

bradgrafelman

Weedpacket wrote:
If it is then there's something very wrong with what you've written

Not to mention the fact(s) that...

There are array indeces without strings, so that PHP first treats the index as a constant, (hopefully) fails to find such a constant, and then treats the index as a string... throwing a nasty warning message your way.
There are "id" fields that have numerical values. Nothing abnormal there, except that you're passing strings in the SQL query. If these are numerical columns (they should be, and if they aren't... why aren't they?), then you should give MySQL a numerical value (meaning no quotes).
"SELECT " is never a good practise - replacing the "" with a column list, e.g. "SELECT column1, column2, column3 FROM ..." is much better as it a) retrieves only the data you need, and b) is more scalable (if you add extra columsn to your table that this script doesn't need, a column list ensures that the script isn't automatically pulling this extra information off the DB and then just not using it).