search

pokemoncentral / wiki-project

Coordinamento dello sviluppo tecnico di Pokémon Central Wiki
https://wiki.pokemoncentral.it
1 stars 0 forks source link

Aggiornamento versi usati nel PokémonInfo #56

Open Snorlite opened 5 years ago

Snorlite commented 5 years ago

I versi usati nella sezione verso del template PokémonInfo sono ancora fermi alla quinta generazione, e direi che sarebbe ora di aggiornarli. Da quello che ho capito solo @mexicat può mettere le mani ai file mp3, quindi è necessario il suo intervento.

Qui ho trovato un elenco completo, su Dropbox ho caricato i rispettivi file .mp3 nella cartella Versi. A differenza dei file attualmente usati, però, sono riportati i nomi dei Pokémon con iniziale minuscola e non il numero di Pokédex... se è un problema mi offro volontario a rinominare i file uno per uno 😅 ho fatto qualche ricerca e ho trovato solo file WAV con problemi di nome simili e incompleti (esempi, fermi a Sole e Luna: 1 e 2 ). Date un occhio anche voi nel caso.

Inoltre, alcuni Pokémon hanno delle forme alternative con dei versi differenti dalla forma originale, e questo non è ancora gestito in alcun modo dal template. La mia idea è di fare in modo di mettere più ▶ interattivi nello stesso box con il nome della forma relativa sotto ciascuno di loro in piccolo. I Pokémon interessati sono:

lucas992x commented 5 years ago

Una cosa non ho capito: i file che hai messo nella cartella condivisa di Dropbox hanno altri problemi o solo quello del nome? Comunque non metterti lì a rinominarli singolarmente, quello se serve si fa con un programma automatico (almeno i Pokémon "standard", poi le forme alternative si sistemeranno a parte).

Snorlite commented 5 years ago

@lucas992 I file su Dropbox sono quelli presi da Showdown, quindi con i nomi e iniziale minuscola e formato mp3.

lucas992x commented 5 years ago

@Snorlite Ok, ma la mia domanda è: il nome è l'unico potenziale problema o ce ne sono altri?

Snorlite commented 5 years ago

@lucas992 Una volta risolto quello penso non ci siano altri problemi, a parte appunto modificare il template in modo da supportare più versi per forme alternative.

davla commented 5 years ago

Quindi c'è da rinominare una schiera di files da nome in minuscolo a numero di dex? Qual'è il problema?

Snorlite commented 5 years ago

@davla I versi aggiornati che ho trovato sono chiamati in base al nome del Pokémon tutto in minuscolo. Se è possibile aggiornare il widget del Verso per fare in modo di prendere come parametro il nome della specie (e "convertendo" in minuscolo l'iniziale del nome della specie inserito nel parametro relativo per leggere ad esempio bulbasaur.mp3 invece di Bulbasaur.mp3) invece del numero Pokédex, allora tutto OK.

Altrimenti, i file vanno rinominati.

lucas992x commented 5 years ago

@Snorlite Rinominare quei file non è un problema, persino io riuscirei a scrivere un programma che lo faccia in automatico, quindi figurati uno esperto come Maze o Flavio.

davla commented 5 years ago

Ma anche la soluzione di Snorlo è ottima. Cioè, si può tranquillamente cambiare il widget, chiamarlo con il nome del Pokémon e farne il lowercase.

@Snorlite i widget sono editabili da tutti gli admin, e non richiedono chissà cosa per scriverli, anche perché li teniamo volutamente piccoli.

Snorlite commented 5 years ago

@davla Ok, allora è deciso, teniamo i nomi. Ho visto un po' come fare.

Il widget non lo modificherei finché @mexicat non carica i file aggiornati, in ogni caso l'unica cosa che modificherei è sostituire $ndex con $specie.

Ho messo nella mia sandbox una versione modificata del PokémonInfo per permettere l'aggiunta di più versi nel caso dei Pokémon elencati nel primo post. Il lowercase l'ho gestito qui e non nel widget. Sentitevi liberi di modificarla nel caso abbia scritto qualche oscenità. Ho inserito i parametri verso2, verso3 e verso4 per inserire gli eventuali suffissi relativi alle forme alternative presenti nel nome del file dopo il trattino -. Sotto i ▶ vengono visualizzati i nomi delle forme relative (nei parametri forme1, forme2, forme3, forme4).

Tldr: image

EDIT: dovrei avere gestito meglio lo spazio insufficiente, prima non andava a capo image

mexicat commented 5 years ago

Confermo che servirebbe averli nel formato attuale (es. 003.mp3)

Snorlite commented 5 years ago

@mexicat Ah, ok. Per eventuali versi delle forme alternative, invece? Come è opportuno gestirle?

mexicat commented 5 years ago

@Snorlite In realtà non avevo letto tutta la discussione 😅 il numero va tenuto nel caso non si modifichi il widget, però se lo volete modificare non c’è problema, posso caricare i file così come sono. (Per le forme non saprei, ma nella cartella vedo che c’è roba tipo 710-average.mp3)

Snorlite commented 5 years ago

@mexicat 710-average nella cartella attualmente usata dal wiki? Perché in quella su Dropbox non ho visto numeri.

In ogni caso il PokémonInfo attuale non gestisce ancora la riproduzione dei versi delle forme alternative, ma nella mia sandbox dovrei avere aggiunto questa funzionalità senza problemi.

Quando caricherai tutti i file che sono su Dropbox sistemo template e widget.

mexicat commented 5 years ago

Ho caricato tutti i file con i nomi al posto dei numeri, tipo https://media.pokemoncentral.it/wiki/versi/abomasnow.mp3

Snorlite commented 5 years ago

@mexicat Grazie, stasera ci guardo ed eventualmente sistemo le cose.

Snorlite commented 5 years ago

@mexicat perdonami ma porygon-z.mp3 dovrebbe chiamarsi porygon-Z.mp3 e hooh.mp3, dovrebbe essere ho-Oh.mp3. Porygon l'avevo corretto in locale ma si vede che Dropbox non me l'ha sincronizzato, il secondo me l'ero proprio dimenticato 😅

Poi per qualche motivo farfetch'd.mp3 non lo prende, nonostante il carattere apostrofo mi sembri lo stesso.

davla commented 5 years ago

Aaaaah. Scusate che mi sono dimenticato dell'esistenza di questo coso.

@Snorlite La prossima volta che fai un parametro con nome italiano ti sputo.

Comunque, come Snorlo ha fatto notare, il caro vecchio Farfetch'd crea immani problemi. Oltre a questio abbiamo i trattini, e cazzi e mazzi vari che potrebbero potenzialmente non essere nomi validi di files in un linux filesystem. Sarebbe meglio tornare ai numeri.

Per la gestione delle forme alternative invoco @flavio-a. Il fatto è che il linguaggio di templating dei widget è meglio di WikiCode (non che ci voglia molto) ed ha i for loops. C'è un modo di avere una lista (concatenata by spaces, per esempio) delle sigle delle forme alternative dato l'ndex, via i moduli interfaccia che a te piacciono molto? Così facendo, il widget pesca i files (opportunamente nominati) delle forme alternative. E magari si riescono a sistemare pure le captions.

flavio-a commented 5 years ago

@davla Non è che i moduli interfaccia mi piacciano tanto, è che sono comodi. Comunque non vedo perché no, basta aggiungere l'adeguata funzione.

davla commented 5 years ago

Muoviti a farlo

flavio-a commented 5 years ago

Requirements needed: vuoi un'interfaccia che mappi ndex (solo ndex o anche nomi? ndex solo "puri" o anche con un'abbr?) in una lista (space separated?) di tutte le abbr delle sue forme alternative (se l'ndex non ha forme alternative lista vuota = stringa vuota o errore di script? se l'ndex ha forme alternative useless vanno considerate o no?)

