getfeaturedlink funktioniert nicht

[psychmaster]

Hallo liebe Gemeinde. Ich habe ein Problem.
Und zwar möchte ich, dass auf der Startseite Links zu Werbung gebucht werden können. Dazu gibt es ein Formular. Alles funktioniert, Bestellung etc, bis auf, dass die Links dann auch wirklich von alleine auf der Startseite angezeigt werden.

Hier einmal ein paar Auszüge.. ist dort irgendwo ein Fehler?

Home.tpl <- hier soll es angezeigt werden

<div style="background:url(/templates/ModernBlue/css/images/briefcase.png) no-repeat scroll 50% 0px;">
{if $settings.fads_available == 'yes'}
  <div class="featured_ads" style="float:left;">
        <div class="title">{$lang.txt.featuredad}</div>
        <div class="fcontent">
            <ul class="featured-ads">
                {if getfeaturedad()}{/if}
            </ul>               
        </div>
  </div>
{/if}
{if $settings.flinks_available == 'yes'}
  <div class="featured_ads">
        <div class="title2">{$lang.txt.featuredlink}</div>
        <div class="fcontent">
            <ul class="featured-ads">
                {if getfeaturedlink()}{/if}
            </ul>               
        </div>
  </div>
{/if}
<div class="clear"></div>
</div> 

flink.php

define("scriptad", 1);
define("ROOTPATH", dirname(__FILE__) . "/");
define("INCLUDES", ROOTPATH . "includes/");
require_once INCLUDES . "global.php";

if (!is_numeric($input->g['id'])) {
  header("location: index.php");
  $db->close();
  exit();
}
else {
  $ad_id = $db->real_escape_string($input->g['id']);
  $res = $db->fetchOne("SELECT COUNT(*) AS NUM FROM featured_link WHERE id=" . $ad_id);

  if ($res != 0) {
    $featuredlink = $db->fetchOne("SELECT url from featured_link WHERE id=" . $ad_id);
    $upd = $db->query("UPDATE featured_link SET clicks=clicks+1 WHERE id=" . $ad_id);
    header("location:" . $featuredlink);
    $db->close();
    exit();
  }
  else {
    header("location: index.php");
    $db->close();
    exit();
  }
}

exit(); 

[Vr3piz4]

Existiert die Funktion getfeaturedlink?

[H4x0r007]

Als kleiner Tipp:

  $res = $db->fetchOne("SELECT COUNT(*) AS NUM FROM featured_link WHERE id=" . $ad_id);

  if ($res != 0) {
    $featuredlink = $db->fetchOne("SELECT url from featured_link WHERE id=" . $ad_id);

Das ist falsch. Es funktioniert zwar in den meisten Fällen, hat aber eine Race Condition. Du prüfst zuerst in einem SQL-Query, ob Daten vorhanden sind und frägst anschließend ungeprüft die Daten ab. Was passiert nun, wenn zwischen den beiden Queries die Daten geändert oder gelöscht werden?

--> Lass den COUNT-Query weg und prüfe den Rückgabewert des eigentlichen Queries. Laut PEAR-Manual gibt fetchOne ein MDB_ERROR Objekt zurück, auf das du prüfen kannst.

Zu deiner Frage: Ich kann ich zunächst nur auf meinen Vorredner verweisen: Gibt es die Funktionen getfeaturedlink und getfeaturedads?