Fehler im Downloadlisten-Modul von J. Clemens (30.05.04)

[pinkernell]

Im Module Downloadbereich ist seit dem 30.05.2004 ein Downloadlisten-Modul, dass einen Fehler aufweist. Zumindest habe ich es erst nach Korrektur des Codes ans Laufen gekriegt.

Der Fehler ist im Output. Ich zitiere den Anfang:

Code: Alles auswählen

<?php
/*********************************************** 
* CONTENIDO MODUL - OUTPUT 
*
* Modulname	:	Downloadliste
* over-worked by Johannes Clemens
* Copyright	:	Johannes Clemens
* Modified	:	2004-05-30
* www.clemens-web.de
************************************************/
# original
/***********************************************
* Modulname   :     Download
* Author      :     4fb - Jan Lengowski/ 4.3.1b mods by J. Matern
* Copyright   :     Contenido - four for business
* Created     :     26-08-2002
* Modified    :     05-07-2003
************************************************/ 
echo "<table width=\"100%\" cellspacing=\"0\" cellpadding=\"0\" border=\"0\"><ul>";
function showfile() {
  $db->query($query);
  $db->next_record();
  $file = $db->f("filename");
  $href =  $cfgClient[$client]["htmlpath"]["frontend"] . $cfgClient[$client]["upload"]. $db->f("dirname") . $db->f("filename");
        $description = $db->f("description");
				$size= $db->f("size");
				$size=$size/1024/1024;
				$size=@round($size,2);
				$size=$size." MB";
  return "<li><a target=\"_blank\" href=\"$href\">$description</a> ($size)</li>";
}

if ("CMS_VALUE[1]" == "on") {
  $s = "CMS_VALUE[3]";
  $query = "SELECT * FROM ".$cfg["tab"]["upl"]." WHERE filetype = 'pdf'"." AND idupl = '$s'";
	echo showfile();
} // end if

if ("CMS_VALUE[4]" == "on") {
  $s = "CMS_VALUE[6]";
  $query = "SELECT * FROM ".$cfg["tab"]["upl"]." WHERE filetype = 'pdf'"." AND idupl = '$s'";
  $db->query($query);
  $db->next_record();
  $file = $db->f("filename");
    $href =  $cfgClient[$client]["htmlpath"]["frontend"] . $cfgClient[$client]["upload"]. $db->f("dirname") . $db->f("filename");
        $description = $db->f("description");
				$size= $db->f("size");
				$size=$size/1024/1024;
				$size=@round($size,2);
				$size=$size." MB";
  echo "<li><a target=\"_blank\" href=\"$href\">$description</a> ($size)</li>";
} // end if

Man werfe die "function showfile" heraus und definiere das erste "if" analog wie alle folgenden "if"s:

Code: Alles auswählen

<?php
/*********************************************** 
* CONTENIDO MODUL - OUTPUT 
*
* Modulname	:	Downloadliste
* over-worked by Johannes Clemens
* Copyright	:	Johannes Clemens
* Modified	:	2004-05-30
* www.clemens-web.de
************************************************/
# original
/***********************************************
* Modulname   :     Download
* Author      :     4fb - Jan Lengowski/ 4.3.1b mods by J. Matern
* Copyright   :     Contenido - four for business
* Created     :     26-08-2002
* Modified    :     05-07-2003
************************************************/ 
echo "<table width=\"100%\" cellspacing=\"0\" cellpadding=\"0\" border=\"5\"><ul>";

if ("CMS_VALUE[1]" == "on") {
  $s = "CMS_VALUE[3]";
  $query = "SELECT * FROM ".$cfg["tab"]["upl"]." WHERE filetype = 'pdf'"." AND idupl = '$s'";
  $db->query($query);
  $db->next_record();
  $file = $db->f("filename");
    $href =  $cfgClient[$client]["htmlpath"]["frontend"] . $cfgClient[$client]["upload"]. $db->f("dirname") . $db->f("filename");
        $description = $db->f("description");
				$size= $db->f("size");
				$size=$size/1024/1024;
				$size=@round($size,2);
				$size=$size." MB";
  echo "<li><a target=\"_blank\" href=\"$href\">$description</a> ($size)</li>";
} // end if

if ("CMS_VALUE[4]" == "on") {
  $s = "CMS_VALUE[6]";
  $query = "SELECT * FROM ".$cfg["tab"]["upl"]." WHERE filetype = 'pdf'"." AND idupl = '$s'";
  $db->query($query);
  $db->next_record();
  $file = $db->f("filename");
    $href =  $cfgClient[$client]["htmlpath"]["frontend"] . $cfgClient[$client]["upload"]. $db->f("dirname") . $db->f("filename");
        $description = $db->f("description");
  echo "<li><a target=\"_blank\" href=\"$href\">$description</a> ($size)</li>";
} // end if

Guido

[emergence]

hab mir das jetzt mal angesehen...

bei deinem outputmodul fehlt

?>

am ende...

[pinkernell]

... bei beiden Zitaten fehlt das ?>. Sind aber auch beides Zitate, also nur Auszüge. Eigentlich gings ja nur um einen Fehler, den ich selbst berichtigen konnte. Und der hatte nicht mit dem ?> zu tun. Aber danke für den Hinweis. Vielleicht könnte ja sonst jemand auf die Idee kommen, es handelte sich hier um vollständige Module.