Problem z wyświetlaniem - error, wyświetlanie newsów na stronie index Opcje

[artiego]

hi
kiedy wchodzimy sobie na strone www.alemiasto.pl [oczywiście to samo jest kiedy dodamy ../index.php] - wyskakuje nam strona, ale bez newsów.. za to pojawia sie tekst
"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '-10,10' at line 1"

A kiedy wpiszemy dokładnie tak jak powina otworzyć się strona.. to jest http://www.alemiasto.pl/index.php?site=mai...e&news=view&p=1 wczytuje się tak jak powinno.

OCB ? W czym tkwi problemo ? Jak to obejść ? - PILNE (IMG:http://forum.php.pl/style_emoticons/default/blink.gif)

ps. system newsów był wykonany na zlecenie, prosze nie komentować kodu html, wiem, ze tam jest brudów dużo, bo strona robiona przez FrontPage. Zdaje sobie z tego sprawee (IMG:http://forum.php.pl/style_emoticons/default/rolleyes.gif) (IMG:http://forum.php.pl/style_emoticons/default/tiredsmiley.gif)

[nospor]

No ale czego od nas oczekujesz? Nie podales kodu php do generowania tych newsów, to ci bledu bezosrednio nie pokazymy.

Cala sprawa rozchodzi się o parametr p w linku. Jak go nie podasz, to do mysql trafiają bzdurne dane do limit. Najprawdopodoniej, ktos przyjąl, ze ten parametr zawsze jest i go zawsze zczytuje, nawet jak go nie ma. A powinno byc, ze jesli go nie ma, to p=1.
Tyle jestem w stanie ci teraz powiedziec. Pokaz kod php, to powiem ci dokladniej (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Adiasz]

parametr p to z chyba id pagera, tak na szybko to wpisz gdzies na poczatkiu index.php linijke

[PHP] pobierz, plaintext 
<?php
 
if((int)$_GET["p"] == 0) {
  $_GET["p"] = 1;
}
 
?>
[PHP] pobierz, plaintext 

powinno pomoc

[nospor]

Skoro juz padl kod, to ja bym go lekko zmodyfikowal. @Adiasz w ten sposob, gdy nie będzie p, php bedzie plul noticami. A po co ma to robic? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) I lepiej nie przypisywac nic do super globali.

[PHP] pobierz, plaintext 
<?php
 
if(empty($_GET['p']) )
  $p = 1;
else
  $p = (int)$_GET['p'];
?>
[PHP] pobierz, plaintext 

edit:
@Adiasz i dlatego też, ja wolalem zaczekac na ten jego kod, by nie pisać nic na darmo i niekoniecznie poprawnego (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Adiasz]

@nospor masz racje, wpisalem wartosc do GETa poniewaz nie znajac kodu artiego nie wiemy czy odwolanie do zmiennej $_GET["p"] jest w pliku index.php czy moze w jakiejs funkcji czy klasie, wpisujac ja do tablicy globalnej mam pewnosc ze zawsze zadziala :-)

[artiego]

czyli skad kod Wam pokazać ? wydaje mi się że chodzi o kod z pliku `view` z systemu newsów. ..
jestli tak to oto on
PLIK VIEW

