Topten Topics

[maverix]

Io questa MOD è molto che l'ho installata e funziona bene, ho però modificato la formattazione e tolto alcuni numeri per migliorare la visibilità........
Recentemente ho scoperto, avevo già notato ma non capivo perfettamente, che nella lista "ultimi messaggi" NON ultimi argomenti, se clicco mi rimanda proprio a quella determinata risposta MA SOLO se facente parte della prima pagina dell'argomento, se invece tale argomento ha per esempio due o più sottopagine cliccando su "ultimi messaggi" rimanda al primo messaggio della prima pagina......
Qualcuno ha notato la stessa cosa? Se così fosse si può rimediare?

[Micogian]

Devi modificare il link al post aggiungendo il post_id.
Nel file topten_topic.php, la riga

Codice: Seleziona tutto

$last_post_link[$n4]		= append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row4['forum_id'] . "&t=" . $row4['topic_id'] .  "#p" . $row4['post_id']);

diventa

Codice: Seleziona tutto

$last_post_link[$n4]		= append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row4['forum_id'] . "&t=" . $row4['topic_id'] . "&p=" . $row4['post_id'] . "#p" . $row4['post_id']);

[maverix]

Bingo, sei Grande...... velocissimo nel rispondere....
Appena ho un attimo provo e ti faccio sapere.
Potrebbe diventare una modifica ufficiale a tutto il pacchetto?

[maverix]

L'ultimo tuo suggerimento non funziona......
Ho fatto la modifica e ora rimanda SEMPRE al primo messaggio dell'argomento.
Ripristinato il vecchio file e il problema è come descritto sopra.

[Micogian]

Strano perchè io ho provato e funziona, Vedi nel mio Forum.
Fai attenzione all'url che esce, dovrebbe essere così:

Codice: Seleziona tutto

viewtopic.php?f=7&t=4906&p=60458#p60458

f sta per forum_id
t sta per topic_id
p sta per post_id (va messo due volte, una per il post, l'altra per la pagina.)

Se è installata la SEO non so come si comporta, questo è l'url normale.
Basta cliccare sul titolo di un post per avere l'url completo per andare a quella pagina.

[maverix]

Codice: Seleziona tutto

http://www.pompone.eu/viewtopic.php?f=11&t=18960#p266418

Questo è quello che esce in un argomento di 2 pagine, ma rimanda al primo messaggio della prima pagina.
MOD normale.

Codice: Seleziona tutto

http://www.pompone.eu/viewtopic.php?f=11&t=18960&p=#p

Questo è quello che esce dopo la modifica con lo stesso argomento...

E questo è il codice modificato

Codice: Seleziona tutto

 $last_post_link[$n4]      = append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row4['forum_id'] . "&t=" . $row4['topic_id'] . "&p=" . $row4['post_id'] . "#p" . $row4['post_id']);

[Micogian]

Se il risultato è questo

Codice: Seleziona tutto

http://www.pompone.eu/viewtopic.php?f=11&t=18960&p=#p

ovvio che non funziona, manca il valore del post_id.
C'è qualcosa che non va nel codice, postami il codice del file "topten_topics.php" che usi tu.

[maverix]

Codice: Seleziona tutto

<?php
// MOD Topten Topics : Topics più recenti, Topics più visti e Posts più recenti 
// created by Micogian (Bingo)
if (!defined('IN_PHPBB'))
{
   exit;
}
// configurazione: numero dei recods da visualizzare nella Lista
$list_rec = 20 ;   // numero di records da estrarre dalla tabella (alcuni topics potrebbero non aver il permesso di lettura)
$list_view = 10 ;  // sostituire il numero della variabile $lista se si desidera diminuire/aumentare il numero dei Topics 

//---------- 10 Topics più Recenti start -----------//
$sql1 = "SELECT tt.topic_id, tt.forum_id, tt.topic_title, tt.topic_time, tt.topic_moved_id, tt.topic_first_poster_name,
    ft.forum_id, ft.forum_name
    FROM " . TOPICS_TABLE . " tt, " . FORUMS_TABLE . " ft 
    WHERE tt.topic_moved_id = 0
    AND tt.forum_id = ft.forum_id
    ORDER BY tt.topic_time DESC LIMIT 0,$list_rec";
$result1 = $db->sql_query($sql1);
    $n1 = 0;
    while ($row1 = $db->sql_fetchrow($result1))
    {
       	if ($auth->acl_get('f_read', $row1['forum_id']) == 1) 
        {
            if ($n1 < $list_view)
			{
				if (strlen($row1['topic_title']) > 28)
				{
				$topic_title1 = substr($row1['topic_title'],0,27) . "...";
				}else{
				$topic_title1 = $row1['topic_title'];
				}
			$last_topic_link[$n1]   		= append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row1['forum_id'] . "&t=" . $row1['topic_id']);
            $last_topic_title[$n1]  		= $row1['topic_title'];
			$last_topic_title_short[$n1]  	= $topic_title1;
			$last_topic_forum[$n1]  		= $row1['forum_name'];
            $last_topic_author[$n1] 		= $row1['topic_first_poster_name'];
            $last_topic_data[$n1]   		= $user->format_date($row1['topic_time'], "|d M|");
			++$n1 ;          	
            }else{
			break ;
			}
        }
    }
//---------- 10 Topics più Recenti end -----------//

//---------- 10 Topics più Visti start -----------//

$sql2 = "SELECT tt.topic_id, tt.forum_id, tt.topic_title, tt.topic_first_poster_name, tt.topic_views,
    ft.forum_id, ft.forum_name 
    FROM " . TOPICS_TABLE . " tt, " . FORUMS_TABLE . " ft
    WHERE tt.forum_id = ft.forum_id
    ORDER BY tt.topic_views DESC LIMIT 0,$list_rec";
