Discuție Modul:Monumente
Adăugare subiectUltimul comentariu: acum 9 ani de Andrei Stroe în subiectul Code review
Code review[modificare sursă]
Aș dori să rog pe oricine are cunoștințe de programare și copyright să treacă un pic prin modulul ăsta si să verifice îndeosebi codul care relizează linkul către formularul de încărcare - încerc să-l setez în așa fel încât pozele cu monumente probabil să fie sub drepturi de autor să ajungă la noi, celelalte să ajungă la commons. Nici eu nu prea știu LUA, așa că a fost scris cam prin trial and error, dar codul mi se pare decent de urmărit. Mulțumesc.--Strainu (دسستي) 18 august 2014 21:46 (EEST)
- Îmi pare rău, nici eu nu știu LUA (sincer să fiu, e prima dată când aud de jucăria asta) și sunt foarte prins zilele astea cu pregătiri de concediu. --Gutza DD+ 19 august 2014 12:43 (EEST)
- Nimeni nu cred că știe LUA foarte bine, dat fiind că e un limbaj relativ nou. Din fericire, codul este și destul de lizibil. Miezul fiecărei metode par a fi instrucțiunile if-elseif, care mie mi se par OK, deși am ceva teamă de faptul că nu se verifică înainte ce conțin acele frame.args și dacă chiar există. Poate i-ar prinde bine niște teste unitare. —Andreidiscuţie 20 august 2014 11:19 (EEST)
- Andrei Stroe, ce zici de un subset din [1] pentru testare? Căsuțele cu roșu, de jos, au câmpul copyright setat la toate cele 3 variante acceptabile (da/nu/an), restul sunt calculate pe baza datării.--Strainu (دسستي) 20 august 2014 12:19 (EEST)
- Da, ar merge. —Andreidiscuţie 20 august 2014 13:08 (EEST)
- Andrei Stroe, ce zici de un subset din [1] pentru testare? Căsuțele cu roșu, de jos, au câmpul copyright setat la toate cele 3 variante acceptabile (da/nu/an), restul sunt calculate pe baza datării.--Strainu (دسستي) 20 august 2014 12:19 (EEST)
- Nimeni nu cred că știe LUA foarte bine, dat fiind că e un limbaj relativ nou. Din fericire, codul este și destul de lizibil. Miezul fiecărei metode par a fi instrucțiunile if-elseif, care mie mi se par OK, deși am ceva teamă de faptul că nu se verifică înainte ce conțin acele frame.args și dacă chiar există. Poate i-ar prinde bine niște teste unitare. —Andreidiscuţie 20 august 2014 11:19 (EEST)