Topic_list - Lista alfabetica delle discussioni

Micogian · Messaggio da **Micogian** » 06/09/2023, 10:33

Sicuro di aver modificato ola variabile $per_page nel file listener.php ? Hai messo la versione 1.0.8 ?
Io avevo messo 4 per default per provare nel mio forum dove non ho molti topics.
Adesso ho messo $per_page = 20 e funziona regolarmente.: https://giannidose.altervista.org/forum ... c.php?t=52
Al limite prova a cancellare la cache, anche se le modifiche ai file php non dovrebbero influire sulla cache, oppure cancella i dati del browser.

ItalianCoverDesign · Messaggio da **ItalianCoverDesign** » 06/09/2023, 15:02

Perdonami. Avevo inviato un post di correzione proprio ieri ma a quanto pare non è andato a buon fine.
Il primo problema l'ho risolto... ero idiota io (è per la cronaca, lo sono ancora) che salvavo il file con le modifiche da un'altra parte, e mettevo sul forum sempre il file originale!
Quindi problema 1 risolto, il secondo rimane.

Ci tengo comunque a dire che hai fatto una grandissima estensione. Complimenti.

Micogian · Messaggio da **Micogian** » 06/09/2023, 22:04

Non saprei, nel mio forum ho dei titoli che iniziano con un numero ma funziona. https://giannidose.altervista.org/forum ... c.php?t=52

Io però ho phpbb 3.3.8 e php 7.3. Non è che puoi provare a modificare temporaneamente il PHP ?

Dovrei provare ad aggiornare in locale dove ho php 8.0

ItalianCoverDesign · Messaggio da **ItalianCoverDesign** » 07/09/2023, 11:41

Dunque, ho provato sia con PHP 7.3, 7.4 e 8.0, ma l'errore rimane.

Codice: Seleziona tutto

Errore Generale
SQL ERROR [ mysqli ]

In aggregated query without GROUP BY, expression #1 of SELECT list contains nonaggregated column 'Sql1573107_4.t.forum_id'; this is incompatible with sql_mode=only_full_group_by [1140]

SQL

SELECT t.forum_id, t.topic_id, count(t.topic_id) AS total FROM avphpbb_topics t WHERE t.forum_id IN(26,27) AND t.topic_moved_id = 0 AND t.topic_last_post_time > 0 ORDER BY t.topic_id

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 1023
CALL: trigger_error()

FILE: [ROOT]/phpbb/db/driver/mysqli.php
LINE: 202
CALL: phpbb\db\driver\driver->sql_error()

FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 345
CALL: phpbb\db\driver\mysqli->sql_query()

FILE: [ROOT]/ext/micogian/topic_list/event/listener.php
LINE: 301
CALL: phpbb\db\driver\factory->sql_query()

FILE: [ROOT]/vendor/symfony/event-dispatcher/EventDispatcher.php
LINE: 214
CALL: micogian\topic_list\event\listener->viewtopic_add()

FILE: [ROOT]/vendor/symfony/event-dispatcher/EventDispatcher.php
LINE: 44
CALL: Symfony\Component\EventDispatcher\EventDispatcher->doDispatch()

FILE: [ROOT]/phpbb/event/dispatcher.php
LINE: 62
CALL: Symfony\Component\EventDispatcher\EventDispatcher->dispatch()

FILE: [ROOT]/phpbb/event/dispatcher.php
LINE: 46
CALL: phpbb\event\dispatcher->dispatch()

FILE: [ROOT]/viewtopic.php
LINE: 2155
CALL: phpbb\event\dispatcher->trigger_event()

Ora, perdona la mia domanda, anche io nel mio forum ho dei thread che iniziano con numero, ma forse no in questa particolare lista. Potrebbe essere causato da quello l'errore?
Chiaramente la mia è un'ipotesi perchè non sono un programmatore e non so da cosa possa dipendere la cosa.

Grazie

Micogian · Messaggio da **Micogian** » 07/09/2023, 13:01

Forse non ho capito bene cosa intendi per "Se invece provo la versione titoli con numeri iniziali ....".
Avevo pensato ai topics che iniziano cn un numero che però mi sembra funzionino.

ItalianCoverDesign · Messaggio da **ItalianCoverDesign** » 07/09/2023, 15:05

Dunque, rispiego dal principio perché magari ho fatto confusione

Installato versione 1.0.8 e funziona come da foto:

screenshot_735.png

