Problema con script "ammap.js"

[sviluppoRobyone]

Prerequisiti

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

Descrivi il problema

Buongiorno,
nella parte admin del tema viene registrato uno script che però non esiste. Lo script in questione è ammap.js del plugin Amministrazione aperta (a riga 391 di inc\actions.php).

Il problema è già stato discusso nella issue #237 ma vorrei tornare su questa versione in quanto saremmo ben disposti a fare una pull request per correggere la registrazione dello script del plugin, ma non riesco a trovare il plugin a cui ci si sta riferendo. La discussione dell'issue parla di un plugin la cui cartella di root è italia-amministrazione-aperta, ma cercando "amministrazione aperta" tra i plugin di WordPress, sono presenti:

• Amministrazione Trasparente la cui cartella di root è amministrazione-trasparente;
• Amministrazione Aperta la cui cartella di root è amministrazione-aperta.

Nessuno dei due ha come root italia-amministrazione-aperta e nessuno dei due contiene un file che si chiami ammap.js. Probabilmente non sto trovando il plugin corretto. Avete per caso maggiori informazioni su come sistemare la cosa?

Per fornire più contesto, saremmo interessati a risolvere questo problema in quanto l'errore 404 dovuto al file mancante causa un leggero ritardo che impedisce a un plugin di terze parti per gestire i media di funzionare correttamente (che è un bug del plugin, ma abbiamo più speranze di risolvere il problema agendo sul tema anziché attendendo una risposta dagli sviluppatori, che abbiamo comunque sollecitato).

Includi link o screenshot

No response

Su quale sisitema operativo stai riscontrando il problema?

Windows

Su quale browser stai riscontrando il problema?

Qualsiasi

Che versione del tema stai utilizzando?

2.11.0

[milesimarco]

Ciao @sviluppoRobyone,
il plugin italia-amministrazione-aperta è un clone di amministrazione-aperta a cui è stato aggiunto il prefisso per integrarlo nel tema come funzionalità integrata.

Non ero al corrente di questa scelta quando è stata implementata e personalmente consiglio di utilizzare il plugin come presente su wordpress.org in quanto solo così vengono ricevuti gli aggiornamenti di funzionalità e di sicurezza - https://it.wordpress.org/plugins/amministrazione-aperta/

Come primo test proverei a disattivare italia-amministrazione-aperta e installare quello rilasciato tramite wordpress.org per verificare che l'anomalia si risolva e che non ci siano disallineamenti sui dati precedentemente caricati.

A presto,
M.

[sviluppoRobyone]

Buongiorno @milesimarco,

Il problema risiede proprio nel fatto che il tema richiama il file amministrazione-aperta/js/ammap.js, che nella repo del plugin italia/italia-amministrazione-aperta esiste, mentre in quella di wordpress.org no.

Tuttavia penso che la questione principale sia un'altra (e che non dipenda direttamente le azioni di @milesimarco): perché il tema registra un file javascript che risiede in un plugin esterno? Non ci sono garanzie che il plugin di amministrazione aperta sia installato.

È vero che nella pagina dei plugin un messaggio dice di installare "Italia - Amministrazione Aperta", ma non ci sono link per l'installazione e comunque la cosa più intuitiva è che una persona cerchi il plugin in wordpress.org e installi la versione che non contiene lo script richiesto dal tema, quindi il problema permarrebbe.

Ponendo anche che uno trovi la repo italia/italia-amministrazione-aperta, il README di tale repo dice solo di estrarre il contenuto dell'archivio nella cartella dei plugin, tuttavia il nome della cartella contenuta nell'archivio è plugin_wp_amministrazione_aperta-... ma il tema cerca il file nella cartella amministrazione-aperta, quindi comunque l'errore rimarrebbe.

Infine, rinominando la cartella in amministrazione-aperta come vorrebbe il tema, WordPress riconduce il plugin a quello in wordpress.org, propone un aggiornamento, e facendolo il plugin viene riscaricato da wordpress.org, che non contiene il file ammap.js, tornando punto e a capo.

La nostra proposta sarebbe quella di rimuovere l'inclusione dello script ammap.js dal tema, e nelle note di versione del tema suggerire di rimuovere il plugin della repo italia/italia-amministrazione-aperta e di installare quello presente in wordpress.org.