[PHP] pobierz, plaintext 
<?php
# 
# STOPKA (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)  ale co będę kolegę reklamował 
#
if(!defined("Index")) {
    exit();
}
#if(!isset($_GET["news"])) {
#    header("Location: ".$_SERVER["REQUEST_URI"]."news=view&p=1");
#    exit();
#}
#if(!isset($_GET["p"])) {
#    header("Location: ".$_SERVER["REQUEST_URI"]."&p=1");
#    exit();
#}
$q = "SELECT * FROM ".DbPrefix."news";
if(isset($_GET["cat"])) {
    $qc = "SELECT CategoryId FROM ".DbPrefix."category WHERE CategoryId = '".$_GET["cat"]."'";
    $mc = mysql_query($qc) or die(mysql_error());
    $nc = mysql_num_rows($mc);
    if($nc == 1) {
        $q .= " WHERE NewCategory = '".$_GET["cat"]."'";
    } else {
        echo "Wybrana kategoria nie istnieje!";
    }
}
$q .= " ORDER BY NewDate DESC LIMIT ".($_GET["p"]-1)*ConfigPage.",".ConfigPage;
$m = mysql_query($q) or die(mysql_error());
$n = mysql_num_rows($m);
if($n == 0) {
    echo "Brak wpisów.";
} else {
    $i = 0;
    while($r = mysql_fetch_array($m)) {
        $i++;
        if(ConfigSize != NULL) {
            if(strlen($r["NewValue"]) >= ConfigSize) {
                $NewValue = substr($r["NewValue"], 0, ConfigSize)
                ."... [<a href=\"".ConfigInclude
                ."news=comment&id=".$r["NewId"]."\">więcej</a>]\n";
            } else {
                $NewValue = $r["NewValue"];
            }
        } else {
            $NewValue = $r["NewValue"];
        }
        $qc = "SELECT * FROM ".DbPrefix."category WHERE CategoryId = '".$r["NewCategory"]."'";
        $mc = mysql_query($qc) or die(mysql_error());
        $rc = mysql_fetch_array($mc);
        $qu = "SELECT UserName, UserContact FROM ".DbPrefix."users WHERE UserId = '".$r["NewUser"]."'";
        $mu = mysql_query($qu) or die(mysql_error());
        $ru = mysql_fetch_array($mu);
        if($r["NewCategory"] == 0) {
            $rc["CategoryTpl"] = ConfigTpl;
            $rc["CategoryId"] = null;
            $rc["CategoryName"] = null;
        }
 
        $CategoryTpl = preg_replace(array(
            "#{KATEGORIA}#is",
            "#{TYTUL}#is",
            "#{DATA}#is",
            "#{TRESC}#is",
            "#{ZDJECIE}#is",
            "#{AUTOR}#is",
            "#{DRUKUJ}#is",
            "#{KOMENTUJ}#is",
            "#{KOMENTARZY}#is",
            "#{ZRODLO}#is",
            ""#{ID}#is
        ), array(
            "<a href=\"".ConfigInclude."news=view&cat=".$rc["CategoryId"]."\">".$rc["CategoryName"]."</a>\n",
            $r["NewTitle"],
            date("d.m.Y H:i:s", $r["NewDate"]),
            $NewValue,
            "<img border=\"0\" align=\"".$r["NewImgp"]."\" src=\"images/".$r["NewImage"]."\">",
            "<a href=\"".$ru["UserContact"]."\">".$ru["UserName"]."</a>",
            "<a href=\"print.php?id=".$r["NewId"]."\" target=\"_blank\">Drukuj</a>",
            "<a href=\"".ConfigInclude."news=comment&id=".$r["NewId"]."\">Komentarze</a>",
            $r["NewComments"],
            $r["NewInfo"],
            $r["NewId"]
        ), $rc["CategoryTpl"]);
        if(isset($_GET["mc"]) && isset($_GET["rok"])) {
            $dt1 = date("m", $r["NewDate"]);
            $dt2 = date("Y", $r["NewDate"]);
            if(($_GET["mc"] == $dt1) && ($_GET["rok"] == $dt2)) {
                echo $CategoryTpl;
            }
        } else {
            echo $CategoryTpl;
        }
        if($n != $i) {
            echo "<br /><br />";
        }
        echo "\n";
    }
    $qp = "SELECT NewId FROM ".DbPrefix."news";
    if(isset($_GET["cat"])) {
        $qp .= " WHERE NewCategory = '".$_GET["cat"]."'";
    }
    $mp = mysql_query($qp) or die(mysql_error());
    $np = mysql_num_rows($mp);
    if($np > ConfigPage) {
        $s = ceil($np/ConfigPage);
        echo "<br /><br />Strony:";
        for($d = 1; $d <= $s; $d++) {
            echo " [<a href=\"".preg_replace("#p=".$_GET["p"]."#is", "p=".$d, $_SERVER["REQUEST_URI"])."\">$d</a>]";
        }
    }
}
 
