[Patch] Soppressione notifiche da utenti ignorati

Problemi riscontrati nell'uso del forum, suggerimenti e critiche per migliorare il servizio.
Avatar utente
Ubuntello
Scoppiettante Seguace
Scoppiettante Seguace
Messaggi: 594
Iscrizione: sabato 27 novembre 2010, 19:42

[Patch] Soppressione notifiche da utenti ignorati

Messaggio da Ubuntello »

La patch che allego e propongo, per quando phpBB sarà aggiornato, fa sì che gli utenti non ricevano notifiche di messaggi pubblici o privati quando il loro autore è un utente ignorato. In pratica ho realizzato quanto proposto da @danilo_g in questa discussione.

Compatibilità garantita: 3.3.13 e 3.3.12. Come per l'ultima patch che ho proposto, evito di perdere del tempo con la versione attualmente usata dal forum, considerando che dev'essere aggiornato.

Importante: dopo aver applicato la patch, sarà necessario rimuovere la cache contenente la precedente definizione della classe modificata:

Codice: Seleziona tutto

rm PHPBB_ROOT/cache/production/container_*
In alternativa, si può ottenere lo stesso effetto rimuovendo la cache dall'ACP, a patto di aver l'accortezza di accedervi preventivamente. E ovviamente andrebbe fatto lo stesso se un giorno si vorrà annullare la patch.
Allegati
dont-send-notifications-from-foes.patch.zip
(1.18 KiB) Scaricato 12 volte
mednafen-it - Gruppo di utenti italiani dell'emulatore Mednafen | Retro games: hidden gems - top titles - genre guides | Recommended games | Configurazione giochi Windows: Wine Staging con esync (dalla 4.6), drive_c +F + DXVK + vkd3d-proton + patch force feedback e force-large-address-aware
Avatar utente
danilo_g
Prode Principiante
Messaggi: 212
Iscrizione: giovedì 6 giugno 2024, 13:02
Desktop: BoiaDE
Distribuzione: LollOS
Località: Memeland

Re: [Patch] Soppressione notifiche da utenti ignorati

Messaggio da danilo_g »

sono grato a @Ubuntello per gli sforzi profusi. :shy: Speriamo che questa patch sia implementabile quanto prima.

PS. hai considerato di sottoporre la patch direttamente all'upstream? (al progetto phpBB) così potrebbero beneficiarne tutti, dalle prossime release.
Resistance is futile, you will be garbage collected.
«Le opinioni sono come i buchi del c*lo, tutti ne hanno uno.» :gh:
Whatever is the way. ¯\_(ツ)_/¯
Avatar utente
Ubuntello
Scoppiettante Seguace
Scoppiettante Seguace
Messaggi: 594
Iscrizione: sabato 27 novembre 2010, 19:42

Re: [Patch] Soppressione notifiche da utenti ignorati

Messaggio da Ubuntello »

danilo_g ha scritto:
domenica 1 settembre 2024, 23:11
PS. hai considerato di sottoporre la patch direttamente all'upstream? (al progetto phpBB) così potrebbero beneficiarne tutti, dalle prossime release.

Sì, l'ho considerato per tutte le patch fatte finora, anche quelle non condivise qui sul forum. Se gli fossero andate bene le sole pull request, lo avrei già fatto, ma visto che richiedono l'utilizzo dei ticket, dovrei prima approfondire le modalità di contributo, anche per quanto riguarda ticket già esistenti e assegnati (non so per questa patch, ma almeno una delle precedenti ricordo che risolve una richiesta rimasta aperta da anni). Insomma, sebbene con bassa priorità, il proposito generale non manca ed è sempre rientrato nel programma. ;)
mednafen-it - Gruppo di utenti italiani dell'emulatore Mednafen | Retro games: hidden gems - top titles - genre guides | Recommended games | Configurazione giochi Windows: Wine Staging con esync (dalla 4.6), drive_c +F + DXVK + vkd3d-proton + patch force feedback e force-large-address-aware
Avatar utente
danilo_g
Prode Principiante
Messaggi: 212
Iscrizione: giovedì 6 giugno 2024, 13:02
Desktop: BoiaDE
Distribuzione: LollOS
Località: Memeland

Re: [Patch] Soppressione notifiche da utenti ignorati

Messaggio da danilo_g »

Ubuntello ha scritto:
lunedì 2 settembre 2024, 9:54
danilo_g ha scritto:
domenica 1 settembre 2024, 23:11
PS. hai considerato di sottoporre la patch direttamente all'upstream? (al progetto phpBB) così potrebbero beneficiarne tutti, dalle prossime release.

