[Haselnuss] Design: Vorschläge, Ideen, etc.

[gandro]

friedrichheinze schrieb:  

s4ndwichMakeR schrieb:  Welche Funktionen-Suche meinst du eigentlich?

php.net search for x in the function list

Jap, an PHP ist sehr vieles dämlich, aber immerhin haben sie eine sehr gute Dokumentation.

[s4ndwichMakeR]

Ach so, die offizielle Referenz ist gemeint. x)

[Blue-Fox]

Version 0.1.0 ist draußen!
Es ist erstmal ein Probe-Release.

http://yabfbb.bplaced.net/haselnuss0.1.0.tar.bz2

[gandro]

Der PHP-Code sieht recht aufgeräumt aus (SQL kann ich mangels Wissen nicht bewerten), das gefällt.

Die Trennung zwischen Ausgabe & Programm dürfte noch etwas radikaler ausfallen (z.B. Login, HTML-Code mit echo ausgeben ist suboptimal).. und content.php könnte man ggf. ne Schleife herum machen, kA), das merkst du aber selber, wenn du später mal am Layout was ändern willst. Eventuell willst du den Krams eher in Funktionen auslagern als in Dateien.

Ansonsten musst du etwas radikaler escapen und auf Sicherheit achten. Der Installer enthält noch eine SQL-Lücke, und beim Login gibts ne non-persistente XSS-Lücke.

[Blue-Fox]

Ich hab content.php jetzt verbessert:

PHP-Code:

<div id="content">
    <p id="box_logo">Aktuelle Einträge</p>
    <?php include "includes/infoboxarray.php";
    do {
        $i += 1;
        if(isset($content["entry".$i."date"]) ) {
            echo $content["entry".$i."date"]." ".$content["entry".$i"title"];
        }
        if($i != 10) {
            i += 1;
            if(isset($content["entry".$i."date"]) ) {
                i -= 1;
                echo "<br />";
            }
        }
    } while($i != 10);
    ?>
</div>
<br />
<br />
<?php 
do {
    $j += 1;
    if(isset($content["entry".$j."content"]) ) {
        echo "<div id=\"content\">";
        echo "<p id=\"box_logo\">".$content["entry".$j."title"]."</p>";
        echo $content["entry1content"];
        echo "</div>";
    }
} while($j != 10);
?>

[chiaki]

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING, expecting ']' in /users/yabfbb/www/includes/content.php on line 7


da bastelt jemand rum

[gandro]

Warum nimmst du nicht eine for-Schleife?

Und $i += auf eine uninitialisierte Variable ist zwar in PHP machbar, aber sehr unklug, weil es ja sein könnte, dass irgendwo $i vorher schon gesetzt wird (dann kannst du in deinem Beispiel auch $j durch $i ersetzen und damit min. 4 Byte Speicher sparen)

[julben]

chiaki schrieb:  Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING, expecting ']' in /users/yabfbb/www/includes/content.php on line 7


da bastelt jemand rum

Ja, die siebte Zeile müsste so aussehen. Denk ich.

PHP-Code:

echo $content["entry".$i."date"]." ".$content["entry".$i."title"]; 


[freaked]

täts anders nennen..klingt zu sehr nach YaBB

[Blue-Fox]

Ja, ich bastel daran rum.
Und ja, ich habe auch typos.
Jetzt ist es gefixt.

Noch ein Typo gefixt.