[RC] Top Ten Topics

[brunino]

Micogian sarebbe carino mettere nel PCA "abilita/disabilita", "in alto/in basso" e numero di record da estrarre.

Nel costruttore non hai messo phpEX...

Inviato con Tapatalk 2 da Android

[Micogian]

Per metterla in basso ho usato il template "index_body_stat_blocks_before.html", come fare per inserire la scelta nel PCA non saprei ma non credo sia una cosa tanto complicata.
Sul numero non saprei, io l'avevo chiamata Topten proprio perchè erano 10 i topics estratti.
Le variabili $root_path e $phpEx le avevo inserite nel costruttore ma mentre la $root_path funzionava, la $phpEx no, temporaneamente ho messo "php".
Non so come la pensate voi, ma per le estensioni che creiamo in ambito di questo Forum perchè non usiamo la cartella "phpbbitalia" come lo staff di phpbb.com usa "phpbb" ?
Tanto l'autore o gli autori vanno indicati nel "composer".
L'estensione Topten sarà inserita in "ext/phpbbitalia/topten/"

[brunino]

Infatti nel pacchetto che rilascio più tardi di questa mod ho messo come directory autore phpbbitalia e directory toptentopics (ho già sistemato composer e gli altri file) poi nel composer si mette quelli che ci hanno lavorato.

Nel costruttore phpEX l'hai dichiarata sotto ma non dentro le tonde. Per quanto riguarda il PCA ora guardo.

Inviato con Tapatalk 2 da Android

[Carlo]

Codice: Seleziona tutto

public function __construct(\phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\auth\auth $auth, \phpbb\user $user)
{
	$this->db = $db;
	$this->template = $template; 
	$this->auth = $auth;
	$this->user = $user;
	$this->root_path = $root_path;
	$this->phpEx = $phpEx;
}

Ti sbagli, nessuna delle 2 ultime variabili va bene. All'interno del costruttore non esistono le variabili $root_path e $phpEx. Devi definirle come argomenti nel servizio (vedi file services.yml), e poi prenderle in input dal costruttore.

Secondo te, $root_path va bene semplicemente perché il valore che assume regolarmente è "./" (cartella attuale), e andrebbe bene in questo caso anche se non viene specificato (in quanto PHP assume come cartella di riferimento, l'attuale). Infatti, quella variabile non esiste, quindi anziché riportare come valore di percorso di phpBB "./" riporta "" (nulla) che va bene comunque. Non so se mi sono spiegato correttamente.

Nella definizione del servizio del listener (file services.yml), devi aggiungere come argomento %core.root_path% e %core.php_ext%.

Infine devi passarle al costruttore in input, sostituendo:

Codice: Seleziona tutto

public function __construct(\phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\auth\auth $auth, \phpbb\user $user) 

con:

Codice: Seleziona tutto

public function __construct(\phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\auth\auth $auth, \phpbb\user $user, $root_path, $phpEx) 

[Micogian]

Probabilmente è una stupidaggine ma per ora non funziona.
Avevo trovato un errore nel file service.yml, non era stato cambiato il nome dell'estensione da "topten" a "toptentopics".
Ho inserito le due variabili, adesso il file service.yml è così:

Codice: Seleziona tutto

services:
    phpbbitalia.toptentopics.listener:
        class: phpbbitalia\toptentopics\event\listener
        arguments:
            - @dbal.conn
            - @template
            - @auth
            - @user
            - @cache
            - @config
            - @content.visibility
            - @dispatcher
            - @pagination
            - @request			
            - %core.root_path%
            - %core.php_ext%

        tags:
            - { name: event.listener }

Il file listener.php è stato così modificato:

Codice: Seleziona tutto

namespace phpbbitalia\toptentopics\event;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
class listener implements
EventSubscriberInterface{
protected $db;
protected $table_prefix;
protected $request;
protected $search_engines;
protected $template;
protected $auth;
protected $user;
protected $root_path;
protected $phpEx;

public function __construct(\phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\auth\auth $auth, \phpbb\user $user, $root_path, $phpEx) 
{
	$this->db = $db;
	$this->template = $template; 
	$this->auth = $auth;
	$this->user = $user;
	$this->root_path = $root_path;
	$this->phpEx	= $phpEx ;
}

Il link

Codice: Seleziona tutto

$last_topic_link[$n1]   		= append_sid("($this->root_path)viewtopic.$this->phpEx", "f=" . $row1['forum_id'] . "&t=" . $row1['topic_id']);

Le variabili $this->root_path e $this->phpEx non vengono interpretate, mi dà il seguente errore

Catchable fatal error: Object of class phpbb\config\db could not be converted to string in D:\xampp\htdocs\actafungorum\actaforum\ext\phpbbitalia\toptentopics\event\listener.php on line 58

La riga 58 è quella del link. Dove sbaglio?

[Carlo]

Nella definizione del servizio hai specificato 11 parametri, mentre il costruttore ne richiede solo 6 in input.

I parametri in entrambi i casi devono coincidere, sia nel numero, sia nell'ordine.

Nello specifico, l'errore si presenta perché nella definizione del servizio, phpBB passa alla tua classe come 5° parametro il servizio cache, ma il costruttore come quinto parametro vuole il percorso della root di phpBB.

Comunque, non credo tu abbia bisogno di tutti quei servizi nella tua estensione, come per esempio il servizio @request.

[brunino]

Rilasciata la prima versione 0.1.0-a1
Leggere il primo post di questo argomento per download e dettagli

[brunino]

Rilasciata la versione 0.1.0-a2
Leggere il primo post di questo argomento per download e dettagli.

Novità:
-> Risolto bug topic in approvazione

[brunino]

Rilasciata la versione 0.1.0-b1
Leggere il primo post di questo argomento per download e dettagli.

Novità:
-> Aggiunti file di lingua

[Ares]

Ottimo lavoro ragazzi!