Sì, l'ho considerato per tutte le patch fatte finora, anche quelle non condivise qui sul forum. Se gli fossero andate bene le sole pull request, lo avrei già fatto, ma visto che richiedono l'utilizzo dei ticket, dovrei prima approfondire le modalità di contributo, anche per quanto riguarda ticket già esistenti e assegnati (non so per questa patch, ma almeno una delle precedenti ricordo che risolve una richiesta rimasta aperta da anni). Insomma, sebbene con bassa priorità, il proposito generale non manca ed è sempre rientrato nel programma. ;)
Capisco. Beh intanto grazie ancora. :)
Resistance is futile, you will be garbage collected.
«Le opinioni sono come i buchi del c*lo, tutti ne hanno uno.» :gh:
Whatever is the way. ¯\_(ツ)_/¯
Avatar utente
Janvitus
Amministratore
Amministratore
Messaggi: 18809
Iscrizione: lunedì 25 aprile 2005, 15:52
Desktop: GNOME Shell / Xfce
Sesso: Maschile
Località: Potenza
Contatti:

Re: [Patch] Soppressione notifiche da utenti ignorati

Messaggio da Janvitus »

Ubuntello ha scritto:
lunedì 2 settembre 2024, 9:54
danilo_g ha scritto:
domenica 1 settembre 2024, 23:11
PS. hai considerato di sottoporre la patch direttamente all'upstream? (al progetto phpBB) così potrebbero beneficiarne tutti, dalle prossime release.

Sì, l'ho considerato per tutte le patch fatte finora, anche quelle non condivise qui sul forum. Se gli fossero andate bene le sole pull request, lo avrei già fatto, ma visto che richiedono l'utilizzo dei ticket, dovrei prima approfondire le modalità di contributo, anche per quanto riguarda ticket già esistenti e assegnati (non so per questa patch, ma almeno una delle precedenti ricordo che risolve una richiesta rimasta aperta da anni). Insomma, sebbene con bassa priorità, il proposito generale non manca ed è sempre rientrato nel programma. ;)
Vedo come si creano estensioni e via, è la via più comoda per tutti.
+Fedora 40 • +Xubuntu 24.04
Avatar utente
Ubuntello
Scoppiettante Seguace
Scoppiettante Seguace
Messaggi: 594
Iscrizione: sabato 27 novembre 2010, 19:42

Re: [Patch] Soppressione notifiche da utenti ignorati

Messaggio da Ubuntello »

Janvitus ha scritto:
venerdì 6 settembre 2024, 10:44
Vedo come si creano estensioni e via, è la via più comoda per tutti.
Di potersi fare, oggettivamente in questo caso si potrebbe anche fare, visto che l'hook necessario non manca (a differenza della penultima patch proposta). Quindi ci pensi tu a studiarti il dovuto e convertire la patch in un'estensione? O c'è un errore di battitura e intendevi dire che dovrei essere io a farlo?
mednafen-it - Gruppo di utenti italiani dell'emulatore Mednafen | Retro games: hidden gems - top titles - genre guides | Recommended games | Configurazione giochi Windows: Wine Staging con esync (dalla 4.6), drive_c +F + DXVK + vkd3d-proton + patch force feedback e force-large-address-aware
Avatar utente
Janvitus
Amministratore
Amministratore
Messaggi: 18809
Iscrizione: lunedì 25 aprile 2005, 15:52
Desktop: GNOME Shell / Xfce
Sesso: Maschile
Località: Potenza
Contatti:

Re: [Patch] Soppressione notifiche da utenti ignorati

Messaggio da Janvitus »

Ubuntello ha scritto:
venerdì 6 settembre 2024, 15:11
Janvitus ha scritto:
venerdì 6 settembre 2024, 10:44
Vedo come si creano estensioni e via, è la via più comoda per tutti.
Di potersi fare, oggettivamente in questo caso si potrebbe anche fare, visto che l'hook necessario non manca (a differenza della penultima patch proposta). Quindi ci pensi tu a studiarti il dovuto e convertire la patch in un'estensione? O c'è un errore di battitura e intendevi dire che dovrei essere io a farlo?
rispondo il ritardo perché ho avuto un lutto in famiglia: ovviamente mi riferivo al fatto che dovresti pensarci tu a creare l'estensione, anche perché il codice è tuo ;)
+Fedora 40 • +Xubuntu 24.04
Scrivi risposta

Ritorna a “Gruppo Forum”

Chi c’è in linea

Visualizzano questa sezione: 0 utenti iscritti e 1 ospite