<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div dir="auto" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Salve </div><div class=""><br class=""></div><div class=""><div class="">Il giorno 16 giu 2017, alle ore 21:34, Pietro Albini <<a href="mailto:pietro@pietroalbini.org" class="">pietro@pietroalbini.org</a>> ha scritto:</div><div><blockquote type="cite" class=""><br class="Apple-interchange-newline"><div class=""><div class="">Allora, code review time :)<br class=""><br class="">Per prima cosa, [……..]<br class=""></div></div></blockquote><br class=""><blockquote type="cite" class=""><div class=""><div class="">+def Partecipa(self, args):<br class="">+ url = normalize_url(args, "<a href="http://wiki.ubuntu-it.org" class="">wiki.ubuntu-it.org</a>", "al wiki di Ubuntu-it")<br class="">+ return '<p class="Partecipa"><font color="red">[...]</font>[...]</p>' % ([...])<br class=""><br class="">Questa funzione non è necessaria, basta che inserisci direttamente<br class="">l'indirizzo quando generi l'output. Le funzioni `forum` ed `help`<br class="">esistono perché filtrano contenuti inseriti dall'utente, ma nel<br class="">tuo caso l'url è hardcoded (anche se sarebbe bello averlo in una<br class="">costante in cima al file).<br class=""><br class="">Inoltre, per favore **non usare il tag <font>**. È deprecato, a volte<br class="">fa casini con il CSS ed in generale non è più usato. Magari rinomina<br class="">la classe in `informazioni-partecipa` e metti `color: #c11` che è più<br class="">leggero come colore.<br class=""><br class="">+ num_arg_depr+=1<br class="">+ anchor = page + '#'<br class="">+ if arg == developing:<br class="">+ anchor += 'Versione_in_sviluppo'<br class="">+ else:<br class="">+ anchor += 'A' + arg # anchors with numbers start with "A"<br class="">+<br class="">+ result_depre.append("%s%s%s " % (<br class="">+ self.formatter.url(on=1, url=anchor, title="[...]" % (arg, codenames[arg])),<br class="">+ arg,<br class="">+ self.formatter.url(on=0)<br class="">+ ))<br class="">+ has_releases = True<br class=""><br class="">Questo codice in `releases` è un duplicato di quello sotto. Non si<br class="">potrebbe unire? Magari rimuovendo anche results_depre che contiene<br class="">gli stessi dati del resto.<br class=""><br class="">Grazie per il contributo comunque!<br class=""><br class="">Pietro,<br class="">Gruppo Sistemisti.<br class=""><br class=""><br class=""><br class=""></div></div></blockquote><br class=""></div><div>……Salve a tutti ripropongo una nuova versione del file Informazioni.py</div><div class=""><br class=""></div><div class="">Allora visti tutti i consigli di Pietro Albini ho rivisto un pò tutto , mi spiego meglio :</div><div class="">molto del codice che avevo scritto era superfluo quindi sono ripartito dall’inizio </div><div class="">e dal file originale ho soltanto aggiunto un paio di cose poi il resto è rimasto tutto uguale </div><div class="">in moda tale da non stravolgere tutto il codice ; comunque ottenendo lo stesso risultato cioè </div><div class="">l’avviso in caso di rilasci non più supportati.</div><div class="">Di seguito riporto le diff dall’originale :</div><div class=""><br class=""></div><div class="">- alla riga 15 (all’inizio del file) ho aggiunto una variabile</div><div class=""><span class="Apple-tab-span" style="white-space: pre;"> </span>´ url_partecipa = […..] ´</div><div class=""><br class=""></div><div class="">- alla riga 163 aggiunto una variabile </div><div class=""><span class="Apple-tab-span" style="white-space: pre;"> </span>´message_allert = […….]´ ==> in modo da poter modificare il messaggio a piacere ( ho inserito quello suggerito da Pietro) </div><div class=""><br class=""></div><div class="">- alla riga 180 ho modificato il condizionale íf´</div><div class=""><span class="Apple-tab-span" style="white-space: pre;"> </span>´ if rilasci is ‘ ‘ : [……] ´</div><div class="">tutto il resto è come l’originale !!</div><div class=""><br class=""></div><div class="">Vorrei fare una proposta e una considerazione:</div><div class="">Penso che quando un utente ricerca e legge una guida lo stia facendo perché ha necessità di installare o </div><div class="">provare qualcosa che gli occorre e , che vedendo un avviso tipo questo tenti comunque di seguire la guida e gli diamo</div><div class="">anche un incentivo ad aiutarci nell’aggiornarla chiedendogli di farlo.</div><div class=""><br class=""></div><div class="">La proposta è che se dopo la supervisione di Pietro o di qualche membro dei sistemisti si reputi che il codice posa andare , e nel frattempo </div><div class="">che venga effettuato un aggiornamento alla macro</div><div class=""><br class=""></div><div class=""><blockquote type="cite" class="">Il giorno 16 giu 2017, alle ore 21:34, Pietro Albini <<a href="mailto:pietro@pietroalbini.org" class="">pietro@pietroalbini.org</a>> ha scritto:</blockquote></div><blockquote type="cite" class="">PS: ho in programma soonish di riscrivere parte della macro per migliorarla e renderla più magica ;) </blockquote><div class=""><br class=""></div><div class=""> e che se anche a tutti i membri della Documentazione vada bene </div><div class="">si possa usare per informare e incentivare l’aggiornamento delle pagine .(viste le minime differenze dall’originale) </div><div class="">Grazie a tutti </div><div class="">Link al file </div><div class=""><br class=""></div><div class="">[1] <a href="http://bazaar.launchpad.net/~21stefano/+junk/macro_wiki/files" class="">http://bazaar.launchpad.net/~21stefano/+junk/macro_wiki/files</a></div><div class=""><br class=""></div><div class="">PS: Prometto di non toccare più codice wiki ;) </div><div>__________________ <br class=""><br class=""></div><div><div class="">Stefano Pica</div><div class=""><a href="mailto:stefano.pica73@gmail.com" class="">stefano.pica73@gmail.com</a></div></div><br class=""></div></div></body></html>