davla commented 5 years ago

Esatto. Ci ho pensato ieri ma poi non ho scritto. Si passa l'ndex su tre cifre, perché esiste come parametro dell'infobox.

Ci sono tre casi in cui dovresti ritornare cose diverse:

Come noti, serve di sapere quali forme alternative hanno un verso diverso. Quindi una nuova table nell'AltFroms/data. Trovati qualcuno da schiavizzare.

davla commented 5 years ago

In realtà, togethr with le sigle mi servirebbero pure i nomi delle forme alternative. Una list of pairs, insomma. Una roba tipo comma per separare le pairs, dash per separare i due items nella pair. O qualsiasi cosa migliore di venga in mente

flavio-a commented 5 years ago

@davla Non puoi fare da solo usando PokémonData.getFormName (o qualcosa del genere)?

flavio-a commented 5 years ago

Tre domande: nella lista di abbr serve anche base? Se sì, in che formato? Nella lista devo mettere tutte le forme o solo quelle con versi diversi dalla forma base?

davla commented 5 years ago
flavio-a commented 5 years ago

Aggiunta in pokemoncentral/wiki-lua-modules@bb169d7c8f06f0db764cc37ed1efa0d53fad6f73. Dai test dovresti anche capire se ho rispettato tutti i requisiti o se mi sono sbagliato.

davla commented 5 years ago

@mexicat Dovresti caricare i nuovi files. Cartella Cries su Dropbox. Ma soprattutto,

Errore nel widget Test: Syntax Error in template "wiki:Test" on line 1 "<!--{section name=form loop=','|explode:$forms}-->" modifier 'explode' not allowed by security setting

Dovrei fare un banale split-by-comma di una stringa. Poi l'escape si fa sui singoli elementi nel loop, tranquillo.

Qui ti dice come fare, https://www.smarty.net/docsv2/en/language.modifiers.tpl, il bullet point the inizia con Secondly - if $security is enabled,

mexicat commented 5 years ago

Ho caricato i file, ma non ho capito cosa mi stai chiedendo di fare e soprattutto in che contesto...

davla commented 5 years ago

L'estensione dei widget usa Smarty templating language, che ha una simpatica cosa di far usare tutte le funzioni PHP come filtri/modifiers/pipes/ilcazzochesichiamano. Ora, l'estensione blocca molte di queste funzioni, che devono essere inserite in una whitelist. A me servirebbe di fare uno split di una stringa, ovverosia explode in PHP. Prenderò tutte le precauzioni del caso per evitare code injections di sorta, ma mi abiliti l'explode nella whitelist? Si dovrebbe fare agile con il link di cui sopra.

Zizork commented 3 years ago

Avete già considerato di mettere accanto ai versi aggiornati quelli originari? Secondo me sarebbe una cosa opportuna che vale la pena fare...