italia / design-comuni-wordpress-theme

Tema Wordpress per i siti dei comuni italiani
GNU Affero General Public License v3.0
33 stars 33 forks source link

C.SI.1.2 - LIBRERIA DI ELEMENTI DI INTERFACCIA #321

Closed tommaso326 closed 5 months ago

tommaso326 commented 1 year ago

Prerequisiti

Descrivi il problema

Sto lavorando su 2 siti in contemporanea. Sono identici, uno la copia dell'altro, in quanto il secondo è stato clonato dal primo. L'unica cosa in cui differiscono sono i dati inseriti. Nell'originale, quello di partenza, sto riscontrando l'errore alla voce 1.2, in allegato uno screen fatto durante la validazione. Nel clone l'errore non sussiste. Nel report dell'originale, l'app di validazione trova la libreria in homepage ma non trova almeno una classe CSS di quelle indicate nella documentazione. Tuttavia, alla voce "Versione in uso", viene riportata la versione della libreria tra virgolette, cosa che non succede nelle altre pagine analizzate (screenshot in allegato).

versione app di valutazione: 2.5.1 versione wp: 6.2.1 versione tema: 1.6.2

Includi link o screenshot

image image

Su quale sisitema operativo stai riscontrando il problema?

Windows

Su quale browser stai riscontrando il problema?

Qualsiasi

Che versione del tema stai utilizzando?

1.6.2

enrimk commented 1 year ago

Forse più un problema del validatore che del tema... Vedo che il validatore legge il numero di versione da controllare prima da una variabile globale js e poi da una custom property css: entrambe possono essere state sovrascritte in qualunque momento, chissà in che punto e chissà da cosa. Nella fattispecie come hotfix forse basterebbe cambiare qui e qui .replaceAll('"', "") in .replaceAll(/["']/, ""), ma non so se ne vale la pena, oltretutto visto che stai usando una vecchia versione del tema.

tommaso326 commented 1 year ago

@enrimk Ho apportato le modifiche che hai menzionato, ma non mi è chiara una cosa. Come compilo il progetto per renderlo un file eseguibile .exe su VSC ?

Ho seguito i passaggi di installazione descritti a questo link: https://github.com/italia/pa-website-validator/tree/8116ef982d3d5cd1735d907ed104be886803309d

E da terminale ricevo questo errore:

PS C:\Users\PincoPalla\Desktop\Dati\Web\Wordpress\pa-website-validator> node dist --type municipality --destination ~/pa-italia-crawler-reports --report nomedelcomune-report --website https://nomedelcomune.cloud/ --scope online --view --accuracy all node:internal/process/esm_loader:74 internalBinding('errors').triggerUncaughtException( ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'C:\Users\PincoPalla\Desktop\Dati\Web\Wordpress\pa-website-validator\node_modules\proxy-agent\' imported from C:\Users\PincoPalla\Desktop\Dati\Web\Wordpress\pa-website-validator\node_modules\@puppeteer\browsers\lib\esm\httpUtil.js at new NodeError (node:internal/errors:329:5) at legacyMainResolve (node:internal/modules/esm/resolve:272:9) at packageResolve (node:internal/modules/esm/resolve:707:14) at moduleResolve (node:internal/modules/esm/resolve:755:18) at Loader.defaultResolve [as _resolve] (node:internal/modules/esm/resolve:869:11) at Loader.resolve (node:internal/modules/esm/loader:88:40) at Loader.getModuleJob (node:internal/modules/esm/loader:241:28) at ModuleWrap. (node:internal/modules/esm/module_job:57:40) at link (node:internal/modules/esm/module_job:56:36) { code: 'ERR_MODULE_NOT_FOUND' }

enrimk commented 1 year ago

A dire il vero, a compilare il validatore in un exe non ci ho provato. Non dovrebbe essere necessario però: è un applicativo node.js, può essere eseguito anche "spacchettato", dopo una build seguendo le procedure descritte nel Readme. Con le ultime versioni, sinceramente, io ho avuto qualche problema con le dipendenze tra pacchetti npm che non ho avuto tempo di risolvere, quindi ho sperimentato solo fino a un paio di versioni fa.

Comunque non saprei se modificare il validatore ti serve realmente... hai poi provato con la nuova versione del tema?

stale[bot] commented 5 months ago

Questo problema è stato automaticamente contrassegnato come non aggiornato perché non ha avuto attività recentemente. Sarà chiuso se non si verificano ulteriori attività. Considerate inoltre l'aggiornamento alle ultime versioni del tema. Grazie.