phpBB.nl

Mandrake Linux schreef:http://www.forum.stophier.nl/viewtop25.php
i heb hem maar er eens gezet
maar dan krijg je een leuke fout melding

SennA schreef:

Mandrake Linux schreef:http://www.forum.stophier.nl/viewtop25.php
i heb hem maar er eens gezet
maar dan krijg je een leuke fout melding

Owh das waar ook

Bij mij staat de table prefix op forum_ :$, en niet op phpbb_. Moet je de query's ff veranderen.

PS: Nu snap ik ook wat "constants.php" doet onderaan met al die tabellen .

Mandrake Linux schreef:ok dat is goed
maar nu krijg ik weer een hele boel fouten
en ik denk dat ik weet wat
ik moet 25 topic openen
maar dit moet jecht veranderen hoor

Mandrake Linux schreef:....
met wat
:S

        $sql2 = "SELECT * FROM forum_posts_text ORDER BY CHAR_LENGTH(post_text) DESC LIMIT 26";
        if( !$result = $db->sql_query($sql2) )
        {
                message_die(GENERAL_ERROR, "Kon top25 data niet verkrijgen", "", __LINE__, __FILE__, $sql2);
        }
        $top_count = $db->sql_numrows($result);

        $top_rows = $db->sql_fetchrowset($result);

	for($i = 1; $i < $top_count; $i++)
	{
			$lengtebericht = strlen($top_rows[$i]['post_text']);
			$post_id = $top_rows[$i]['post_id'];
		$sql =  mysql_query("SELECT * FROM forum_posts WHERE post_id='".$top_rows[$i]['post_id']."'");
		$list = mysql_fetch_object($sql);
		$sql3 = mysql_query("SELECT * FROM forum_topics WHERE topic_id='".$list->topic_id."'");
		$show = mysql_fetch_object($sql3);
			$topic = $show->topic_title;
		$sql4 = mysql_query("SELECT * FROM forum_users WHERE user_id='".$list->poster_id."'");
		while($record = mysql_fetch_object($sql4))
			$auteur = $record->username;

        $sql2 = "SELECT * FROM ".$table_prefix."posts_text ORDER BY CHAR_LENGTH(post_text) DESC LIMIT 26";
        if( !$result = $db->sql_query($sql2) )
        {
                message_die(GENERAL_ERROR, "Kon top25 data niet verkrijgen", "", __LINE__, __FILE__, $sql2);
        }
        $top_count = $db->sql_numrows($result);

        $top_rows = $db->sql_fetchrowset($result);

	for($i = 1; $i < $top_count; $i++)
	{
			$lengtebericht = strlen($top_rows[$i]['post_text']);
			$post_id = $top_rows[$i]['post_id'];
		$sql =  mysql_query("SELECT * FROM ".$table_prefix."posts WHERE post_id='".$post_id."'");
		$list = mysql_fetch_object($sql);
		$sql3 = mysql_query("SELECT * FROM ".$table_prefix."topics WHERE topic_id='".$list->topic_id."'");
		$show = mysql_fetch_object($sql3);
			$topic = $show->topic_title;
		$sql4 = mysql_query("SELECT * FROM ".$table_prefix."users WHERE user_id='".$list->poster_id."'");
		$record = mysql_fetch_object($sql4);
			$auteur = $record->username;

$sql4 = mysql_query("SELECT * FROM forum_users WHERE user_id='".$list->poster_id."'"); 
      while($record = mysql_fetch_object($sql4)) 
         $auteur = $record->username;

WebSiteNet schreef:Je hebt de standaard php mysql functies gebruikt.

Dit zal niet echt werken. Je moet met het $db object werken.

• Geef een $page_title op voor je de header include, dan krijgt je pagina een titel
• Gebruik de constanten voor je tabelnamen, dus
  

  Code: Selecteer alles

  "SELECT * FROM " . POSTS_TEXT_TABLE . "

  in plaats van
  

  Code: Selecteer alles

  "SELECT * FROM forum_posts_text

  je bent dan ook van het prefix probleem af
• Je begint je for() loop met 1, en geeft daarom een SQL limit aan van 26, ipv 25. Als je de het middelste stuk van je loop nu met een <= vergelijking schrijft kan de limit wel 25 blijven.
• Waarom heb je een compleet deel van de pagina er dubbel in staan, de tweede keer tussen comments? Je zal het vast nodig gehad hebben bij het testen, maar let hier wel op als je de mod online zet voor het publiek.
• Je gebruikt 4 query's waar je met ééntje genoeg hebt.
  

  Code: Selecteer alles

  SELECT u.user_id, u.username, t.topic_title, pt.post_text
  FROM " . USERS_TABLE . " u, " . TOPICS_TABLE . " t, " . POSTS_TABLE . " p, " . POSTS_TEXT_TABLE . " pt
  WHERE pt.post_id = p.post_id
  AND p.topic_id = t.topic_id
  AND t.topic_poster = u.user_id
  ORDER BY CHAR_LENGTH(post_text) DESC
  LIMIT 26;

• Voor query 3 vraag je de gegevens opeens op met mysql_fetch_object(), waarom? Ook moet je consequent met het $db object werken, dus zoals je dat bij query 2 hebt gedaan.
• Je begint en eindigt 3 keer een <? .. ?> PHP gedeelte, dit is onnodig omdat het allemaal PHP is en het veroorzaakt onnodige spaties en returns in je source. Als je zou werken met cookie functies zou het je zelfs fatale errors opleveren.
• In het onderste stuk schrijf je een HTML tabel in je PHP bestand, doe dit met je template.
• Maak links van de usernames, aan de hand van het user_id uit de query.
• Schrijf je links, zoals die naar viewtopic, in je PHP bestand en haal ze eerst door append_sid(). Maak ook gebruik van $phpEx en de constante POST_POST_URL voor p=.
• Ik zie in een van de onderste comments een http://www.reev.nl/forum/ url. Slordig!

mosymuis schreef:Allereerst raad ik je aan de officiële MOD template te gebruiken, net als een kloppend versie nummer. Neem voor dat en andere handige tutorials eens een kijkje in de phpbb.com knowledge database:
http://www.phpbb.com/kb/list_articles.php?category_id=7

mosymuis schreef: Verder nog wat tips voor je mod:

• Geef een $page_title op voor je de header include, dan krijgt je pagina een titel
• Gebruik de constanten voor je tabelnamen, dus
  

  Code: Selecteer alles

  "SELECT * FROM " . POSTS_TEXT_TABLE . "

  in plaats van
  

  Code: Selecteer alles

  "SELECT * FROM forum_posts_text

  je bent dan ook van het prefix probleem af

mosymuis schreef: [*] Je begint je for() loop met 1, en geeft daarom een SQL limit aan van 26, ipv 25. Als je de het middelste stuk van je loop nu met een <= vergelijking schrijft kan de limit wel 25 blijven.

mosymuis schreef: [*] Waarom heb je een compleet deel van de pagina er dubbel in staan, de tweede keer tussen comments? Je zal het vast nodig gehad hebben bij het testen, maar let hier wel op als je de mod online zet voor het publiek.

mosymuis schreef: [*] Je gebruikt 4 query's waar je met ééntje genoeg hebt.

Code: Selecteer alles

SELECT u.user_id, u.username, t.topic_title, pt.post_text
FROM " . USERS_TABLE . " u, " . TOPICS_TABLE . " t, " . POSTS_TABLE . " p, " . POSTS_TEXT_TABLE . " pt
WHERE pt.post_id = p.post_id
AND p.topic_id = t.topic_id
AND t.topic_poster = u.user_id
ORDER BY CHAR_LENGTH(post_text) DESC
LIMIT 26;

[*] Voor query 3 vraag je de gegevens opeens op met mysql_fetch_object(), waarom? Ook moet je consequent met het $db object werken, dus zoals je dat bij query 2 hebt gedaan.
[*] Je begint en eindigt 3 keer een <? .. ?> PHP gedeelte, dit is onnodig omdat het allemaal PHP is en het veroorzaakt onnodige spaties en returns in je source. Als je zou werken met cookie functies zou het je zelfs fatale errors opleveren.
[*] In het onderste stuk schrijf je een HTML tabel in je PHP bestand, doe dit met je template.
[*] Maak links van de usernames, aan de hand van het user_id uit de query.
[*] Schrijf je links, zoals die naar viewtopic, in je PHP bestand en haal ze eerst door append_sid(). Maak ook gebruik van $phpEx en de constante POST_POST_URL voor p=.
[*] Ik zie in een van de onderste comments een http://www.reev.nl/forum/ url. Slordig![/list]