dci_get_breadcrumb_label per costruzione etichetta breadcrumb ha parti letterali

[mfrasca]

Prerequisiti

• Ho già cercato tra le issue la presenza di un problema simile
• Ho letto le linee guida di contribuzione

Descrivi il problema

Mi trovo con un progetto ancora praticamente vuoto, entro nella pagina "Documenti e Dati", provo a seguire una delle categorie già presenti. Sono perplesso sul codice della funzione:

/**
 * restituisce label per costruzione breadcrumb
 * @param $name
 * @param string $type
 * @return mixed|string
 */
function dci_get_breadcrumb_label($name , $type = 'term') {
    $terms = array(
        'comunicato stampa' => 'Comunicati',
        'news' => 'Notizie',
        'avviso' => 'Avvisi'
    );

    if ($terms[$name]) {
        return $terms[$name];
    }

    return ucfirst($name);
}

Secondo me il codice va migliorato, per due motivi, ed inizio da quello più semplice. Il test logico sulla presenza del $name nell'array $terms così com'è scritto mi causa un warning per ogni valore fuori dai tre previsti nel codice. Si potrebbe migliorare aggiungendo un isset al test logico:

    if (isset($terms[$name])) {
        return $terms[$name];
    }

Però quello che davvero mi lascia perplesso è la presenza letterale (hard-coded) dell'array $terms, che a mio vedere andrebbe levato e riformulato come colonna extra nella base dati. In pratica qui vogliamo tradurre un nome in un testo breve da includere fra i breadcrumb, probabilmente dal singolare al plurale, probabilmente abbreviando. Secondo me è una operazione simile alla creazione (che non facciamo in automatico!) dello slug corrispondente al nome, e direi che andrebbe fatta aggiungendo una colonna alla tabella di database dove sono questi nomi e dove l'amministratore può aggiungere nomi, con slug e descrizioni.

Ci manca di offrire all'amministratore di definire il breadcrumb_name.

Includi link o screenshot

No response

Su quale sisitema operativo stai riscontrando il problema?

Linux

Su quale browser stai riscontrando il problema?

Firefox

Che versione del tema stai utilizzando?

1.12.4

[enrimk]

È anche peggio di così.
A ben vedere, l'intera dci_get_breadcrumb_label() è inutile.
L'unico punto in tutta la codebase in cui viene usata è nelle breadcrumb.

design-comuni-wordpress-theme/inc/breadcrumb.php

Line 443 in c88ddf9

$this->items[] = __(dci_get_breadcrumb_label($term_name), "design_comuni_italia");

Lì, le viene passato il titolo del term, che non corrisponderà mai alla stringa su cui fa il confronto. Che è in minuscolo, ma con spazi... Cos'è? Non è un titolo, non è uno slug.
Infatti, i nomi delle tax tipi_notizia sono normati dal modello Comuni senza possibilità di eccezione, sono fissi, e sono definiti all'installazione del tema (anche lì, hardcoded; ma con qualche ragione):

design-comuni-wordpress-theme/inc/vocabolario.php

Lines 404 to 408 in 88bfa16

	$tipi_notizia_arr = [
	'Notizie',
	'Comunicati',
	'Avvisi'
	];

E lo stesso per i tipi_documento.
In pratica, usata così, tutta la funzione si riduce a ucfirst($name), che è comunque ridondante, perché il titolo della tassonomia la maiuscola già dovrebbe averla.