?>
[PHP] pobierz, plaintext 

PLIK INDEX (nie mojej strony, tylko skryptu)

[PHP] pobierz, plaintext 
<?php
 
define("Index", true);
require_once("includes/config.php");
 
mysql_connect(DbHost, DbUser, DbPassword) or die(mysql_error());
mysql_select_db(DbName) or die(mysql_error());
 
$q = "SELECT * FROM ".DbPrefix."config";
$m = mysql_query($q) or die(mysql_error());
while($r = mysql_fetch_array($m)) {
    define("Config".$r["ConfigName"], $r["ConfigValue"]);
}
 
$q = "SELECT * FROM ".DbPrefix."banlist WHERE BanIp = '".$_SERVER["REMOTE_ADDR"]."'";
$m = mysql_query($q) or die(mysql_error());
$n = mysql_num_rows($m);
if($n == 1) {
    exit("Zostałe&para; zbanowany!");
}
 
if(!isset($_GET["news"]) || $_GET["news"] == "view") {
    require_once("includes/view.php");
} elseif(!isset($_GET["news"]) || $_GET["news"] == "admin") {
    require_once("admin/index.php");    
} elseif(isset($_GET["news"]) && $_GET["news"] != "admin" && $_GET["news"] != "view") {
    require_once("includes/".$_GET["news"].".php");
} else {
    require_once("includes/view.php");
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował artiego 30.03.2006, 10:58:43

[nospor]

czyli skad kod Wam pokazać ?

(IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Kod, w którym wyswietlasz te newsy. No i inny, jesli jest potrzebny. Ty to wogole robiles? Wiesz mniej więcej o czym mowimy?

[artiego]

@nospor - pisałem skrypt newsów był pisany na zlecenie.

VIEW to plik który odpowiada za wyświetlanie newsa

na podstronie na której wyświetlają się newsy mam wrzucone

[PHP] pobierz, plaintext 
<?php
# --- N E W S --- #
$dir = "news/index.php";
# --------------------- #
if(!is_file($dir)) {
   echo "Plik <b>".$dir."</b> nie istnieje!\n";
} else {
   include_once($dir);
}
# --- N E W S --- #
?>
[PHP] pobierz, plaintext 

[nospor]

@nospor - pisałem skrypt newsów był pisany na zlecenie

Tak, wiem. czytalem to. tylko nie napisale czy to zlecono Tobie, czy ty zlecileś komus (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

No niestety za bardzo nie wiesz o co nam chodzilo. Kod co pokazales nam nie pomoże. Pewnie bardziej by sie przydal o ten plik: "news/index.php"

edit: na początku pierwszego skrytpu co pokazaleś (plik view), daj tak:

[PHP] pobierz, plaintext 
<?php
if(empty($_GET['p']) )
  $p = 1;
else
  $p = (int)$_GET['p'];
 
?>
[PHP] pobierz, plaintext 

a potem wszystkie wystąpienia $_GET["p"] zamien na $p

[artiego]

a j no i piknie ! (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

DZIKI WIELGACHNE ZA POMOC (IMG:http://forum.php.pl/style_emoticons/default/Rkingsmiley.png)
Można na was polegać - w sumie szybka reakcja na problemm no i co wazne - skuteczna - brawo brawo. Będe o Was pamietał w moich potyczkach.

PS. A tak na marginesie zapraszam na ALemiasto - serwis o mieście, którego oficjalne otwarcie o1.o4.2oo6 ! (IMG:http://forum.php.pl/style_emoticons/default/cool.gif)

Mogę dać z odetchnięciem (IMG:http://forum.php.pl/style_emoticons/default/offtopic.gif)