cioè perfettamente (o quasi, spiego più avanti)
ma, se scarico i due file aggiuntivi da sostituire, quelli della versione per topics con numeri iniziali nel titolo (file da sostituire), mi genera gli errori del post precedente, sia da PHP 7.3 fino alla versione attuale, la 8.2.

Ora, io non ho ben capito cosa fanno esattamente questi 2 file all'estensione, e li volevo provare, ma da quanto vedo, fa già tutto quello che mi serve, con una sola postilla da segnalare, cioè, dove si vede la lettera di raggruppamento dei thread nell'elenco (A, B, C, ecc) per quelli che iniziano con numero, appare solo "0" al posto di "0-9":

screenshot_737.png

Io immagino che i 2 file modifichino l'estensione per per raggruppare tutti i thread anche per numero: quelli che iniziano per "0", per "1", per "2", e così via. Quindi se i 2 file aggiuntivi fanno questo, a me non serve.

Chiedo solo di sapere come far apparire al posto di quello "0", un "0-9".

Spero di esservi riuscito a spiegarmi stavolta. Grazie come sempre del tuo tempo.

Micogian · Messaggio da **Micogian** » 07/09/2023, 16:07

Ma io non so quali sono questi due file da sostituire per i topics con iniziali nel titolo, forse non sono miei e se sono miei saranno per qualcuno che mi avrà chiesto una modifica ma al momento non ricordo. Può darsi che abbia fatto qualche modifica che poi non ho aggiornato.
Dove li hai scaricati questi file ? Io gli aggiornamenti dell'estensione li carico su GitHub e al momento l'ultima versione è la 1.0.8.
Che poi nella Lista compaia l'intestazione "0" e non "0-9" è normale, non mi sembra un grosso problema, vuole solo indicare che il primo carattere è un numero.
Altra caratteristica dell'estensione è che se il totale dei records è minore del valore "$per_page" nella pagina compaiono dei tasti alfabetici che portano, nella Lista, direttamente al carattere selezionato.

PS: adesso ho visto che la modifica è inserita nel primo post, ma è probabile che poi non l'abbia aggiornata.
Alla fine ho fatto una ricerca e risulta che l'utente "eniac" , in data 20/5/2021, mi avevo chiesto una modifica ai topics con numero iniziale nel titolo perchè nel suo Forum aveva molti topics di questo tipo e preferiva dividerli per numero iniziale e non inseriti nel gruppo "0".
Sinceramente non ho aggiornato questa versione.
Il problema potrebbe dipendere dal valore "replies" che sono le risposte al topic. A suo tempo PHPBB questo valore lo aveva inserito nella tabella "topics" ma poi, non ricordo da quale versione di PHPBB, aveva tolto il campo "replies" dalla tabella "topics" inserendo al suo posto un codice che questo valore lo calcolava. Nella versione standard dell'estensione avevo inserito il calcolo ma nella versione con numeri i9niziali non l'ho fatto.
Adesso nella prima pagina ho inserito un avviso che quella versione non è aggiornata. In pratica quella versione suddivideva i topics con iniziale numerica in base al primo numero, salvo casi particolari non mi sembra una modifica necessaria.

ItalianCoverDesign · Messaggio da **ItalianCoverDesign** » 07/09/2023, 16:41

Si, primo post.
Volendo modificare solo quella dicitura, c'è un modo rapido e veloce per inserire "0-9" al posto di "0"?

Micogian · Messaggio da **Micogian** » 07/09/2023, 16:57

Dalla riga 274 del file "listener.php" c'è questo codice

Codice: Seleziona tutto

        if (strchr("0123456789", $row1['first_char']))
        {
           $first_char[$i] = '0';
        }else if (strchr($string, $row1['first_char']) )
        {
           $first_char[$i] = $row1['first_char'];
        }else{
           $first_char[$i]    = "@";
        }

Basta modificare la riga "$first_char[$i] = '0';" con "$first_char[$i] = '0-9';" ma poi non so se questo crea qualche altro problema perchè "$firsts_char[$i]" dovrebbe essere un carattere singolo.

ItalianCoverDesign · Messaggio da **ItalianCoverDesign** » 07/09/2023, 17:09

Grazie.
Allora lo metto in prova sul mio forum di prova. Se vedo anomalie, le segnalo subito qui. Per ora, grazie!

EDIT: Modifiche fatte. attualmente sembra OK

screenshot_738.png

phpBB-Italia.it

Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni

Re: Topic_list - Lista alfabetica delle discussioni