[2.0.21] Another News Mod

Zelf bezig aan een modificatie? Wij kijken graag mee..
Forumregels

Sinds 1 januari 2009 wordt phpBB2 niet meer ondersteund.
Onderstaande informatie is verouderd en dient uitsluitend als archief.
phpBB2.0.x
Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

[2.0.21] Another News Mod

Bericht door Jan » 25 jun 2006, 01:13

  • 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
Downloaden doe je [HIER]
Bij deze zie je ineens m'n site + forum eens.
Laatst gewijzigd door Jan op 16 sep 2006, 18:42, 4 keer totaal gewijzigd.
Vriendelijke groet,
Jan
Gratis Forum - Gratis Blog - Blokart

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 25 jun 2006, 01:14

Ik heb 'm ook al aangekondigd + toegevoegd op de Mod-sectie op phpBB.com
Wat vinden jullie er trouwens van?

Gebruikersavatar
hendry
Berichten: 629
Lid geworden op: 04 sep 2005, 10:41
Contacteer:

Bericht door hendry » 25 jun 2006, 05:10

wat doet deze mod als ik vragen mag
haai

DjChrissoow
Berichten: 13
Lid geworden op: 20 dec 2005, 14:29
Locatie: utrecht/NL
Contacteer:

Bericht door DjChrissoow » 25 jun 2006, 08:10

Gaeeft het nieuws ? :wink: :?:

Gebruikersavatar
Paul
Beheerder
Beheerder
Berichten: 20297
Lid geworden op: 23 okt 2003, 11:38
Locatie: Utrecht
Contacteer:

Bericht door Paul » 25 jun 2006, 08:21

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 :D). 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?

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 25 jun 2006, 09:27

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 :D). 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?
Jah, hij werkt zonder bijkomende problemen, dat is het belangrijkste, toch? Al de rest van die portal mod has ik niet nodig

Gebruikersavatar
Paul
Beheerder
Beheerder
Berichten: 20297
Lid geworden op: 23 okt 2003, 11:38
Locatie: Utrecht
Contacteer:

Bericht door Paul » 25 jun 2006, 09:39

Nee, dat is naar mijn mening niet het belangerijkste. Het belangerijkste als mod auteur is dat je niet veel overbodige code gebruikt. Zie het ook niet als afkraking van je mod mijn bericht, maar als opbouwende kritiek ;)

Gebruikersavatar
Stef
Berichten: 9080
Lid geworden op: 04 jun 2003, 20:47

Bericht door Stef » 25 jun 2006, 11:19

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
Dan had je ze toch kunnen verwijderen? Een modification moet makkelijk te installeren zijn, niet teveel rotzooi met zich meebrengen en gemakkelijk aan te passen zijn.

Zo'n mod komt de mod sectie van phpBB niet eens in. :-)

Gebruikersavatar
Bee
Berichten: 13403
Lid geworden op: 29 aug 2004, 10:30

Bericht door Bee » 25 jun 2006, 11:28

Welcome to phpBB 2 Installation :?:

Er is iets mis met je download.
... Maar ik modereer (nog) niet.

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 13:37

Is het goed dat ik 'm bijwerk.
Zouden jullie 'm erna nogeens willen controleren?

Gebruikersavatar
Stef
Berichten: 9080
Lid geworden op: 04 jun 2003, 20:47

Bericht door Stef » 27 jun 2006, 13:55

Tuurlijk, we willen dat er een mooie mod uit komt.

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 14:08

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

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 14:09

paul schreef:Het is eigenlijk een verbouwde ezportal (En met uit de headers niet eens een vermelding naar smartor :)).
Had ik inderdaad moeten doen, ga me er niet uit goedpraten, maar in de volgende zit ie inbegrepen.

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 14:24

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

Gebruikersavatar
Bee
Berichten: 13403
Lid geworden op: 29 aug 2004, 10:30

Bericht door Bee » 27 jun 2006, 14:28

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
Al eens gehoord van simple_header.tpl en simple_footer.tpl? Die kan je gewoon bereiken met page_header.tpl, door voor de aanroep van het bestand in je PHP bestand dit te plaatsen:

Code: Selecteer alles

$gen_simple_header = TRUE;
Daarmee worden de simple_header.tpl en simple_footer.tpl gebruikt, die bijvoorbeeld ook gebruikt worden bij het onderwerp voorbeeld, en de privebericht popup.
Jan-phpBBfm schreef:Komt deze al wat beter in de buurt?
http://janmarques.be.tt/2.rar
Helaas kwam mijn opmerking te laat om hem mee te nemen :(

Code: Selecteer alles

$page_title = $lang['Newssite'];
Die lang kom ik niet tegen in de modinstructies?
... Maar ik modereer (nog) niet.

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 14:37

Dus delete ik news_header.tpl, news_header.php en news_tail.php en verwijs ik in announcements.php naar page_header.tpl ??

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 14:49

Nop, de news_header moet blijven.
Nu heb ik deze genomen als page_header.php met ervoor

Code: Selecteer alles

$gen_simple_header = TRUE;
ervoor.
Indien ik dit met de gewone page_header.php deed, had het hele forum geen header.

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 14:53

Download met veranderingen ff hier aub:
http://www.janmarques.be.tt/3.rar

Gebruikersavatar
Paul
Beheerder
Beheerder
Berichten: 20297
Lid geworden op: 23 okt 2003, 11:38
Locatie: Utrecht
Contacteer:

Bericht door Paul » 27 jun 2006, 14:57

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.

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.

er moet na:

Code: Selecteer alles

$template->pparse('body');
nog

Code: Selecteer alles

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

Gebruikersavatar
Jan
Berichten: 1987
Lid geworden op: 30 mei 2006, 20:30
Locatie: De kelder van Dhr. J. Fritzl
Contacteer:

Bericht door Jan » 27 jun 2006, 15:34

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.
Done, thanks
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.

Heb ik in m'n install.mod laten toevoegen
paul schreef:er moet na:

Code: Selecteer alles

$template->pparse('body');
nog

Code: Selecteer alles

include($phpbb_root_path . 'includes/page_tail.' $phpEx);
Als ik dat toevoeg, krijgt ie een string error.
Indien ie er niet staat, is er geen enkel probleem. Ik laat dat dus maar even zo.

Alles samen: http://janmarques.be.tt/4.rar

Gesloten