Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug sugli script polito.it #1

Open
LucaMannella opened this issue Jul 23, 2015 · 8 comments
Open

Bug sugli script polito.it #1

LucaMannella opened this issue Jul 23, 2015 · 8 comments

Comments

@LucaMannella
Copy link

Ciao, vorrei segnalarti un bug che ho riscontrato utilizzando il tuo script su Firefox con sistema operativo Xubuntu e altre distribuzioni Ubuntu.
Se è presente una riga nella pagina delle valutazioni provvisorie ma questa riga non ha voto (esito Respinto o Assente) lo script visualizza "?" al posto di visualizzare il numero (ad esempio 1 nel mio caso).
Non so esattamente se sia una tua scelta stilistica o un bug ma nel dubbio ho preferito segnalartelo.
Direi che questo sia dovuto al fatto che lo script in questa situazione non entra all'interno del ciclo for nel ramo else quindi penso che per qualche motivo lo script non riesca a reperire la corretta lunghezza della pagina in questi casi, correggimi se sbaglio...
Spero che la segnalazione ti sia gradita, un saluto!

@fgiobergia
Copy link
Owner

Ciao, scusa se ti rispondo solo ora. Ho cambiato abbastanza sostanzialmente lo script "valutazioni provvisorie", e credo che il problema a cui ti riferivi dovrebbe essere risolto ora (inoltre, adesso funziona anche con il nuovo layout del sito). Se hai l'occasione, provalo e fammi sapere!

@LucaMannella
Copy link
Author

Tranquillo!
Lo scaricherò e lo proverò senz'altro!

Tra l'altro volevo suggerirti di modificare i nomi degli script da "nome_script.js" a "nome_script.user.js".
In questo modo alcuni plugin come "greasemonkey" riconoscono automaticamente che si tratta di uno userscript e permettono di importarlo direttamente con un click.
Comunque vedi tu come preferisci!

Il giorno 26 feb 2016, alle ore 23:57, Flavio Giobergia notifications@github.com ha scritto:

Ciao, scusa se ti rispondo solo ora. Ho cambiato abbastanza sostanzialmente lo script "valutazioni provvisorie", e credo che il problema a cui ti riferivi dovrebbe essere risolto ora (inoltre, adesso funziona anche con il nuovo layout del sito). Se hai l'occasione, provalo e fammi sapere!


Reply to this email directly or view it on GitHub.

@fgiobergia
Copy link
Owner

Ottimo, appena hai occasione di provarlo fammi sapere, così posso chiudere l'issue :)

Per il .user.js, non ne ero al corrente! Ho provveduto a rinominare i file, e provando con tampermonkey, funziona!

(Tra l'altro, la butto lì: ho anche appena pubblicato un nuovo userscript, buttagli un'occhiata se ti va)

@LucaMannella
Copy link
Author

Tra l'altro mi sono reso conto solo ora che l'issue è rimasto associato all'intero package e non al relativo userscript e che potevo essere più chiaro nel spiegarti la mia supposizione... Dovevo essere di fretta quando l'ho scritto scusami!
Ad ogni modo ho provato lo script su diversi pc e non sembra dare problemi sia con il nuovo che con il vecchio layout, nel caso te lo segnalo, magari anche solo con un pm!

Ho provato il tuo nuovo script oggi su Xubuntu sui pc del labinf e mi è piaciuto, forse tra i tuoi script è quello meno "utile" però comunque lo trovo carino.
Provandolo però adesso a casa sul mio netbook con una distribuzione Lubuntu non so perchè ma non mi funziona, lo script prepara la griglia ma non riesce a riempirla con i grafici...
Ti allego qualche immagine (anche se non dicono nulla più che le mie parole)
2016-03-04-023302_1024x600_scrot
2016-03-04-023307_1024x600_scrot
2016-03-04-023339_1024x600_scrot
2016-03-04-023345_1024x600_scrot

Comunque grazie per i tuoi script, li trovo molto interessanti

@fgiobergia
Copy link
Owner

Uhm. Decisamente strano! A occhio sembra che la parte di compattazione dei mesi funziona (correggimi se sbaglio, ma immagino tu abbia dato esami a Feb 2015, Jul 2015, Sep 2015 e Jan 2016), quindi dovrebbe riuscire ad accedere al libretto senza problemi.

È possibile che si verifichi qualche errore nell'elaborazione dei dati, e quindi lo script generi un errore e venga interrotto. Riesci ad aprire la console JS (Chrome, Firefox) e vedere se sono riportati errori?

@LucaMannella LucaMannella changed the title Bug sul numero visualizzato Bug sugli script polito.it Mar 7, 2016
@LucaMannella
Copy link
Author

Sì esatto, ho dato esami in tutti e 4 i mesi da te elencati (anche a febbraio in realtà ma non mi sono ancora stati registrati quindi tutto nella norma).
Ho provato ad aprire la console sia su Firefox, che su Chromium (anche se non ho tanta dimestichezza con questo browser) e errori non ne segna, eccetto il consiglio di usare un algoritmo di crittografia più robusto di SHA-1 che ovviamente non ti riguarda.
Posso provare a leggere o debuggare il codice quando trovo un po' di tempo se vuoi, in questo momento sono un po' arrugginito di programmazione web e penso che ci metterei un bel po'...

Tra l'altro mi è venuta in mente un'altra cosa che potrei segnalarti relativa allo script Depenna che più che un bug è qualcosa di cui magari non hai voluto o pensato di tener conto.
Quando un insegnamento viene leggermente modificato può succedere che ti venga riportato nel portale della didattica come insegnamento mutuato (ti allego un immagine di come appare nel portale).

2016-03-07-145218_1280x1024_scrot

In questa situazione, anche quando l'esame è superato, lo script depenna non lo riconosce come tale (in quanto le stringhe sono diverse) e non lo depenna dal carico didattico.

@olimaster
Copy link

Ciao tutti scusate l'ignoranza ma come si usa questo script? grazie in anticipo per le vostre risposte.

@LucaMannella
Copy link
Author

Ciao @olimaster,
per poter usare degli script utente ti occorre un plugin adatto come Greasemonkey per Firefox o Tampermonkey per Chrome.

Ti linko una pagina in cui potrai trovare un po' di informazioni https://github.com/OpenUserJs/OpenUserJS.org/wiki/Userscript-beginners-HOWTO
Non è un procedimento complicato importare questi script, troverai molte guide sul web!

Ciao a presto e scusa il ritardo nella risposta! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants
@fgiobergia @LucaMannella @olimaster and others