phpBB.nl

• Classification: 51 Add-On's
  
  MOD Name: Another News Mod
  Author: Jan Marques
  MOD Description: Allows you to put the latest topics in one or more forums in a frame on your main webpage
  
  MOD Version: 1.0.0.
  Installation Level: Easy
  Installation Time: ~ 1 minute

paul schreef:Het is een soort topics anywhere voor op je forum.
Het is eigenlijk een verbouwde ezportal (En met uit de headers niet eens een vermelding naar smartor ).
Verder snap ik niet waarom er speciaal een nieuws_header.php/nieuws_footer.php inzit? nieuwsheader.php bevat dingen die nooit gebruikt zullen worden op die pagina, en is kwa verschil niet verschillend met page_header.php. Verder produceert de mod nu geen valid html. Je sluit </html> en </body> niet af, doordat je geen footer parsed.
Fetchposts.php bevat nog onnodige dingen van portal.php. (fetch poll ).
verder maak je geen gebruik van de $lang vars, wat wel verplicht is (Anders komt hij niet eens in de moddb ).

Code: Selecteer alles

define('SHOW_ONLINE', true);

Waarom defineer je die variabelen? Weet je wel wat hij doet? Hij zorgt ervoor dat je who's online weergeeft op die pagina(En dus voor extra querys ), maar die weergeef jij helemaal niet

Code: Selecteer alles

$page_title = $lang['Home'];

Nog een restje ezportal,
$lang['home'] bestaat standard niet, en zal geen titel geven.

Code: Selecteer alles

$template->assign_block_vars('welcome_text', array());

en weer (niet gebruikte) ezportal code

verder snap ik dit stuk code niet waarom je het gebruikt:

Code: Selecteer alles

if(!isset($HTTP_GET_VARS['article']))
{
	$template->assign_block_vars('welcome_text', array());

	$fetchposts = phpbb_fetch_posts($CFG['news_forum'], $CFG['number_of_news'], $CFG['news_length']);

	for ($i = 0; $i < count($fetchposts); $i++)
	{
		if( $fetchposts[$i]['striped'] == 1 )
		{
			$open_bracket = '[ ';
			$close_bracket = ' ]';
			$read_full = $lang['Read_Full'];
		}
		else
		{
			$open_bracket = '';
			$close_bracket = '';
			$read_full = '';
		}

		$template->assign_block_vars('fetchpost_row', array(
			'TITLE' => $fetchposts[$i]['topic_title'],
			'POSTER' => $fetchposts[$i]['username'],
			'TIME' => $fetchposts[$i]['topic_time'],
			'TEXT' => $fetchposts[$i]['post_text'],
			'REPLIES' => $fetchposts[$i]['topic_replies'],
			'U_VIEW_COMMENTS' => append_sid('viewtopic.' . $phpEx . '?t=' . $fetchposts[$i]['topic_id']),
			'U_POST_COMMENT' => append_sid('posting.' . $phpEx . '?mode=reply&t=' . $fetchposts[$i]['topic_id']),
			'U_READ_FULL' => append_sid('portal.' . $phpEx . '?article=' . $i),
			'L_READ_FULL' => $read_full,
			'OPEN' => $open_bracket,
			'CLOSE' => $close_bracket)
		);
	}
}
else
{
	$fetchposts = phpbb_fetch_posts($CFG['news_forum'], $CFG['number_of_news'], 0);

	$i = intval($HTTP_GET_VARS['article']);

	$template->assign_block_vars('fetchpost_row', array(
		'TITLE' => $fetchposts[$i]['topic_title'],
		'POSTER' => $fetchposts[$i]['username'],
		'TIME' => $fetchposts[$i]['topic_time'],
		'TEXT' => $fetchposts[$i]['post_text'],
		'REPLIES' => $fetchposts[$i]['topic_replies'],
		'U_VIEW_COMMENTS' => append_sid('viewtopic.' . $phpEx . '?t=' . $fetchposts[$i]['topic_id']),
		'U_POST_COMMENT' => append_sid('posting.' . $phpEx . '?mode=reply&t=' . $fetchposts[$i]['topic_id'])
		)
	);
}

(Ook van ezportal ja ). Waarom verwijs je niet gewoon direct naar viewtopic? Dat lijkt me veel logischer, als in het iframe dat bericht te laten zien (Of in nieuw venster).
Verder snap ik niet waarom je de standaard lengte van het bericht zo hoog zet? Stel je nou is voor, je laat als nieuwscategorie hier phpbb2.0.x support gelden, en in de eerste 10 topics starts staat viewtopic.php in code blokken? dan wordt mijn scherm zo groot, dat ik bijna 10 keer een viewtopic zie
Verder zie ik op jouw site geen voorbeeld?

Jan-phpBBfm schreef:Jah, hij werkt zonder bijkomende problemen, dat is het belangrijkste, toch? Al de rest van die portal mod has ik niet nodig

paul schreef:Verder snap ik niet waarom er speciaal een nieuws_header.php/nieuws_footer.php inzit?

paul schreef:Het is eigenlijk een verbouwde ezportal (En met uit de headers niet eens een vermelding naar smartor ).

Jan-phpBBfm schreef:

paul schreef:Verder snap ik niet waarom er speciaal een nieuws_header.php/nieuws_footer.php inzit?

Om in frame de login, memberlist, logo niet weer te geven, en toch de CSS niet te verwijderen

$gen_simple_header = TRUE;

Jan-phpBBfm schreef:Komt deze al wat beter in de buurt?
http://janmarques.be.tt/2.rar

Code: Selecteer alles

$page_title = $lang['Newssite'];

paul schreef:

Code: Selecteer alles

$page_title = $lang['Newssite'];

Moet zijn

Code: Selecteer alles

$page_title = $lang['Home'];

Code: Selecteer alles

include($phpbb_root_path . 'includes/news_header.'.$phpEx);

Beter is:

Code: Selecteer alles

$gen_simple_header = TRUE;
include($phpbb_root_path . 'includes/page_header.'.$phpEx);

En dan kan news_header.php/tpl ook weg.

paul schreef:

Code: Selecteer alles

$read_full = $lang['Read_Full'];

bestaat standaard zover ik weet niet in het taalkpakket.

Code: Selecteer alles

'U_READ_FULL' => append_sid('portal.' . $phpEx . '?article=' . $i),

Werkt alleen als je portal.php hebt, je moet de url aanpassen.

paul schreef:er moet na:

Code: Selecteer alles

$template->pparse('body');

nog

Code: Selecteer alles

include($phpbb_root_path . 'includes/page_tail.' $phpEx);