$result2 = $db->sql_query($sql2);
    $n2 = 0 ;
    while ($row2 = $db->sql_fetchrow($result2))
    {
		if ($auth->acl_get('f_read', $row2['forum_id']) == 1)
		{
			if ($n2 < $list_view)
			{
				if (strlen($row2['topic_title']) > 28)
				{
				$topic_title2 = substr($row2['topic_title'],0,27) . "...";
				}else{
				$topic_title2 = $row2['topic_title'];
				}
			$view_topic_link[$n2]   		= append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row2['forum_id'] . "&t=" . $row2['topic_id']);
			$view_topic_title[$n2]			= $row2['topic_title'];
			$view_topic_title_short[$n2]  	= $topic_title2;	
			$view_topic_forum[$n2]  		= $row2['forum_name'];		
			$view_topic_author[$n2] 		= $row2['topic_first_poster_name'];
			$view_topic_views[$n2]  		= $row2['topic_views'];
			++$n2 ;
			}else{
			break ;
			}
		}
	}
//---------- 10 Topics più visti end -----------//

//---------- 10 Ultimi posts start -----------//
$sql4 = "SELECT tt.topic_id, tt.forum_id, tt.topic_moved_id, tt.topic_last_post_id, tt.topic_last_poster_id, tt.topic_last_poster_name, tt.topic_last_post_subject, tt.topic_last_post_time,
    ft.forum_id, ft.forum_name
    FROM " . TOPICS_TABLE . " tt, " . FORUMS_TABLE . " ft 
    WHERE tt.topic_moved_id = 0
    AND tt.forum_id = ft.forum_id
    ORDER BY tt.topic_last_post_time DESC LIMIT 0,$list_rec";
$result4 = $db->sql_query($sql4);
    $n4 = 0;
    while ($row4 = $db->sql_fetchrow($result4))
    {
       	if ($auth->acl_get('f_read', $row4['forum_id']) == 1) 
        {
            if ($n4 < $list_view)
			{
				$post_subject = str_replace("Re: ", "", $row4['topic_last_post_subject']) ;
				if (strlen($post_subject) > 28)
				{
				$post_title = substr($post_subject,0,25) . "...";
				}else{
				$post_title = $post_subject ;
				}
			    $last_post_link[$n4]      = append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row4['forum_id'] . "&t=" . $row4['topic_id'] . "&p=" . $row4['post_id'] . "#p" . $row4['post_id']);
			//$last_post_link[$n4]   		= append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row4['forum_id'] . "&t=" . $row4['topic_id'] . "p#" . $row4['topic_last_post_id']);
			$last_post_title[$n4] 		= $row4['topic_last_post_subject'];
			$last_post_title_short[$n4] = $post_title;
			$last_post_forum[$n4]  		= $row4['forum_name'];
			$last_post_author[$n4] 		= $row4['topic_last_poster_name'];
			++$n4 ;          	
			}else{
			break ;
			}
        }
    }
//---------- 10 Ultimi posts end -----------//

// Crea l'array "topten_list" che contiene le Variabili per il Template
for ($x = 0; $x < $list_view; ++$x)
{
    $template->assign_block_vars('topten_list',array(
       'LAST_TOPIC_LINK'         => $last_topic_link[$x],
       'LAST_TOPIC_TITLE'         => $last_topic_title[$x],
       'LAST_TOPIC_TITLE_SHORT'   => strtolower($last_topic_title_short[$x]),
       'LAST_TOPIC_FORUM'         => $last_topic_forum[$x],
       'LAST_TOPIC_AUTHOR'         => $last_topic_author[$x],
       'LAST_TOPIC_DATA'         => $last_topic_data[$x],

       'VIEW_TOPIC_LINK'         => $view_topic_link[$x],
       'VIEW_TOPIC_TITLE'         => $view_topic_title[$x],
       'VIEW_TOPIC_TITLE_SHORT'   => strtolower($view_topic_title_short[$x]),
       'VIEW_TOPIC_FORUM'         => $view_topic_forum[$x],
       'VIEW_TOPIC_AUTHOR'         => $view_topic_author[$x],
       'VIEW_TOPIC_VIEWS'         => $view_topic_views[$x],

       'LAST_POST_LINK'         => $last_post_link[$x],
       'LAST_POST_TITLE'         => $last_post_title[$x],
       'LAST_POST_TITLE_SHORT'      => strtolower($last_post_title_short[$x]),
       'LAST_POST_FORUM'         => $last_post_forum[$x],
       'LAST_POST_AUTHOR'         => $last_post_author[$x]
       ));
}
?>

Forse non ti ricorderai ma io avevo fatto qualche modifica di formattazione delle liste togliendo numeri e maiuscolo....può essere questo? Non ricordo però in quale file del pacchetto l'ho fatta......

[Micogian]

Ho capito, nel mio Forum ho ancora la versione che estraeva i dati dalla tabella POSTS, quindi ho a disposizione il post_id.
Con la nuova versione il numero dell'ultimo post c'è ma si chiama topic_last_post_id, quindi la modifica da fare è questa:

Codice: Seleziona tutto

$last_post_link[$n4]      = append_sid("{$phpbb_root_path}viewtopic.$phpEx", "f=" . $row4['forum_id'] . "&t=" . $row4['topic_id'] . "&p=" . $row4['topic_last_post_id'] . "#p" . $row4['topic_last_post_id']);

[maverix]

Grande Bingo..........hai fatto centro....
Ora funziona e per quanto mi riguarda la MOD è perfetta.
Pensi sia una modifica da aggionrnare nel pacchetto........?